[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH for-4.12 v2 04/17] xen/arm: p2m: Introduce p2m_is_valid and use it



Hi Stefano

On 06/12/2018 22:02, Stefano Stabellini wrote:
On Wed, 5 Dec 2018, Julien Grall wrote:
On 04/12/2018 23:50, Stefano Stabellini wrote:
On Tue, 4 Dec 2018, Julien Grall wrote:
The LPAE format allows to store information in an entry even with the
valid bit unset. In a follow-up patch, we will take advantage of this
feature to re-purpose the valid bit for generating a translation fault
even if an entry contains valid information.

So we need a different way to know whether an entry contains valid
information. It is possible to use the information hold in the p2m_type
to know for that purpose. Indeed all entries containing valid
information will have a valid p2m type (i.e p2m_type != p2m_invalid).

This patch introduces a new helper p2m_is_valid, which implements that
idea, and replace most of lpae_is_valid call with the new helper. The ones
remaining are for TLBs handling and entries accounting.

With the renaming there are 2 others changes required:
      - Generate table entry with a valid p2m type
      - Detect new mapping for proper stats accounting

Signed-off-by: Julien Grall <julien.grall@xxxxxxx>

Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

(This patch doesn't apply to master, please rebase)

Why are you trying to apply to master? This series (as most of my series) are
based on staging at the time it was sent. I tried to apply this patch today on
staging and I didn't find any issue.

No problems then, I thought the series was based on an older tree, but
instead it was on step ahead.

I tend to base everything on staging because master sometimes far ahead :). I just realized that I haven't pushed a branch with this series applied. It is now pushed:

https://xenbits.xen.org/git-http/people/julieng/xen-unstable.git
branch cacheflush/v2

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.