On 05.06.19 17:39, Andrej Utz wrote:
Second parameter to MemRegion must be its end (inclusive).

Fixes: 5fe206927c05 ("tools: Implement ACPI IVRS table parser")
Signed-off-by: Andrej Utz <[email protected]>
---
  pyjailhouse/sysfs_parser.py | 5 ++++-
  1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/pyjailhouse/sysfs_parser.py b/pyjailhouse/sysfs_parser.py
index fa32ba48..83222c4a 100644
--- a/pyjailhouse/sysfs_parser.py
+++ b/pyjailhouse/sysfs_parser.py
@@ -481,7 +481,10 @@ def parse_ivrs(pcidevices, ioapics):
                        'regions. The memory at 0x%x will be mapped accessible '
                        'to all devices.' % mem_addr)
- regions.append(MemRegion(mem_addr, mem_len, 'ACPI IVRS', comment))
+            regions.append(
+                MemRegion(mem_addr, mem_addr + mem_len - 1, 'ACPI IVRS',
+                    comment))
+
          elif type == 0x40:
              raise RuntimeError(
                  'You board uses IVRS Rev. 2 feature Jailhouse doesn\'t '


Thanks, applied, fixing up the whitespace warning.

Jan

--
Siemens AG, Corporate Technology, CT RDA IOT SES-DE
Corporate Competence Center Embedded Linux

--
You received this message because you are subscribed to the Google Groups 
"Jailhouse" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jailhouse-dev/6c82b0ff-36a8-4916-acc0-f1dc32d8b6eb%40siemens.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to