Re: [dm-devel] [PATCH 05/45] mtip32xx: remove the call to fsync_bdev on removal

2020-11-30 Thread Johannes Thumshirn
Looks good, Reviewed-by: Johannes Thumshirn -- dm-devel mailing list dm-devel@redhat.com https://www.redhat.com/mailman/listinfo/dm-devel

Re: [dm-devel] [PATCH 05/45] mtip32xx: remove the call to fsync_bdev on removal

2020-11-29 Thread Hannes Reinecke
On 11/28/20 5:14 PM, Christoph Hellwig wrote: del_gendisk already calls fsync_bdev for every partition, no need to do this twice. Signed-off-by: Christoph Hellwig Reviewed-by: Jan Kara --- drivers/block/mtip32xx/mtip32xx.c | 15 --- drivers/block/mtip32xx/mtip32xx.h | 2 -- 2

[dm-devel] [PATCH 05/45] mtip32xx: remove the call to fsync_bdev on removal

2020-11-28 Thread Christoph Hellwig
del_gendisk already calls fsync_bdev for every partition, no need to do this twice. Signed-off-by: Christoph Hellwig Reviewed-by: Jan Kara --- drivers/block/mtip32xx/mtip32xx.c | 15 --- drivers/block/mtip32xx/mtip32xx.h | 2 -- 2 files changed, 17 deletions(-) diff --git a/driver

Re: [dm-devel] [PATCH 05/45] mtip32xx: remove the call to fsync_bdev on removal

2020-11-25 Thread Jan Kara
On Tue 24-11-20 14:27:11, Christoph Hellwig wrote: > del_gendisk already calls fsync_bdev for every partition, no need > to do this twice. > > Signed-off-by: Christoph Hellwig Makes sense to me. You can add: Reviewed-by: Jan Kara

[dm-devel] [PATCH 05/45] mtip32xx: remove the call to fsync_bdev on removal

2020-11-24 Thread Christoph Hellwig
del_gendisk already calls fsync_bdev for every partition, no need to do this twice. Signed-off-by: Christoph Hellwig --- drivers/block/mtip32xx/mtip32xx.c | 15 --- drivers/block/mtip32xx/mtip32xx.h | 2 -- 2 files changed, 17 deletions(-) diff --git a/drivers/block/mtip32xx/mtip32