[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] x86/ACPI: also print address space for PM1x fields



Jan Beulich wrote:

At least one vendor is in the process of making systems available where
these live in MMIO, not in I/O port space.

Signed-off-by: Jan Beulich<jbeulich@xxxxxxxx>


Probably an abundance of caution to keep everything platform related in sync between dom0 and Xen. I agree this shouldn't be necessary in this case and I'm okay with this simplification. Frankly I doubt anyone has to specify this option on any halfway sane box anyway.

Acked-by: Keir Fraser <keir@xxxxxxx>


--- a/xen/arch/x86/acpi/boot.c
+++ b/xen/arch/x86/acpi/boot.c
@@ -402,11 +402,15 @@ acpi_fadt_parse_sleep_info(struct acpi_t
  acpi_fadt_copy_address(pm1b_evt, pm1b_event, pm1_event);

  printk(KERN_INFO PREFIX
-        "SLEEP INFO: pm1x_cnt[%"PRIx64",%"PRIx64"], "
-        "pm1x_evt[%"PRIx64",%"PRIx64"]\n",
+        "SLEEP INFO: pm1x_cnt[%d:%"PRIx64",%d:%"PRIx64"], "
+        "pm1x_evt[%d:%"PRIx64",%d:%"PRIx64"]\n",
+        acpi_sinfo.pm1a_cnt_blk.space_id,
         acpi_sinfo.pm1a_cnt_blk.address,
+        acpi_sinfo.pm1b_cnt_blk.space_id,
         acpi_sinfo.pm1b_cnt_blk.address,
+        acpi_sinfo.pm1a_evt_blk.space_id,
         acpi_sinfo.pm1a_evt_blk.address,
+        acpi_sinfo.pm1b_evt_blk.space_id,
         acpi_sinfo.pm1b_evt_blk.address);

  /* Now FACS... */



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.