Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-07 Thread Laszlo Ersek
On 02/07/17 22:02, Phil Dennis-Jordan wrote: > On 7 February 2017 at 20:54, Laszlo Ersek wrote: >> I filed . > > That looks nice and thorough. > >>> Your EDK2 patch >> >> For the record: >> [1] https://lists.01.org/pipermail/edk2-devel/2017-Fe

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-07 Thread Phil Dennis-Jordan
On 7 February 2017 at 20:54, Laszlo Ersek wrote: > On 02/06/17 17:30, Phil Dennis-Jordan wrote: >> Thanks for the in-depth reply - apologies for abandoning the thread >> for a few days. Looks like a bunch of things have already been hashed >> out, but I have a few more comments: > > Vice versa...

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-07 Thread Laszlo Ersek
On 02/06/17 17:30, Phil Dennis-Jordan wrote: > Thanks for the in-depth reply - apologies for abandoning the thread > for a few days. Looks like a bunch of things have already been hashed > out, but I have a few more comments: Vice versa... Sorry about responding a bit late. Yesterday I severely ov

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-06 Thread Laszlo Ersek
On 02/06/17 17:44, Phil Dennis-Jordan wrote: > On 31 January 2017 at 20:08, Laszlo Ersek wrote: >> On 01/31/17 19:17, Michael S. Tsirkin wrote: >>> On Tue, Jan 31, 2017 at 05:28:57PM +0100, Laszlo Ersek wrote: The ACPI 6.1 spec says, - DSDT: [...] If the X_DSDT field contains a non-

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-06 Thread Phil Dennis-Jordan
On 31 January 2017 at 20:08, Laszlo Ersek wrote: > On 01/31/17 19:17, Michael S. Tsirkin wrote: >> On Tue, Jan 31, 2017 at 05:28:57PM +0100, Laszlo Ersek wrote: >>> The ACPI 6.1 spec says, >>> >>> - DSDT: [...] If the X_DSDT field contains a non-zero value then this >>> field must be zero. >>> -

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-06 Thread Phil Dennis-Jordan
Thanks for the in-depth reply - apologies for abandoning the thread for a few days. Looks like a bunch of things have already been hashed out, but I have a few more comments: On 31 January 2017 at 17:28, Laszlo Ersek wrote: > On 01/31/17 15:58, Michael S. Tsirkin wrote: >> On Tue, Jan 31, 2017 at

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-02 Thread Michael S. Tsirkin
On Tue, Jan 31, 2017 at 04:04:58PM +, Phil Dennis-Jordan wrote: > > On Tue, 31 Jan 2017 at 16:41, Igor Mammedov wrote: > > On Tue, 31 Jan 2017 16:58:22 +0200 > "Michael S. Tsirkin" wrote: > > > On Tue, Jan 31, 2017 at 03:31:46PM +0100, Phil Dennis-Jordan wrote: > > > On 18

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-01 Thread Laszlo Ersek
On 02/01/17 17:17, Michael S. Tsirkin wrote: > On Wed, Feb 01, 2017 at 05:03:38PM +0100, Laszlo Ersek wrote: >> On 02/01/17 16:16, Igor Mammedov wrote: >>> On Wed, 1 Feb 2017 14:03:52 +0100 >>> Laszlo Ersek wrote: >>> On 02/01/17 13:52, Laszlo Ersek wrote: > On 02/01/17 12:37, Igor Mammed

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-01 Thread Michael S. Tsirkin
On Wed, Feb 01, 2017 at 05:03:38PM +0100, Laszlo Ersek wrote: > On 02/01/17 16:16, Igor Mammedov wrote: > > On Wed, 1 Feb 2017 14:03:52 +0100 > > Laszlo Ersek wrote: > > > >> On 02/01/17 13:52, Laszlo Ersek wrote: > >>> On 02/01/17 12:37, Igor Mammedov wrote: > On Tue, 31 Jan 2017 20:17:02

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-01 Thread Laszlo Ersek
On 02/01/17 16:16, Igor Mammedov wrote: > On Wed, 1 Feb 2017 14:03:52 +0100 > Laszlo Ersek wrote: > >> On 02/01/17 13:52, Laszlo Ersek wrote: >>> On 02/01/17 12:37, Igor Mammedov wrote: On Tue, 31 Jan 2017 20:17:02 +0200 "Michael S. Tsirkin" wrote: > On Tue, Jan 31, 2017 a

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-01 Thread Igor Mammedov
On Wed, 1 Feb 2017 14:03:52 +0100 Laszlo Ersek wrote: > On 02/01/17 13:52, Laszlo Ersek wrote: > > On 02/01/17 12:37, Igor Mammedov wrote: > >> On Tue, 31 Jan 2017 20:17:02 +0200 > >> "Michael S. Tsirkin" wrote: > >> > >>> On Tue, Jan 31, 2017 at 05:28:57PM +0100, Laszlo Ersek wrote: > >>>

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-01 Thread Michael S. Tsirkin
On Wed, Feb 01, 2017 at 12:37:39PM +0100, Igor Mammedov wrote: > On Tue, 31 Jan 2017 20:17:02 +0200 > "Michael S. Tsirkin" wrote: > > > On Tue, Jan 31, 2017 at 05:28:57PM +0100, Laszlo Ersek wrote: > > > The ACPI 6.1 spec says, > > > > > > - DSDT: [...] If the X_DSDT field contains a non-zero va

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-01 Thread Laszlo Ersek
On 02/01/17 13:52, Laszlo Ersek wrote: > On 02/01/17 12:37, Igor Mammedov wrote: >> On Tue, 31 Jan 2017 20:17:02 +0200 >> "Michael S. Tsirkin" wrote: >> >>> On Tue, Jan 31, 2017 at 05:28:57PM +0100, Laszlo Ersek wrote: The ACPI 6.1 spec says, - DSDT: [...] If the X_DSDT field contai

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-01 Thread Laszlo Ersek
On 02/01/17 12:37, Igor Mammedov wrote: > On Tue, 31 Jan 2017 20:17:02 +0200 > "Michael S. Tsirkin" wrote: > >> On Tue, Jan 31, 2017 at 05:28:57PM +0100, Laszlo Ersek wrote: >>> The ACPI 6.1 spec says, >>> >>> - DSDT: [...] If the X_DSDT field contains a non-zero value then this >>> field must

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-02-01 Thread Igor Mammedov
On Tue, 31 Jan 2017 20:17:02 +0200 "Michael S. Tsirkin" wrote: > On Tue, Jan 31, 2017 at 05:28:57PM +0100, Laszlo Ersek wrote: > > The ACPI 6.1 spec says, > > > > - DSDT: [...] If the X_DSDT field contains a non-zero value then this > > field must be zero. > > - X_DSDT: [...] If the DSDT field

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-31 Thread Laszlo Ersek
On 01/31/17 19:17, Michael S. Tsirkin wrote: > On Tue, Jan 31, 2017 at 05:28:57PM +0100, Laszlo Ersek wrote: >> The ACPI 6.1 spec says, >> >> - DSDT: [...] If the X_DSDT field contains a non-zero value then this >> field must be zero. >> - X_DSDT: [...] If the DSDT field contains a non-zero value

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-31 Thread Michael S. Tsirkin
On Tue, Jan 31, 2017 at 05:28:57PM +0100, Laszlo Ersek wrote: > The ACPI 6.1 spec says, > > - DSDT: [...] If the X_DSDT field contains a non-zero value then this > field must be zero. > - X_DSDT: [...] If the DSDT field contains a non-zero value then this > field must be zero. But that's only

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-31 Thread Laszlo Ersek
On 01/31/17 15:58, Michael S. Tsirkin wrote: > On Tue, Jan 31, 2017 at 03:31:46PM +0100, Phil Dennis-Jordan wrote: >> On 18 January 2017 at 18:19, Igor Mammedov wrote: >>> On Wed, 18 Jan 2017 18:30:59 +0200 >>> "Michael S. Tsirkin" wrote: >>> On Wed, Jan 18, 2017 at 12:45:54PM +0100, Phil De

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-31 Thread Igor Mammedov
On Tue, 31 Jan 2017 16:04:58 + Phil Dennis-Jordan wrote: > On Tue, 31 Jan 2017 at 16:41, Igor Mammedov wrote: > > > On Tue, 31 Jan 2017 16:58:22 +0200 > > "Michael S. Tsirkin" wrote: > > > > > On Tue, Jan 31, 2017 at 03:31:46PM +0100, Phil Dennis-Jordan wrote: > > > > On 18 January 201

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-31 Thread Phil Dennis-Jordan
On Tue, 31 Jan 2017 at 16:41, Igor Mammedov wrote: > On Tue, 31 Jan 2017 16:58:22 +0200 > "Michael S. Tsirkin" wrote: > > > On Tue, Jan 31, 2017 at 03:31:46PM +0100, Phil Dennis-Jordan wrote: > > > On 18 January 2017 at 18:19, Igor Mammedov > wrote: > > > > On Wed, 18 Jan 2017 18:30:59 +0200 >

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-31 Thread Igor Mammedov
On Tue, 31 Jan 2017 16:58:22 +0200 "Michael S. Tsirkin" wrote: > On Tue, Jan 31, 2017 at 03:31:46PM +0100, Phil Dennis-Jordan wrote: > > On 18 January 2017 at 18:19, Igor Mammedov wrote: > > > On Wed, 18 Jan 2017 18:30:59 +0200 > > > "Michael S. Tsirkin" wrote: > > > > > >> On Wed, Jan 18,

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-31 Thread Michael S. Tsirkin
On Tue, Jan 31, 2017 at 03:31:46PM +0100, Phil Dennis-Jordan wrote: > On 18 January 2017 at 18:19, Igor Mammedov wrote: > > On Wed, 18 Jan 2017 18:30:59 +0200 > > "Michael S. Tsirkin" wrote: > > > >> On Wed, Jan 18, 2017 at 12:45:54PM +0100, Phil Dennis-Jordan wrote: > > [...] > > > >> > I suspec

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-31 Thread Phil Dennis-Jordan
On 18 January 2017 at 18:19, Igor Mammedov wrote: > On Wed, 18 Jan 2017 18:30:59 +0200 > "Michael S. Tsirkin" wrote: > >> On Wed, Jan 18, 2017 at 12:45:54PM +0100, Phil Dennis-Jordan wrote: > [...] > >> > I suspect more might be involved in enabling ACPI 2.0, and it should >> > probably be an op

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-27 Thread Paolo Bonzini
On 27/01/2017 14:57, Igor Mammedov wrote: >> +fadt->reset_register.space_id = 1; >> +fadt->reset_register.bit_width = 8; >> +fadt->reset_register.address= ICH9_RST_CNT_IOPORT; > cpu_to_le64 > > + specify explicitly all fields of GAS structure: reset_register > > + is r

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-27 Thread Igor Mammedov
On Thu, 26 Jan 2017 14:43:04 +0100 Phil Dennis-Jordan wrote: > On 23 January 2017 at 12:12, Igor Mammedov wrote: > >> For reference, my approach to filling out the Xdsdt/Xfacs fields in > >> build_fadt() is essentially the same as for the 32-bit variants from > >> rev1: > >> > >> unsigned xfacs_

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-26 Thread Phil Dennis-Jordan
On 23 January 2017 at 12:12, Igor Mammedov wrote: >> For reference, my approach to filling out the Xdsdt/Xfacs fields in >> build_fadt() is essentially the same as for the 32-bit variants from >> rev1: >> >> unsigned xfacs_offset = (char *)&fadt->Xfacs - table_data->data; >> bios_linker_loader_add

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-23 Thread Igor Mammedov
On Thu, 19 Jan 2017 19:09:47 +0100 Phil Dennis-Jordan wrote: > On 18 January 2017 at 17:30, Michael S. Tsirkin wrote: > > I think what's important is the Fadt format revision. That one was 1 for > > 1.0b and 3 for 2.0. > > > > See page 112, Table 5-5 Fixed ACPI Description Table Format in acpi

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-19 Thread Phil Dennis-Jordan
On 18 January 2017 at 17:30, Michael S. Tsirkin wrote: > I think what's important is the Fadt format revision. That one was 1 for 1.0b > and 3 for 2.0. > > See page 112, Table 5-5 Fixed ACPI Description Table Format in acpi spec > 1.0b. > > Now look at page 110 in spec 2.0, this time > "Table 5-8

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-18 Thread Igor Mammedov
On Wed, 18 Jan 2017 18:30:59 +0200 "Michael S. Tsirkin" wrote: > On Wed, Jan 18, 2017 at 12:45:54PM +0100, Phil Dennis-Jordan wrote: [...] > > I suspect more might be involved in enabling ACPI 2.0, and it should > > probably be an option so as to avoid regressions. I don't know what the > > be

Re: [Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-18 Thread Michael S. Tsirkin
On Wed, Jan 18, 2017 at 12:45:54PM +0100, Phil Dennis-Jordan wrote: > About 2 years ago, Reza Jelveh submitted essentially this same patch: > https://lists.gnu.org/archive/html/qemu-devel/2015-03/msg05832.html > > It adds the reset register defined in ACPI 2.0 to the x86 FADT, which fixes > reboo

[Qemu-devel] [PATCH RFC] acpi: add reset register to fadt

2017-01-18 Thread Phil Dennis-Jordan
About 2 years ago, Reza Jelveh submitted essentially this same patch: https://lists.gnu.org/archive/html/qemu-devel/2015-03/msg05832.html It adds the reset register defined in ACPI 2.0 to the x86 FADT, which fixes rebooting for Darwin/OS X/macOS guests. I'm trying to revive this as part of an ef