From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752830AbdAZHQm (ORCPT ); Thu, 26 Jan 2017 02:16:42 -0500 Received: from mail.linuxfoundation.org ([140.211.169.12]:52234 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751787AbdAZHQl (ORCPT ); Thu, 26 Jan 2017 02:16:41 -0500 Date: Thu, 26 Jan 2017 08:16:53 +0100 From: "gregkh@linuxfoundation.org" To: Dexuan Cui Cc: Stephen Hemminger , "driverdev-devel@linuxdriverproject.org" , KY Srinivasan , Haiyang Zhang , Stephen Hemminger , "olaf@aepfle.de" , Rolf Neugebauer , "jasowang@redhat.com" , "linux-kernel@vger.kernel.org" , "apw@canonical.com" Subject: Re: [PATCH] Drivers: hv: vmbus: finally fix hv_need_to_signal_on_read() Message-ID: <20170126071653.GA26979@kroah.com> References: <20170124080818.48c26a6c@xeon-e3> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jan 26, 2017 at 06:10:19AM +0000, Dexuan Cui wrote: > > From: Stephen Hemminger [mailto:stephen@networkplumber.org] > > Sent: Wednesday, January 25, 2017 00:08 > > To: Dexuan Cui > > Cc: gregkh@linuxfoundation.org; driverdev-devel@linuxdriverproject.org; KY > > Srinivasan ; Haiyang Zhang ; > > Stephen Hemminger ; olaf@aepfle.de; Rolf > > Neugebauer ; jasowang@redhat.com; linux- > > kernel@vger.kernel.org; apw@canonical.com > > Subject: Re: [PATCH] Drivers: hv: vmbus: finally fix hv_need_to_signal_on_read() > > > > On Tue, 24 Jan 2017 06:54:46 +0000 > > Dexuan Cui wrote: > > > > > +static inline void > > > +init_cached_read_index(struct vmbus_channel *channel) > > > +{ > > > + struct hv_ring_buffer_info *rbi = &channel->inbound; > > > + > > > + rbi->cached_read_index = rbi->ring_buffer->read_index; > > > +} > > > > Looks good thanks. This should go in right away. Which versions are impacted? > > Should it also go to stable? > > Yes, it needs to go to stable. > I have Cc-ed in the patch's changelog, so it should be > included in the stable tree automatically. > > As I checked against the kernels listed on the homapage of www.kernel.org, the > below versions are impacted: > v3.16.39 > v3.18.47 > v4.1.38 > v4.8.17 > v4.9.5 > v4.10-rc5 > > It's interesting v4.4.44 is not impacted, but actually it needs both the 2 patches: > i.e. this patch, and the previous one: > Commit a389fcfd2cb5 ("Drivers: hv: vmbus: Fix signaling logic in hv_need_to_signal_on_read()") That patch does not apply to the 4.4-stable tree, which is why it was not included there. If you feel it should be included, please provide a backport and send it to the stable@vger.kernel.org mailing list. thanks, greg k-h