Messages in this thread | | | From | KY Srinivasan <> | Subject | RE: [PATCH 1/1] vmbus: hvsock: add proper sync for vmbus_hvsock_device_unregister() | Date | Wed, 11 Oct 2017 13:55:26 +0000 |
| |
Thanks Dan. Will do.
K. Y
> -----Original Message----- > From: Dan Carpenter [mailto:dan.carpenter@oracle.com] > Sent: Wednesday, October 11, 2017 2:42 AM > To: KY Srinivasan <kys@microsoft.com> > Cc: gregkh@linuxfoundation.org; linux-kernel@vger.kernel.org; > devel@linuxdriverproject.org; olaf@aepfle.de; apw@canonical.com; > vkuznets@redhat.com; jasowang@redhat.com; > leann.ogasawara@canonical.com; marcelo.cerri@canonical.com; Stephen > Hemminger <sthemmin@microsoft.com>; Haiyang Zhang > <haiyangz@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, kys@exchange.microsoft.com > wrote: > > From: Dexuan Cui <decui@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 <decui@microsoft.com> > > Cc: Haiyang Zhang <haiyangz@microsoft.com> > > Cc: Stephen Hemminger <sthemmin@microsoft.com> > > Signed-off-by: K. Y. Srinivasan <kys@microsoft.com> > > --- > > drivers/hv/channel_mgmt.c | 5 ++++- > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > regards, > dan carpenter
| |