lkml.org 
[lkml]   [2018]   [Nov]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.18 056/171] zram: close udev startup race condition as default groups
    Date
    4.18-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    commit fef912bf860e upstream.
    commit 98af4d4df889 upstream.

    I got a report from Howard Chen that he saw zram and sysfs race(ie,
    zram block device file is created but sysfs for it isn't yet)
    when he tried to create new zram devices via hotadd knob.

    v4.20 kernel fixes it by [1, 2] but it's too large size to merge
    into -stable so this patch fixes the problem by registering defualt
    group by Greg KH's approach[3].

    This patch should be applied to every stable tree [3.16+] currently
    existing from kernel.org because the problem was introduced at 2.6.37
    by [4].

    [1] fef912bf860e, block: genhd: add 'groups' argument to device_add_disk
    [2] 98af4d4df889, zram: register default groups with device_add_disk()
    [3] http://kroah.com/log/blog/2013/06/26/how-to-create-a-sysfs-file-correctly/
    [4] 33863c21e69e9, Staging: zram: Replace ioctls with sysfs interface

    Cc: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>
    Cc: Hannes Reinecke <hare@suse.com>
    Tested-by: Howard Chen <howardsoc@google.com>
    Signed-off-by: Minchan Kim <minchan@kernel.org>
    Reviewed-by: Hannes Reinecke <hare@suse.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/block/zram/zram_drv.c | 26 ++++++--------------------
    1 file changed, 6 insertions(+), 20 deletions(-)

    diff --git a/drivers/block/zram/zram_drv.c b/drivers/block/zram/zram_drv.c
    index af7cb8e618fe..363b9102ebb0 100644
    --- a/drivers/block/zram/zram_drv.c
    +++ b/drivers/block/zram/zram_drv.c
    @@ -1637,6 +1637,11 @@ static const struct attribute_group zram_disk_attr_group = {
    .attrs = zram_disk_attrs,
    };

    +static const struct attribute_group *zram_disk_attr_groups[] = {
    + &zram_disk_attr_group,
    + NULL,
    +};
    +
    /*
    * Allocate and initialize new zram device. the function returns
    * '>= 0' device_id upon success, and negative value otherwise.
    @@ -1717,24 +1722,15 @@ static int zram_add(void)

    zram->disk->queue->backing_dev_info->capabilities |=
    (BDI_CAP_STABLE_WRITES | BDI_CAP_SYNCHRONOUS_IO);
    + disk_to_dev(zram->disk)->groups = zram_disk_attr_groups;
    add_disk(zram->disk);

    - ret = sysfs_create_group(&disk_to_dev(zram->disk)->kobj,
    - &zram_disk_attr_group);
    - if (ret < 0) {
    - pr_err("Error creating sysfs group for device %d\n",
    - device_id);
    - goto out_free_disk;
    - }
    strlcpy(zram->compressor, default_compressor, sizeof(zram->compressor));

    zram_debugfs_register(zram);
    pr_info("Added device: %s\n", zram->disk->disk_name);
    return device_id;

    -out_free_disk:
    - del_gendisk(zram->disk);
    - put_disk(zram->disk);
    out_free_queue:
    blk_cleanup_queue(queue);
    out_free_idr:
    @@ -1763,16 +1759,6 @@ static int zram_remove(struct zram *zram)
    mutex_unlock(&bdev->bd_mutex);

    zram_debugfs_unregister(zram);
    - /*
    - * Remove sysfs first, so no one will perform a disksize
    - * store while we destroy the devices. This also helps during
    - * hot_remove -- zram_reset_device() is the last holder of
    - * ->init_lock, no later/concurrent disksize_store() or any
    - * other sysfs handlers are possible.
    - */
    - sysfs_remove_group(&disk_to_dev(zram->disk)->kobj,
    - &zram_disk_attr_group);
    -
    /* Make sure all the pending I/O are finished */
    fsync_bdev(bdev);
    zram_reset_device(zram);
    --
    2.17.1


    \
     
     \ /
      Last update: 2018-11-19 18:53    [W:4.065 / U:0.224 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site