Re: [Qemu-devel] [PULL for-2.8 0/1] ipxe: update to 20161108 snapshot.

[Laszlo Ersek]

On 11/10/16 16:45, Stefan Hajnoczi wrote:
> On Thu, Nov 10, 2016 at 2:10 PM, Laszlo Ersek <address@hidden> wrote:
>> On 11/10/16 11:48, Stefan Hajnoczi wrote:
>>> On Wed, Nov 09, 2016 at 10:09:24AM +0100, Gerd Hoffmann wrote:
>>>>   Hi,
>>>>
>>>> This rebases ipxe to latest master.  ipxe qemu builds will not use
>>>> fxsave/fxrestore any more, which caused problems on older intel cpus
>>>> due to kvm not emulating these instructions.
>>>>
>>>> Special thanks to Laszlo for pushing these patches to ipxe upstream,
>>>> they finally landed in ipxe master yesterday.
>>>>
>>>> please pull,
>>>>   Gerd
>>>>
>>>> The following changes since commit 
>>>> 207faf24c58859f5240f66bf6decc33b87a1776e:
>>>>
>>>>   Merge remote-tracking branch 'pm215/tags/pull-target-arm-20161107' into 
>>>> staging (2016-11-07 14:02:15 +0000)
>>>>
>>>> are available in the git repository at:
>>>>
>>>>
>>>>   git://git.kraxel.org/qemu tags/pull-ipxe-20161109-1
>>>>
>>>> for you to fetch changes up to 129fa54c734f4dbaf8d3bb9ca47283a2add2e4dc:
>>>>
>>>>   ipxe: update to 20161108 snapshot (2016-11-09 09:49:33 +0100)
>>>>
>>>> ----------------------------------------------------------------
>>>> ipxe: update to 20161108 snapshot.
>>>>
>>>> ----------------------------------------------------------------
>>>> Gerd Hoffmann (1):
>>>>       ipxe: update to 20161108 snapshot
>>>>
>>>>  pc-bios/efi-e1000.rom    | Bin 209408 -> 209920 bytes
>>>>  pc-bios/efi-e1000e.rom   | Bin 209408 -> 209920 bytes
>>>>  pc-bios/efi-eepro100.rom | Bin 209920 -> 209920 bytes
>>>>  pc-bios/efi-ne2k_pci.rom | Bin 208384 -> 208896 bytes
>>>>  pc-bios/efi-pcnet.rom    | Bin 208384 -> 208896 bytes
>>>>  pc-bios/efi-rtl8139.rom  | Bin 211456 -> 212480 bytes
>>>>  pc-bios/efi-virtio.rom   | Bin 211456 -> 212480 bytes
>>>>  pc-bios/efi-vmxnet3.rom  | Bin 205312 -> 206848 bytes
>>>>  roms/ipxe                |   2 +-
>>>>  9 files changed, 1 insertion(+), 1 deletion(-)
>>>>
>>>
>>> Thanks, applied to my staging tree:
>>> https://github.com/stefanha/qemu/commits/staging
>>>
>>> Stefan
>>>
>>
>> Thank you, Gerd and Stefan.
>>
>> Stefan, if your staging tree is eligible for rebasing, I propose to add
>> the following pointers to the commit message (on patch #1):
>>
>> Ref: https://bugs.archlinux.org/task/50778
>> Ref: https://bugs.launchpad.net/qemu/+bug/1623276
>> Ref: https://bugzilla.proxmox.com/show_bug.cgi?id=1182
>> Ref: https://bugzilla.redhat.com/show_bug.cgi?id=1356762
> 
> Sorry, it's too late.  I've already pushed the staging tree to master.
> 
> BTW I will not modify commits from pull requests.  Gerd would have had
> to send a v2.  That way the merge is bit-for-bit the tree from the
> pull request.
> 
> Stefan
> 

No problem -- since you pushed the patch to master, I can now flip the
status on LP#1623276, and name the commit by hash.

Thank you!
Laszlo