From patchwork Fri Jul 20 20:19:38 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Colin Ian King X-Patchwork-Id: 172341 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from chlorine.canonical.com (chlorine.canonical.com [91.189.94.204]) by ozlabs.org (Postfix) with ESMTP id 846B22C03EC for ; Sat, 21 Jul 2012 06:19:46 +1000 (EST) Received: from localhost ([127.0.0.1] helo=chlorine.canonical.com) by chlorine.canonical.com with esmtp (Exim 4.71) (envelope-from ) id 1SsJfs-0003Tg-VH for incoming@patchwork.ozlabs.org; Fri, 20 Jul 2012 20:19:45 +0000 Received: from youngberry.canonical.com ([91.189.89.112]) by chlorine.canonical.com with esmtp (Exim 4.71) (envelope-from ) id 1SsJfq-0003TS-Ps for fwts-devel@lists.ubuntu.com; Fri, 20 Jul 2012 20:19:42 +0000 Received: from 74-95-45-185-oregon.hfc.comcastbusiness.net ([74.95.45.185] helo=localhost) by youngberry.canonical.com with esmtpsa (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1SsJfp-0004eo-Sy for fwts-devel@lists.ubuntu.com; Fri, 20 Jul 2012 20:19:42 +0000 From: Colin King To: fwts-devel@lists.ubuntu.com Subject: [PATCH] acpi: acpidump: make arrays static Date: Fri, 20 Jul 2012 13:19:38 -0700 Message-Id: <1342815578-20139-1-git-send-email-colin.king@canonical.com> X-Mailer: git-send-email 1.7.10.4 X-BeenThere: fwts-devel@lists.ubuntu.com X-Mailman-Version: 2.1.13 Precedence: list List-Id: Firmware Test Suite Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Sender: fwts-devel-bounces@lists.ubuntu.com Errors-To: fwts-devel-bounces@lists.ubuntu.com From: Colin Ian King A whole load of tables should be static rather than on the stack. Signed-off-by: Colin Ian King Acked-by: Keng-Yu Lin Acked-by: Ivan Hu --- src/acpi/acpidump/acpidump.c | 55 +++++++++++++++++++++--------------------- 1 file changed, 27 insertions(+), 28 deletions(-) diff --git a/src/acpi/acpidump/acpidump.c b/src/acpi/acpidump/acpidump.c index 636631a..5c70bf3 100644 --- a/src/acpi/acpidump/acpidump.c +++ b/src/acpi/acpidump/acpidump.c @@ -294,7 +294,7 @@ static void acpi_dump_gas(fwts_framework *fw, fwts_acpidump_field *info, void *d "Unknown", }; - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_STRF(" addr_space_id", fwts_acpi_gas, address_space_id, acpi_dump_gas_address_space_id), FIELD_UINT(" reg_bit_width", fwts_acpi_gas, register_bit_width), FIELD_UINT(" reg_bit_offset", fwts_acpi_gas, register_bit_offset), @@ -313,7 +313,7 @@ static void acpi_dump_gas(fwts_framework *fw, fwts_acpidump_field *info, void *d static void acpidump_hdr(fwts_framework *fw, fwts_acpi_table_header *hdr, int length) { - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_STR ("Signature", fwts_acpi_table_header, signature), FIELD_UINT("Length", fwts_acpi_table_header, length), FIELD_UINT("Revision", fwts_acpi_table_header, revision), @@ -386,7 +386,7 @@ static void acpidump_bert(fwts_framework *fw, fwts_acpi_table_info *table) int n = length - sizeof(fwts_acpi_table_bert); - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_UINT("Region Length", fwts_acpi_table_bert, boot_error_region_length), FIELD_UINT("Region Addr", fwts_acpi_table_bert, boot_error_region), FIELD_UINT("Boot Status", fwts_acpi_table_bert, boot_status), @@ -430,8 +430,7 @@ static void acpidump_ecdt(fwts_framework *fw, fwts_acpi_table_info *table) fwts_acpi_table_ecdt *ecdt = (fwts_acpi_table_ecdt*)data; int n = length - sizeof(fwts_acpi_table_ecdt); - - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_GAS ("EC_CONTROL",fwts_acpi_table_ecdt, ec_control), FIELD_GAS ("EC_DATA", fwts_acpi_table_ecdt, ec_data), FIELD_UINT("UID", fwts_acpi_table_ecdt, uid), @@ -502,7 +501,7 @@ static void acpidump_erst(fwts_framework *fw, fwts_acpi_table_info *table) fwts_acpi_table_erst *erst = (fwts_acpi_table_erst*)data; - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_UINT("Serialization Hdr. Size", fwts_acpi_table_erst, serialization_header_size), FIELD_UINT("Insrtuction Entry Count", fwts_acpi_table_erst, instruction_entry_count), FIELD_END @@ -536,7 +535,7 @@ static void acpidump_facs(fwts_framework *fw, fwts_acpi_table_info *table) uint8_t *data = (uint8_t *)table->data; size_t length = table->length; - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_STR ("Signature", fwts_acpi_table_facs, signature), FIELD_UINT("Length", fwts_acpi_table_facs, length), FIELD_UINT("H/W Signature", fwts_acpi_table_facs, hardware_signature), @@ -557,7 +556,7 @@ static void acpidump_hpet(fwts_framework *fw, fwts_acpi_table_info *table) uint8_t *data = (uint8_t *)table->data; size_t length = table->length; - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_UINT("Event Timer ID", fwts_acpi_table_hpet, event_timer_block_id), FIELD_BITF(" Hardware Rev", fwts_acpi_table_hpet, event_timer_block_id, 8, 0), FIELD_BITF(" Num Comparitors", fwts_acpi_table_hpet, event_timer_block_id, 5, 8), @@ -582,7 +581,7 @@ static void acpidump_fadt(fwts_framework *fw, fwts_acpi_table_info *table) uint8_t *data = (uint8_t *)table->data; size_t length = table->length; - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_UINT("FACS Address", fwts_acpi_table_fadt, firmware_control), FIELD_UINT("DSDT Address", fwts_acpi_table_fadt, dsdt), FIELD_UINT("Model", fwts_acpi_table_fadt, reserved), @@ -673,7 +672,7 @@ static void acpidump_rsdp(fwts_framework *fw, fwts_acpi_table_info *table) uint8_t *data = (uint8_t *)table->data; size_t length = table->length; - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_STR ("Signature", fwts_acpi_table_rsdp, signature), FIELD_UINT("Checksum", fwts_acpi_table_rsdp, checksum), FIELD_STR ("OEM ID", fwts_acpi_table_rsdp, oem_id), @@ -717,7 +716,7 @@ static void acpidump_sbst(fwts_framework *fw, fwts_acpi_table_info *table) uint8_t *data = (uint8_t *)table->data; size_t length = table->length; - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_UINT("Warn. Energy Level", fwts_acpi_table_sbst, warning_energy_level), FIELD_UINT("Low Energy Level", fwts_acpi_table_sbst, low_energy_level), FIELD_UINT("Crit. Energy Level", fwts_acpi_table_sbst, critical_energy_level), @@ -757,7 +756,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) int i = 0; size_t offset = 0; - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_UINT("Local APIC Address", fwts_acpi_table_madt, lapic_address), FIELD_UINT("Flags", fwts_acpi_table_madt, flags), FIELD_BITF(" PCAT_COMPAT", fwts_acpi_table_madt, flags, 1, 0), @@ -782,7 +781,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) switch (hdr->type) { case 0: { - fwts_acpidump_field fields_processor_local_apic[] = { + static fwts_acpidump_field fields_processor_local_apic[] = { FIELD_UINT(" ACPI CPU ID", fwts_acpi_madt_processor_local_apic, acpi_processor_id), FIELD_UINT(" APIC ID", fwts_acpi_madt_processor_local_apic, apic_id), FIELD_UINT(" Flags", fwts_acpi_madt_processor_local_apic, flags), @@ -795,7 +794,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) } break; case 1: { - fwts_acpidump_field fields_io_apic[] = { + static fwts_acpidump_field fields_io_apic[] = { FIELD_UINT(" I/O APIC ID", fwts_acpi_madt_io_apic, io_apic_id), FIELD_UINT(" I/O APIC Addr", fwts_acpi_madt_io_apic, io_apic_phys_address), FIELD_UINT(" Global IRQ Base", fwts_acpi_madt_io_apic, global_irq_base), @@ -807,7 +806,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) } break; case 2: { - fwts_acpidump_field fields_madt_interrupt_override[] = { + static fwts_acpidump_field fields_madt_interrupt_override[] = { FIELD_UINT(" Bus", fwts_acpi_madt_interrupt_override, bus), FIELD_UINT(" Source", fwts_acpi_madt_interrupt_override, source), FIELD_UINT(" Gbl Sys Int", fwts_acpi_madt_interrupt_override, gsi), @@ -820,7 +819,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) } break; case 3: { - fwts_acpidump_field fields_madt_nmi[] = { + static fwts_acpidump_field fields_madt_nmi[] = { FIELD_UINT(" Flags", fwts_acpi_madt_nmi, flags), FIELD_UINT(" Gbl Sys Int", fwts_acpi_madt_nmi, gsi), FIELD_END @@ -831,7 +830,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) } break; case 4: { - fwts_acpidump_field fields_madt_local_apic_nmi[] = { + static fwts_acpidump_field fields_madt_local_apic_nmi[] = { FIELD_UINT(" ACPI CPU ID", fwts_acpi_madt_local_apic_nmi, acpi_processor_id), FIELD_UINT(" Flags", fwts_acpi_madt_local_apic_nmi, flags), FIELD_UINT(" Local APIC LINT", fwts_acpi_madt_local_apic_nmi, local_apic_lint), @@ -843,7 +842,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) } break; case 5: { - fwts_acpidump_field fields_madt_local_apic_addr_override[] = { + static fwts_acpidump_field fields_madt_local_apic_addr_override[] = { FIELD_UINT(" Local APIC Addr", fwts_acpi_madt_local_apic_addr_override, address), FIELD_END }; @@ -853,7 +852,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) } break; case 6: { - fwts_acpidump_field fields_madt_io_sapic[] = { + static fwts_acpidump_field fields_madt_io_sapic[] = { FIELD_UINT(" I/O SAPIC ID", fwts_acpi_madt_io_sapic, io_sapic_id), FIELD_UINT(" Gbl Sys Int", fwts_acpi_madt_io_sapic, gsi), FIELD_UINT(" I/O SAPIC Addr", fwts_acpi_madt_io_sapic, address), @@ -866,7 +865,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) break; case 7: { fwts_acpi_madt_local_sapic *local_sapic = (fwts_acpi_madt_local_sapic*)data; - fwts_acpidump_field fields_madt_local_sapic[] = { + static fwts_acpidump_field fields_madt_local_sapic[] = { FIELD_UINT(" ACPI CPU ID", fwts_acpi_madt_local_sapic, acpi_processor_id), FIELD_UINT(" Local SAPIC ID", fwts_acpi_madt_local_sapic, local_sapic_id), FIELD_UINT(" Local SAPIC EID", fwts_acpi_madt_local_sapic, local_sapic_eid), @@ -882,7 +881,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) } break; case 8: { - fwts_acpidump_field fields_madt_local_sapic[] = { + static fwts_acpidump_field fields_madt_local_sapic[] = { FIELD_UINT(" Flags", fwts_acpi_madt_platform_int_source, flags), FIELD_UINT(" Type", fwts_acpi_madt_platform_int_source, type), FIELD_UINT(" Processor ID", fwts_acpi_madt_platform_int_source, processor_id), @@ -898,7 +897,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) } break; case 9: { - fwts_acpidump_field fields_madt_local_x2apic[] = { + static fwts_acpidump_field fields_madt_local_x2apic[] = { FIELD_UINT(" x2APIC ID", fwts_acpi_madt_local_x2apic, x2apic_id), FIELD_UINT(" Flags", fwts_acpi_madt_local_x2apic, flags), FIELD_UINT(" Processor UID", fwts_acpi_madt_local_x2apic, processor_uid), @@ -910,7 +909,7 @@ static void acpidump_madt(fwts_framework *fw, fwts_acpi_table_info *table) } break; case 10: { - fwts_acpidump_field fields_madt_local_x2apic_nmi[] = { + static fwts_acpidump_field fields_madt_local_x2apic_nmi[] = { FIELD_UINT(" Flags", fwts_acpi_madt_local_x2apic_nmi, flags), FIELD_UINT(" Processor UID", fwts_acpi_madt_local_x2apic_nmi, processor_uid), FIELD_UINT(" LINT#", fwts_acpi_madt_local_x2apic_nmi, local_x2apic_lint), @@ -940,7 +939,7 @@ static void acpidump_mcfg(fwts_framework *fw, fwts_acpi_table_info *table) int n; int i; - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_UINT("Base Address", fwts_acpi_table_mcfg, base_address), FIELD_UINT("Base Reserved", fwts_acpi_table_mcfg, base_reserved), FIELD_END @@ -1012,7 +1011,7 @@ static void acpidump_srat(fwts_framework *fw, fwts_acpi_table_info *table) while (ptr < data + length) { switch (*ptr) { case 0: { - fwts_acpidump_field fields_lasa[] = { + static fwts_acpidump_field fields_lasa[] = { FIELD_UINT(" Proximity [7:0]", fwts_acpi_table_slit_local_apic_sapic_affinity, proximity_domain_0), FIELD_UINT(" APIC ID", fwts_acpi_table_slit_local_apic_sapic_affinity, apic_id), FIELD_UINT(" Flags", fwts_acpi_table_slit_local_apic_sapic_affinity, flags), @@ -1030,7 +1029,7 @@ static void acpidump_srat(fwts_framework *fw, fwts_acpi_table_info *table) } break; case 1: { - fwts_acpidump_field fields_ma[] = { + static fwts_acpidump_field fields_ma[] = { FIELD_UINT(" Prox. Domain", fwts_acpi_table_slit_memory_affinity, proximity_domain), FIELD_UINT(" Base Addr Lo", fwts_acpi_table_slit_memory_affinity, base_addr_lo), FIELD_UINT(" Base Addr Hi", fwts_acpi_table_slit_memory_affinity, base_addr_hi), @@ -1046,7 +1045,7 @@ static void acpidump_srat(fwts_framework *fw, fwts_acpi_table_info *table) } break; case 2: { - fwts_acpidump_field fields_xa[] = { + static fwts_acpidump_field fields_xa[] = { FIELD_UINT(" Prox. Domain", fwts_acpi_table_slit_local_x2apic_affinity, proximity_domain), FIELD_UINT(" X2APIC ID", fwts_acpi_table_slit_local_x2apic_affinity, x2apic_id), FIELD_UINT(" Flags", fwts_acpi_table_slit_local_x2apic_affinity, flags), @@ -1071,7 +1070,7 @@ static void acpidump_tcpa(fwts_framework *fw, fwts_acpi_table_info *table) uint8_t *data = (uint8_t *)table->data; size_t length = table->length; - fwts_acpidump_field fields[] = { + static fwts_acpidump_field fields[] = { FIELD_UINT("Reserved", fwts_acpi_table_tcpa, reserved), FIELD_UINT("Log Zone Length", fwts_acpi_table_tcpa, log_zone_length), FIELD_UINT("Log Zone Address", fwts_acpi_table_tcpa, log_zone_addr),