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

Re: [Xen-devel] [PATCH for-4.5 v7 05/21] xen: Relocate p2m_access_t into common and swap the order



>>> On 17.09.14 at 22:51, <tklengyel@xxxxxxxxxxxxx> wrote:
> We swap the order of the enum of types n ... rwx, as to have rwx at 0. This 
> allows
> us to memset the permission store to 0, without having to specify rwx as the 
> default
> permission when memaccess is not in use. This also has performance benefits 
> for
> non-memaccess paths, as now comparison is to 0 when checking if memaccess is 
> in use,
> which is usually faster.

I'd say often instead of usually here.

And then the description leaves open whether any dependency on
rwx now being first is getting introduced (no memset() is being added
here, yet wherever an explicit one would get added a respective
BUILD_BUG_ON() should be put along with it).

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

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