On Thu, 6 Feb 2025 13:04:53 +0000 Lorenzo Stoakes <lorenzo.stoakes@xxxxxxxxxx> wrote: > On Wed, Feb 05, 2025 at 10:15:17PM -0800, SeongJae Park wrote: > > Optimize redundant mmap lock operations from process_madvise() by > > directly doing the mmap locking first, and then the remaining works for > > all ranges in the loop. > > > > Reviewed-by: Shakeel Butt <shakeel.butt@xxxxxxxxx> > > Signed-off-by: SeongJae Park <sj@xxxxxxxxxx> > > Reviewed-by: Lorenzo Stoakes <lorenzo.stoakes@xxxxxxxxxx> > > But please fixup the comment as below. Maybe Andrew could do it if I am > giving enough info to go on, otherwise it could be a quick fix-patch? I > attach a lazy attempt at a fix-patch in case that's useful. Thank you for your reviews and this fixup! Since Andrew already picked this patch in mm-unstable, I will keep my eyes on the tree and make some action if needed, to ensure your suggestion is applied. Thanks, SJ [...]