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 PG_translat

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 4

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 == PG_ref

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 PG_trans

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 PG_refco

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 by

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 replaced