On Mon, Nov 27, 2023 at 02:21:00PM +0800, Yu Kuai wrote: > From: Yu Kuai <yukuai3@xxxxxxxxxx> > > Changes in v2: > - split different portions into different patches, as greg k-h > suggested. > - use container_of() instead of "bdev + 1" to get the address of > bd_inode in the new helper, as grep k-h suggested. You might have misinterpreted gregkh - in your place I would rather do a one-patch never-rebased branch (introduction of bdev_inode() in form that returns bdev->bd_inode), with followup in your branch that switches it to your variant. Then conversions of ->bd_inode users, to be either picked by individual filesystems of staying in your branch. Any filesystem tree could merge from your never-rebased branch, after which they could switch their ->bd_inode uses to the new helper, without introducing any bisection hazards or interdependencies. After the next -rc1, once all ->bd_inode users are gone from the tree - remove the field.
