mirror of
git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-09-18 22:14:16 +00:00
md/raid1: Use blk_opf_t for read and write operations
Use the type blk_opf_t for read and write operations instead of int. This
patch does not affect the generated code but fixes the following sparse
warning:
drivers/md/raid1.c:1993:60: sparse: sparse: incorrect type in argument 5 (different base types)
expected restricted blk_opf_t [usertype] opf
got int rw
Cc: Song Liu <song@kernel.org>
Cc: Jens Axboe <axboe@kernel.dk>
Fixes: 3c5e514db5
("md/raid1: Use the new blk_opf_t type")
Cc: stable@vger.kernel.org # v6.0+
Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202401080657.UjFnvQgX-lkp@intel.com/
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Song Liu <song@kernel.org>
Link: https://lore.kernel.org/r/20240108001223.23835-1-bvanassche@acm.org
This commit is contained in:
parent
f9cfe7e7f9
commit
7dab24554d
1 changed files with 6 additions and 6 deletions
|
@ -1968,12 +1968,12 @@ static void end_sync_write(struct bio *bio)
|
|||
}
|
||||
|
||||
static int r1_sync_page_io(struct md_rdev *rdev, sector_t sector,
|
||||
int sectors, struct page *page, int rw)
|
||||
int sectors, struct page *page, blk_opf_t rw)
|
||||
{
|
||||
if (sync_page_io(rdev, sector, sectors << 9, page, rw, false))
|
||||
/* success */
|
||||
return 1;
|
||||
if (rw == WRITE) {
|
||||
if (rw == REQ_OP_WRITE) {
|
||||
set_bit(WriteErrorSeen, &rdev->flags);
|
||||
if (!test_and_set_bit(WantReplacement,
|
||||
&rdev->flags))
|
||||
|
@ -2090,7 +2090,7 @@ static int fix_sync_read_error(struct r1bio *r1_bio)
|
|||
rdev = conf->mirrors[d].rdev;
|
||||
if (r1_sync_page_io(rdev, sect, s,
|
||||
pages[idx],
|
||||
WRITE) == 0) {
|
||||
REQ_OP_WRITE) == 0) {
|
||||
r1_bio->bios[d]->bi_end_io = NULL;
|
||||
rdev_dec_pending(rdev, mddev);
|
||||
}
|
||||
|
@ -2105,7 +2105,7 @@ static int fix_sync_read_error(struct r1bio *r1_bio)
|
|||
rdev = conf->mirrors[d].rdev;
|
||||
if (r1_sync_page_io(rdev, sect, s,
|
||||
pages[idx],
|
||||
READ) != 0)
|
||||
REQ_OP_READ) != 0)
|
||||
atomic_add(s, &rdev->corrected_errors);
|
||||
}
|
||||
sectors -= s;
|
||||
|
@ -2321,7 +2321,7 @@ static void fix_read_error(struct r1conf *conf, struct r1bio *r1_bio)
|
|||
!test_bit(Faulty, &rdev->flags)) {
|
||||
atomic_inc(&rdev->nr_pending);
|
||||
r1_sync_page_io(rdev, sect, s,
|
||||
conf->tmppage, WRITE);
|
||||
conf->tmppage, REQ_OP_WRITE);
|
||||
rdev_dec_pending(rdev, mddev);
|
||||
}
|
||||
}
|
||||
|
@ -2335,7 +2335,7 @@ static void fix_read_error(struct r1conf *conf, struct r1bio *r1_bio)
|
|||
!test_bit(Faulty, &rdev->flags)) {
|
||||
atomic_inc(&rdev->nr_pending);
|
||||
if (r1_sync_page_io(rdev, sect, s,
|
||||
conf->tmppage, READ)) {
|
||||
conf->tmppage, REQ_OP_READ)) {
|
||||
atomic_add(s, &rdev->corrected_errors);
|
||||
pr_info("md/raid1:%s: read error corrected (%d sectors at %llu on %pg)\n",
|
||||
mdname(mddev), s,
|
||||
|
|
Loading…
Add table
Reference in a new issue