[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <fec59d4d-898d-447c-b4fb-e9d055550f96@web.de>
Date: Thu, 15 Aug 2024 11:55:44 +0200
From: Markus Elfring <Markus.Elfring@....de>
To: Kemeng Shi <shikemeng@...weicloud.com>, linux-ext4@...r.kernel.org,
Andreas Dilger <adilger.kernel@...ger.ca>, Theodore Ts'o <tytso@....edu>
Cc: LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/7] ext4: avoid buffer_head leak in ext4_mark_inode_used
> 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
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?
Regards,
Markus
Powered by blists - more mailing lists