<div dir="ltr">Thanks Stefan, I will update the SRU with all related commits</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, May 18, 2021 at 2:42 PM Stefan Bader <<a href="mailto:stefan.bader@canonical.com">stefan.bader@canonical.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 18.05.21 04:56, Gerald Yang wrote:<br>
> BugLink: <a href="https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1910976" rel="noreferrer" target="_blank">https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1910976</a><br>
<br>
Above should be <a href="https://bugs.launchpad.net/bugs/1910976" rel="noreferrer" target="_blank">https://bugs.launchpad.net/bugs/1910976</a> but that could be done <br>
when applying. However there seem to be multiple references to this patch <br>
following its introduction upstream (see below). That does not sound like just <br>
picking this one patch is a good idea.<br>
<br>
-Stefan<br>
<br>
> <br>
> === SRU Justification ===<br>
> [Impact]<br>
> To get better performance for dm-crypt in some cases, bypass kcryptd<br>
> workqueue can reduce the overhead in context switch between kworkers,<br>
> cherry-pick commit 39d42fa96ba1b7d2544db3f8ed5da8fb0d5cb877 from mainline<br>
> kernel, and bypass kcryptd workqueue is not enabled by default<br>
> <br>
> [Fix]<br>
> Add flags to bypass kcryptd workqueue<br>
> <br>
> [Test]<br>
> create dm-crypt and setup DM_CRYPT_NO_READ_WORKQUEUE and<br>
> DM_CRYPT_NO_WRITE_WORKQUEUE, read/write data and run perf record to see<br>
> if read/write to the encrypted device will bypass kcryptd workqueue<br>
> <br>
> [Regression Potential]<br>
> Low, this feature is disabled by default, need to enable manually<br>
> <br>
> Ignat Korchagin (1):<br>
>    dm crypt: add flags to optionally bypass kcryptd workqueues<br>
> <br>
>   drivers/md/dm-crypt.c | 50 ++++++++++++++++++++++++++++++++++++-------<br>
>   1 file changed, 42 insertions(+), 8 deletions(-)<br>
> <br>
<br>
 > git log --oneline --grep "Fixes: 39d42fa96ba1"<br>
c87a95dc28b1 dm crypt: defer decryption to a tasklet if interrupts disabled<br>
8e14f610159d dm crypt: do not call bio_endio() from the dm-crypt tasklet<br>
d68b29584c25 dm crypt: use GFP_ATOMIC when allocating crypto requests from softirq<br>
8abec36d1274 dm crypt: do not wait for backlogged crypto request completion in <br>
softirq<br>
4a5caa4af0df dm crypt: document new no_workqueue flags<br>
<br>
</blockquote></div>