[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CABHrTi+pMjYX4jqaaXS=X02o7CbWBqdMMwcyrxo=BywV18n7WQ@mail.gmail.com>
Date: Wed, 28 Mar 2018 12:18:06 -0700
From: "Joshua R. Poulson" <jrp@....org>
To: Stephen Hemminger <stephen@...workplumber.org>
Cc: "K. Y. Srinivasan" <kys@...hange.microsoft.com>,
KY Srinivasan <kys@...rosoft.com>, gregkh@...uxfoundation.org,
linux-kernel@...r.kernel.org, devel@...uxdriverproject.org,
olaf@...fle.de, Andy Whitcroft <apw@...onical.com>,
Vitaly Kuznetsov <vkuznets@...hat.com>, jasowang@...hat.com,
Leann Ogasawara <leann.ogasawara@...onical.com>,
Marcelo Henrique Cerri <marcelo.cerri@...onical.com>,
Stephen Hemminger <sthemmin@...rosoft.com>,
Michael Kelley <mhkelley@...look.com>,
Stable <stable@...r.kernel.org>
Subject: Re: [PATCH 1/1] Drivers: hv: vmbus: Fix ring buffer signaling
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v4.16-rc7&id=655296c8bbeffcf020558c4455305d597a73bde1
On Wed, Mar 28, 2018 at 12:01 PM, Stephen Hemminger
<stephen@...workplumber.org> wrote:
> On Sun, 4 Mar 2018 22:24:08 -0700
> kys@...hange.microsoft.com wrote:
>
>> From: Michael Kelley <mhkelley@...look.com>
>>
>> Fix bugs in signaling the Hyper-V host when freeing space in the
>> host->guest ring buffer:
>>
>> 1. The interrupt_mask must not be used to determine whether to signal
>> on the host->guest ring buffer
>> 2. The ring buffer write_index must be read (via hv_get_bytes_to_write)
>> *after* pending_send_sz is read in order to avoid a race condition
>> 3. Comparisons with pending_send_sz must treat the "equals" case as
>> not-enough-space
>> 4. Don't signal if the pending_send_sz feature is not present. Older
>> versions of Hyper-V that don't implement this feature will poll.
>>
>> Fixes: 03bad714a161 ("vmbus: more host signalling avoidance")
>>
>> Cc: Stable <stable@...r.kernel.org> # 4.14 and above
>> Signed-off-by: Michael Kelley <mhkelley@...look.com>
>> Signed-off-by: K. Y. Srinivasan <kys@...rosoft.com>
>
> What ever happened to this patch? It doesn't seem to be in char-misc, upstream,
> or stable kernel tree yet.
>
>
Powered by blists - more mailing lists