Re: [Qemu-devel] [PATCH 1/3] hw/sd/pxa2xx_mmci: convert to SysBusDevice object

2015-12-04 Thread Kevin O'Connor
On Fri, Dec 04, 2015 at 12:55:07PM +, Peter Maydell wrote: > On 4 December 2015 at 12:50, Markus Armbruster wrote: > > To finally answer your question: the proper owner of the property > > connecting the backend is the frontend half of the split device. So, if > > we have separate device mode

Re: [Qemu-devel] [PATCH 1/3] hw/sd/pxa2xx_mmci: convert to SysBusDevice object

2015-12-04 Thread Kevin O'Connor
On Fri, Dec 04, 2015 at 10:50:21AM -0800, Peter Crosthwaite wrote: > > FWIW, I don't think the SD card will be qdevified because it doesn't > > need a bus. It's similar indeed to SerialState, which was supposed to > > be the poster child of QOM embedding and never got QOMified. > > SD is a bus in

Re: [Qemu-devel] [PATCH 1/3] hw/sd/pxa2xx_mmci: convert to SysBusDevice object

2015-12-06 Thread Kevin O'Connor
On Sun, Dec 06, 2015 at 04:02:14PM -0800, Peter Crosthwaite wrote: > On Fri, Dec 4, 2015 at 11:24 AM, Kevin O'Connor wrote: > > On Fri, Dec 04, 2015 at 10:50:21AM -0800, Peter Crosthwaite wrote: > >> > FWIW, I don't think the SD card will be qdevified because it d

Re: [Qemu-devel] [PATCH 4/8] bcm2835_emmc: add bcm2835 MMC/SD controller

2015-12-09 Thread Kevin O'Connor
On Wed, Dec 09, 2015 at 10:54:38AM -0800, Peter Crosthwaite wrote: > On Wed, Dec 9, 2015 at 10:17 AM, Andrew Baumann > wrote: > >> From: Peter Crosthwaite [mailto:crosthwaitepe...@gmail.com] > >> Sent: Tuesday, 8 December 2015 23:40 > >> On Tue, Dec 8, 2015 at 10:19 PM, Andrew Baumann > >> wrote:

Re: [Qemu-devel] [PATCH 05/10] hw/sd/sdhci.c: Update to use SDBus APIs

2015-12-11 Thread Kevin O'Connor
On Fri, Dec 11, 2015 at 04:37:06PM +, Peter Maydell wrote: > Update the SDHCI code to use the new SDBus APIs. > > This commit introduces the new command line options required > to connect a disk to sdhci-pci: > > -device sdhci-pci -drive id=mydrive,[...] -device sd,drive=mydrive I can't rev

Re: [Qemu-devel] about the 'mptable' issue

2016-01-14 Thread Kevin O'Connor
CC'ing the seabios mailing list. On Thu, Jan 14, 2016 at 10:36:07AM +, Li, Liang Z wrote: > Correct something. > The actual parameter for QEMU in our test case is: > 'qemu-systerm-x86_64 -enable-kvm -smp 20 -m 2048 -no-acpi -monitor stdio > -drive file=/mnt/centos6u6.qcow, if=none, id=fo

Re: [Qemu-devel] [V3 3/4] hw/i386: ACPI table for AMD IO MMU

2016-01-14 Thread Kevin O'Connor
On Thu, Jan 14, 2016 at 12:09:46PM +0200, Michael S. Tsirkin wrote: > On Thu, Jan 14, 2016 at 11:04:27AM +0300, David Kiarie wrote: > > Add IVRS table for AMD IO MMU. Also reverve MMIO > > reserve? > > > region for IO MMU via ACPI > > > It does not look like you reserve anything. > > Pls add a

Re: [Qemu-devel] [V3 3/4] hw/i386: ACPI table for AMD IO MMU

2016-01-14 Thread Kevin O'Connor
On Thu, Jan 14, 2016 at 07:29:40PM +0300, David kiarie wrote: > On Thu, Jan 14, 2016 at 7:19 PM, Jan Kiszka wrote: > > On 2016-01-14 17:09, David kiarie wrote: > >> On Thu, Jan 14, 2016 at 6:42 PM, Jan Kiszka wrote: > >>> On 2016-01-14 16:39, Michael S. Tsirkin wrote: > On Thu, Jan 14, 2016

Re: [Qemu-devel] [PATCH v2] Add optionrom compatible with fw_cfg DMA version

2016-01-18 Thread Kevin O'Connor
On Mon, Jan 18, 2016 at 05:22:09PM +0100, Marc Marí wrote: > On Mon, 18 Jan 2016 14:42:06 + > Stefan Hajnoczi wrote: > > Was there a technical reason why linuxboot.S cannot be extended > > (e.g. a size limit)? > > I don't think there's a technical reason. It is a lot simpler to write > the f

Re: [Qemu-devel] [PATCH] SeaBios: Fix reset procedure reentrancy problem on qemu-kvm platform

2015-12-18 Thread Kevin O'Connor
On Fri, Dec 18, 2015 at 03:04:58AM +, Gonglei (Arei) wrote: > Hi Kevin & Paolo, > > Luckily, I reproduced this problem last night. And I got the below log when > SeaBIOS is stuck. [...] > [2015-12-18 10:38:10] gonglei: finish while [...] > <...>-31509 [035] 154753.180077: kvm_exit: reason

Re: [Qemu-devel] [PATCH] SeaBios: Fix reset procedure reentrancy problem on qemu-kvm platform

2015-12-19 Thread Kevin O'Connor
On Sat, Dec 19, 2015 at 12:03:15PM +, Gonglei (Arei) wrote: > Maybe the root cause is not NMI but INTR, so yield() can open hardware > interrupt, > And then execute interrupt handler, but the interrupt handler make the SeaBIOS > stack broken, so that the BSP can't execute the instruction and o

Re: [Qemu-devel] [PATCH] SeaBios: Fix reset procedure reentrancy problem on qemu-kvm platform

2015-12-20 Thread Kevin O'Connor
On Sun, Dec 20, 2015 at 09:49:54AM +, Gonglei (Arei) wrote: > > From: Kevin O'Connor [mailto:ke...@koconnor.net] > > Sent: Saturday, December 19, 2015 11:12 PM > > On Sat, Dec 19, 2015 at 12:03:15PM +, Gonglei (Arei) wrote: > > > Maybe the root cause is

Re: [Qemu-devel] [PATCH] SeaBios: Fix reset procedure reentrancy problem on qemu-kvm platform

2015-12-21 Thread Kevin O'Connor
On Mon, Dec 21, 2015 at 09:41:32AM +, Gonglei (Arei) wrote: > When the gurb of OS is booting, then the softirq and C function send_disk_op() > may use extra stack of SeaBIOS. If we inject a NMI, romlayout.S: > irqentry_extrastack > is invoked, and the extra stack will be used again. And the st

Re: [Qemu-devel] [PATCH] SeaBios: Fix reset procedure reentrancy problem on qemu-kvm platform

2015-12-22 Thread Kevin O'Connor
On Tue, Dec 22, 2015 at 03:15:26AM +, Xulei (Stone) wrote: > Hi, Kevin, > Can you tell how to reset/reboot this VM, if it goes to the handle_hwpic1() > on its booting procedure? I mean, usually, SeaBIOS would not go to > handle_hwpic routine. But in my test case, SeaBIOS calls handle_hwpic whe

Re: [Qemu-devel] [PATCH] SeaBios: Fix reset procedure reentrancy problem on qemu-kvm platform

2015-12-22 Thread Kevin O'Connor
On Tue, Dec 22, 2015 at 02:14:12AM +, Gonglei (Arei) wrote: > > From: Kevin O'Connor [mailto:ke...@koconnor.net] > > Sent: Tuesday, December 22, 2015 2:47 AM > > To: Gonglei (Arei) > > Cc: Xulei (Stone); Paolo Bonzini; qemu-devel; seab...@seabios.org; > > Hua

Re: [Qemu-devel] [PATCH] SeaBios: Fix reset procedure reentrancy problem on qemu-kvm platform

2015-12-23 Thread Kevin O'Connor
On Wed, Dec 23, 2015 at 06:40:12AM +, Gonglei (Arei) wrote: > > From: Kevin O'Connor [mailto:ke...@koconnor.net] > > On Tue, Dec 22, 2015 at 02:14:12AM +, Gonglei (Arei) wrote: > > > Sorry, it doesn't work. What's worse is we cannot

Re: [Qemu-devel] [PATCH RFC] fw-cfg: support writeable blobs

2016-02-22 Thread Kevin O'Connor
On Mon, Feb 22, 2016 at 02:41:38PM +0200, Michael S. Tsirkin wrote: > Useful to send guest data back to QEMU. > The write interface is restricted to DMA. > > Suggested-by: Kevin O'Connor > Signed-off-by: Michael S. Tsirkin > --- > > hw/lm32/lm32_hwsetup.h|

Re: [Qemu-devel] [RFC PATCH v4] fw/pci: Add support for mapping Intel IGD via QEMU

2016-02-23 Thread Kevin O'Connor
On Tue, Feb 16, 2016 at 02:39:27PM -0700, Alex Williamson wrote: > QEMU provides two fw_cfg files to support IGD. The first holds the > OpRegion data which holds the Video BIOS Table (VBT). This needs to > be copied into reserved memory and the address stored in the ASL > Storage register of the

Re: [Qemu-devel] [PATCH v7 10/10] hw/pci-bridge: format special OFW unit address for PXB host

2015-06-24 Thread Kevin O'Connor
fore the comma is dynamically > taken from the main host bridge, similarly to sysbus_get_fw_dev_path(). > > Cc: Kevin O'Connor > Cc: Michael S. Tsirkin > Cc: Marcel Apfelbaum > Signed-off-by: Laszlo Ersek > --- > > Notes: > v7: > - implement the f

Re: [Qemu-devel] [SeaBIOS] [PATCH 03/18] virtio: add struct vp_device

2015-06-29 Thread Kevin O'Connor
On Mon, Jun 29, 2015 at 10:53:25AM +0200, Gerd Hoffmann wrote: > For virtio 1.0 support we will need more state than just the (legacy > mode) ioaddr for each virtio-pci device. Prepare for that by adding > a new struct for it. For now it carries the ioaddr only. > > Signed-off-by: Gerd Hoffmann

Re: [Qemu-devel] [SeaBIOS] [PATCH 07/18] virtio: add version 1.0 read/write macros

2015-06-29 Thread Kevin O'Connor
On Mon, Jun 29, 2015 at 10:53:29AM +0200, Gerd Hoffmann wrote: > Add macros to read/write registers of virtio-1.0 regions. > > Signed-off-by: Gerd Hoffmann > --- > src/hw/virtio-pci.h | 76 > + > 1 file changed, 76 insertions(+) > > diff --gi

Re: [Qemu-devel] [SeaBIOS] [PATCH 05/18] virtio: add version 1.0 structs and #defines

2015-06-29 Thread Kevin O'Connor
On Mon, Jun 29, 2015 at 10:53:27AM +0200, Gerd Hoffmann wrote: > Signed-off-by: Gerd Hoffmann > --- > src/hw/virtio-pci.h | 59 > > src/hw/virtio-ring.h | 5 + > 2 files changed, 64 insertions(+) > > diff --git a/src/hw/virtio-pci.h b/s

Re: [Qemu-devel] [SeaBIOS] [PATCH 10/18] virtio: add version 1.0 support to vp_get_isr

2015-06-29 Thread Kevin O'Connor
On Mon, Jun 29, 2015 at 10:53:32AM +0200, Gerd Hoffmann wrote: > Signed-off-by: Gerd Hoffmann > --- > src/hw/virtio-pci.c | 12 > src/hw/virtio-pci.h | 6 +- > 2 files changed, 13 insertions(+), 5 deletions(-) > > diff --git a/src/hw/virtio-pci.c b/src/hw/virtio-pci.c > index b

Re: [Qemu-devel] [SeaBIOS] [PATCH 07/18] virtio: add version 1.0 read/write macros

2015-06-29 Thread Kevin O'Connor
On Mon, Jun 29, 2015 at 03:46:54PM +0200, Gerd Hoffmann wrote: > On Mo, 2015-06-29 at 09:02 -0400, Kevin O'Connor wrote: > > On Mon, Jun 29, 2015 at 10:53:29AM +0200, Gerd Hoffmann wrote: > > > Add macros to read/write registers of virtio-1.0 regions. > > > &g

Re: [Qemu-devel] [SeaBIOS] [PATCH 05/18] virtio: add version 1.0 structs and #defines

2015-06-29 Thread Kevin O'Connor
On Mon, Jun 29, 2015 at 03:35:59PM +0200, Gerd Hoffmann wrote: > Hi, > > > > +typedef struct virtio_pci_common_cfg { > > > +/* About the whole device. */ > > > +u32 device_feature_select; /* read-write */ > > > > If I understand these structs correctly, they don't reside in memory, >

Re: [Qemu-devel] [SeaBIOS] [PATCH 00/18] virtio: add version 1.0 support

2015-06-29 Thread Kevin O'Connor
On Mon, Jun 29, 2015 at 10:53:22AM +0200, Gerd Hoffmann wrote: > Hi, > > This patch series adds virtio 1.0 support to the virtio blk and scsi > drivers in seabios. With this series applied seabios happily boots > in virtio 1.0 mode from both transitional and modern devices. Thanks. The series

Re: [Qemu-devel] [SeaBIOS] [PATCH 10/18] virtio: add version 1.0 support to vp_get_isr

2015-06-29 Thread Kevin O'Connor
On Mon, Jun 29, 2015 at 04:33:54PM +0200, Gerd Hoffmann wrote: > Hi, > > > > +u8 vp_get_isr(struct vp_device *vp) > > > +{ > > > +u8 isr; > > > + > > > +if (vp->use_modern) { > > > +vp_modern_read(vp->isr, virtio_pci_isr, isr, isr); > > > +} else { > > > +isr = inb(vp

Re: [Qemu-devel] [SeaBIOS] [PATCH v2 03/22] virtio: add struct vp_device

2015-06-30 Thread Kevin O'Connor
On Tue, Jun 30, 2015 at 10:38:54AM +0200, Gerd Hoffmann wrote: > For virtio 1.0 support we will need more state than just the (legacy > mode) ioaddr for each virtio-pci device. Prepare for that by adding > a new struct for it. For now it carries the ioaddr only. > > Signed-off-by: Gerd Hoffmann

Re: [Qemu-devel] [SeaBIOS] [PATCH v2 02/22] virtio: run drivers in 32bit mode

2015-06-30 Thread Kevin O'Connor
On Tue, Jun 30, 2015 at 10:38:53AM +0200, Gerd Hoffmann wrote: > virtio version 1.0 registers can (and actually do in the qemu > implementation) live in mmio space. So we must run the blk and > scsi virtio drivers in 32bit mode, otherwise we can't access them. > > This also allows to drop a bunch

Re: [Qemu-devel] [SeaBIOS] [PATCH v2 02/22] virtio: run drivers in 32bit mode

2015-07-01 Thread Kevin O'Connor
On Wed, Jul 01, 2015 at 03:50:50PM +0200, Michael S. Tsirkin wrote: > On Wed, Jul 01, 2015 at 02:30:29PM +0200, Gerd Hoffmann wrote: > > On Mi, 2015-07-01 at 10:08 +0200, Michael S. Tsirkin wrote: > > > On Tue, Jun 30, 2015 at 10:38:53AM +0200, Gerd Hoffmann wrote: > > > > virtio version 1.0 regist

Re: [Qemu-devel] [SeaBIOS] [PATCH v3 00/25] virtio: add version 1.0 support, move to 32bit

2015-07-01 Thread Kevin O'Connor
On Wed, Jul 01, 2015 at 04:40:45PM +0200, Gerd Hoffmann wrote: > Hi, > > This patch series adds virtio 1.0 support to the virtio blk and scsi > drivers in seabios. With this series applied seabios happily boots > in virtio 1.0 mode from both transitional and modern devices. This > series also

Re: [Qemu-devel] [RFC 10/10] fdc: change default drive to 288

2015-07-05 Thread Kevin O'Connor
On Tue, Jun 30, 2015 at 09:20:40PM -0400, John Snow wrote: > The 2.88 drive is more suitable as a default because > it can still read 1.44 images correctly, but the reverse > is not true. > > Since there exist virtio-win drivers that are shipped on > 2.88 floppy images, this patch will allow VMs b

Re: [Qemu-devel] [PATCH] esp: enable busmaster

2012-11-20 Thread Kevin O'Connor
On Tue, Nov 20, 2012 at 06:33:41PM +0100, Paolo Bonzini wrote: > The HBA does DMA, thus we must enable the busmaster bit, > otherwise dma access will fail with recent qemu versions. Thanks - I applied this. -Kevin

Re: [Qemu-devel] SeaBIOS release for QEMU 1.3 (was Re: [PATCH] esp: enable busmaster)

2012-11-20 Thread Kevin O'Connor
On Tue, Nov 20, 2012 at 06:42:25PM +0100, Paolo Bonzini wrote: > Il 20/11/2012 18:33, Paolo Bonzini ha scritto: > > The HBA does DMA, thus we must enable the busmaster bit, > > otherwise dma access will fail with recent qemu versions. > > > > Signed-off-by: Paolo Bonzini > > As usual we forget t

Re: [Qemu-devel] [SeaBIOS] SeaBIOS release for QEMU 1.3 (was Re: [PATCH] esp: enable busmaster)

2013-01-07 Thread Kevin O'Connor
On Mon, Jan 07, 2013 at 03:29:33PM +0100, Gerd Hoffmann wrote: > Hi, > > >> Kevin, are you okay with tagging SeaBIOS master as 1.7.2 one of these > >> days? Other opinions? > > > > A new release can be produced if desired. Given that there isn't much > > new since 1.7.1 I too would prefer to

Re: [Qemu-devel] [SeaBIOS] SeaBIOS release for QEMU 1.3 (was Re: [PATCH] esp: enable busmaster)

2013-01-07 Thread Kevin O'Connor
On Mon, Jan 07, 2013 at 06:51:03PM -0500, Brad Smith wrote: > On Mon, Jan 07, 2013 at 06:34:01PM -0500, Kevin O'Connor wrote: > > On Mon, Jan 07, 2013 at 03:29:33PM +0100, Gerd Hoffmann wrote: > > > Hi, > > > > > > >> Kevin, are you okay wi

Re: [Qemu-devel] [SeaBIOS PATCH v2] boot: support strict boot and make it configurable

2013-01-08 Thread Kevin O'Connor
On Wed, Dec 26, 2012 at 08:55:40AM +0800, Amos Kong wrote: > Current seabios will try to boot from selected devices first, > if they are all failed, seabios will also try to boot from > un-selected devices. > > For example: > @ qemu-kvm -boot order=n,menu=on ... > > Guest will boot from network f

Re: [Qemu-devel] [SeaBIOS PATCH v2] boot: support strict boot and make it configurable

2013-01-12 Thread Kevin O'Connor
On Wed, Jan 09, 2013 at 01:03:37PM +0800, Amos Kong wrote: > On Tue, Jan 08, 2013 at 11:00:42PM -0500, Kevin O'Connor wrote: > > On Wed, Dec 26, 2012 at 08:55:40AM +0800, Amos Kong wrote: > > > Current seabios will try to boot from selected devices first, > > > if t

[Qemu-devel] [ANNOUNCE] SeaBIOS 1.7.2

2013-01-18 Thread Kevin O'Connor
The 1.7.2 version of SeaBIOS has now been released. For more information on the release, please see: http://seabios.org/Releases New in this release: * Support for ICH9 host chipset ("q35") on emulators * Support for booting from LSI MegaRAID SAS controllers * Support for using the ACPI PM tim

Re: [Qemu-devel] [RESEND PATCH v2] pciinit: Enable default VGA device

2013-03-19 Thread Kevin O'Connor
On Mon, Mar 18, 2013 at 09:07:21PM -0600, Alex Williamson wrote: > As QEMU gains PCI bridge and PCIe root port support, we won't always > find the VGA device on the root bus. We therefore need to add support > to find and enable a VGA device and the path to it through the VGA > Enable support in t

Re: [Qemu-devel] [SeaBIOS] [PATCH v15 2/2] patch dsdt to use passed-in pvpanic ioport

2013-03-21 Thread Kevin O'Connor
On Thu, Mar 21, 2013 at 05:08:34PM +0800, Hu Tao wrote: > Signed-off-by: Hu Tao I don't think it is a good idea to dynamically modify the DSDT. We've been using the SSDT for that. In any case, I think this would be a good candidate for merging after the ACPI stuff is moved into QEMU. -Kevin

Re: [Qemu-devel] qemu / seabios ACPI table interface

2013-03-22 Thread Kevin O'Connor
On Fri, Mar 22, 2013 at 05:09:53PM +0100, Laszlo Ersek wrote: > I'm confused. What are the requirements? Here's my suggested implementation: - Have qemu create the ACPI tables in new fw_cfg "file" entries; one "file" per table. Have QEMU put ACPI tables grouped in /etc/acpi/ - for example /e

Re: [Qemu-devel] [PATCH v3] pciinit: Enable default VGA device

2013-03-22 Thread Kevin O'Connor
On Wed, Mar 20, 2013 at 10:58:47AM -0600, Alex Williamson wrote: > As QEMU gains PCI bridge and PCIe root port support, we won't always > find the VGA device on the root bus. We therefore need to add support > to find and enable a VGA device and the path to it through the VGA > Enable support in t

Re: [Qemu-devel] [SeaBIOS] [PATCH v16] Add pvpanic device driver

2013-03-29 Thread Kevin O'Connor
On Fri, Mar 29, 2013 at 04:18:44PM +0800, Hu Tao wrote: > pvpanic device is used to notify host(qemu) when guest panic happens. Thanks. However, we're planning a move of ACPI tables from SeaBIOS to QEMU. I think this should wait until after the move. -Kevin

Re: [Qemu-devel] [SeaBIOS] [PATCH v16] Add pvpanic device driver

2013-03-30 Thread Kevin O'Connor
On Fri, Mar 29, 2013 at 02:49:12PM +0100, Paolo Bonzini wrote: > Il 29/03/2013 14:33, Kevin O'Connor ha scritto: > > On Fri, Mar 29, 2013 at 04:18:44PM +0800, Hu Tao wrote: > >> pvpanic device is used to notify host(qemu) when guest panic happens. > > > > Than

Re: [Qemu-devel] [PATCH 2/2] Provide ACPI SSDT table for TPM device

2013-04-01 Thread Kevin O'Connor
On Mon, Apr 01, 2013 at 03:05:55PM -0400, Corey Bryant wrote: > On 03/28/2013 05:03 AM, Paolo Bonzini wrote: > >There is work on moving ACPI tables to QEMU. Please work with the other > >developers (Kevin of course, and Michael and Laszlo who I have CCed) on > >this. > > Kevin, Do you have a pref

Re: [Qemu-devel] [SeaBIOS] [PATCH v16] Add pvpanic device driver

2013-04-01 Thread Kevin O'Connor
On Sun, Mar 31, 2013 at 05:34:10PM +0300, Gleb Natapov wrote: > On Sat, Mar 30, 2013 at 09:20:09AM -0400, Kevin O'Connor wrote: > > On Fri, Mar 29, 2013 at 02:49:12PM +0100, Paolo Bonzini wrote: > > > Il 29/03/2013 14:33, Kevin O'Connor ha scritto: > > > >

Re: [Qemu-devel] [PATCH 00/11] build ACPI MADT for fw_cfg clients

2013-04-04 Thread Kevin O'Connor
On Thu, Apr 04, 2013 at 09:52:31AM +0200, Laszlo Ersek wrote: > On 04/03/13 22:05, Anthony Liguori wrote: > > Laszlo Ersek writes: > >> Any chance patches 01 to 09 could be considered? Esp. 06 which removes > >> an out-of-bounds access (an innocent-looking one, admittedly). > >> > >> I'm OK too if

[Qemu-devel] SeaBIOS v1.7.2.2 stable release

2013-05-27 Thread Kevin O'Connor
: Set _DRIVER_OK flag before scsi target scanning Kevin O'Connor (1): Cache boot-fail-wait to avoid romfile access after POST. boot.c| 15 +-- virtio-scsi.c |5 +++-- virtio-scsi.h |4 ++-- 3 files changed, 14 insertions(+), 10 deletions(-)

Re: [Qemu-devel] [SeaBIOS PATCH] boot: fix fw_dev_path pattern for q35-pcihost

2013-05-28 Thread Kevin O'Connor
On Tue, May 28, 2013 at 08:28:14PM +0800, Amos Kong wrote: > Bootindex string passed from qemu: > /q35-pcihost@i0cf8/ethernet@2/ethernet-phy@0 > > We match pci domain by "/pci@i0cf8" in SeaBIOS, but fw_dev_path prefix > of q35 is "/q35-pcihost@i0cf8". So bootindex in qemu commandline > doesn't wo

Re: [Qemu-devel] KVM call agenda for 2013-05-28

2013-05-28 Thread Kevin O'Connor
On Thu, May 23, 2013 at 03:41:32PM +0300, Michael S. Tsirkin wrote: > Juan is not available now, and Anthony asked for > agenda to be sent early. > So here comes: > > Agenda for the meeting Tue, May 28: > > - Generating acpi tables I didn't see any meeting notes, but I thought it would be worthw

Re: [Qemu-devel] [SeaBIOS] SeaBIOS v1.7.2.2 stable release

2013-05-28 Thread Kevin O'Connor
On Mon, May 27, 2013 at 05:11:46PM +0400, Michael Tokarev wrote: > 27.05.2013 16:59, Kevin O'Connor wrote: > > A new stable release of SeaBIOS (version 1.7.2.2) has been tagged. > > This release contains bug fixes. > > > > The release is available via git: >

Re: [Qemu-devel] [SeaBIOS] KVM call agenda for 2013-05-28

2013-05-29 Thread Kevin O'Connor
On Wed, May 29, 2013 at 11:18:03AM -0500, Anthony Liguori wrote: > Gerd Hoffmann writes: > > On 05/29/13 01:53, Kevin O'Connor wrote: > >> Raised > >> that QOM interface should be sufficient. > > > > Agree on this one. Ideally the acpi table gene

Re: [Qemu-devel] [SeaBIOS PATCH] boot: fix fw_dev_path pattern for q35-pcihost

2013-05-29 Thread Kevin O'Connor
On Wed, May 29, 2013 at 10:33:54AM +0800, Amos Kong wrote: > On Tue, May 28, 2013 at 06:59:02PM -0400, Kevin O'Connor wrote: > > On Tue, May 28, 2013 at 08:28:14PM +0800, Amos Kong wrote: > > > -#define FW_PCI_DOMAIN "/pci@i0cf8" > > > +#define FW_PCI_

Re: [Qemu-devel] [coreboot] [SeaBIOS] SeaBIOS v1.7.2.2 stable release

2013-05-29 Thread Kevin O'Connor
On Wed, May 29, 2013 at 09:27:16AM +0200, Gerd Hoffmann wrote: > While talking about releases: There are quite some changes accumulated > in master, time to cut a new release I think. Given that sorting the > acpi table issue seems to take more time than expected, how about freeze > + call for pa

Re: [Qemu-devel] [PATCH v7 1/4] isapc: Fix non-KVM qemu boot (read/write memory for isapc BIOS)

2013-05-30 Thread Kevin O'Connor
On Wed, May 29, 2013 at 01:27:24AM -0700, Jordan Justen wrote: > The isapc machine with seabios currently requires the BIOS region > to be read/write memory rather than read-only memory. > > KVM currently cannot support the BIOS as a ROM region, but qemu > in non-KVM mode can. Based on this, isapc

Re: [Qemu-devel] KVM call agenda for 2013-05-28

2013-05-30 Thread Kevin O'Connor
On Tue, May 28, 2013 at 07:53:09PM -0400, Kevin O'Connor wrote: > There were discussions on potentially introducing a middle component > to generate the tables. Coreboot was raised as a possibility, and > David thought it would be okay to use coreboot for both OVMF and > SeaBIOS.

Re: [Qemu-devel] KVM call agenda for 2013-05-28

2013-05-31 Thread Kevin O'Connor
On Fri, May 31, 2013 at 07:58:36AM -0500, Anthony Liguori wrote: > Kevin O'Connor writes: > > Given the objections to implementing ACPI directly in QEMU, one > > possible way forward would be to split the current SeaBIOS rom into > > two roms: "qvmloader" and

Re: [Qemu-devel] [SeaBIOS] KVM call agenda for 2013-05-28

2013-05-31 Thread Kevin O'Connor
On Fri, May 31, 2013 at 10:13:34AM +0200, Peter Stuge wrote: > Kevin O'Connor wrote: > > one possible way forward would be to split the current SeaBIOS rom > > into two roms: "qvmloader" and "seabios". The "qvmloader" would do > > the qe

Re: [Qemu-devel] [PATCH v7 1/4] isapc: Fix non-KVM qemu boot (read/write memory for isapc BIOS)

2013-05-31 Thread Kevin O'Connor
On Fri, May 31, 2013 at 02:48:17PM +0200, Paolo Bonzini wrote: > Il 31/05/2013 04:06, Kevin O'Connor ha scritto: > > On Wed, May 29, 2013 at 01:27:24AM -0700, Jordan Justen wrote: > >> > The isapc machine with seabios currently requires the BIOS region > >> &g

Re: [Qemu-devel] [SeaBIOS] What's the impact of enlarging IDE_TIMEOUT ?

2013-06-01 Thread Kevin O'Connor
On Fri, May 31, 2013 at 03:18:56AM +, Gonglei (Arei) wrote: >IDE_TIMEOUT is defined 32s. But we encountered its timeout in >some cases, and then loading disk failed in VM. In order to >reduce the probability of timeout, we want to enlarge the >IDE_TIMEOUT, such as 120s. We ver

Re: [Qemu-devel] [PATCH] qemu: piix: PCI bridge ACPI hotplug support

2013-06-10 Thread Kevin O'Connor
On Mon, Jun 10, 2013 at 04:45:53PM -0500, Anthony Liguori wrote: > This discussion comes down to two things I think: (a) our existing > firmware interface is pretty poor (b) we are duplicating work because of > firmware licensing. > > We can fix (a) and there's lots of value in doing that in terms

Re: [Qemu-devel] [PATCH] qemu: piix: PCI bridge ACPI hotplug support

2013-06-10 Thread Kevin O'Connor
On Mon, Jun 10, 2013 at 06:34:29PM -0500, Anthony Liguori wrote: > Kevin O'Connor writes: > > > On Mon, Jun 10, 2013 at 04:45:53PM -0500, Anthony Liguori wrote: > >> This discussion comes down to two things I think: (a) our existing > >> firmware interface is

Re: [Qemu-devel] [PATCH] qemu: piix: PCI bridge ACPI hotplug support

2013-06-10 Thread Kevin O'Connor
On Mon, Jun 10, 2013 at 07:51:55PM -0500, Anthony Liguori wrote: > I think that we can pretty much touch a table once pulling all of the > info from QOM and then from a SeaBIOS point of view, never have to > touch it again. Thanks. I do think it would help if you could go through the details of a

Re: [Qemu-devel] [PATCH] qemu: piix: PCI bridge ACPI hotplug support

2013-06-10 Thread Kevin O'Connor
On Mon, Jun 10, 2013 at 08:25:15PM -0500, Anthony Liguori wrote: > On Mon, Jun 10, 2013 at 8:19 PM, Kevin O'Connor wrote: > > I do understand your desire to pass this stuff as parameters, but I > > really don't see it as feasible. I'm hoping that if you can w

Re: [Qemu-devel] [SeaBIOS] solaris x86 in qemu? [bisected]

2013-06-13 Thread Kevin O'Connor
> seabios: update to 1.7.0 > > Update roms/seabios and pc-bios/bios.bin to the 1.7.0 release. > Most noticable new feature is virtio-scsi support. > > Signed-off-by: Gerd Hoffmann > > > So I went on and tried to bisect seabios (previous version in qemu

Re: [Qemu-devel] [SeaBIOS] Next release

2013-07-07 Thread Kevin O'Connor
On Sun, Jul 07, 2013 at 06:01:08PM +0300, Michael S. Tsirkin wrote: > On Mon, Jul 01, 2013 at 05:20:46PM -0400, Kevin O'Connor wrote: > > As previously discussed, I'll be generating the next seabios release > > in the next couple of days. If there are any outstanding de

Re: [Qemu-devel] [SeaBIOS] Next release

2013-07-07 Thread Kevin O'Connor
On Sun, Jul 07, 2013 at 06:50:44PM +0300, Michael S. Tsirkin wrote: > On Sun, Jul 07, 2013 at 11:14:03AM -0400, Kevin O'Connor wrote: > > On Sun, Jul 07, 2013 at 06:01:08PM +0300, Michael S. Tsirkin wrote: > > > On Mon, Jul 01, 2013 at 05:20:46PM -0400, Kevin O'Connor

Re: [Qemu-devel] [PATCH v3 05/16] pc: Make -no-fd-bootchk stick across boot order changes

2013-07-07 Thread Kevin O'Connor
On Tue, Jun 18, 2013 at 01:39:25PM +0200, Markus Armbruster wrote: > Anthony Liguori writes: > > Markus Armbruster writes: > > > >> Option -no-fd-bootchk asks the BIOS to attempt booting from a floppy > >> even when the boot sector signature isn't there, by setting a bit in > >> RTC CMOS. It was

[Qemu-devel] [ANNOUNCE] SeaBIOS 1.7.3

2013-07-07 Thread Kevin O'Connor
The 1.7.3 version of SeaBIOS has now been released. For more information on the release, please see: http://seabios.org/Releases New in this release: * Initial support for using SeaBIOS as a UEFI Compatibility Support Module (CSM) * Support for detecting and using ACPI reboot ports. * By def

Re: [Qemu-devel] seabios 1.7.3 breaks booting windows 7? (was: [ANNOUNCE] SeaBIOS 1.7.3)

2013-07-08 Thread Kevin O'Connor
On Mon, Jul 08, 2013 at 05:23:40PM -0400, Cole Robinson wrote: > On 07/07/2013 11:41 PM, Kevin O'Connor wrote: > > The 1.7.3 version of SeaBIOS has now been released. For more > > information on the release, please see: > > Just gave it a spin, breaks booting a Windo

Re: [Qemu-devel] seabios 1.7.3 breaks booting windows 7?

2013-07-08 Thread Kevin O'Connor
On Mon, Jul 08, 2013 at 08:18:42PM -0400, Cole Robinson wrote: > On 07/08/2013 07:18 PM, Kevin O'Connor wrote: > > On Mon, Jul 08, 2013 at 05:23:40PM -0400, Cole Robinson wrote: > >> On 07/07/2013 11:41 PM, Kevin O'Connor wrote: > >>> The 1.7.3 version of

Re: [Qemu-devel] [SeaBIOS] [PATCH v2 1/5] linker: utility to patch in-memory ROM files

2013-07-14 Thread Kevin O'Connor
On Sun, Jul 07, 2013 at 06:42:35PM +0300, Michael S. Tsirkin wrote: > Add ability for a ROM file to point to > it's image in memory. When file is in memory, > add utility that can patch it, storing > pointers to one file within another file. Thanks. See my comments below. [...] > --- /dev/null >

Re: [Qemu-devel] [SeaBIOS] [PATCH v2 4/5] acpi: load and link tables from /etc/acpi/

2013-07-14 Thread Kevin O'Connor
On Sun, Jul 07, 2013 at 06:42:43PM +0300, Michael S. Tsirkin wrote: > Load files in /etc/acpi/, link them using > a linker script and use for acpi tables, including the RSDP. > Presense of RSDP in this directory completely disables > generating and loading legacy acpi tables. > > Signed-off-by: Mi

Re: [Qemu-devel] [SeaBIOS] [PATCH 11/12] Migrate 64bit entries to 64bit pci regions

2012-04-26 Thread Kevin O'Connor
On Thu, Apr 26, 2012 at 07:12:22PM +1200, Alexey Korolev wrote: > On 26/04/12 03:29, Gerd Hoffmann wrote: > >> Okay - I missed that. I think the patches look okay to be committed - > >> any additional changes can be made on top. Gerd - do you have any > >> comments? > > These are certainly no blo

Re: [Qemu-devel] [SeaBIOS] [seabios patch 1/5] pci: init all devices

2012-05-04 Thread Kevin O'Connor
On Fri, May 04, 2012 at 10:21:23AM +0200, Gerd Hoffmann wrote: > seabios used to initialize root bus devices only, with this patch > devices behind pci bridges are initialized too. This allows to boot > from virtio devices behind pci bridges. > > Signed-off-by: Gerd Hoffmann > --- > src/pciinit

Re: [Qemu-devel] [SeaBIOS] [PATCH 11/12] Migrate 64bit entries to 64bit pci regions

2012-05-04 Thread Kevin O'Connor
On Wed, May 02, 2012 at 03:42:51PM +0200, Gerd Hoffmann wrote: > Hi, > > > Is this just a matter of removing the "if (pci_bdf_to_bus(pci->bdf) != > > 0) break" from pci_bios_init_devices()? > > Seems to do the trick, at least the disks connected appear in the boot > menu now and the seabios log

Re: [Qemu-devel] [SeaBIOS] [seabios patch 0/5] dynamic pci i/o windows

2012-05-04 Thread Kevin O'Connor
On Fri, May 04, 2012 at 10:21:22AM +0200, Gerd Hoffmann wrote: > Hi, > > This patch series makes the PCI I/O windows runtime-configurable via > qemu firmware config interface. Main advantage is that we can size and > shuffle around the PCI i/O windows according to the amount of memory the > vir

Re: [Qemu-devel] [SeaBIOS] [seabios patch 0/5] dynamic pci i/o windows

2012-05-04 Thread Kevin O'Connor
On Fri, May 04, 2012 at 04:01:56PM +0200, Gerd Hoffmann wrote: > On 05/04/12 15:18, Kevin O'Connor wrote: > > On Fri, May 04, 2012 at 10:21:22AM +0200, Gerd Hoffmann wrote: > >> Hi, > >> > >> This patch series makes the PCI I/O windows runtime-configurabl

Re: [Qemu-devel] [SeaBIOS] [seabios patch 0/5] dynamic pci i/o windows

2012-05-04 Thread Kevin O'Connor
On Fri, May 04, 2012 at 10:46:00AM -0400, Kevin O'Connor wrote: > On Fri, May 04, 2012 at 04:01:56PM +0200, Gerd Hoffmann wrote: > > On 05/04/12 15:18, Kevin O'Connor wrote: > > > On Fri, May 04, 2012 at 10:21:22AM +0200, Gerd Hoffmann wrote: > > >> Hi, &

Re: [Qemu-devel] [SeaBIOS] [seabios patch 1/5] pci: init all devices

2012-05-04 Thread Kevin O'Connor
On Fri, May 04, 2012 at 05:35:10PM +0200, Gerd Hoffmann wrote: > On 05/04/12 15:15, Kevin O'Connor wrote: > > On Fri, May 04, 2012 at 10:21:23AM +0200, Gerd Hoffmann wrote: > >> seabios used to initialize root bus devices only, with this patch > >> devices behind

Re: [Qemu-devel] [SeaBIOS] [seabios patch 0/5] dynamic pci i/o windows

2012-05-06 Thread Kevin O'Connor
On Mon, May 07, 2012 at 01:58:18PM +1200, Alexey Korolev wrote: > Hi, > Tried these patches today on Win2008 x64 guest with 64bit devices. > I've got BSOD on boot. I guess windows don't like changes in _CRS. Hrmm, I went to test this, and found that the "Fix 64bit PCI issues on Windows" patch caus

Re: [Qemu-devel] [SeaBIOS] : Memory problem with Qemu/SeaBIOS

2012-05-07 Thread Kevin O'Connor
On Mon, May 07, 2012 at 08:58:45AM -0700, Alain Ribière wrote: > Hello, > > I have a memory trouble with an old DOS OS from Digital Research called > Concurrent DOS. > It worked fine till Qemu v0.11.1. > I use Qemu to make a legacy application run without any modification. > > Starting with Qemu

Re: [Qemu-devel] Re : [SeaBIOS] : Memory problem with Qemu/SeaBIOS

2012-05-08 Thread Kevin O'Connor
On Tue, May 08, 2012 at 09:47:56AM -0700, Alain Ribière wrote: > Thanks for your answer. > > Here is the debug log in attachement. > I disabled the debug on the screen shot I sent because I noticed I could get > a bit more memory without it. > With the debug on, I got : > Banked Window  416K at 3

Re: [Qemu-devel] Re : Re : [SeaBIOS] : Memory problem with Qemu/SeaBIOS

2012-05-09 Thread Kevin O'Connor
On Wed, May 09, 2012 at 08:57:53AM -0700, Alain Ribière wrote: > Tanks for your answer. > > > I have to apologize, I made a mistake. I rechecked my application and used > git bisect. > So I found that this is not a SeaBIOS issue (actually it's more a C-DOS > issues...). My problem comes from th

Re: [Qemu-devel] Re : Re : [SeaBIOS] : Memory problem with Qemu/SeaBIOS

2012-05-09 Thread Kevin O'Connor
On Wed, May 09, 2012 at 08:44:45PM -0400, Kevin O'Connor wrote: > On Wed, May 09, 2012 at 08:57:53AM -0700, Alain Ribière wrote: > > I wonder what is the CDOS system at 0D4000... It seems a bit strange. > > Very odd. This is still controlled by seabios. Indeed, if I a

Re: [Qemu-devel] [SeaBIOS] [seabios patch 1/5] pci: init all devices

2012-05-12 Thread Kevin O'Connor
On Fri, May 04, 2012 at 05:35:10PM +0200, Gerd Hoffmann wrote: > On 05/04/12 15:15, Kevin O'Connor wrote: > > On Fri, May 04, 2012 at 10:21:23AM +0200, Gerd Hoffmann wrote: > >> seabios used to initialize root bus devices only, with this patch > >> devices behind

Re: [Qemu-devel] [PATCH 12/12] Fix 64bit PCI issues on Windows

2012-05-12 Thread Kevin O'Connor
On Tue, Apr 24, 2012 at 06:26:30PM +1200, Alexey Korolev wrote: > This patch solves issues on Windows guests, when 64bit > BAR's are present. It is also helpful on Linux guests > when use_crs kernel boot option is set. Unfortunately, this patch causes problems for WinXP, so I reverted it. -Kevin

Re: [Qemu-devel] [PATCH 2/2] Get system state configuration from QEMU and patcth DSDT with it.

2012-05-14 Thread Kevin O'Connor
On Mon, May 14, 2012 at 03:35:23PM +0300, Gleb Natapov wrote: > QEMU may want to disable guest's S3/S4 support and it wants to distinguish > between regular powerdown and S4 powerdown. To support that new fw_cfg > option was added that passes supported system states and what value should > guest us

Re: [Qemu-devel] [PATCH 2/2] Get system state configuration from QEMU and patcth DSDT with it.

2012-05-15 Thread Kevin O'Connor
On Tue, May 15, 2012 at 11:06:05AM +0300, Gleb Natapov wrote: > On Mon, May 14, 2012 at 09:43:19PM -0400, Kevin O'Connor wrote: > > On Mon, May 14, 2012 at 03:35:23PM +0300, Gleb Natapov wrote: > > > QEMU may want to disable guest's S3/S4 support and it wants to disti

Re: [Qemu-devel] [PATCH 2/2] Get system state configuration from QEMU and patcth DSDT with it.

2012-05-16 Thread Kevin O'Connor
On Wed, May 16, 2012 at 04:46:57PM +0300, Gleb Natapov wrote: > On Tue, May 15, 2012 at 07:18:10PM -0400, Kevin O'Connor wrote: > > As in the other recent discussion, a struct can be built by the BIOS > > and a pointer passed in via a dynamic SSDT (eg, BDAT). Whatever data &g

Re: [Qemu-devel] [PATCH 2/2] Get system state configuration from QEMU and patcth DSDT with it.

2012-05-16 Thread Kevin O'Connor
On Wed, May 16, 2012 at 05:50:31PM +0200, Paolo Bonzini wrote: > Il 16/05/2012 15:46, Gleb Natapov ha scritto: > > I saw that, but I don't get why doing it this way instead of defining > > the object in AML and patching it? I can define Name(S4VL, 0x2) and path > > 0x2 to whatever QEMU wants me to

Re: [Qemu-devel] [PATCH 3/3] Get system state configuration from QEMU and patch DSDT with it.

2012-05-20 Thread Kevin O'Connor
On Sun, May 20, 2012 at 04:39:01PM +0300, Avi Kivity wrote: > What about > > If (Fcfg(...)) { > Method()... > } > > ? > > (i.e.. define the method conditionally at runtime) As Gleb points out, this wont work. AML defines a static device/method/variable tree heirarchy. Only the ret

Re: [Qemu-devel] [PATCH 3/3] Get system state configuration from QEMU and patch DSDT with it.

2012-05-20 Thread Kevin O'Connor
On Sun, May 20, 2012 at 06:15:44PM +0300, Gleb Natapov wrote: > On Sun, May 20, 2012 at 05:46:46PM +0300, Avi Kivity wrote: > > On 05/20/2012 05:43 PM, Gleb Natapov wrote: > > > > > > > > Or it can be a fixed address in low memory, or a scratch register in > > > > hardware. > > > > > > > Both wil

Re: [Qemu-devel] [PATCH 3/3] Get system state configuration from QEMU and patch DSDT with it.

2012-05-20 Thread Kevin O'Connor
On Sun, May 20, 2012 at 07:25:40PM +0300, Avi Kivity wrote: > On 05/20/2012 07:16 PM, Kevin O'Connor wrote: > > > Here we in agreement, and I was against patching till it was unavoidable, > > > but than pci hotplug started using it, and afterwards processor > &g

Re: [Qemu-devel] [PATCH 1/3] Fix aml_name_string() to recognize block name modifiers.

2012-05-22 Thread Kevin O'Connor
On Sun, May 20, 2012 at 12:03:38PM +0300, Gleb Natapov wrote: > > Signed-off-by: Gleb Natapov The patch series looks okay to me. Let me know when the corresponding qemu patches are committed. -Kevin

Re: [Qemu-devel] [SeaBIOS] solaris x86 in qemu? [bisected]

2013-06-14 Thread Kevin O'Connor
On Fri, Jun 14, 2013 at 06:14:00PM +0400, Michael Tokarev wrote: > 14.06.2013 16:36, Gerd Hoffmann wrote: > > Hi, > > > >> Hmm. Speaking of the splitting. Does the current bios include the > >> the tables which were split into separate files? > > > > Yes, they are in out/ too after building s

Re: [Qemu-devel] [SeaBIOS] solaris x86 in qemu? [bisected]

2013-06-16 Thread Kevin O'Connor
On Sun, Jun 16, 2013 at 02:07:39PM +0400, Michael Tokarev wrote: > 14.06.2013 03:47, Kevin O'Connor wrote: > > On Fri, Jun 14, 2013 at 12:32:22AM +0400, Michael Tokarev wrote: > >> 13.06.2013 11:51, Michael Tokarev wrote: > >>> In order to verify some build is

Re: [Qemu-devel] [PATCH] Probe PCI existence

2011-09-22 Thread Kevin O'Connor
On Wed, Sep 21, 2011 at 08:16:21AM +0200, Jan Kiszka wrote: > From: Jan Kiszka > > This prevents lockups when trying to allocate PCI resources on an > ISA-only system like QEMU can emulate. > > Signed-off-by: Jan Kiszka Looks okay to me. -Kevin

Re: [Qemu-devel] Hang when using 9p mounts after last Seabios update

2011-09-27 Thread Kevin O'Connor
On Thu, Sep 22, 2011 at 12:45:11PM +0100, Daniel P. Berrange wrote: > I'm doing some experimentation with application sandboxing and running > QEMU with the following command line: > > exec $QEMU \ > -m 64 \ > -nographic \ > -nodefconfig \ > -nodefaults \ > -kernel ./kernel-2

<    1   2   3   4   5   6   7   8   >