Re: [PATCH 1/9] hyperv: Convert from tasklet to BH workqueue

From: Allen <allen.lkml@gmail.com>
To: Allen Pais <apais@linux.microsoft.com>
Cc: linux-kernel@vger.kernel.org, tj@kernel.org,
	keescook@chromium.org,  vkoul@kernel.org, marcan@marcan.st,
	sven@svenpeter.dev,  florian.fainelli@broadcom.com,
	rjui@broadcom.com, sbranden@broadcom.com,  paul@crapouillou.net,
	Eugeniy.Paltsev@synopsys.com,  manivannan.sadhasivam@linaro.org,
	vireshk@kernel.org, Frank.Li@nxp.com,  leoyang.li@nxp.com,
	zw@zh-kernel.org, wangzhou1@hisilicon.com,  haijie1@huawei.com,
	shawnguo@kernel.org, s.hauer@pengutronix.de,
	 sean.wang@mediatek.com, matthias.bgg@gmail.com,
	 angelogioacchino.delregno@collabora.com, afaerber@suse.de,
	 logang@deltatee.com, daniel@zonque.org,
	haojian.zhuang@gmail.com,  robert.jarzmik@free.fr,
	andersson@kernel.org, konrad.dybcio@linaro.org,
	 orsonzhai@gmail.com, baolin.wang@linux.alibaba.com,
	zhang.lyra@gmail.com,  patrice.chotard@foss.st.com,
	linus.walleij@linaro.org, wens@csie.org,
	 jernej.skrabec@gmail.com, peter.ujfalusi@gmail.com,
	kys@microsoft.com,  haiyangz@microsoft.com, wei.liu@kernel.org,
	decui@microsoft.com,  jassisinghbrar@gmail.com,
	mchehab@kernel.org, maintainers@bluecherrydvr.com,
	 aubin.constans@microchip.com, ulf.hansson@linaro.org,
	manuel.lauss@gmail.com,  mirq-linux@rere.qmqm.pl,
	jh80.chung@samsung.com, oakad@yahoo.com,
	 hayashi.kunihiko@socionext.com, mhiramat@kernel.org,
	brucechang@via.com.tw,  HaraldWelte@viatech.com,
	pierre@ossman.eu, duncan.sands@free.fr,
	 stern@rowland.harvard.edu, oneukum@suse.com,
	 openipmi-developer@lists.sourceforge.net,
	dmaengine@vger.kernel.org,  asahi@lists.linux.dev,
	linux-arm-kernel@lists.infradead.org,
	 linux-rpi-kernel@lists.infradead.org,
	linux-mips@vger.kernel.org,  imx@lists.linux.dev,
	linuxppc-dev@lists.ozlabs.org,
	 linux-mediatek@lists.infradead.org,
	linux-actions@lists.infradead.org,
	 linux-arm-msm@vger.kernel.org, linux-riscv@lists.infradead.org,
	 linux-sunxi@lists.linux.dev, linux-tegra@vger.kernel.org,
	 linux-hyperv@vger.kernel.org, linux-rdma@vger.kernel.org,
	 linux-media@vger.kernel.org, linux-mmc@vger.kernel.org,
	 linux-omap@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
	 linux-s390@vger.kernel.org, netdev@vger.kernel.org,
	linux-usb@vger.kernel.org
Subject: Re: [PATCH 1/9] hyperv: Convert from tasklet to BH workqueue
Date: Wed, 3 Apr 2024 09:43:57 -0700	[thread overview]
Message-ID: <CAOMdWSLavg27YZgnfE2QHjO=4RNmFx_7veAURaPG_=qWX=KMVA@mail.gmail.com> (raw)
In-Reply-To: <20240327160314.9982-2-apais@linux.microsoft.com>

> The only generic interface to execute asynchronously in the BH context is
> tasklet; however, it's marked deprecated and has some design flaws. To
> replace tasklets, BH workqueue support was recently added. A BH workqueue
> behaves similarly to regular workqueues except that the queued work items
> are executed in the BH context.
>
> This patch converts drivers/hv/* from tasklet to BH workqueue.
>
> Based on the work done by Tejun Heo <tj@kernel.org>
> Branch: https://git.kernel.org/pub/scm/linux/kernel/git/tj/wq.git for-6.10
>
> Signed-off-by: Allen Pais <allen.lkml@gmail.com>
> ---
>  drivers/hv/channel.c      |  8 ++++----
>  drivers/hv/channel_mgmt.c |  5 ++---
>  drivers/hv/connection.c   |  9 +++++----
>  drivers/hv/hv.c           |  3 +--
>  drivers/hv/hv_balloon.c   |  4 ++--
>  drivers/hv/hv_fcopy.c     |  8 ++++----
>  drivers/hv/hv_kvp.c       |  8 ++++----
>  drivers/hv/hv_snapshot.c  |  8 ++++----
>  drivers/hv/hyperv_vmbus.h |  9 +++++----
>  drivers/hv/vmbus_drv.c    | 19 ++++++++++---------
>  include/linux/hyperv.h    |  2 +-
>  11 files changed, 42 insertions(+), 41 deletions(-)

Wei,

 I need to send out a v2 as I did not include the second patch that
updates drivers/pci/controller/pci-hyperv.c

Thanks.