<div dir="ltr">Hi Adrien,<div><br></div><div>Thanks for working on the SRU. FWIW, I tested the packages from your PPA, looks good to me and results match previous tests performed.</div><div>Let me know if I can do any further testing or assist in any way to expedite the SRU.<br></div><div><br></div><div>Regards,</div><div>Rafael</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Sep 9, 2023 at 1:50 AM Adrien Nader <<a href="mailto:adrien@notk.org">adrien@notk.org</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">Hi Rafael,<br>
<br>
On Fri, Sep 01, 2023, Rafael Lopez wrote:<br>
> Hi Adrien,<br>
> <br>
> I added some results to LP#2009544 for your consideration, including tests<br>
> on mantic and lunar.<br>
> <br>
> I also tested a couple of good-looking patches in addition to PR#18151 [1].<br>
> Summary below, see LP for detailed results and PPAs with patches.<br>
> <br>
> PR#18151 + PR#17881 [2] - up to ~2x improvement over [1]<br>
> PR#18151 + PR#17921 [3] - up to ~2.5x improvement over [1]<br>
> PR#18151 + PR#17881 [2] + PR#17921 [3] - up to ~4x improvement over [1]<br>
> (based on user CPU time)<br>
> <br>
> PR#17921 [3] might be a good candidate to SRU in addition to [1] at some<br>
> stage. It is a moderate change in Jammy as there are a number of<br>
> pre-requisite patches including refactoring. However, all said<br>
> pre-requisites are merged into 3.0 branch upstream and already included in<br>
> both Lunar and Mantic (you can cleanly cherry pick PR#17921 for those). The<br>
> performance increase is noticeable. [3] itself is not merged to 3.0<br>
> upstream even though the pre-reqs are, see [4] for a comment on this. Seems<br>
> to fit what you described regarding upstream (not) backporting performance<br>
> patches into 3.0. It is merged into 3.1.<br>
> <br>
> PR#17881 [2] also brings noticeable improvement, though less, and has<br>
> nearly all the pre-reqs in 3.0 (and mantic/lunar). Looks like one [5] is<br>
> missing which was not backported to the 3.0 branch, though it is in 3.1.<br>
> <br>
> I haven't analysed the patches in any detail, wanted to see if they were<br>
> worth the effort performance wise first.<br>
> <br>
> Rafael<br>
> <br>
> [1] <a href="https://github.com/openssl/openssl/pull/18151" rel="noreferrer" target="_blank">https://github.com/openssl/openssl/pull/18151</a><br>
> [2] <a href="https://github.com/openssl/openssl/pull/17881" rel="noreferrer" target="_blank">https://github.com/openssl/openssl/pull/17881</a><br>
> [3] <a href="https://github.com/openssl/openssl/pull/17921" rel="noreferrer" target="_blank">https://github.com/openssl/openssl/pull/17921</a><br>
> [4] <a href="https://github.com/openssl/openssl/pull/20421#issuecomment-1457743122" rel="noreferrer" target="_blank">https://github.com/openssl/openssl/pull/20421#issuecomment-1457743122</a><br>
> [5] <a href="https://github.com/openssl/openssl/pull/6127" rel="noreferrer" target="_blank">https://github.com/openssl/openssl/pull/6127</a><br>
<br>
Thanks a lot for digging into this.<br>
<br>
I thought a bit about the best way forward and I came up with three or<br>
four big steps:<br>
<br>
1- SRU [1] to Jammy in addition to bugfixes<br>
2- Include [2] and [3] in the next Ubuntu release (also next LTS)<br>
   Per policy these cannot go straight into an SRU I think. Moreover<br>
   there are a lot of patches already for Jammy which can make resolving<br>
   conflict more difficult and error-prone, and while [3] from<br>
   openssl-3.1 applies almost completely cleanly, [2] doesn't.<br>
3- After OO is released, SRU [2] and [3] to Jammy<br>
4- Bonus: SRU the same version as OO to Jammy (_maybe_ it will make more<br>
   sense to skip step 3 but I can't predict it)<br>
<br>
Unfortunately we have to wait until the OO release for [2] and [3].<br>
<br>
I've created a PPA for step 1 at<br>
<a href="https://launchpad.net/~adrien-n/+archive/ubuntu/openssl-jammy-sru" rel="noreferrer" target="_blank">https://launchpad.net/~adrien-n/+archive/ubuntu/openssl-jammy-sru</a> . It<br>
should be ready besides last-minute debian/changelog fixes (trimming an<br>
entry and removing ~ppa*, and triple-checking the version number).<br>
<br>
I need to trigger as many autopkgtests in the PPA as I can fit, which is<br>
something I intend to do this evening and/or over the week-end but I'm<br>
pretty confident.<br>
<br>
-- <br>
Adrien<br>
</blockquote></div>