Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 19 Jun 2020 15:56:14 +0000
From:   Wei Liu <wei.liu@kernel.org>
To:     "Andrea Parri (Microsoft)" <parri.andrea@gmail.com>
Cc:     "K . Y . Srinivasan" <kys@microsoft.com>,
        Haiyang Zhang <haiyangz@microsoft.com>,
        Stephen Hemminger <sthemmin@microsoft.com>,
        Wei Liu <wei.liu@kernel.org>,
        Michael Kelley <mikelley@microsoft.com>,
        linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/8] Drivers: hv: vmbus: Miscellaneous cleanups
Message-ID: <20200619155613.uc3lvcpfz3lb5lzf@liuwe-devbox-debian-v2>
References: <20200617164642.37393-1-parri.andrea@gmail.com>
 <20200619153954.qlrsgtreboqkn7xg@liuwe-devbox-debian-v2>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200619153954.qlrsgtreboqkn7xg@liuwe-devbox-debian-v2>
User-Agent: NeoMutt/20180716
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Fri, Jun 19, 2020 at 03:39:54PM +0000, Wei Liu wrote:
> On Wed, Jun 17, 2020 at 06:46:34PM +0200, Andrea Parri (Microsoft) wrote:
> > Hi all,
> > 
> > I went back to my "cleanup list" recently and I wrote these patches:
> > here you can find, among other things,
> > 
> >   1) the removal of the fields 'target_vp' and 'numa_node' from the
> >      channel data structure, as suggested by Michael back in May;
> > 
> >   2) various cleanups for channel->lock, which is actually *removed
> >      by the end of this series!  ;-)
> > 
> > I'm sure there is room for further "cleanups",  ;-) but let me check
> > if these (relatively small) changes make sense first...
> > 
> > Thanks,
> >   Andrea
> > 
> > Andrea Parri (Microsoft) (8):
> >   Drivers: hv: vmbus: Remove the target_vp field from the vmbus_channel
> >     struct
> >   Drivers: hv: vmbus: Remove the numa_node field from the vmbus_channel
> >     struct
> >   Drivers: hv: vmbus: Replace cpumask_test_cpu(, cpu_online_mask) with
> >     cpu_online()
> >   Drivers: hv: vmbus: Remove unnecessary channel->lock critical sections
> >     (sc_list readers)
> >   Drivers: hv: vmbus: Use channel_mutex in channel_vp_mapping_show()
> >   Drivers: hv: vmbus: Remove unnecessary channel->lock critical sections
> >     (sc_list updaters)
> >   scsi: storvsc: Introduce the per-storvsc_device spinlock
> >   Drivers: hv: vmbus: Remove the lock field from the vmbus_channel
> >     struct
> 
> I've queued these up to hyperv-next except for the scsi patch.
> 

Andrea just pointed out that the last patch can't build without the scsi
patch, so I've only applied only the first 6 patches for now.

Wei.

> Thanks,
> Wei.