qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Alex Williamson <alex.williamson@redhat.com>
Cc: qemu-devel@nongnu.org, stefanha@linux.vnet.ibm.com, mcb30@ipxe.org
Subject: Re: [Qemu-devel] [PATCH v2 0/3] pc-bios: Use iPXE ROMs
Date: Mon, 11 Apr 2011 14:48:31 -0500	[thread overview]
Message-ID: <4DA35B0F.3040904@codemonkey.ws> (raw)
In-Reply-To: <20110411192841.21414.46756.stgit@s20.home>

On 04/11/2011 02:35 PM, Alex Williamson wrote:
> This series replaces our current gPXE based PXE ROMs with iPXE
> versions from the iPXE project (http://ipxe.org).  This version
> adds ipxe to our submodules so it can be easily included in
> releases.  I'm still including a script for updating these,
> perhaps someone better with Makefiles can eventually adopt this
> to a build target.
>
> This email series is mainly for reference, there's too much
> renaming and replacing binary files to send out to the mailing
> list.  I'll strip out the binaries here so the rest can be
> reviewed.  For the real code, please pull:
>
> git://github.com/awilliam/qemu.git (ipxe branch)
>
> Thanks to Anthony for already setting up an ipxe mirror.
> Thanks,

Looks good to me.  How different is this from what we've been shipping?  
Have you tested PXE boot from the builtin TFTP server and from an 
external one (like dnsmasq)?

Regards,

Anthony Liguori

> Alex
>
> ---
>
> Alex Williamson (3):
>        PXE: Refresh all PXE ROMs from the ipxe submodule
>        PXE: Use consistent naming for PXE ROMs
>        Add ipxe submodule
>
>
>   .gitmodules                        |    3 +
>   Makefile                           |   16 +++---
>   hw/e1000.c                         |    2 -
>   hw/eepro100.c                      |    2 -
>   hw/ne2000.c                        |    2 -
>   hw/pcnet-pci.c                     |    2 -
>   hw/rtl8139.c                       |    2 -
>   hw/virtio-pci.c                    |    2 -
>   pc-bios/README                     |   19 +++----
>   pc-bios/gpxe-eepro100-80861209.rom |  Bin
>   pc-bios/pxe-e1000.bin              |  Bin
>   pc-bios/pxe-e1000.rom              |  Bin
>   pc-bios/pxe-eepro100.rom           |  Bin
>   pc-bios/pxe-ne2k_pci.bin           |  Bin
>   pc-bios/pxe-ne2k_pci.rom           |  Bin
>   pc-bios/pxe-pcnet.bin              |  Bin
>   pc-bios/pxe-pcnet.rom              |  Bin
>   pc-bios/pxe-rtl8139.bin            |  Bin
>   pc-bios/pxe-rtl8139.rom            |  Bin
>   pc-bios/pxe-virtio.bin             |  Bin
>   pc-bios/pxe-virtio.rom             |  Bin
>   roms/ipxe                          |    1
>   scripts/refresh-pxe-roms.sh        |   99 ++++++++++++++++++++++++++++++++++++
>   23 files changed, 126 insertions(+), 24 deletions(-)
>   delete mode 100644 pc-bios/gpxe-eepro100-80861209.rom
>   delete mode 100644 pc-bios/pxe-e1000.bin
>   create mode 100644 pc-bios/pxe-e1000.rom
>   create mode 100644 pc-bios/pxe-eepro100.rom
>   delete mode 100644 pc-bios/pxe-ne2k_pci.bin
>   create mode 100644 pc-bios/pxe-ne2k_pci.rom
>   delete mode 100644 pc-bios/pxe-pcnet.bin
>   create mode 100644 pc-bios/pxe-pcnet.rom
>   delete mode 100644 pc-bios/pxe-rtl8139.bin
>   create mode 100644 pc-bios/pxe-rtl8139.rom
>   delete mode 100644 pc-bios/pxe-virtio.bin
>   create mode 100644 pc-bios/pxe-virtio.rom
>   create mode 160000 roms/ipxe
>   create mode 100755 scripts/refresh-pxe-roms.sh

  parent reply	other threads:[~2011-04-12  1:39 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-11 19:35 [Qemu-devel] [PATCH v2 0/3] pc-bios: Use iPXE ROMs Alex Williamson
2011-04-11 19:36 ` [Qemu-devel] [PATCH v2 1/3] Add ipxe submodule Alex Williamson
2011-04-11 19:37 ` [Qemu-devel] [PATCH v2 2/3] PXE: Use consistent naming for PXE ROMs Alex Williamson
2011-04-11 19:38 ` [Qemu-devel] [PATCH v2 3/3] PXE: Refresh all PXE ROMs from the ipxe submodule Alex Williamson
2011-04-11 19:48 ` Anthony Liguori [this message]
2011-04-11 19:57   ` [Qemu-devel] [PATCH v2 0/3] pc-bios: Use iPXE ROMs Alex Williamson
2011-04-11 20:10     ` Alex Williamson
2011-04-12  9:38       ` Stefan Hajnoczi
2011-04-18 17:25     ` Alex Williamson
2011-04-18 17:34       ` Anthony Liguori
2011-04-18 20:43         ` Stefan Weil
2011-04-11 21:13 ` Stefan Weil
2011-04-11 21:26   ` Alex Williamson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4DA35B0F.3040904@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=alex.williamson@redhat.com \
    --cc=mcb30@ipxe.org \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@linux.vnet.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).