Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2013-01-17 Thread TeLeMan
On Thu, May 12, 2011 at 10:44 PM, Michael Tokarev m...@tls.msk.ru wrote: This patch almost rewrites acpi_table_add() function (but still leaves it using old get_param_value() interface). The result is that it's now possible to specify whole table (together with a header) in an external file,

Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2013-01-17 Thread Michael Tokarev
Wow you're replying to an old post... ;) 17.01.2013 13:50, TeLeMan wrote: On Thu, May 12, 2011 at 10:44 PM, Michael Tokarev m...@tls.msk.ru wrote: This patch almost rewrites acpi_table_add() function [] + +/* now read in the data files, reallocating buffer as needed */ + +for (f =

Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2011-07-30 Thread Blue Swirl
Thanks, applied. On Fri, Jul 29, 2011 at 4:49 AM, Isaku Yamahata yamah...@valinux.co.jp wrote: On Fri, Jul 15, 2011 at 07:51:43PM +0300, Blue Swirl wrote: On Fri, Jul 15, 2011 at 6:18 PM, John Baboval john.babo...@virtualcomputer.com wrote: Is there something I can do to help take this patch

Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2011-07-30 Thread John Baboval
Thanks. Somehow completely missed Blue's response back on the 15th. Glad to see this in. When using this for SLIC I had to patch the OEM ID fields in the other tables to match at runtime in order to make windows licensing happy. But thats a BIOS change, not something in QEMU On Jul 30,

Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2011-07-28 Thread Isaku Yamahata
On Fri, Jul 15, 2011 at 07:51:43PM +0300, Blue Swirl wrote: On Fri, Jul 15, 2011 at 6:18 PM, John Baboval john.babo...@virtualcomputer.com wrote: Is there something I can do to help take this patch the rest of the way? I'd hate to see it die because of a style issue and a compiler warning.

Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2011-07-28 Thread Michael Tokarev
29.07.2011 05:49, Isaku Yamahata wrote: On Fri, Jul 15, 2011 at 07:51:43PM +0300, Blue Swirl wrote: On Fri, Jul 15, 2011 at 6:18 PM, John Baboval john.babo...@virtualcomputer.com wrote: Is there something I can do to help take this patch the rest of the way? I'd hate to see it die because of

Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2011-07-15 Thread John Baboval
Is there something I can do to help take this patch the rest of the way? I'd hate to see it die because of a style issue and a compiler warning. -John On 06/15/2011 02:19 PM, Blue Swirl wrote: On Sat, Jun 11, 2011 at 1:29 AM, Michael Tokarevm...@tls.msk.ru wrote: I've given up on this one.

Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2011-07-15 Thread Blue Swirl
On Fri, Jul 15, 2011 at 6:18 PM, John Baboval john.babo...@virtualcomputer.com wrote: Is there something I can do to help take this patch the rest of the way? I'd hate to see it die because of a style issue and a compiler warning. There's also suspicious missing 'break' statement. How about

Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2011-07-06 Thread John Baboval
Sorry for coming late to this thread. I've tested this patch, after fixing the format specifier, and it works. (Though I did test with Xen, and not KVM.) It's quite convenient. I'm planning on including it in the build we ship with our product. It would be nice to see the style issues

Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2011-06-15 Thread Blue Swirl
On Sat, Jun 11, 2011 at 1:29 AM, Michael Tokarev m...@tls.msk.ru wrote: I've given up on this one.  Personally I don't need this stuff for my win7 guests since I can hack either bios or the O/S loader to include all the necessary verifications for the win7 activation to work.  I tried to make

Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2011-06-12 Thread Isaku Yamahata
No more comments on the patch? All comments were addressed and there is no comments so far. And this patch only adds one more option to -acpi, thus it's quite safe to merge this patch. So please pick this patch up. Anyway who is the maintainer of acpi.c(or acpi related files)? According to

Re: [Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2011-06-10 Thread Michael Tokarev
I've given up on this one. Personally I don't need this stuff for my win7 guests since I can hack either bios or the O/S loader to include all the necessary verifications for the win7 activation to work. I tried to make this process to be legal (no hacks or cracks needed) and easy for others,

[Qemu-devel] [PATCH] v6 revamp acpitable parsing and allow to specify complete (headerful) table

2011-06-06 Thread Michael Tokarev
This patch almost rewrites acpi_table_add() function (but still leaves it using old get_param_value() interface). The result is that it's now possible to specify whole table (together with a header) in an external file, instead of just data portion, with a new file= parameter, but at the same time