lkml.org 
[lkml]   [2014]   [May]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[RFC PATCH v6 1/2 ] Use lock_device_hotplug() in cpu_probe_store() and cpu_release_store()
    Date
    While auditing the usage of lock_device_hotplug_sysfs() for implementing
    it in another way in following patch, it seems to me that the code here
    is to add/remove device, and the files probe/release themselves won't be
    removed.

    lock_device_hotplug_sysfs() is used to solve the deadlocks of s_active of
    the "online" file and the hotplug related locks.

    So if one process trying to remove the device, the sysfs files for this device,
    including "online" will be removed after device_hotplug_lock is grabbed.
    So for another process which might concurrently write to the "online" file,
    after it grabs s_active of the "online", it couldn't directly lock
    device_hotplug_lock, instead, in *_sysfs() version, try lock is used, and if
    it fails, the whole syscall is restarted.

    But here the s_active of "probe/release" are not involved in the above process,
    as they are not to be removed. So we could use lock_device_hotplug() here.

    Cc: Tejun Heo <tj@kernel.org>
    Cc: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
    Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    Cc: Toshi Kani <toshi.kani@hp.com>
    Signed-off-by: Li Zhong <zhong@linux.vnet.ibm.com>
    ---
    drivers/base/cpu.c | 8 ++------
    1 file changed, 2 insertions(+), 6 deletions(-)

    diff --git a/drivers/base/cpu.c b/drivers/base/cpu.c
    index 006b1bc..9483225 100644
    --- a/drivers/base/cpu.c
    +++ b/drivers/base/cpu.c
    @@ -88,9 +88,7 @@ static ssize_t cpu_probe_store(struct device *dev,
    ssize_t cnt;
    int ret;

    - ret = lock_device_hotplug_sysfs();
    - if (ret)
    - return ret;
    + lock_device_hotplug();

    cnt = arch_cpu_probe(buf, count);

    @@ -106,9 +104,7 @@ static ssize_t cpu_release_store(struct device *dev,
    ssize_t cnt;
    int ret;

    - ret = lock_device_hotplug_sysfs();
    - if (ret)
    - return ret;
    + lock_device_hotplug();

    cnt = arch_cpu_release(buf, count);

    --
    1.8.1.4


    \
     
     \ /
      Last update: 2014-05-09 11:21    [W:4.350 / U:0.008 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site