mbox

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

Message ID 1478682565-27496-1-git-send-email-kraxel@redhat.com
State New
Headers show

Pull-request

git://git.kraxel.org/qemu tags/pull-ipxe-20161109-1

Message

Gerd Hoffmann Nov. 9, 2016, 9:09 a.m. UTC
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(-)

Comments

Stefan Hajnoczi Nov. 10, 2016, 10:48 a.m. UTC | #1
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
Stefan Hajnoczi Nov. 10, 2016, 11:23 a.m. UTC | #2
On Wed, Nov 09, 2016 at 10:09:24AM +0100, Gerd Hoffmann wrote:
> 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,
This pull request broke the staging tree because
git.qemu-project.org/ipxe.git didn't have the commit you pointed to yet.

You don't need to do anything now since I have manually kicked the ipxe
mirror.

In the future please only send submodule pull requests after the commits
appear in our mirror repositories.

Thanks,
Stefan
Laszlo Ersek Nov. 10, 2016, 2:10 p.m. UTC | #3
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

If you think that's overkill, then please consider adding just the
Launchpad reference.

Thanks!
Laszlo
Stefan Hajnoczi Nov. 10, 2016, 3:45 p.m. UTC | #4
On Thu, Nov 10, 2016 at 2:10 PM, Laszlo Ersek <lersek@redhat.com> 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
Laszlo Ersek Nov. 10, 2016, 4:30 p.m. UTC | #5
On 11/10/16 16:45, Stefan Hajnoczi wrote:
> On Thu, Nov 10, 2016 at 2:10 PM, Laszlo Ersek <lersek@redhat.com> 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