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

Re: [Xen-devel] [PATCH 2/7] xen/arm: Remove flush_xen_text_tlb_local()



Julien,

Sorry, reply format is broken. It should be as following:

On 26.04.19 16:50, Andrii Anisov wrote:
To be honest, I am not entirely sure whether the isb() is necessary
here. At worst, an entry of an existing mappings is not cached with the
SCTLR_EL2.WXN set. This may only defer the problem is an entry is
incorrectly configured before the flush. This is done early at boot (and
will be done much earlier in [1]).


Well, following this logic we may finish with TLB flush is not needed here at 
all. Because entries, already reached TLB, were configured before, and are 
believed valid. All the next will have WXN set.
But in this case, ISB is needed as an exact point from which all new TLB 
entries will have WXN.


So I am kind of tempted to drop the
isb() here.

--
Sincerely,
Andrii Anisov.

_______________________________________________
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®.