commit e144bb7af49ca8756b7222a75811f3b85b0bc1f5 Author: Gerd Hoffmann kraxel@redhat.com Date: Mon Jun 3 16:30:18 2013 +0200
usb: add xhci support $subject says all. Support for usb3 streams is not implemented yet, otherwise it is fully functional. Tested all usb devices supported by qemu (keyboard, storage, usb hubs), except for usb attached scsi in usb3 mode (which needs streams). Tested on qemu only, tagged with QEMU_HARDWARE because of that. Testing with physical hardware to be done.
That commit made seabios size (default qemu config, gcc 4.7+) jump from 128k to 256k in size because the code didn't fit into 128k any more.
Most likely the failure isn't related to xhci at all, but to the size change.
You can try to turn off some features (hardware support) you don't need to make the bios image smaller. 1.7.4 also has a config option to explicitly set the image size you want.
IIRC xen combines seabios and hvmloader into a single 256k image somehow, so it might make sense to set the seabios image size to something between 128k and 256k. But better ask the xen people for details here.
HTH, Gerd
(adding xen-devel too) On Thu, 2014-02-06 at 09:31 +0100, Gerd Hoffmann wrote:
commit e144bb7af49ca8756b7222a75811f3b85b0bc1f5 Author: Gerd Hoffmann kraxel@redhat.com Date: Mon Jun 3 16:30:18 2013 +0200
usb: add xhci support $subject says all. Support for usb3 streams is not implemented yet, otherwise it is fully functional. Tested all usb devices supported by qemu (keyboard, storage, usb hubs), except for usb attached scsi in usb3 mode (which needs streams). Tested on qemu only, tagged with QEMU_HARDWARE because of that. Testing with physical hardware to be done.That commit made seabios size (default qemu config, gcc 4.7+) jump from 128k to 256k in size because the code didn't fit into 128k any more.
Most likely the failure isn't related to xhci at all, but to the size change.
You can try to turn off some features (hardware support) you don't need to make the bios image smaller. 1.7.4 also has a config option to explicitly set the image size you want.
IIRC xen combines seabios and hvmloader into a single 256k image somehow, so it might make sense to set the seabios image size to something between 128k and 256k. But better ask the xen people for details here.
I think this was fixed in Xen with http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=5f2875739beef3a75c7a7e85...
I suppose this should be backported to the 4.3.x stable branch.
Ian.
On 06.02.14 at 09:36, Ian Campbell ijc@hellion.org.uk wrote:
(adding xen-devel too) On Thu, 2014-02-06 at 09:31 +0100, Gerd Hoffmann wrote:
commit e144bb7af49ca8756b7222a75811f3b85b0bc1f5 Author: Gerd Hoffmann kraxel@redhat.com Date: Mon Jun 3 16:30:18 2013 +0200
usb: add xhci support $subject says all. Support for usb3 streams is not implemented yet, otherwise it is fully functional. Tested all usb devices supported by qemu (keyboard, storage, usb hubs), except for usb attached scsi in usb3 mode (which needs streams). Tested on qemu only, tagged with QEMU_HARDWARE because of that. Testing with physical hardware to be done.That commit made seabios size (default qemu config, gcc 4.7+) jump from 128k to 256k in size because the code didn't fit into 128k any more.
Most likely the failure isn't related to xhci at all, but to the size change.
You can try to turn off some features (hardware support) you don't need to make the bios image smaller. 1.7.4 also has a config option to explicitly set the image size you want.
IIRC xen combines seabios and hvmloader into a single 256k image somehow, so it might make sense to set the seabios image size to something between 128k and 256k. But better ask the xen people for details here.
A patch allowing the size to be other than a power of 2 was rejected. And I vaguely seem to recall that you actually participated in that?
I think this was fixed in Xen with http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=5f2875739beef3a75c7a7e85 79b6cbcb464e61b3
I suppose this should be backported to the 4.3.x stable branch.
It was backported already, and is part of 4.3.1.
Jan
On Thu, 2014-02-06 at 10:01 +0000, Jan Beulich wrote:
On 06.02.14 at 09:36, Ian Campbell ijc@hellion.org.uk wrote:
(adding xen-devel too) On Thu, 2014-02-06 at 09:31 +0100, Gerd Hoffmann wrote:
commit e144bb7af49ca8756b7222a75811f3b85b0bc1f5 Author: Gerd Hoffmann kraxel@redhat.com Date: Mon Jun 3 16:30:18 2013 +0200
usb: add xhci support $subject says all. Support for usb3 streams is not implemented yet, otherwise it is fully functional. Tested all usb devices supported by qemu (keyboard, storage, usb hubs), except for usb attached scsi in usb3 mode (which needs streams). Tested on qemu only, tagged with QEMU_HARDWARE because of that. Testing with physical hardware to be done.That commit made seabios size (default qemu config, gcc 4.7+) jump from 128k to 256k in size because the code didn't fit into 128k any more.
Most likely the failure isn't related to xhci at all, but to the size change.
You can try to turn off some features (hardware support) you don't need to make the bios image smaller. 1.7.4 also has a config option to explicitly set the image size you want.
IIRC xen combines seabios and hvmloader into a single 256k image somehow, so it might make sense to set the seabios image size to something between 128k and 256k. But better ask the xen people for details here.
A patch allowing the size to be other than a power of 2 was rejected. And I vaguely seem to recall that you actually participated in that?
I think this was fixed in Xen with http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=5f2875739beef3a75c7a7e85 79b6cbcb464e61b3
I suppose this should be backported to the 4.3.x stable branch.
It was backported already, and is part of 4.3.1.
Oops, gitweb didn't find it.
Debian only has 4.3.0 so that makes sense...
Ian.
Hi,
IIRC xen combines seabios and hvmloader into a single 256k image somehow, so it might make sense to set the seabios image size to something between 128k and 256k. But better ask the xen people for details here.
A patch allowing the size to be other than a power of 2 was rejected. And I vaguely seem to recall that you actually participated in that?
seabios by default still picks the smallest possible power-of-two size. If you need something else you can simply set the new CONFIG_ROM_SIZE option to whatever you want, including non-power-of-two sizes.
cheers, Gerd
Thank you all for your quick, helpful replies! This was about to become a headache for us, so the identification of a fix so quickly is fantastic.
On Thu, 2014-02-06 at 02:31 -0600, Gerd Hoffmann wrote:
That commit made seabios size (default qemu config, gcc 4.7+) jump from 128k to 256k in size because the code didn't fit into 128k any more.
Makes sense. Thanks for the explanation.
On Thu, 2014-02-06 at 02:37 -0600, Ian Campbell wrote:
I think this was fixed in Xen with http://xenbits.xen.org/gitweb/?p=xen.git;a=commit;h=5f2875739beef3a75c7a7e85...
I have confirmed that applying that patch to our Xen sources fixes the problem.
I've filed a bug with Debian's Xen package on this matter, with the hope they'll backport that commit. http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=737905
-
Gerd Hoffmann -
Ian Campbell -
Jan Beulich -
Moyer, Keith