<div dir="ltr"><div dir="ltr">Hi Tim,<div>the change set comes indeed form the s390x/KVM developers at IBM and is signed by the upstream maintainer - so it's fully backed by them.</div><div><br></div><div>The tests are - like you assumed - not trivial, hence unit tests were written and run.</div><div><br></div><div>It's always a balance on what to incl. in this submission to the kernel team's mailing-list and what I leave at the Launchpad bug.<br></div><div>The LP bug contains in comment #1 (lower part) some "Notes on backport<s>",that briefly describe the what and what.<br>I think that's what you were roughly looking for: <a href="https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1962831/comments/1">https://bugs.launchpad.net/ubuntu/+source/linux/+bug/1962831/comments/1</a><br>(One may find some more details like stats and a lengthy SRU Justification).</div><div><br></div><div>I understand you that you want more details on the backport as part of the PR submission - will keep that in mind for the next time ...</div><div><span style="color:rgb(51,51,51);font-family:monospace;font-size:12px"><br></span></div><div><span style="color:rgb(51,51,51);font-family:monospace;font-size:12px">Thx, for your feedback and ACK,</span></div><div><span style="color:rgb(51,51,51);font-family:monospace;font-size:12px">Bye, Frank</span></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 4, 2022 at 1:54 PM Tim Gardner <<a href="mailto:tim.gardner@canonical.com">tim.gardner@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">Acked-by: Tim Gardner <<a href="mailto:tim.gardner@canonical.com" target="_blank">tim.gardner@canonical.com</a>><br>
<br>
In theory these patches look OK since you'd have to be an s390 KVM <br>
expert to understand the changes. I'd really like to see simple <br>
explanations for the backports in the future. Just a short explanation <br>
of how conflicts were resolved would be nice.<br>
<br>
There are no test results in the bug report. I just have to assume this <br>
pile works, 'cause I guess its going to be your problem if it doesn't.<br>
<br>
rtg<br>
<br>
On 3/4/22 04:31, <a href="mailto:frank.heimes@canonical.com" target="_blank">frank.heimes@canonical.com</a> wrote:<br>
>   <a href="https://git.launchpad.net/~fheimes/+git/lp1962831/" rel="noreferrer" target="_blank">https://git.launchpad.net/~fheimes/+git/lp1962831/</a>  16c0809cf1012e68279a8936a482c1d63cc4d14c<br>
<br>
-- <br>
-----------<br>
Tim Gardner<br>
Canonical, Inc<br>
</blockquote></div></div>