Re: [RFC PATCH 3/4] mm: add zbud flag to page flags

2013-08-08 Thread Krzysztof Kozlowski
Hi, On wto, 2013-08-06 at 09:58 -0700, Dave Hansen wrote: > On 08/05/2013 11:42 PM, Krzysztof Kozlowski wrote: > > +#ifdef CONFIG_ZBUD > > + /* Allocated by zbud. Flag is necessary to find zbud pages to unuse > > +* during migration/compaction. > > +*/ > > + PG_zbud, > > +#endif > >

Re: [RFC PATCH 3/4] mm: add zbud flag to page flags

2013-08-08 Thread Krzysztof Kozlowski
Hi, On wto, 2013-08-06 at 09:58 -0700, Dave Hansen wrote: On 08/05/2013 11:42 PM, Krzysztof Kozlowski wrote: +#ifdef CONFIG_ZBUD + /* Allocated by zbud. Flag is necessary to find zbud pages to unuse +* during migration/compaction. +*/ + PG_zbud, +#endif Do you _really_

Re: [RFC PATCH 3/4] mm: add zbud flag to page flags

2013-08-07 Thread Krzysztof Kozlowski
On wto, 2013-08-06 at 09:58 -0700, Dave Hansen wrote: > On 08/05/2013 11:42 PM, Krzysztof Kozlowski wrote: > > +#ifdef CONFIG_ZBUD > > + /* Allocated by zbud. Flag is necessary to find zbud pages to unuse > > +* during migration/compaction. > > +*/ > > + PG_zbud, > > +#endif > > Do

Re: [RFC PATCH 3/4] mm: add zbud flag to page flags

2013-08-07 Thread Krzysztof Kozlowski
On wto, 2013-08-06 at 09:58 -0700, Dave Hansen wrote: On 08/05/2013 11:42 PM, Krzysztof Kozlowski wrote: +#ifdef CONFIG_ZBUD + /* Allocated by zbud. Flag is necessary to find zbud pages to unuse +* during migration/compaction. +*/ + PG_zbud, +#endif Do you _really_ need

Re: [RFC PATCH 3/4] mm: add zbud flag to page flags

2013-08-06 Thread Dave Hansen
On 08/05/2013 11:42 PM, Krzysztof Kozlowski wrote: > +#ifdef CONFIG_ZBUD > + /* Allocated by zbud. Flag is necessary to find zbud pages to unuse > + * during migration/compaction. > + */ > + PG_zbud, > +#endif Do you _really_ need an absolutely new, unshared page flag? The zbud

Re: [RFC PATCH 3/4] mm: add zbud flag to page flags

2013-08-06 Thread Dave Hansen
On 08/05/2013 11:42 PM, Krzysztof Kozlowski wrote: +#ifdef CONFIG_ZBUD + /* Allocated by zbud. Flag is necessary to find zbud pages to unuse + * during migration/compaction. + */ + PG_zbud, +#endif Do you _really_ need an absolutely new, unshared page flag? The zbud code