Message ID | 20211025025426.2815424-1-ming.lei@redhat.com (mailing list archive) |
---|---|
Headers | show |
Series | zram: fix two races and one zram leak | expand |
On Mon, Oct 25, 2021 at 10:54:22AM +0800, Ming Lei wrote: > Hello, > > Fixes three issues reported by Luis Chamberlain with one simpler approach: > > - race between between zram_reset_device() and disksize_store() (1/4) > > - zram leak during unloading module, which is one race between resetting > and removing device (2/4) > > - race between zram_remove and disksize_store (3/4) > > Also replace replace fsync_bdev with sync_blockdev since no one opens > it.(4/4) > > V3: > - no code change > - update commit log or comment as Luis suggested > - add reviewed-by tag > > V2: > - take another approach to avoid failing of zram_remove() > - add patch to address race between zram_reset_device() and > disksize_store() > > > Ming Lei (4): > zram: fix race between zram_reset_device() and disksize_store() > zram: don't fail to remove zram during unloading module > zram: avoid race between zram_remove and disksize_store > zram: replace fsync_bdev with sync_blockdev Andrew Morton usually takes zram patches so Ccing him. Acked-by: Minchan Kim <minchan@kernel.org> for all patches in this thread.
Hello Andrew Morton and Jens, On Mon, Oct 25, 2021 at 09:30:07AM -0700, Minchan Kim wrote: > On Mon, Oct 25, 2021 at 10:54:22AM +0800, Ming Lei wrote: > > Hello, > > > > Fixes three issues reported by Luis Chamberlain with one simpler approach: > > > > - race between between zram_reset_device() and disksize_store() (1/4) > > > > - zram leak during unloading module, which is one race between resetting > > and removing device (2/4) > > > > - race between zram_remove and disksize_store (3/4) > > > > Also replace replace fsync_bdev with sync_blockdev since no one opens > > it.(4/4) > > > > V3: > > - no code change > > - update commit log or comment as Luis suggested > > - add reviewed-by tag > > > > V2: > > - take another approach to avoid failing of zram_remove() > > - add patch to address race between zram_reset_device() and > > disksize_store() > > > > > > Ming Lei (4): > > zram: fix race between zram_reset_device() and disksize_store() > > zram: don't fail to remove zram during unloading module > > zram: avoid race between zram_remove and disksize_store > > zram: replace fsync_bdev with sync_blockdev > > Andrew Morton usually takes zram patches so Ccing him. > > Acked-by: Minchan Kim <minchan@kernel.org> > > for all patches in this thread. Any chance to make it in v5.16? Thanks, Ming
On Mon, 25 Oct 2021 10:54:22 +0800, Ming Lei wrote: > Fixes three issues reported by Luis Chamberlain with one simpler approach: > > - race between between zram_reset_device() and disksize_store() (1/4) > > - zram leak during unloading module, which is one race between resetting > and removing device (2/4) > > [...] Applied, thanks! [1/4] zram: fix race between zram_reset_device() and disksize_store() commit: 6f1637795f2827d36aec9e0246487f5852e8abf7 [2/4] zram: don't fail to remove zram during unloading module commit: 8c54499a59b026a3dc2afccf6e1b36d5700d2fef [3/4] zram: avoid race between zram_remove and disksize_store commit: 5a4b653655d554b5f51a5d2252882708c56a6f7e [4/4] zram: replace fsync_bdev with sync_blockdev commit: 00c5495c54f785beb0f6a34f7a3674d3ea0997d5 Best regards,