Re: [PATCH v2 1/9] mm: introduce pmd_install() helper
From: Qi Zheng <hidden>
Date: 2021-08-26 03:04:37
Also in:
linux-mm, lkml
On 2021/8/26 AM12:32, David Hildenbrand wrote:
On 25.08.21 18:20, Qi Zheng wrote:quoted
On 2021/8/25 AM12:26, David Hildenbrand wrote:quoted
On 19.08.21 05:18, Qi Zheng wrote:quoted
Currently we have three times the same few lines repeated in theReviewed-by: David Hildenbrand <redacted>Thanks for your review, I will add this to the patch v3.quoted
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