Thread (28 messages) 28 messages, 5 authors, 2020-05-13

Re: [PATCH 01/15] nfblock: use gendisk private_data

From: Geert Uytterhoeven <geert@linux-m68k.org>
Date: 2020-05-09 10:38:17
Also in: linux-bcache, linux-m68k, linux-raid, linuxppc-dev, lkml, nvdimm

Hi Christoph,

On Fri, May 8, 2020 at 6:16 PM Christoph Hellwig [off-list ref] wrote:
Signed-off-by: Christoph Hellwig <hch@lst.de>
Thanks for your patch!
quoted hunk ↗ jump to hunk
--- a/arch/m68k/emu/nfblock.c
+++ b/arch/m68k/emu/nfblock.c
@@ -61,7 +61,7 @@ struct nfhd_device {

 static blk_qc_t nfhd_make_request(struct request_queue *queue, struct bio *bio)
 {
-       struct nfhd_device *dev = queue->queuedata;
+       struct nfhd_device *dev = bio->bi_disk->private_data;
        struct bio_vec bvec;
        struct bvec_iter iter;
        int dir, len, shift;
@@ -122,7 +122,6 @@ static int __init nfhd_init_one(int id, u32 blocks, u32 bsize)
        if (dev->queue == NULL)
                goto free_dev;

-       dev->queue->queuedata = dev;
        blk_queue_logical_block_size(dev->queue, bsize);

        dev->disk = alloc_disk(16);
@@ -136,6 +135,7 @@ static int __init nfhd_init_one(int id, u32 blocks, u32 bsize)
        sprintf(dev->disk->disk_name, "nfhd%u", dev_id);
        set_capacity(dev->disk, (sector_t)blocks * (bsize / 512));
        dev->disk->queue = dev->queue;
+       dev->disk->private_data = dev;
This is already set above, just before the quoted sprintf() call.
        add_disk(dev->disk);
Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help