[PATCH 0/2] block: Delete bio_set_prio() and bio_prio()

John Garry posted 2 patches 1 year, 2 months ago
drivers/block/rnbd/rnbd-srv.c | 2 +-
drivers/md/bcache/movinggc.c  | 2 +-
drivers/md/bcache/writeback.c | 2 +-
drivers/md/dm-verity-fec.c    | 6 +++---
drivers/md/dm-verity-target.c | 4 ++--
fs/bcachefs/move.c            | 6 +++---
include/linux/bio.h           | 3 ---
7 files changed, 11 insertions(+), 14 deletions(-)
[PATCH 0/2] block: Delete bio_set_prio() and bio_prio()
Posted by John Garry 1 year, 2 months ago
Macros bio_set_prio() and bio_prio() do nothing special in terms of
setting and getting the bio io prio member, so just delete them.

Prior to commit 43b62ce3ff0a, they would actually encode and decode the
prio in the now-deleted bi_rw member.

John Garry (2):
  block: Delete bio_prio()
  block: Delete bio_set_prio()

 drivers/block/rnbd/rnbd-srv.c | 2 +-
 drivers/md/bcache/movinggc.c  | 2 +-
 drivers/md/bcache/writeback.c | 2 +-
 drivers/md/dm-verity-fec.c    | 6 +++---
 drivers/md/dm-verity-target.c | 4 ++--
 fs/bcachefs/move.c            | 6 +++---
 include/linux/bio.h           | 3 ---
 7 files changed, 11 insertions(+), 14 deletions(-)

-- 
2.31.1
Re: [PATCH 0/2] block: Delete bio_set_prio() and bio_prio()
Posted by Jens Axboe 1 year, 2 months ago
On Mon, 02 Dec 2024 11:19:55 +0000, John Garry wrote:
> Macros bio_set_prio() and bio_prio() do nothing special in terms of
> setting and getting the bio io prio member, so just delete them.
> 
> Prior to commit 43b62ce3ff0a, they would actually encode and decode the
> prio in the now-deleted bi_rw member.
> 
> John Garry (2):
>   block: Delete bio_prio()
>   block: Delete bio_set_prio()
> 
> [...]

Applied, thanks!

[1/2] block: Delete bio_prio()
      commit: 099d214fc7abc3fec0f38d10bec31ac7acce8d13
[2/2] block: Delete bio_set_prio()
      commit: 77cfdf838d8467d3ca44058caff7c1727080efb2

Best regards,
-- 
Jens Axboe
Re: [PATCH 0/2] block: Delete bio_set_prio() and bio_prio()
Posted by Chaitanya Kulkarni 1 year, 2 months ago
On 12/2/24 03:19, John Garry wrote:
> Macros bio_set_prio() and bio_prio() do nothing special in terms of
> setting and getting the bio io prio member, so just delete them.
>
> Prior to commit 43b62ce3ff0a, they would actually encode and decode the
> prio in the now-deleted bi_rw member.

Looks good.

Reviewed-by: Chaitanya Kulkarni <kch@nvidia.com>

-ck