Re: [PATCH] mm/rmap: synchronize lock and unlock target in anon_vma_clone
Barry Song <[email protected]>
| Newsgroups | gmane.linux.kernel,gmane.linux.kernel.mm |
|---|---|
| Message-ID | <CAGsJ_4xtF3awozoxJTHPu7nREiiJ09Gg1W8tY3B=8Zg_io2wOg@mail.gmail.com> |
On Sun, Aug 16, 2026 at 7:51 PM Seohyun Kim <[email protected]> wrote: > > On 8/15/26 14:21, Barry Song wrote: > > > On Fri, Aug 14, 2026 at 2:33 PM Eric Kim <[email protected]> wrote: [...] > You're right. My original commit message does come as confusing > because it sounds like there's an actual lock/unlock target mismatch. > (I'm not a native speaker too) My intent was to match the variable > name for locking and unlocking, and since active_anon_vma is the > implemented alias for src->anon_vma, anon_vma_clone() should > consistently use active_anon_vma. Should I send a v2 to clarify the commit > message? Yes, please, if possible. :-) Best Regards Barry