Re: [Xen-devel] [PATCH RFC 2/2] x86/mm: PG_translate implies PG_refcounts

2017-08-24 Thread Tim Deegan
At 11:05 +0100 on 24 Aug (1503572736), Wei Liu wrote: > On Thu, Aug 24, 2017 at 11:02:33AM +0100, Tim Deegan wrote: > > At 16:58 +0100 on 23 Aug (1503507504), Wei Liu wrote: > > > After 404595352 ("x86/paging: Enforce PG_external == PG_translate == > > > PG_refcounts"), PG_refcounts and

Re: [Xen-devel] [PATCH RFC 2/2] x86/mm: PG_translate implies PG_refcounts

2017-08-24 Thread Wei Liu
On Thu, Aug 24, 2017 at 11:09:38AM +0100, Andrew Cooper wrote: > On 24/08/17 11:07, Wei Liu wrote: > > On Thu, Aug 24, 2017 at 11:05:36AM +0100, Wei Liu wrote: > >> On Thu, Aug 24, 2017 at 11:02:33AM +0100, Tim Deegan wrote: > >>> At 16:58 +0100 on 23 Aug (1503507504), Wei Liu wrote: > After

Re: [Xen-devel] [PATCH RFC 2/2] x86/mm: PG_translate implies PG_refcounts

2017-08-24 Thread Andrew Cooper
On 24/08/17 11:07, Wei Liu wrote: > On Thu, Aug 24, 2017 at 11:05:36AM +0100, Wei Liu wrote: >> On Thu, Aug 24, 2017 at 11:02:33AM +0100, Tim Deegan wrote: >>> At 16:58 +0100 on 23 Aug (1503507504), Wei Liu wrote: After 404595352 ("x86/paging: Enforce PG_external == PG_translate ==

Re: [Xen-devel] [PATCH RFC 2/2] x86/mm: PG_translate implies PG_refcounts

2017-08-24 Thread Wei Liu
On Thu, Aug 24, 2017 at 11:05:36AM +0100, Wei Liu wrote: > On Thu, Aug 24, 2017 at 11:02:33AM +0100, Tim Deegan wrote: > > At 16:58 +0100 on 23 Aug (1503507504), Wei Liu wrote: > > > After 404595352 ("x86/paging: Enforce PG_external == PG_translate == > > > PG_refcounts"), PG_refcounts and

Re: [Xen-devel] [PATCH RFC 2/2] x86/mm: PG_translate implies PG_refcounts

2017-08-24 Thread Wei Liu
On Thu, Aug 24, 2017 at 11:02:33AM +0100, Tim Deegan wrote: > At 16:58 +0100 on 23 Aug (1503507504), Wei Liu wrote: > > After 404595352 ("x86/paging: Enforce PG_external == PG_translate == > > PG_refcounts"), PG_refcounts and PG_translate always need to be set > > together. > > > > Squash

Re: [Xen-devel] [PATCH RFC 2/2] x86/mm: PG_translate implies PG_refcounts

2017-08-24 Thread Tim Deegan
At 16:58 +0100 on 23 Aug (1503507504), Wei Liu wrote: > After 404595352 ("x86/paging: Enforce PG_external == PG_translate == > PG_refcounts"), PG_refcounts and PG_translate always need to be set > together. > > Squash PG_refcounts to simplify code. All calls paging_mode_refcounts > are replaced

Re: [Xen-devel] [PATCH RFC 2/2] x86/mm: PG_translate implies PG_refcounts

2017-08-23 Thread Wei Liu
On Wed, Aug 23, 2017 at 04:58:24PM +0100, Wei Liu wrote: > After 404595352 ("x86/paging: Enforce PG_external == PG_translate == > PG_refcounts"), PG_refcounts and PG_translate always need to be set > together. > > Squash PG_refcounts to simplify code. All calls paging_mode_refcounts > are

[Xen-devel] [PATCH RFC 2/2] x86/mm: PG_translate implies PG_refcounts

2017-08-23 Thread Wei Liu
After 404595352 ("x86/paging: Enforce PG_external == PG_translate == PG_refcounts"), PG_refcounts and PG_translate always need to be set together. Squash PG_refcounts to simplify code. All calls paging_mode_refcounts are replaced by calls to paging_mode_translate. Signed-off-by: Wei Liu