Re: QEMU 6.2.0 and rhbz#1999878
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.hender...@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.hender...@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~ > >
Re: QEMU 6.2.0 and rhbz#1999878
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.hender...@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~
Re: QEMU 6.2.0 and rhbz#1999878
On Fri, Dec 3, 2021 at 4:37 PM Richard W.M. Jones 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 > > +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 > >
Re: QEMU 6.2.0 and rhbz#1999878
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 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 > +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~
Re: QEMU 6.2.0 and rhbz#1999878
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 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 > > +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
Re: QEMU 6.2.0 and rhbz#1999878
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 > +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
