Hi Shakeel, On Wed, 29 Jan 2025 11:22:29 -0800 Shakeel Butt <shakeel.butt@xxxxxxxxx> wrote: > On Thu, Jan 16, 2025 at 05:30:54PM -0800, SeongJae Park wrote: > > process_madvise() calls do_madvise() for each address range. Then, each > > do_madvise() invocation holds and releases same mmap_lock. Optimize the > > redundant lock operations by splitting do_madvise() internal logics > > including the mmap_lock operations, and calling the small logics > > directly from process_madvise() in a sequence that removes the redundant > > locking. > > > > You skipped the evaluation section which was present in the v1. If you > decide to post a new version, please include that otherwise we can ask > Andrew to pickup the evaluation section from the v1 [1]. > > [1] https://lore.kernel.org/all/20250111004618.1566-1-sj@xxxxxxxxxx/ Sure. I will run the evaluation again on the new version and include the result on the next version. Also, thank you very much for reviews :) Thanks, SJ