linux-kernel - Re: [PATCH 9/9] mmc: Convert from tasklet to BH workqueue

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: <7e618af0-51a7-4941-a386-0ac68c66d358@microchip.com>
Date: Mon, 3 Jun 2024 14:38:22 +0200
From: Aubin Constans <aubin.constans@...rochip.com>
To: Allen Pais <apais@...ux.microsoft.com>, <linux-kernel@...r.kernel.org>
CC: <tj@...nel.org>, <keescook@...omium.org>, <vkoul@...nel.org>,
	<marcan@...can.st>, <sven@...npeter.dev>, <florian.fainelli@...adcom.com>,
	<rjui@...adcom.com>, <sbranden@...adcom.com>, <paul@...pouillou.net>,
	<Eugeniy.Paltsev@...opsys.com>, <manivannan.sadhasivam@...aro.org>,
	<vireshk@...nel.org>, <Frank.Li@....com>, <leoyang.li@....com>,
	<zw@...kernel.org>, <wangzhou1@...ilicon.com>, <haijie1@...wei.com>,
	<shawnguo@...nel.org>, <s.hauer@...gutronix.de>, <sean.wang@...iatek.com>,
	<matthias.bgg@...il.com>, <angelogioacchino.delregno@...labora.com>,
	<afaerber@...e.de>, <logang@...tatee.com>, <daniel@...que.org>,
	<haojian.zhuang@...il.com>, <robert.jarzmik@...e.fr>, <andersson@...nel.org>,
	<konrad.dybcio@...aro.org>, <orsonzhai@...il.com>,
	<baolin.wang@...ux.alibaba.com>, <zhang.lyra@...il.com>,
	<patrice.chotard@...s.st.com>, <linus.walleij@...aro.org>, <wens@...e.org>,
	<jernej.skrabec@...il.com>, <peter.ujfalusi@...il.com>, <kys@...rosoft.com>,
	<haiyangz@...rosoft.com>, <wei.liu@...nel.org>, <decui@...rosoft.com>,
	<jassisinghbrar@...il.com>, <mchehab@...nel.org>,
	<maintainers@...echerrydvr.com>, <ulf.hansson@...aro.org>,
	<manuel.lauss@...il.com>, <mirq-linux@...e.qmqm.pl>,
	<jh80.chung@...sung.com>, <oakad@...oo.com>,
	<hayashi.kunihiko@...ionext.com>, <mhiramat@...nel.org>,
	<brucechang@....com.tw>, <HaraldWelte@...tech.com>, <pierre@...man.eu>,
	<duncan.sands@...e.fr>, <stern@...land.harvard.edu>, <oneukum@...e.com>,
	<openipmi-developer@...ts.sourceforge.net>, <dmaengine@...r.kernel.org>,
	<asahi@...ts.linux.dev>, <linux-arm-kernel@...ts.infradead.org>,
	<linux-rpi-kernel@...ts.infradead.org>, <linux-mips@...r.kernel.org>,
	<imx@...ts.linux.dev>, <linuxppc-dev@...ts.ozlabs.org>,
	<linux-mediatek@...ts.infradead.org>, <linux-actions@...ts.infradead.org>,
	<linux-arm-msm@...r.kernel.org>, <linux-riscv@...ts.infradead.org>,
	<linux-sunxi@...ts.linux.dev>, <linux-tegra@...r.kernel.org>,
	<linux-hyperv@...r.kernel.org>, <linux-rdma@...r.kernel.org>,
	<linux-media@...r.kernel.org>, <linux-mmc@...r.kernel.org>,
	<linux-omap@...r.kernel.org>, <linux-renesas-soc@...r.kernel.org>,
	<linux-s390@...r.kernel.org>, <netdev@...r.kernel.org>,
	<linux-usb@...r.kernel.org>
Subject: Re: [PATCH 9/9] mmc: Convert from tasklet to BH workqueue

On 27/03/2024 17:03, Allen Pais wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
> 
> 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/infiniband/* from tasklet to BH workqueue.
> 
> Based on the work done by Tejun Heo <tj@...nel.org>
> Branch: https://git.kernel.org/pub/scm/linux/kernel/git/tj/wq.git for-6.10
> 
> Signed-off-by: Allen Pais <allen.lkml@...il.com>
> ---
>   drivers/mmc/host/atmel-mci.c                  | 35 ++++-----
[...]

For atmel-mci, judging from a few simple tests, performance is preserved.
E.g. writing to a SD Card on the SAMA5D3-Xplained board:
time dd if=/dev/zero of=/opt/_del_me bs=4k count=64k

      Base 6.9.0 : 0.07user 5.05system 0:18.92elapsed 27%CPU
   Patched 6.9.0+: 0.12user 4.92system 0:18.76elapsed 26%CPU

However, please resolve what checkpatch is complaining about:
scripts/checkpatch.pl --strict 
PATCH-9-9-mmc-Convert-from-tasklet-to-BH-workqueue.mbox

   WARNING: please, no space before tabs
   #72: FILE: drivers/mmc/host/atmel-mci.c:367:
   +^Istruct work_struct ^Iwork;$

Same as discussions on the USB patch[1] and others in this series, I am 
also in favour of "workqueue" or similar in the comments, rather than 
just "work".

Apart from that:
Tested-by: Aubin Constans <aubin.constans@...rochip.com>
Acked-by: Aubin Constans <aubin.constans@...rochip.com>

Thanks.

[1]: 
https://lore.kernel.org/linux-mmc/CAOMdWSLipPfm3OZTpjZz4uF4M+E_8QAoTeMcKBXawLnkTQx6Jg@mail.gmail.com/