On Mon, Jul 28, 2025 at 04:58:51PM +0800, Zorro Lang wrote: > On Sat, Jul 12, 2025 at 07:42:48PM +0530, Ojaswin Mujoo wrote: > > From: "Ritesh Harjani (IBM)" <ritesh.list@xxxxxxxxx> > > > > This tests uses fio to first create a file with mixed mappings. Then it > > does atomic writes using aio dio with parallel jobs to the same file > > with mixed mappings. This forces the filesystem allocator to allocate > > extents over mixed mapping regions to stress FS block allocators. > > > > Co-developed-by: Ritesh Harjani (IBM) <ritesh.list@xxxxxxxxx> > > Signed-off-by: Ritesh Harjani (IBM) <ritesh.list@xxxxxxxxx> > > Signed-off-by: Ojaswin Mujoo <ojaswin@xxxxxxxxxxxxx> > > --- > > This patch looks good to me, just the subject: > "generic/1227: Add atomic write test using fio verify on file mixed mappings" > > generally if we write a new test case, we don't use a temporary case number > in commit subject, you can just write as "generic: add atomic write test using ..." > > Other patches (with new test cases) refer to this. > > With this change, > Reviewed-by: Zorro Lang <zlang@xxxxxxxxxx> Hi Zorro, thanks for pointing it out. I'll make the change in next revision. Regards, Ojaswin
