[GIT PULL] ->driverfs_dev removal for 4.8

[GIT PULL] ->driverfs_dev removal for 4.8

[Williams, Dan J]

Hi Jens, please pull from:

  git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm for-4.8/block

...to receive:

The removal of ->driverfs_dev in favor of just passing the parent
device in as a parameter to add_disk().  See below, it has received a
"Reviewed-by" from Christoph, Bart, and Johannes.

It is also a pre-requisite for Fam Zheng's work to cleanup gendisk
uevents vs attribute visibility [1].  We would extend device_add_disk()
to take an attribute_group list.

This is based off a branch of block.git/for-4.8/drivers and has
received a positive build success notification from the kbuild robot
across several configs.

[1]: "gendisk: Generate uevent after attribute available"
http://marc.info/?l=linux-virtualization&m=146725201522201&w=2

---

The following changes since commit 1b57e66384e2d21150301e68078526fac5680a16:

  drbd: correctly handle failed crypto_alloc_hash (2016-06-13 21:43:08 -0600)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm for-4.8/block

for you to fetch changes up to 52c44d93c26f5a76068c0a8cc83bb8f56f38043d:

  block: remove ->driverfs_dev (2016-06-27 12:26:08 -0700)

----------------------------------------------------------------
Dan Williams (5):
      block: introduce device_add_disk()
      mmc: move 'parent' tracking to mmc_blk_data
      um: track 'parent' device in a local variable
      block: convert to device_add_disk()
      block: remove ->driverfs_dev

 arch/powerpc/sysdev/axonram.c       |  3 +--
 arch/um/drivers/ubd_kern.c          |  5 +++--
 block/genhd.c                       | 18 +++++++++---------
 drivers/block/cciss.c               |  3 +--
 drivers/block/floppy.c              |  3 +--
 drivers/block/mtip32xx/mtip32xx.c   |  5 ++---
 drivers/block/ps3disk.c             |  3 +--
 drivers/block/ps3vram.c             |  3 +--
 drivers/block/rsxx/dev.c            |  4 +---
 drivers/block/skd_main.c            |  8 +++-----
 drivers/block/sunvdc.c              |  3 +--
 drivers/block/virtio_blk.c          |  3 +--
 drivers/block/xen-blkfront.c        |  3 +--
 drivers/ide/ide-cd.c                |  3 +--
 drivers/ide/ide-gd.c                |  3 +--
 drivers/memstick/core/ms_block.c    |  3 +--
 drivers/memstick/core/mspro_block.c |  3 +--
 drivers/mmc/card/block.c            |  5 +++--
 drivers/mtd/mtd_blkdevs.c           |  4 +---
 drivers/nvdimm/blk.c                |  3 +--
 drivers/nvdimm/btt.c                |  3 +--
 drivers/nvdimm/bus.c                |  2 +-
 drivers/nvdimm/pmem.c               |  3 +--
 drivers/nvme/host/core.c            |  3 +--
 drivers/s390/block/dasd_genhd.c     |  3 +--
 drivers/s390/block/dcssblk.c        |  3 +--
 drivers/s390/block/scm_blk.c        |  3 +--
 drivers/scsi/sd.c                   |  3 +--
 drivers/scsi/sr.c                   |  3 +--
 include/linux/genhd.h               |  8 ++++++--
 30 files changed, 50 insertions(+), 72 deletions(-)


commit e63a46bef01ff3064f44dba145833284fb6adeec
Author: Dan Williams <dan.j.williams@intel.com>
Date:   Wed Jun 15 18:17:27 2016 -0700

    block: introduce device_add_disk()
    
    In preparation for removing the ->driverfs_dev member of a gendisk, add
    an api that takes the parent device as a parameter to add_disk().  For
    now this maintains the status quo of WARN()ing on failure, but not
    return a error code.
    
    Reviewed-by: Christoph Hellwig <hch@lst.de>
    Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
    Reviewed-by: Bart Van Assche <bart.vanassche@sandisk.com>
    Signed-off-by: Dan Williams <dan.j.williams@intel.com>

commit 307d8e6f248558b184a2513593dbbeb08406a4fa
Author: Dan Williams <dan.j.williams@intel.com>
Date:   Mon Jun 20 10:40:44 2016 -0700

    mmc: move 'parent' tracking to mmc_blk_data
    
    In preparation for the removal of 'driverfs_dev' from 'struct gendisk',
    carry this data in mmc_blk_data. It is used for registration of parent
    disks and partitions.
    
    Cc: Ulf Hansson <ulf.hansson@linaro.org>
    Cc: Bart Van Assche <bart.vanassche@sandisk.com>
    Signed-off-by: Dan Williams <dan.j.williams@intel.com>

commit d72a57835c92f6d51863b0ad6d7d45e3859e24d4
Author: Dan Williams <dan.j.williams@intel.com>
Date:   Mon Jun 20 10:44:32 2016 -0700

    um: track 'parent' device in a local variable
    
    In preparation for the removal of 'driverfs_dev' from 'struct gendisk'
    use a local variable to track the parented vs un-parented case in
    ubd_disk_register().
    
    Cc: Jeff Dike <jdike@addtoit.com>
    Cc: Richard Weinberger <richard@nod.at>
    Cc: Bart Van Assche <bart.vanassche@sandisk.com>
    Signed-off-by: Dan Williams <dan.j.williams@intel.com>

commit 0d52c756a665adc032c791307bc55e392b0186b3
Author: Dan Williams <dan.j.williams@intel.com>
Date:   Wed Jun 15 19:44:20 2016 -0700

    block: convert to device_add_disk()
    
    For block drivers that specify a parent device, convert them to use
    device_add_disk().
    
    This conversion was done with the following semantic patch:
    
        @@
        struct gendisk *disk;
        expression E;
        @@
    
        - disk->driverfs_dev = E;
        ...
        - add_disk(disk);
        + device_add_disk(E, disk);
    
        @@
        struct gendisk *disk;
        expression E1, E2;
        @@
    
        - disk->driverfs_dev = E1;
        ...
        E2 = disk;
        ...
        - add_disk(E2);
        + device_add_disk(E1, E2);
    
    ...plus some manual fixups for a few missed conversions.
    
    Cc: Jens Axboe <axboe@fb.com>
    Cc: Keith Busch <keith.busch@intel.com>
    Cc: Michael S. Tsirkin <mst@redhat.com>
    Cc: David Woodhouse <dwmw2@infradead.org>
    Cc: David S. Miller <davem@davemloft.net>
    Cc: James Bottomley <James.Bottomley@hansenpartnership.com>
    Cc: Ross Zwisler <ross.zwisler@linux.intel.com>
    Cc: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
    Cc: Martin K. Petersen <martin.petersen@oracle.com>
    Reviewed-by: Christoph Hellwig <hch@lst.de>
    Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
    Signed-off-by: Dan Williams <dan.j.williams@intel.com>

commit 52c44d93c26f5a76068c0a8cc83bb8f56f38043d
Author: Dan Williams <dan.j.williams@intel.com>
Date:   Wed Jun 15 19:43:07 2016 -0700

    block: remove ->driverfs_dev
    
    Now that all drivers that specify a ->driverfs_dev have been converted
    to device_add_disk(), the pointer can be removed from struct gendisk.
    
    Cc: Jens Axboe <axboe@fb.com>
    Cc: Ross Zwisler <ross.zwisler@linux.intel.com>
    Reviewed-by: Christoph Hellwig <hch@lst.de>
    Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
    Signed-off-by: Dan Williams <dan.j.williams@intel.com>

Re: [GIT PULL] ->driverfs_dev removal for 4.8

[Jens Axboe]

On 07/08/2016 03:56 PM, Williams, Dan J wrote:
> Hi Jens, please pull from:
>
>    git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm for-4.8/block
>
> ...to receive:
>
> The removal of ->driverfs_dev in favor of just passing the parent
> device in as a parameter to add_disk().  See below, it has received a
> "Reviewed-by" from Christoph, Bart, and Johannes.
>
> It is also a pre-requisite for Fam Zheng's work to cleanup gendisk
> uevents vs attribute visibility [1].  We would extend device_add_disk()
> to take an attribute_group list.
>
> This is based off a branch of block.git/for-4.8/drivers and has
> received a positive build success notification from the kbuild robot
> across several configs.
>
> [1]: "gendisk: Generate uevent after attribute available"
> http://marc.info/?l=linux-virtualization&m=146725201522201&w=2

Pulled, thanks Dan.

-- 
Jens Axboe