[PATCH] [HVMLOADER] fixup compiler warnings and constant usage

[Ben Thomas <bthomas@virtualiron.com>]

[-- Attachment #1: Type: text/plain, Size: 564 bytes --]

This patch fixes up some compiler warnings from the hvmloader build.
And, while I was there, I removed the hard constants from the
strncpy functions.

Signed-off-by: Ben Thomas (bthomas@virtualiron.com)

-- 
------------------------------------------------------------------------
Ben Thomas                                         Virtual Iron Software
bthomas@virtualiron.com                            Tower 1, Floor 2
978-849-1214                                       900 Chelmsford Street
                                                    Lowell, MA 01851

[-- Attachment #2: tools-hvmloader_acpi_build_warnings.patch --]
[-- Type: text/x-patch, Size: 3038 bytes --]

diff -r 0444f009ccbb tools/firmware/hvmloader/acpi/build.c
--- a/tools/firmware/hvmloader/acpi/build.c	Mon Mar 12 15:48:11 2007 -0400
+++ b/tools/firmware/hvmloader/acpi/build.c	Mon Mar 12 15:53:28 2007 -0400
@@ -57,8 +57,8 @@ int construct_madt(struct acpi_20_madt *
     memset(madt, 0, sizeof(*madt));
     madt->header.signature    = ACPI_2_0_MADT_SIGNATURE;
     madt->header.revision     = ACPI_2_0_MADT_REVISION;
-    strncpy(madt->header.oem_id, ACPI_OEM_ID, 6);
-    strncpy(madt->header.oem_table_id, ACPI_OEM_TABLE_ID, 8);
+    strncpy((char *)madt->header.oem_id, ACPI_OEM_ID, sizeof(madt->header.oem_id));
+    strncpy((char *)madt->header.oem_table_id, ACPI_OEM_TABLE_ID, sizeof(madt->header.oem_table_id));
     madt->header.oem_revision = ACPI_OEM_REVISION;
     madt->header.creator_id   = ACPI_CREATOR_ID;
     madt->header.creator_revision = ACPI_CREATOR_REVISION;
@@ -131,8 +131,8 @@ int construct_hpet(struct acpi_20_hpet *
     memset(hpet, 0, sizeof(*hpet));
     hpet->header.signature    = ACPI_2_0_HPET_SIGNATURE;
     hpet->header.revision     = ACPI_2_0_HPET_REVISION;
-    strncpy(hpet->header.oem_id, ACPI_OEM_ID, 6);
-    strncpy(hpet->header.oem_table_id, ACPI_OEM_TABLE_ID, 8);
+    strncpy((char *)hpet->header.oem_id, ACPI_OEM_ID, sizeof(hpet->header.oem_id));
+    strncpy((char *)hpet->header.oem_table_id, ACPI_OEM_TABLE_ID, sizeof(hpet->header.oem_table_id));
     hpet->header.oem_revision = ACPI_OEM_REVISION;
     hpet->header.creator_id   = ACPI_CREATOR_ID;
     hpet->header.creator_revision = ACPI_CREATOR_REVISION;
@@ -161,8 +161,8 @@ int construct_processor_objects(uint8_t 
     hdr = (struct acpi_header *)p;
     hdr->signature = ASCII32('S','S','D','T');
     hdr->revision  = 2;
-    strncpy(hdr->oem_id, ACPI_OEM_ID, 6);
-    strncpy(hdr->oem_table_id, ACPI_OEM_TABLE_ID, 8);
+    strncpy((char *)hdr->oem_id, ACPI_OEM_ID, sizeof(hdr->oem_id));
+    strncpy((char *)hdr->oem_table_id, ACPI_OEM_TABLE_ID, sizeof(hdr->oem_table_id));
     hdr->oem_revision = ACPI_OEM_REVISION;
     hdr->creator_id = ACPI_CREATOR_ID;
     hdr->creator_revision = ACPI_CREATOR_REVISION;
@@ -195,7 +195,7 @@ int construct_processor_objects(uint8_t 
     }
 
     /* NameString */
-    strncpy(p, "\\_PR_", 5);
+    strncpy((char *)p, "\\_PR_", strlen("\\_PR_"));
     p += 5;
 
     /*
@@ -263,8 +263,8 @@ int construct_secondary_tables(uint8_t *
         tcpa->header.signature = ACPI_2_0_TCPA_SIGNATURE;
         tcpa->header.length    = sizeof(*tcpa);
         tcpa->header.revision  = ACPI_2_0_TCPA_REVISION;
-        strncpy(tcpa->header.oem_id, ACPI_OEM_ID, 6);
-        strncpy(tcpa->header.oem_table_id, ACPI_OEM_TABLE_ID, 8);
+        strncpy((char *)tcpa->header.oem_id, ACPI_OEM_ID, sizeof(tcpa->header.oem_id));
+        strncpy((char *)tcpa->header.oem_table_id, ACPI_OEM_TABLE_ID, sizeof(tcpa->header.oem_table_id));
         tcpa->header.oem_revision = ACPI_OEM_REVISION;
         tcpa->header.creator_id   = ACPI_CREATOR_ID;
         tcpa->header.creator_revision = ACPI_CREATOR_REVISION;

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel