Messages in this thread | | | From | KY Srinivasan <> | Subject | RE: [PATCH] Drivers: hv: vmbus: incorrect device name is printed when child device is unregistered | Date | Fri, 14 Jun 2013 21:36:07 +0000 |
| |
> -----Original Message----- > From: Greg KH (gregkh@linuxfoundation.org) > [mailto:gregkh@linuxfoundation.org] > Sent: Friday, June 14, 2013 12:23 PM > To: KY Srinivasan > Cc: Fernando Soto; linux-kernel@vger.kernel.org; Haiyang Zhang > Subject: Re: [PATCH] Drivers: hv: vmbus: incorrect device name is printed when > child device is unregistered > > On Fri, Jun 14, 2013 at 02:13:44AM +0000, KY Srinivasan wrote: > > > > > > > -----Original Message----- > > > From: Fernando Soto [mailto:fsoto@bluecatnetworks.com] > > > Sent: Thursday, June 13, 2013 8:11 PM > > > To: linux-kernel@vger.kernel.org > > > Cc: KY Srinivasan; Haiyang Zhang > > > Subject: [PATCH] Drivers: hv: vmbus: incorrect device name is printed when > child > > > device is unregistered > > > > > > From: Fernando Soto <fsoto@bluecatnetworks.com> > > > Please CC me, I am not subscribed to the list. > > > > > > 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 > > > > > > Signed-off-by: Fernando M Soto <fsoto@bluecatnetworks.com> > > Acked-by: K. Y. Srinivasan <kys@microsoft.com> > > > > > --- linux-3.10-rc5/drivers/hv/vmbus_drv.c.orig 2013-06-13 > 19:20:55.359511352 - > > > 0400 > > > +++ linux-3.10-rc5/drivers/hv/vmbus_drv.c 2013-06-13 19:21:39.299511695 - > > > 0400 > > > @@ -698,14 +698,14 @@ int vmbus_device_register(struct hv_devi > > > */ > > > void vmbus_device_unregister(struct hv_device *device_obj) > > > { > > > + pr_info("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)); > > No, please just remove these entirely, or make them pr_debug(), you > shouldn't clutter up the kernel log for stuff like this.
Agreed. Fernando, do you want to submit a patch as suggested by Greg.
K. Y > > thanks, > > greg k-h >
-- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |