* Re: QEMU 6.2.0 and rhbz#1999878 [not found] <CAJzYwARYDA+E4wrszx-F1D_9+VAYB2dU=M-LtvzPJey02gu2qg@mail.gmail.com> @ 2021-12-03 19:37 ` Richard W.M. Jones 2021-12-03 20:35 ` Eduardo Lima 0 siblings, 1 reply; 6+ messages in thread From: Richard W.M. Jones @ 2021-12-03 19:37 UTC (permalink / raw) To: Eduardo Lima Cc: peter.maydell, berrange, sw, richard.henderson, qemu-devel, f4bug On Fri, Dec 03, 2021 at 04:20:23PM -0300, Eduardo Lima wrote: > Hi Rich, > > Can you confirm if the patch you added for qemu in Fedora has still not been > merged upstream? I could not find it on the git source tree. > > +Patch2: 0001-tcg-arm-Reduce-vector-alignment-requirement-for-NEON.patch > +From 1331e4eec016a295949009b4360c592401b089f7 Mon Sep 17 00:00:00 2001 > +From: Richard Henderson <richard.henderson@linaro.org> > +Date: Sun, 12 Sep 2021 10:49:25 -0700 > +Subject: [PATCH] tcg/arm: Reduce vector alignment requirement for NEON https://bugzilla.redhat.com/show_bug.cgi?id=1999878 https://lists.nongnu.org/archive/html/qemu-devel/2021-09/msg01028.html The patch I posted wasn't correct (or meant to be), it was just a workaround. However I think you're right - I don't believe the original problem was ever fixed. Let's see what upstreams says ... Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com virt-p2v converts physical machines to virtual machines. Boot with a live CD or over the network (PXE) and turn machines into KVM guests. http://libguestfs.org/virt-v2v ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: QEMU 6.2.0 and rhbz#1999878 2021-12-03 19:37 ` QEMU 6.2.0 and rhbz#1999878 Richard W.M. Jones @ 2021-12-03 20:35 ` Eduardo Lima 2021-12-03 21:03 ` Richard W.M. Jones 0 siblings, 1 reply; 6+ messages in thread From: Eduardo Lima @ 2021-12-03 20:35 UTC (permalink / raw) To: Richard W.M. Jones Cc: peter.maydell, Daniel Berrange, sw, richard.henderson, qemu-devel, f4bug [-- Attachment #1: Type: text/plain, Size: 1769 bytes --] On Fri, Dec 3, 2021 at 4:37 PM Richard W.M. Jones <rjones@redhat.com> wrote: > On Fri, Dec 03, 2021 at 04:20:23PM -0300, Eduardo Lima wrote: > > Hi Rich, > > > > Can you confirm if the patch you added for qemu in Fedora has still not > been > > merged upstream? I could not find it on the git source tree. > > > > +Patch2: 0001-tcg-arm-Reduce-vector-alignment-requirement-for-NEON.patch > > +From 1331e4eec016a295949009b4360c592401b089f7 Mon Sep 17 00:00:00 2001 > > +From: Richard Henderson <richard.henderson@linaro.org> > > +Date: Sun, 12 Sep 2021 10:49:25 -0700 > > +Subject: [PATCH] tcg/arm: Reduce vector alignment requirement for NEON > > https://bugzilla.redhat.com/show_bug.cgi?id=1999878 > https://lists.nongnu.org/archive/html/qemu-devel/2021-09/msg01028.html > > The patch I posted wasn't correct (or meant to be), it was just a > workaround. However I think you're right - I don't believe the > original problem was ever fixed. > Yes, I saw that your original patch had been replaced by this new one I mentioned, so I thought it was the correct solution, but I could not find this new one on the repository as well. At the moment I kept it as part of 6.2.0 build, which I am just about to push to rawhide. It builds locally, and I am only waiting for the scratch-build to finish. https://koji.fedoraproject.org/koji/taskinfo?taskID=79556515 Thanks, Eduardo. > > Let's see what upstreams says ... > > Rich. > > -- > Richard Jones, Virtualization Group, Red Hat > http://people.redhat.com/~rjones > Read my programming and virtualization blog: http://rwmj.wordpress.com > virt-p2v converts physical machines to virtual machines. Boot with a > live CD or over the network (PXE) and turn machines into KVM guests. > http://libguestfs.org/virt-v2v > > [-- Attachment #2: Type: text/html, Size: 3926 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: QEMU 6.2.0 and rhbz#1999878 2021-12-03 20:35 ` Eduardo Lima @ 2021-12-03 21:03 ` Richard W.M. Jones 2021-12-03 22:00 ` Richard Henderson 0 siblings, 1 reply; 6+ messages in thread From: Richard W.M. Jones @ 2021-12-03 21:03 UTC (permalink / raw) To: Eduardo Lima Cc: peter.maydell, Daniel Berrange, sw, richard.henderson, qemu-devel, f4bug On Fri, Dec 03, 2021 at 05:35:41PM -0300, Eduardo Lima wrote: > > > On Fri, Dec 3, 2021 at 4:37 PM Richard W.M. Jones <rjones@redhat.com> wrote: > > On Fri, Dec 03, 2021 at 04:20:23PM -0300, Eduardo Lima wrote: > > Hi Rich, > > > > Can you confirm if the patch you added for qemu in Fedora has still not > been > > merged upstream? I could not find it on the git source tree. > > > > +Patch2: 0001-tcg-arm-Reduce-vector-alignment-requirement-for-NEON.patch > > +From 1331e4eec016a295949009b4360c592401b089f7 Mon Sep 17 00:00:00 2001 > > +From: Richard Henderson <richard.henderson@linaro.org> > > +Date: Sun, 12 Sep 2021 10:49:25 -0700 > > +Subject: [PATCH] tcg/arm: Reduce vector alignment requirement for NEON > > https://bugzilla.redhat.com/show_bug.cgi?id=1999878 > https://lists.nongnu.org/archive/html/qemu-devel/2021-09/msg01028.html > > The patch I posted wasn't correct (or meant to be), it was just a > workaround. However I think you're right - I don't believe the > original problem was ever fixed. > > Yes, I saw that your original patch had been replaced by this new > one I mentioned, so I thought it was the correct solution, but I > could not find this new one on the repository as well. Oh I see, it was indeed replaced by Richard Henderson's patch: https://src.fedoraproject.org/rpms/qemu/blob/rawhide/f/0001-tcg-arm-Reduce-vector-alignment-requirement-for-NEON.patch > At the moment I kept it as part of 6.2.0 build, which I am just about to push > to rawhide. It builds locally, and I am only waiting for the scratch-build to > finish. Yes looks like we need to keep it, and get it upstream too. Thanks, Rich. > https://koji.fedoraproject.org/koji/taskinfo?taskID=79556515 > > Thanks, Eduardo. > > > > > Let's see what upstreams says ... > > Rich. > > -- > Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/ > ~rjones > Read my programming and virtualization blog: http://rwmj.wordpress.com > virt-p2v converts physical machines to virtual machines. Boot with a > live CD or over the network (PXE) and turn machines into KVM guests. > http://libguestfs.org/virt-v2v > > -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com virt-top is 'top' for virtual machines. Tiny program with many powerful monitoring features, net stats, disk stats, logging, etc. http://people.redhat.com/~rjones/virt-top ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: QEMU 6.2.0 and rhbz#1999878 2021-12-03 21:03 ` Richard W.M. Jones @ 2021-12-03 22:00 ` Richard Henderson 2021-12-04 0:09 ` Richard Henderson 0 siblings, 1 reply; 6+ messages in thread From: Richard Henderson @ 2021-12-03 22:00 UTC (permalink / raw) To: Richard W.M. Jones, Eduardo Lima Cc: peter.maydell, sw, Daniel Berrange, qemu-devel, f4bug On 12/3/21 1:03 PM, Richard W.M. Jones wrote: > On Fri, Dec 03, 2021 at 05:35:41PM -0300, Eduardo Lima wrote: >> >> >> On Fri, Dec 3, 2021 at 4:37 PM Richard W.M. Jones <rjones@redhat.com> wrote: >> >> On Fri, Dec 03, 2021 at 04:20:23PM -0300, Eduardo Lima wrote: >> > Hi Rich, >> > >> > Can you confirm if the patch you added for qemu in Fedora has still not >> been >> > merged upstream? I could not find it on the git source tree. >> > >> > +Patch2: 0001-tcg-arm-Reduce-vector-alignment-requirement-for-NEON.patch >> > +From 1331e4eec016a295949009b4360c592401b089f7 Mon Sep 17 00:00:00 2001 >> > +From: Richard Henderson <richard.henderson@linaro.org> >> > +Date: Sun, 12 Sep 2021 10:49:25 -0700 >> > +Subject: [PATCH] tcg/arm: Reduce vector alignment requirement for NEON >> >> https://bugzilla.redhat.com/show_bug.cgi?id=1999878 >> https://lists.nongnu.org/archive/html/qemu-devel/2021-09/msg01028.html >> >> The patch I posted wasn't correct (or meant to be), it was just a >> workaround. However I think you're right - I don't believe the >> original problem was ever fixed. >> >> Yes, I saw that your original patch had been replaced by this new >> one I mentioned, so I thought it was the correct solution, but I >> could not find this new one on the repository as well. > > Oh I see, it was indeed replaced by Richard Henderson's patch: > > https://src.fedoraproject.org/rpms/qemu/blob/rawhide/f/0001-tcg-arm-Reduce-vector-alignment-requirement-for-NEON.patch > >> At the moment I kept it as part of 6.2.0 build, which I am just about to push >> to rawhide. It builds locally, and I am only waiting for the scratch-build to >> finish. > > Yes looks like we need to keep it, and get it upstream too. Whoops. That dropped through the cracks. I'll queue that now-ish. r~ ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: QEMU 6.2.0 and rhbz#1999878 2021-12-03 22:00 ` Richard Henderson @ 2021-12-04 0:09 ` Richard Henderson 2021-12-09 16:22 ` Eduardo Lima 0 siblings, 1 reply; 6+ messages in thread From: Richard Henderson @ 2021-12-04 0:09 UTC (permalink / raw) To: Richard W.M. Jones, Eduardo Lima Cc: peter.maydell, sw, Daniel Berrange, qemu-devel, f4bug On 12/3/21 2:00 PM, Richard Henderson wrote: >> Oh I see, it was indeed replaced by Richard Henderson's patch: >> >> https://src.fedoraproject.org/rpms/qemu/blob/rawhide/f/0001-tcg-arm-Reduce-vector-alignment-requirement-for-NEON.patch >> >> >>> At the moment I kept it as part of 6.2.0 build, which I am just about to push >>> to rawhide. It builds locally, and I am only waiting for the scratch-build to >>> finish. >> >> Yes looks like we need to keep it, and get it upstream too. > > Whoops. That dropped through the cracks. > I'll queue that now-ish. https://patchew.org/QEMU/20210912174925.200132-1-richard.henderson@linaro.org/ Ah right, I was supposed to test your kernel and never got there. Plus it never got any r-b's. Rebase was smooth and regression testing went ok on cortex-a57 host. r~ ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: QEMU 6.2.0 and rhbz#1999878 2021-12-04 0:09 ` Richard Henderson @ 2021-12-09 16:22 ` Eduardo Lima 0 siblings, 0 replies; 6+ messages in thread From: Eduardo Lima @ 2021-12-09 16:22 UTC (permalink / raw) To: Richard Henderson Cc: Peter Maydell, Daniel Berrange, qemu-devel, sw, f4bug, Richard W.M. Jones [-- Attachment #1: Type: text/plain, Size: 1058 bytes --] Thanks all, I saw the patch has been merged and is part of rc4. I'm removing it from the fedora package. On Fri, Dec 3, 2021 at 9:09 PM Richard Henderson < richard.henderson@linaro.org> wrote: > On 12/3/21 2:00 PM, Richard Henderson wrote: > >> Oh I see, it was indeed replaced by Richard Henderson's patch: > >> > >> > https://src.fedoraproject.org/rpms/qemu/blob/rawhide/f/0001-tcg-arm-Reduce-vector-alignment-requirement-for-NEON.patch > >> > >> > >>> At the moment I kept it as part of 6.2.0 build, which I am just about > to push > >>> to rawhide. It builds locally, and I am only waiting for the > scratch-build to > >>> finish. > >> > >> Yes looks like we need to keep it, and get it upstream too. > > > > Whoops. That dropped through the cracks. > > I'll queue that now-ish. > > > https://patchew.org/QEMU/20210912174925.200132-1-richard.henderson@linaro.org/ > > Ah right, I was supposed to test your kernel and never got there. > Plus it never got any r-b's. > > Rebase was smooth and regression testing went ok on cortex-a57 host. > > > r~ > > [-- Attachment #2: Type: text/html, Size: 1891 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-12-09 16:24 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <CAJzYwARYDA+E4wrszx-F1D_9+VAYB2dU=M-LtvzPJey02gu2qg@mail.gmail.com>
2021-12-03 19:37 ` QEMU 6.2.0 and rhbz#1999878 Richard W.M. Jones
2021-12-03 20:35 ` Eduardo Lima
2021-12-03 21:03 ` Richard W.M. Jones
2021-12-03 22:00 ` Richard Henderson
2021-12-04 0:09 ` Richard Henderson
2021-12-09 16:22 ` Eduardo Lima
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).