Bug#584074: qemu-kvm: please use etherboot-qemu package for PXE roms

2010-08-01 Thread Michael Tokarev
01.06.2010 07:17, Vagrant Cascadian wrote:
 Package: qemu-kvm
 Version: 0.12.4+dfsg-1~bpo50+1
 Severity: important
 Tags: patch


 please consider using the newly introduced etherboot-qemu package to provide
 PXE capable boot roms, rather than shipping copies of etherboot roms included
 into the qemu-kvm package at build time. this makes it hard to track down the
 sources that the etherboot roms were generated with, amoung other issues.

What is interesting is that the pxe roms provided with
the original upstream sources behaves differently from
the ones in etherboot.  Namely, all etherboot variants
tries to boot twice:

-
 Starting SeaBIOS (version 0.5.1-20100801_125707-gandalf)

 Booting from virtio-net.zrom 5.4.4 (GPL) ether...
 ROM segment 0xc900 length 0x8000 reloc 0x
 Etherboot 5.4.4 (GPL) http://etherboot.org
 Drivers: VIRTIO-NET   Images: NBI ELF PXE   Exports: PXE
 Protocols: DHCP TFTP
 Relocating _text from: [00087780,0009f310) to [07ee8470,07f0)
 Boot from (N)etwork or (Q)uit?

 Probing pci nic...
 Probing isa nic...
 sleep
 Boot from (N)etwork or (Q)uit?

 Probing pci nic...
 [virtio-net]I/O address 0xc020, IRQ #11
 MAC address 52:54:00:12:34:56
 Searching for server (DHCP)
-

This is new etherboot-qemu package, the same happens
with the boot roms copied from older etherboot package.

There's even a bugreport about this, see http://bugs.debian.org/585170

 the following patch that should address the issue, although you might consider
 making etherboot-qemu a recommends rather than a depends.

Well, if etherboot-qemu package isn't installed,
we'll have a bunch of dangling symlinks, which is
not good.  I don't have any problems with the
depends, esp. having in mind the dependency is
quite small.

But I wonder what is the difference between the
original etherboot and this stripped-down etherboot-qemu.
I see two points: etherboot-qemu is a subset of
full etherboot (but it is still a separate package),
and etherboot-qemu does not compress the boot roms.
The latter can be dealt with in the qemu, i think -
by uncompressing them automatically on the fly, it
should be easy to implement.  So.. Is there a good
reason to introduce this stripped-down version?

Thanks!

/mjt



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#584074: qemu-kvm: please use etherboot-qemu package for PXE roms

2010-08-01 Thread Vagrant Cascadian
On Sun, Aug 01, 2010 at 01:19:13PM +0400, Michael Tokarev wrote:
 01.06.2010 07:17, Vagrant Cascadian wrote:
  please consider using the newly introduced etherboot-qemu package to provide
  PXE capable boot roms, rather than shipping copies of etherboot roms 
  included
  into the qemu-kvm package at build time. this makes it hard to track down 
  the
  sources that the etherboot roms were generated with, amoung other issues.
 
 What is interesting is that the pxe roms provided with
 the original upstream sources behaves differently from
 the ones in etherboot.  

i believe upstream ships gPXE roms in recent versions (at least, if qemu-kvm is
following qemu). we will hopefully have uploaded gPXE to NEW before the end of
debcconf.


 But I wonder what is the difference between the
 original etherboot and this stripped-down etherboot-qemu.
 I see two points: etherboot-qemu is a subset of
 full etherboot (but it is still a separate package),
 and etherboot-qemu does not compress the boot roms.

correct. to be clear, the etherboot roms are identical to the corresponding
compressed roms shipped in the full package.

 The latter can be dealt with in the qemu, i think - by uncompressing them
 automatically on the fly, it should be easy to implement.  

nobody has proposed patches to qemu or qemu-kvm to handle that, as far as i
know.

 So.. Is there a good reason to introduce this stripped-down version?

etherboot-qemu is already in the archive, so it has already been introduced.

depending on etherboot-qemu works better than shipping whatever roms happen to
be in the etherboot package at qemu-kvm build time, which is the current state.

it honestly seems a simpler approach to me to have a package that ships a
smaller number of uncompressed roms than patching qemu/qemu-kvm to support
compressed roms and is *trivial* to implement given that etherboot-qemu is
already in the archive.

live well,
  vagrant



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#584074: qemu-kvm: please use etherboot-qemu package for PXE roms

2010-05-31 Thread Vagrant Cascadian
Package: qemu-kvm
Version: 0.12.4+dfsg-1~bpo50+1
Severity: important
Tags: patch

please consider using the newly introduced etherboot-qemu package to provide
PXE capable boot roms, rather than shipping copies of etherboot roms included
into the qemu-kvm package at build time. this makes it hard to track down the
sources that the etherboot roms were generated with, amoung other issues.

should also make the packaging a little simpler, too.

the following patch that should address the issue, although you might consider
making etherboot-qemu a recommends rather than a depends.

commit b76cbe96dd25ec5d225848652e2430db1cfb5c38
Author: Vagrant Cascadian vagr...@freegeek.org
Date:   Mon May 31 19:51:39 2010 -0700

Depend on etherboot-qemu package instead of copying etherboot roms at build
time.

diff --git a/debian/control b/debian/control
index 14ae422..9107543 100644
--- a/debian/control
+++ b/debian/control
@@ -7,7 +7,7 @@ Build-Depends: debhelper (= 7.2.3~), pkg-config, quilt (= 
0.40), autotools-dev
  bzip2, uuid-dev, zlib1g-dev, libsdl1.2-dev, libasound2-dev, 
libcurl4-gnutls-dev,
  libgnutls-dev, libncurses5-dev, libvdeplug2-dev, libpulse-dev, 
libbluetooth-dev,
  nasm, texi2html, bcc, iasl, device-tree-compiler [powerpc], python,
- etherboot [i386 amd64], libbrlapi-dev, libpci-dev, libsasl2-dev,
+ libbrlapi-dev, libpci-dev, libsasl2-dev,
  libaio-dev, linux-libc-dev (= 2.6.32)
 Standards-Version: 3.8.4
 Homepage: http://www.linux-kvm.org/
@@ -17,7 +17,7 @@ Vcs-Git: git://git.debian.org/git/collab-maint/qemu-kvm.git
 Package: qemu-kvm
 Architecture: i386 amd64
 Pre-Depends: adduser
-Depends: ${shlibs:Depends}, ${misc:Depends}, iproute, bridge-utils, python
+Depends: ${shlibs:Depends}, ${misc:Depends}, iproute, bridge-utils, python, 
etherboot-qemu
 Recommends: linux-image
 Suggests: debootstrap, vde2, samba
 Provides: kvm
diff --git a/debian/qemu-kvm.links b/debian/qemu-kvm.links
new file mode 100644
index 000..d95c174
--- /dev/null
+++ b/debian/qemu-kvm.links
@@ -0,0 +1,6 @@
+usr/lib/etherboot/rtl8139.rom usr/share/kvm/pxe-rtl8139.bin
+usr/lib/etherboot/e1000-82540em.rom usr/share/kvm/pxe-e1000.bin
+usr/lib/etherboot/ne.rom usr/share/kvm/pxe-ne2k_isa.bin
+usr/lib/etherboot/virtio-net.rom usr/share/kvm/pxe-virtio.bin
+usr/lib/etherboot/rtl8029.rom usr/share/kvm/pxe-ne2k_pci.bin
+usr/lib/etherboot/pcnet32.rom usr/share/kvm/pxe-pcnet.bin
diff --git a/debian/rules b/debian/rules
index 66bbda0..4016c75 100755
--- a/debian/rules
+++ b/debian/rules
@@ -174,21 +174,6 @@ ifeq (x86,$(BASE_ARCH))
$(tbdir)/vgabios.bin
install -m0644 kvm/vgabios/VGABIOS-lgpl-latest.cirrus.bin \
$(tbdir)/vgabios-cirrus.bin
-
-# pxe roms are x86-specific
-# etherboot_5.4.4-1 moved roms from /usr/share/etherboot to /usr/lib/etherboot
-# We can have versioned build-depends once 5.4.4-1 enters testing.
-   set -e; brl=; \
-   for d in /usr/lib/etherboot /usr/share/etherboot; do \
-   if [ -d $$d ]; then brl=$$d; break; fi; done; \
-   cp -p $$brl/e1000-82540em.zrom.gz $(tbdir)/pxe-e1000.bin.gz; \
-   cp -p $$brl/rtl8029.zrom.gz $(tbdir)/pxe-ne2k_pci.bin.gz; \
-   cp -p $$brl/pcnet32.zrom.gz $(tbdir)/pxe-pcnet.bin.gz; \
-   cp -p $$brl/rtl8139.zrom.gz $(tbdir)/pxe-rtl8139.bin.gz; \
-   if [ -f $$brl/virtio-net.zrom.gz ]; then \
- cp -p $$brl/virtio-net.zrom.gz $(tbdir)/pxe-virtio.bin.gz; \
-   fi
-   gzip -d $(tbdir)/pxe-*.bin.gz
 endif
 
 ifeq (powerpc,$(BASE_ARCH))


live well,
  vagrant



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org