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

2012-05-20 Thread Gleb Natapov

Signed-off-by: Gleb Natapov 
---
 tools/acpi_extract.py |6 +-
 1 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/tools/acpi_extract.py b/tools/acpi_extract.py
index 5f613e4..8038269 100755
--- a/tools/acpi_extract.py
+++ b/tools/acpi_extract.py
@@ -121,7 +121,11 @@ def aml_name_string(offset):
 if (aml[offset] != 0x08):
 die( "Name offset 0x%x: expected 0x08 actual 0x%x" %
  (offset, aml[offset]));
-return offset + 1;
+offset += 1
+# Block Name Modifier. Skip it.
+if (aml[offset] == 0x5c or aml[offset] == 0x5e):
+offset += 1
+return offset;
 
 # Given data offset, find dword const offset
 def aml_data_dword_const(offset):
-- 
1.7.7.3




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

2012-06-20 Thread Gleb Natapov
On Tue, May 22, 2012 at 09:23:03PM -0400, Kevin O'Connor wrote:
> 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.
> 
It is committed now: 459ae5ea5ad682c2b3220beb244d4102c1a4e332

--
Gleb.



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

2012-06-21 Thread Kevin O'Connor
On Wed, Jun 20, 2012 at 04:08:41PM +0300, Gleb Natapov wrote:
> On Tue, May 22, 2012 at 09:23:03PM -0400, Kevin O'Connor wrote:
> > 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.
> > 
> It is committed now: 459ae5ea5ad682c2b3220beb244d4102c1a4e332

Thanks.  I committed your seabios patches.

-Kevin



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