Thanks Dan. Will do. K. Y
> -----Original Message----- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Wednesday, October 11, 2017 2:42 AM > To: KY Srinivasan <k...@microsoft.com> > Cc: gre...@linuxfoundation.org; linux-kernel@vger.kernel.org; > de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; > vkuzn...@redhat.com; jasow...@redhat.com; > leann.ogasaw...@canonical.com; marcelo.ce...@canonical.com; Stephen > Hemminger <sthem...@microsoft.com>; Haiyang Zhang > <haiya...@microsoft.com> > 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, k...@exchange.microsoft.com > wrote: > > From: Dexuan Cui <de...@microsoft.com> > > > > 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 <de...@microsoft.com> > > Cc: Haiyang Zhang <haiya...@microsoft.com> > > Cc: Stephen Hemminger <sthem...@microsoft.com> > > Signed-off-by: K. Y. Srinivasan <k...@microsoft.com> > > --- > > drivers/hv/channel_mgmt.c | 5 ++++- > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > regards, > dan carpenter