From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <stable-owner@vger.kernel.org>
Received: from mail-pg0-f53.google.com ([74.125.83.53]:35145 "EHLO
        mail-pg0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1751030AbdCRQDP (ORCPT
        <rfc822;stable@vger.kernel.org>); Sat, 18 Mar 2017 12:03:15 -0400
Received: by mail-pg0-f53.google.com with SMTP id b129so56842197pgc.2
        for <stable@vger.kernel.org>; Sat, 18 Mar 2017 09:03:14 -0700 (PDT)
Date: Sat, 18 Mar 2017 09:03:07 -0700
From: Stephen Hemminger <stephen@networkplumber.org>
To: Dexuan Cui <decui@microsoft.com>
Cc: "gregkh@linuxfoundation.org" <gregkh@linuxfoundation.org>,
        KY Srinivasan <kys@microsoft.com>,
        Haiyang Zhang <haiyangz@microsoft.com>,
        "devel@linuxdriverproject.org" <devel@linuxdriverproject.org>,
        "Stephen Hemminger" <sthemmin@microsoft.com>,
        "stable@vger.kernel.org" <stable@vger.kernel.org>
Subject: Re: [PATCH] vmbus: fix missed ring events on boot
Message-ID: <20170318090307.3397bb5a@xeon-e3>
In-Reply-To: <MWHPR03MB266978D371312DE1777B7469BF380@MWHPR03MB2669.namprd03.prod.outlook.com>
References: <20170318001951.20401-1-sthemmin@microsoft.com>
        <MWHPR03MB266978D371312DE1777B7469BF380@MWHPR03MB2669.namprd03.prod.outlook.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: stable-owner@vger.kernel.org
List-ID: <stable.vger.kernel.org>

On Sat, 18 Mar 2017 07:46:21 +0000
Dexuan Cui <decui@microsoft.com> wrote:

> > From: devel [mailto:driverdev-devel-bounces@linuxdriverproject.org] On
> > Behalf Of Stephen Hemminger
> > @@ -609,7 +609,15 @@ static int vmbus_close_internal(struct
> > @@ -480,20 +492,12 @@ static void vmbus_process_offer(struct  
> 
> Hi Stephen,
> Thanks for the patch!
> 
> hv_event_tasklet_enable() is called in 4 places in 4.10:
> 
> vmbus_close_internal, hv_process_channel_removal and 
> vmbus_process_offer (twice).
> 
> The patch only fixes 2 of the 4 ones.
> Should we fix all of them?
> 
> Thanks,
> -- Dexuan

I am testing a simpler patch that just fixes hv_event_tasklet_enable