seabios for qemu 1.5

[Gerd Hoffmann]

Hi,
> b44a7be17bdd270ea029a8e2ec0c2e80c6cd0444 virtio-scsi: Pack struct virtio_scsi_{req_cmd,resp_cmd}
> 5a7730db57ab0715223421e65b54fb50d6fefe5c virtio-scsi: Set _DRIVER_OK flag before scsi target scanning
> Anthony + Gerd, can we expect an rev to pc-bios/bios.bin soon to pick up
> these seabios changes for vhost-scsi..?

I don't feel like placing a seabios git snapshot into qemu.

I think we should just cherry-pick everything we need for qemu 1.5 into
the 1.7.2-stable branch, roll out a 1.7.2.2 release and put that into
qemu 1.5.

So, what will be needed?

  (1) The two commits listed above.
  (2) The patch for the pvpanic device (assuming it gets
      merged for 1.5).

Anything else?

Hard freeze for qemu 1.5 is next Wednesday.

cheers,
  Gerd

PS: For qemu 1.6 (which will hopefully include the acpi table
    reorganization) we will need a new release from the master
    branch and thus a bit more careful release planning.

[Qemu-devel] seabios for qemu 1.5

[Amos Kong]

On Wed, Apr 24, 2013 at 07:59:36AM +0200, Gerd Hoffmann wrote:
> Hi,
> > b44a7be17bdd270ea029a8e2ec0c2e80c6cd0444 virtio-scsi: Pack struct virtio_scsi_{req_cmd,resp_cmd}
> > 5a7730db57ab0715223421e65b54fb50d6fefe5c virtio-scsi: Set _DRIVER_OK flag before scsi target scanning
> > Anthony + Gerd, can we expect an rev to pc-bios/bios.bin soon to pick up
> > these seabios changes for vhost-scsi..?
>
> I don't feel like placing a seabios git snapshot into qemu.
>
> I think we should just cherry-pick everything we need for qemu 1.5 into
> the 1.7.2-stable branch, roll out a 1.7.2.2 release and put that into
> qemu 1.5.
>
> So, what will be needed?
>
> (1) The two commits listed above.
>   (2) The patch for the pvpanic device (assuming it gets
>       merged for 1.5).
>
> Anything else?

Hi Gerd,

This bug fix is needed:

| commit 11a7234491cb2a027b0fa5e82af38a3e78b44c80
| Author: Kevin O'Connor <[e-mail]>
| Date:   Tue Mar 5 17:52:21 2013 +0800
| 
|     Cache boot-fail-wait to avoid romfile access after POST.
| 
|     Memory allocated with malloc_tmp() can't be used after the POST phase.
|     So, access boot-fail-wait in post phase and store it for the boot
|     phase to use.  This fixes the regression introduced by commit
|     59d6ca52.
> Hard freeze for qemu 1.5 is next Wednesday.
>
> cheers,
>   Gerd
>
> PS: For qemu 1.6 (which will hopefully include the acpi table
>     reorganization) we will need a new release from the master
>     branch and thus a bit more careful release planning.

[Paolo Bonzini]

Il 24/04/2013 07:59, Gerd Hoffmann ha scritto:
> Hi,
> > b44a7be17bdd270ea029a8e2ec0c2e80c6cd0444 virtio-scsi: Pack struct virtio_scsi_{req_cmd,resp_cmd}
> > 5a7730db57ab0715223421e65b54fb50d6fefe5c virtio-scsi: Set _DRIVER_OK flag before scsi target scanning
> > Anthony + Gerd, can we expect an rev to pc-bios/bios.bin soon to pick up
> > these seabios changes for vhost-scsi..?
>
> I don't feel like placing a seabios git snapshot into qemu.
>
> I think we should just cherry-pick everything we need for qemu 1.5 into
> the 1.7.2-stable branch, roll out a 1.7.2.2 release and put that into
> qemu 1.5.
>
> So, what will be needed?
>
> (1) The two commits listed above.
>   (2) The patch for the pvpanic device (assuming it gets
>       merged for 1.5).
>
> Anything else?
>
> Hard freeze for qemu 1.5 is next Wednesday.

It's just a few patches, so we can do it in -rc1 too.

Paolo
> cheers,
>   Gerd
>
> PS: For qemu 1.6 (which will hopefully include the acpi table
>     reorganization) we will need a new release from the master
>     branch and thus a bit more careful release planning.

[Qemu-devel] seabios for qemu 1.5

[Amos Kong]

On Wed, Apr 24, 2013 at 02:11:26PM +0800, Amos Kong wrote:
> On Wed, Apr 24, 2013 at 07:59:36AM +0200, Gerd Hoffmann wrote:
> > Hi,
> > > b44a7be17bdd270ea029a8e2ec0c2e80c6cd0444 virtio-scsi: Pack struct virtio_scsi_{req_cmd,resp_cmd}
> > > 5a7730db57ab0715223421e65b54fb50d6fefe5c virtio-scsi: Set _DRIVER_OK flag before scsi target scanning
> > > Anthony + Gerd, can we expect an rev to pc-bios/bios.bin soon to pick up
> > > these seabios changes for vhost-scsi..?
> >
> > I don't feel like placing a seabios git snapshot into qemu.
> >
> > I think we should just cherry-pick everything we need for qemu 1.5 into
> > the 1.7.2-stable branch, roll out a 1.7.2.2 release and put that into
> > qemu 1.5.
> >
> > So, what will be needed?
> >
> > (1) The two commits listed above.
> >   (2) The patch for the pvpanic device (assuming it gets
> >       merged for 1.5).
> >
> > Anything else?
>
> Hi Gerd,
>
> This bug fix is needed:
>
> | commit 11a7234491cb2a027b0fa5e82af38a3e78b44c80
> | Author: Kevin O'Connor <[e-mail]>
> | Date:   Tue Mar 5 17:52:21 2013 +0800
> | 
> |     Cache boot-fail-wait to avoid romfile access after POST.
> | 
> |     Memory allocated with malloc_tmp() can't be used after the POST phase.
> |     So, access boot-fail-wait in post phase and store it for the boot
> |     phase to use.  This fixes the regression introduced by commit
> |     59d6ca52.

This patch doesn't contain in seabios-1.7.2.1, it wasn't contained in
latest bios.bin of qemu, it caused 'strict boot' doesn't work.

 # qemu -boot strict=on

What can we do for this? push Kevin add this patch in stable branch
and re-built bios.bin?
> > Hard freeze for qemu 1.5 is next Wednesday.
> >
> > cheers,
> >   Gerd
> >
> > PS: For qemu 1.6 (which will hopefully include the acpi table
> >     reorganization) we will need a new release from the master
> >     branch and thus a bit more careful release planning.

[Qemu-devel] seabios for qemu 1.5

[Gerd Hoffmann]

Hi,
> > | commit 11a7234491cb2a027b0fa5e82af38a3e78b44c80
> > | Author: Kevin O'Connor <[e-mail]>
> > | Date:   Tue Mar 5 17:52:21 2013 +0800
> > | 
> > |     Cache boot-fail-wait to avoid romfile access after POST.
> > | 
> > |     Memory allocated with malloc_tmp() can't be used after the POST phase.
> > |     So, access boot-fail-wait in post phase and store it for the boot
> > |     phase to use.  This fixes the regression introduced by commit
> > |     59d6ca52.
>
> This patch doesn't contain in seabios-1.7.2.1, it wasn't contained in
> latest bios.bin of qemu, it caused 'strict boot' doesn't work.
>
> # qemu -boot strict=on
>
> What can we do for this? push Kevin add this patch in stable branch
> and re-built bios.bin?

It's in the 1.7.2-stable branch meanwhile.  1.7.2.2 isn't tagged yet
though.  It's going slower than usual, seems Kevin is a bit busy with
non-seabios things these days.

Once we have a release released I'll put updates into master and the
stable-1.5 branch.

cheers,
  Gerd