Re: [SeaBIOS] [Qemu-devel] [Qemu-stable] problems with freeBSD

2013-03-06 Thread Michael Tokarev
07.03.2013 10:12, Doug Goldstein wrote: > On Wed, Mar 6, 2013 at 7:58 PM, Peter Stuge wrote: >> Yeah, it is very common. Lots of distributions patch their >> toolchains such that they don't compile firmware code correctly. >> >> Firmware isn't a common target, so I understand that it happens. >>

[SeaBIOS] [PATCH 2/2] acpi: Eliminate BDAT parameter passing to DSDT code.

2013-03-06 Thread Kevin O'Connor
The "BDAT" construct is the only ACPI mechanism that relies on SeaBIOS reserved memory. Replace it with the SSDT based template system. Signed-off-by: Kevin O'Connor --- src/acpi-dsdt-pci-crs.dsl | 61 +-- src/acpi.c| 39 ++

[SeaBIOS] [PATCH 1/2] Rename src/ssdt-susp.dsl to src/ssdt-misc.dsl.

2013-03-06 Thread Kevin O'Connor
Signed-off-by: Kevin O'Connor --- Makefile | 2 +- src/acpi.c| 8 src/ssdt-misc.dsl | 38 ++ src/ssdt-susp.dsl | 38 -- 4 files changed, 43 insertions(+), 43 deletions(-) create mode 100644 src/ss

[SeaBIOS] [PATCH 0/2][RFC] Remove BDAT from ACPI interface

2013-03-06 Thread Kevin O'Connor
This patch changes SeaBIOS to pass the PCI regions via a dynamically updated SSDT instead of via the BDAT memory reference system. This change will likely make it easier to port the ACPI tables to QEMU. This patch has only been lightly tested. Gerd - I know you ran various tests when originally

Re: [SeaBIOS] [Qemu-devel] problems with freeBSD

2013-03-06 Thread Peter Stuge
Laszlo Ersek wrote: > Going out on a limb, I suspect qemu commit 5f876756 instead. .. > I think the gcc version Anthony was using miscompiled SeaBIOS Yeah, it is very common. Lots of distributions patch their toolchains such that they don't compile firmware code correctly. Firmware isn't a common

Re: [SeaBIOS] [Qemu-devel] problems with freeBSD

2013-03-06 Thread Laszlo Ersek
On 03/07/13 01:53, Kevin O'Connor wrote: > On Thu, Mar 07, 2013 at 12:12:08AM +0100, Aurelien Jarno wrote: >> On Wed, Mar 06, 2013 at 08:21:11AM +, Dietmar Maurer wrote: >>> Using qemu 1.4.0: >>> >>> # qemu -hda test.raw -m 512 -cdrom >>> pfSense-LiveCD-2.0.2-RELEASE-amd64-20121207-2239.iso >>

Re: [SeaBIOS] Build errors with Clang 3.2: src/ioport.h:126:18: error: invalid operand for instruction

2013-03-06 Thread Kevin O'Connor
On Tue, Mar 05, 2013 at 03:20:57PM -0800, H. Peter Anvin wrote: > On 03/04/2013 03:58 PM, Kevin O'Connor wrote: > > On Mon, Mar 04, 2013 at 04:15:51PM +0100, Paul Menzel wrote: > >> to build SeaBIOS, Clang throws the following errors. > > [...] > >> In file included from src/ata.c:9: > >>

Re: [SeaBIOS] [Seabios PATCH v2] make reboot-timeout to static for using it after POST phase

2013-03-06 Thread Kevin O'Connor
On Tue, Mar 05, 2013 at 05:52:21PM +0800, Amos Kong wrote: > From: Kevin O'Connor > > Memory allocated with malloc_tmp() can't be used after the POST > phase. The reboot-timeout inside romfile could not be loaded in > boot_fail(). The patch saved reboot-timeout to a static variable, > it fixed th

Re: [SeaBIOS] [Qemu-devel] problems with freeBSD

2013-03-06 Thread Kevin O'Connor
On Thu, Mar 07, 2013 at 12:12:08AM +0100, Aurelien Jarno wrote: > On Wed, Mar 06, 2013 at 08:21:11AM +, Dietmar Maurer wrote: > > Using qemu 1.4.0: > > > > # qemu -hda test.raw -m 512 -cdrom > > pfSense-LiveCD-2.0.2-RELEASE-amd64-20121207-2239.iso > > > > Results in: > > > > trap 12: page f

Re: [SeaBIOS] [Qemu-devel] problems with freeBSD

2013-03-06 Thread Aurelien Jarno
On Wed, Mar 06, 2013 at 08:21:11AM +, Dietmar Maurer wrote: > Using qemu 1.4.0: > > # qemu -hda test.raw -m 512 -cdrom > pfSense-LiveCD-2.0.2-RELEASE-amd64-20121207-2239.iso > > Results in: > > trap 12: page fault while in kernel mode > ... > stopped at x86bios_emu_rdw+0x2f: movzwl (%rbx),%

Re: [SeaBIOS] [coreboot] i915 status on x60.

2013-03-06 Thread Denis 'GNUtoo' Carikli
On Sun, 3 Mar 2013 22:58:46 -0500 ron minnich wrote: > setgtt is only for hardware with a gtt. Again, I'm not familiar with > the hardware on this laptop. Does it have a gtt? it seems so(from the xf86-video-intel mentioned before): if (IS_G4X(pI830)) gtt = (unsigned char *)

Re: [SeaBIOS] Build errors with Clang 3.2: src/ioport.h:126:18: error: invalid operand for instruction

2013-03-06 Thread Paul Menzel
Am Mittwoch, den 06.03.2013, 11:49 +0100 schrieb Paul Menzel: > Am Montag, den 04.03.2013, 18:58 -0500 schrieb Kevin O'Connor: > > On Mon, Mar 04, 2013 at 04:15:51PM +0100, Paul Menzel wrote: > > > to build SeaBIOS, Clang throws the following errors. > > [...] > > > In file included from sr

Re: [SeaBIOS] Build errors with Clang 3.2: src/ioport.h:126:18: error: invalid operand for instruction

2013-03-06 Thread H. Peter Anvin
On 03/04/2013 03:58 PM, Kevin O'Connor wrote: > On Mon, Mar 04, 2013 at 04:15:51PM +0100, Paul Menzel wrote: >> to build SeaBIOS, Clang throws the following errors. > [...] >> In file included from src/ata.c:9: >> src/ioport.h:126:18: error: invalid operand for instruction >>

Re: [SeaBIOS] [PATCH 4/3] wakeup: only reset the CPU

2013-03-06 Thread Paolo Bonzini
Il 05/03/2013 20:12, Laszlo Ersek ha scritto: > On 03/05/13 17:59, David Woodhouse wrote:> On Tue, 2013-03-05 at 17:03 +0100, > Paolo Bonzini wrote: >>> Resuming from suspend-to-RAM should not reset all devices. Only the >>> CPU should get a reset signal. >> >> Hm... on reflection, I don't actual

Re: [SeaBIOS] Build errors with Clang 3.2: src/ioport.h:126:18: error: invalid operand for instruction

2013-03-06 Thread Paul Menzel
Am Montag, den 04.03.2013, 18:58 -0500 schrieb Kevin O'Connor: > On Mon, Mar 04, 2013 at 04:15:51PM +0100, Paul Menzel wrote: > > to build SeaBIOS, Clang throws the following errors. > [...] > > In file included from src/ata.c:9: > > src/ioport.h:126:18: error: invalid operand for i

Re: [SeaBIOS] [PATCH 4/3] wakeup: only reset the CPU

2013-03-06 Thread Laszlo Ersek
On 03/05/13 20:25, Paolo Bonzini wrote: > Il 05/03/2013 20:12, Laszlo Ersek ha scritto: >> On 03/05/13 17:59, David Woodhouse wrote:> On Tue, 2013-03-05 at 17:03 >> +0100, Paolo Bonzini wrote: Resuming from suspend-to-RAM should not reset all devices. Only the CPU should get a reset sig

[SeaBIOS] [PATCH v2] pciinit: Enable default VGA device

2013-03-06 Thread Alex Williamson
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 the PCI bridge control register. Signed-off-by: Alex Williamson --- src

Re: [SeaBIOS] [PATCH 4/3] wakeup: only reset the CPU

2013-03-06 Thread Laszlo Ersek
On 03/05/13 17:59, David Woodhouse wrote:> On Tue, 2013-03-05 at 17:03 +0100, Paolo Bonzini wrote: >> Resuming from suspend-to-RAM should not reset all devices. Only the >> CPU should get a reset signal. > > Hm... on reflection, I don't actually know if this is true. > > Perhaps we *should* reset

Re: [SeaBIOS] [Seabios PATCH] make reboot-timeout to static for using it after POST phase

2013-03-06 Thread li guang
在 2013-03-05二的 10:06 +0800,Amos Kong写道: > From: Kevin O'Connor > > Memory allocated with malloc_tmp() can't be used after the POST > phase. The reboot-timeout inside romfile could not be loaded in > boot_fail(). The patch saved reboot-timeout to a static variable, > it fixed the regression bug in