Thanks Dan. Will do. K. Y > -----Original Message----- > From: Dan Carpenter [mailto:dan.carpenter@xxxxxxxxxx] > Sent: Wednesday, October 11, 2017 2:42 AM > To: KY Srinivasan <kys@xxxxxxxxxxxxx> > Cc: gregkh@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; > devel@xxxxxxxxxxxxxxxxxxxxxx; olaf@xxxxxxxxx; apw@xxxxxxxxxxxxx; > vkuznets@xxxxxxxxxx; jasowang@xxxxxxxxxx; > leann.ogasawara@xxxxxxxxxxxxx; marcelo.cerri@xxxxxxxxxxxxx; Stephen > Hemminger <sthemmin@xxxxxxxxxxxxx>; Haiyang Zhang > <haiyangz@xxxxxxxxxxxxx> > Subject: Re: [PATCH 1/1] vmbus: hvsock: add proper sync for > vmbus_hvsock_device_unregister() > > On Tue, Oct 10, 2017 at 10:38:51PM -0700, kys@xxxxxxxxxxxxxxxxxxxxxx > wrote: > > From: Dexuan Cui <decui@xxxxxxxxxxxxx> > > > > Without the patch, vmbus_hvsock_device_unregister() can destroy the > device > > prematurely when close() is called, and can cause NULl dereferencing or > > potential data loss (the last portion of the data stream may be dropped > > prematurely). > > > > Please consider this for 4.14. > > Put these meta comments under the --- cut off line. They don't add > any value to the final upstream git log. > > > > > Signed-off-by: Dexuan Cui <decui@xxxxxxxxxxxxx> > > Cc: Haiyang Zhang <haiyangz@xxxxxxxxxxxxx> > > Cc: Stephen Hemminger <sthemmin@xxxxxxxxxxxxx> > > Signed-off-by: K. Y. Srinivasan <kys@xxxxxxxxxxxxx> > > --- > > drivers/hv/channel_mgmt.c | 5 ++++- > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > regards, > dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
