Re: [PATCH v3 1/2] mm: introduce page reference manipulation functions

Joonsoo Kim
2016-02-25T02:11:34Z

On Wed, Feb 24, 2016 at 05:53:33PM -0800, Andrew Morton wrote:
> On Thu, 25 Feb 2016 09:34:55 +0900 Joonsoo Kim <iamjoonsoo.kim@lge.com> wrote:
>
> > >
> > > The patches will be a bit of a pain to maintain but surprisingly they
> > > apply OK at present. It's possible that by the time they hit upstream,
> > > some direct ->_count references will still be present and it will
> > > require a second pass to complete the conversion.
> >
> > In fact, the patch doesn't change direct ->_count reference for
> > *read*. That's the reason that it is surprisingly OK at present.
> >
> > It's a good idea to change direct ->_count reference even for read.
> > How about changing it in rc2 after mering this patch in rc1?
>
> Sounds fair enough.
>
> Although I'm counting only 11 such sites so perhaps we just go ahead
> and do it?

Okay. It's less than I thought. I will do it soon.

Thanks.