On (23/04/04 17:05), Christoph Hellwig wrote:
>
> Switch on the bio operation in zram_submit_bio and only call into
> __zram_make_request for read and write operations.
>
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
Reviewed-by: Sergey Senozhatsky <senozhatsky@xxxxxxxxxxxx>
[..]
> @@ -1996,7 +1987,19 @@ static void zram_submit_bio(struct bio *bio)
> {
> struct zram *zram = bio->bi_bdev->bd_disk->private_data;
>
> - __zram_make_request(zram, bio);
> + switch (bio_op(bio)) {
> + case REQ_OP_READ:
> + case REQ_OP_WRITE:
> + __zram_make_request(zram, bio);
> + break;
> + case REQ_OP_DISCARD:
> + case REQ_OP_WRITE_ZEROES:
> + zram_bio_discard(zram, bio);
> + return;
Super nitpick, break;
> + default:
> + WARN_ON_ONCE(1);
> + bio_endio(bio);
> + }
> }
>
> static void zram_slot_free_notify(struct block_device *bdev,
> --
> 2.39.2
>
