Re: [PATCH v2 1/9] mm: introduce pmd_install() helper

[Qi Zheng]

On 2021/8/26 AM12:32, David Hildenbrand wrote:
> On 25.08.21 18:20, Qi Zheng wrote:
> > On 2021/8/25 AM12:26, David Hildenbrand wrote:
> > > On 19.08.21 05:18, Qi Zheng wrote:
> > > > Currently we have three times the same few lines repeated in the
> > >
> > > Reviewed-by: David Hildenbrand <david@xxxxxxxxxx>
> >
> > Thanks for your review, I will add this to the patch v3.
> >
> > > That's mostly unrelated to the remaining part of the series and can be
> > > picked up early.
> >
> > The implementation of subsequent patches depends on pmd_install().
> > So I am worried that if this patch is submitted as a separate patch,
> > subsequent patches will not be updated until this patch is merged.
> > What do you think?
>
> Usually I tend to send cleanups out independently, and then just base 
> the other series on top of the other series.

LGTM, I will submit [PATCH v2 1/9] and [PATCH v2 2/9] as a separate
patch series.

> I'll have some more comments in reply to v2. It's fairly hard to revie > because you do a lot of complicated stuff in only a handful of patches
> :) I'll try to think of something reasonable on how to split this up to 
> make it easier to digest.

Thank you very much, and I look forward to your suggestions. At the same
time, I'll also find ways to organize the code more clearly and
concisely, and add documentation to explain the new APIs.

Thanks,
Qi