mbox series

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

Message ID 20241202111957.2311683-1-john.g.garry@oracle.com (mailing list archive)
Headers show
Series block: Delete bio_set_prio() and bio_prio() | expand

Message

John Garry Dec. 2, 2024, 11:19 a.m. UTC 
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(-)

Comments

Chaitanya Kulkarni Dec. 3, 2024, 2:10 a.m. UTC | #1 
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
Jens Axboe Dec. 3, 2024, 1:42 p.m. UTC | #2 
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,