linux-kernel - Re: [PATCH 0/8] Drivers: hv: vmbus: Miscellaneous cleanups

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20200619153954.qlrsgtreboqkn7xg@liuwe-devbox-debian-v2>
Date:   Fri, 19 Jun 2020 15:39:54 +0000
From:   Wei Liu <wei.liu@...nel.org>
To:     "Andrea Parri (Microsoft)" <parri.andrea@...il.com>
Cc:     "K . Y . Srinivasan" <kys@...rosoft.com>,
        Haiyang Zhang <haiyangz@...rosoft.com>,
        Stephen Hemminger <sthemmin@...rosoft.com>,
        Wei Liu <wei.liu@...nel.org>,
        Michael Kelley <mikelley@...rosoft.com>,
        linux-hyperv@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 0/8] Drivers: hv: vmbus: Miscellaneous cleanups

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.

Thanks,
Wei.