on 8/15/2024 5:55 PM, Markus Elfring wrote: >> Release inode_bitmap_bh from ext4_read_inode_bitmap in >> ext4_mark_inode_used to avoid buffer_head leak. >> By the way, remove unneeded goto for invalid ino when inode_bitmap_bh >> is NULL. > > 1. I suggest to split such changes into separate update steps. > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.11-rc3#n81 It's acceptable to me, but I'm not sure if it worth separate patches to others. I will do separate in next version if no person is against this. > > 2. How do you think about to add any tags (like “Fixes” and “Cc”) accordingly? > > 3. Would you like to append parentheses to any function names? Thanks for remind me of these. I will improve the series in next version. Thanks, Kemeng > > Regards, > Markus >
