lkml.org 
[lkml]   [2015]   [Apr]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.4 150/176] Drivers: hv: vmbus: incorrect device name is printed when child device is unregistered
    Date
    From: Fernando Soto <fsoto@bluecatnetworks.com>

    3.4.107-rc1 review patch. If anyone has any objections, please let me know.

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


    commit 84672369ffb98a51d4ddf74c20a23636da3ad615 upstream.

    Whenever a device is unregistered in vmbus_device_unregister (drivers/hv/vmbus_drv.c), the device name in the log message may contain garbage as the memory has already been freed by the time pr_info is called. Log example:
    [ 3149.170475] hv_vmbus: child device àõsèè0_5 unregistered

    By logging the message just before calling device_unregister, the correct device name is printed:
    [ 3145.034652] hv_vmbus: child device vmbus_0_5 unregistered

    Also changing register & unregister messages to debug to avoid unnecessarily cluttering the kernel log.

    Signed-off-by: Fernando M Soto <fsoto@bluecatnetworks.com>
    Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    Cc: Joseph Salisbury <joseph.salisbury@canonical.com>
    Signed-off-by: Zefan Li <lizefan@huawei.com>
    ---
    drivers/hv/vmbus_drv.c | 8 ++++----
    1 file changed, 4 insertions(+), 4 deletions(-)

    diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c
    index 10619b3..1ec309d 100644
    --- a/drivers/hv/vmbus_drv.c
    +++ b/drivers/hv/vmbus_drv.c
    @@ -681,7 +681,7 @@ int vmbus_device_register(struct hv_device *child_device_obj)
    if (ret)
    pr_err("Unable to register child device\n");
    else
    - pr_info("child device %s registered\n",
    + pr_debug("child device %s registered\n",
    dev_name(&child_device_obj->device));

    return ret;
    @@ -693,14 +693,14 @@ int vmbus_device_register(struct hv_device *child_device_obj)
    */
    void vmbus_device_unregister(struct hv_device *device_obj)
    {
    + pr_debug("child device %s unregistered\n",
    + dev_name(&device_obj->device));
    +
    /*
    * Kick off the process of unregistering the device.
    * This will call vmbus_remove() and eventually vmbus_device_release()
    */
    device_unregister(&device_obj->device);
    -
    - pr_info("child device %s unregistered\n",
    - dev_name(&device_obj->device));
    }


    --
    1.9.1


    \
     
     \ /
      Last update: 2015-04-09 11:41    [W:4.119 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site