[PATCH] acpi: pcc: New test to sanity check PCC, (LP: #863175)

Keng-Yu Lin kengyu at canonical.com
Thu Nov 22 06:21:28 UTC 2012


On Wed, Nov 21, 2012 at 1:05 AM, Colin King <colin.king at canonical.com> wrote:
> From: Colin Ian King <colin.king at canonical.com>
>
> This adds a test to sanity check the Processor Clocking Control (PCC).
> PCC seems to be found on just a few HP Proliant machines at the moment,
> for example:
>
>   HP ProLiant DL380 G6
>   HP ProLiant DL380 G7
>   HP ProLiant BL460c G7
>   HP ProLiant BL460c G7
>
> and probably a bunch more HP kit besides.  I've only found 4
> different machines that have the PCCH out of ~6000 kernel related
> bugs filed in LaunchPad, so this really is for a small handful
> of machines.
>
> This test evaluates PCCH and sanity checks the data structures.
> We can go a bit further and determine the PCC Header and memory
> map this and sanity check this too, however, for this initial
> version I have disabled this functionality because I've not got
> any hardware to test this on and also I am concerned that reading
> an incorrectly specified header region may create some issues.
> I will enable this once I can test this on real hardware.
>
> For more information on PCC, see:
> http://acpica.org/download/Processor-Clocking-Control-v1p0.pdf
>
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>  src/Makefile.am    |    1 +
>  src/acpi/pcc/pcc.c |  469 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 470 insertions(+)
>  create mode 100644 src/acpi/pcc/pcc.c
>
> diff --git a/src/Makefile.am b/src/Makefile.am
> index e82c7a9..478c522 100644
> --- a/src/Makefile.am
> +++ b/src/Makefile.am
> @@ -36,6 +36,7 @@ fwts_SOURCES = main.c \
>         acpi/lid/lid.c \
>         acpi/powerbutton/powerbutton.c \
>         acpi/wmi/wmi.c \
> +       acpi/pcc/pcc.c \
>         bios/ebda_region/ebda_region.c \
>         bios/ebdadump/ebdadump.c \
>         bios/mtrr/mtrr.c \
> diff --git a/src/acpi/pcc/pcc.c b/src/acpi/pcc/pcc.c
> new file mode 100644
> index 0000000..c3b6a38
> --- /dev/null
> +++ b/src/acpi/pcc/pcc.c
> @@ -0,0 +1,469 @@
> +/*
> + * Copyright (C) 2010-2012 Canonical
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License
> + * as published by the Free Software Foundation; either version 2
> + * of the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write to the Free Software
> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
> + *
> + */
> +#include "fwts.h"
> +
> +#include <stdio.h>
> +#include <stdlib.h>
> +#include <string.h>
> +#include <unistd.h>
> +#include <inttypes.h>
> +
> +/* acpica headers */
> +#include "acpi.h"
> +#include "fwts_acpi_method.h"
> +
> +/*
> + *  This test does some sanity checking on the PCC interface,
> + *  see http://acpica.org/download/Processor-Clocking-Control-v1p0.pdf
> + */
> +
> +#define PCC_HDR_SIGNATURE      0x24504343      /* $PCC */
> +
> +/*
> + * For the moment, we turn this off as I am concerned that reads of this region
> + * may cause issues.
> + */
> +#define CHECK_PCC_HDR          0
> +
> +typedef struct {
> +       uint8_t  descriptor;
> +       uint8_t  length;
> +       uint8_t  space_id;
> +       uint8_t  resource_usage;
> +       uint8_t  type_specific;
> +       uint64_t granularity;
> +       uint64_t minimum;
> +       uint64_t maximum;
> +       uint64_t translation_offset;
> +       uint64_t address_length;
> +} __attribute__ ((packed)) fwts_pcc_memory_resource;
> +
> +typedef struct  {
> +       uint8_t  descriptor;
> +       uint16_t length;
> +       uint8_t  space_id;
> +       uint8_t  bit_width;
> +       uint8_t  bit_offset;
> +       uint8_t  access_size;
> +       uint64_t address;
> +} __attribute__ ((packed)) fwts_pcc_register_resource;
> +
> +typedef struct {
> +       uint32_t signature;
> +       uint16_t length;
> +       uint8_t  major;
> +       uint8_t  minor;
> +       uint32_t features;
> +       uint16_t command;
> +       uint16_t status;
> +       uint32_t latency;
> +       uint32_t minimum_time;
> +       uint32_t maximum_time;
> +       uint32_t nominal;
> +       uint32_t throttled_frequency;
> +       uint32_t minimum_frequency;
> +} __attribute__ ((packed)) fwts_pcc_header;
> +
> +/*
> + *  pcc_init()
> + *     initialize ACPI
> + */
> +static int pcc_init(fwts_framework *fw)
> +{
> +       if (fwts_method_init(fw) != FWTS_OK)
> +               return FWTS_ERROR;
> +
> +       return FWTS_OK;
> +}
> +
> +/*
> + *  pcc_deinit
> + *     de-intialize ACPI
> + */
> +static int pcc_deinit(fwts_framework *fw)
> +{
> +       return fwts_method_deinit(fw);
> +}
> +
> +#if CHECK_PCC_HDR
> +static void pcc_check_pcc_header(
> +       fwts_framework *fw,
> +       uint64_t addr,
> +       uint64_t length,
> +       bool *failed)
> +{
> +       fwts_pcc_header *hdr;
> +
> +       hdr = (fwts_pcc_header *)fwts_mmap((off_t)addr, (size_t)length);
> +       if (hdr == NULL) {
> +               fwts_log_info(fw, "Failed to memory map PCC header 0x%" PRIx64
> +                       "..0x%" PRIx64 ".", addr, addr + length);
> +               return;
> +       }
> +
> +       fwts_log_info_verbatum(fw, "PCC header at 0x%" PRIx64 ".", addr);
> +       fwts_log_info_verbatum(fw, "  Signature:          0x%" PRIx32, hdr->signature);
> +       fwts_log_info_verbatum(fw, "  Length:             0x%" PRIx16, hdr->length);
> +       fwts_log_info_verbatum(fw, "  Major:              0x%" PRIx8,  hdr->major);
> +       fwts_log_info_verbatum(fw, "  Minor:              0x%" PRIx8,  hdr->minor);
> +       fwts_log_info_verbatum(fw, "  Features:           0x%" PRIx32, hdr->features);
> +       fwts_log_info_verbatum(fw, "  Commend:            0x%" PRIx16, hdr->command);
> +       fwts_log_info_verbatum(fw, "  Status:             0x%" PRIx16, hdr->status);
> +       fwts_log_info_verbatum(fw, "  Latency:            0x%" PRIx32, hdr->latency);
> +       fwts_log_info_verbatum(fw, "  Minimum Time:       0x%" PRIx32, hdr->minimum_time);
> +       fwts_log_info_verbatum(fw, "  Maximum Time:       0x%" PRIx32, hdr->maximum_time);
> +       fwts_log_info_verbatum(fw, "  Nominal:            0x%" PRIx32, hdr->nominal);
> +       fwts_log_info_verbatum(fw, "  Throttled Freq.:    0x%" PRIx32, hdr->throttled_frequency);
> +       fwts_log_info_verbatum(fw, "  Minimum Freq.:      0x%" PRIx32, hdr->minimum_frequency);
> +
> +       fwts_munmap(hdr, (size_t)length);
> +       fwts_log_nl(fw);
> +
> +       if (hdr->signature != PCC_HDR_SIGNATURE) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHdrSignatureError",
> +                       "The PCC Header Signature is not a valid PCC signature, was expecting "
> +                       "0x%" PRIx32 " ($PCC), got instead 0x%" PRIx32,
> +                       PCC_HDR_SIGNATURE, hdr->signature);
> +               *failed = true;
> +       }
> +}
> +#endif
> +
> +static void pcc_check_shared_memory_region(
> +       fwts_framework *fw,
> +       const char *name,
> +       ACPI_OBJECT *pcc_obj,
> +       bool *failed)
> +{
> +       fwts_pcc_memory_resource *pcc_mr;
> +
> +       if (pcc_obj->Type != ACPI_TYPE_BUFFER) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHElementZeroNotBuffer",
> +                       "PCCH object %s returned a package with element zero "
> +                       "was not an ACPI_BUFFER. This does not conform to the "
> +                       "PCC specification.", name);
> +               *failed = true;
> +               return;
> +       }
> +
> +       if (pcc_obj->Buffer.Pointer == NULL) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHElementZeroBufferNull",
> +                       "PCCH object %s returned a package with element zero "
> +                       "which was an ACPI_BUFFER, however, the buffer pointer "
> +                       "was NULL. This does not conform to the PCC "
> +                       "specification.", name);
> +               *failed = true;
> +               return;
> +       }
> +
> +       if (pcc_obj->Buffer.Length < sizeof(fwts_pcc_memory_resource)) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHMemoryResourceIllegalSize",
> +                       "PCCH object %s returned a PCC memory resource buffer "
> +                       "which was the wrong size. Got %" PRIu32 " bytes, "
> +                       "expected %zu bytes.", name,
> +                       pcc_obj->Buffer.Length, sizeof(fwts_pcc_memory_resource));
> +               *failed = true;
> +               return;
> +       }
> +
> +       pcc_mr = (fwts_pcc_memory_resource *)pcc_obj->Buffer.Pointer;
> +
> +       fwts_log_info_verbatum(fw, "PCC Memory Resource (Shared Memory Region) for %s:", name);
> +       fwts_log_info_verbatum(fw, "  Descriptor:         0x%" PRIx8, pcc_mr->descriptor);
> +       fwts_log_info_verbatum(fw, "  Length:             0x%" PRIx8, pcc_mr->length);
> +       fwts_log_info_verbatum(fw, "  Space ID:           0x%" PRIx8, pcc_mr->space_id);
> +       fwts_log_info_verbatum(fw, "  Resource Usage:     0x%" PRIx8, pcc_mr->resource_usage);
> +       fwts_log_info_verbatum(fw, "  Type Specific:      0x%" PRIx8, pcc_mr->type_specific);
> +       fwts_log_info_verbatum(fw, "  Minimum:            0x%" PRIx64, pcc_mr->minimum);
> +       fwts_log_info_verbatum(fw, "  Maximum:            0x%" PRIx64, pcc_mr->maximum);
> +       fwts_log_info_verbatum(fw, "  Translation Offset: 0x%" PRIx64, pcc_mr->translation_offset);
> +       fwts_log_info_verbatum(fw, "  Address Length:     0x%" PRIx64, pcc_mr->address_length);
> +
> +       if (pcc_mr->space_id != ACPI_ADR_SPACE_SYSTEM_MEMORY) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCMemoryResourceSpaceIdWrongType",
> +                       "PCC Memory Resource Space ID is of the wrong type, got 0x%" PRIx8
> +                       ", expected to get type 0x%" PRIx8 " (ACPI_ADR_SPACE_SYSTEM_MEMORY).",
> +                       pcc_mr->space_id, ACPI_ADR_SPACE_SYSTEM_MEMORY);
> +               *failed = true;
> +       }
> +
> +       if (pcc_mr->length == 0) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCMemoryResourceAddrLength",
> +                       "PCC Memory Resource Address Length is zero, this is clearly incorrect.");
> +               *failed = true;
> +       }
> +
> +       if (pcc_mr->minimum & 4095) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCMemoryNotPageAligned",
> +                       "PCC Memory Resource Minumum Address is not page aligned. It must "
> +                       "start on a 4K page boundary.");
> +               *failed = true;
> +       }
> +
> +       /* TODO: We should also check if the region is in the e820 region too */
> +
> +       if (pcc_mr->minimum == 0) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCMemoryResourceMinAddr",
> +                       "PCC Memory Resource Minimum Address is zero, this is clearly incorrect.");
> +               *failed = true;
> +       }
> +
> +       if (pcc_mr->maximum == 0) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCMemoryResourceMaxAddr",
> +                       "PCC Memory Resource Maximum Address is zero, this is clearly incorrect.");
> +               *failed = true;
> +       }
> +
> +       if (pcc_mr->minimum >= pcc_mr->maximum) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCMemoryResourceMinMaxAddrError",
> +                       "PCC Memory Resource Minimum Address should be less than "
> +                       "the Maximum Address: Min: 0x%" PRIx64 ", Max: 0x%" PRIx64,
> +                       pcc_mr->minimum, pcc_mr->maximum);
> +               *failed = true;
> +       }
> +
> +       fwts_log_nl(fw);
> +
> +#if CHECK_PCC_HDR
> +       pcc_check_pcc_header(fw, pcc_mr->minimum, pcc_mr->length, failed);
> +#endif
> +}
> +
> +static void pcc_check_doorbell_address(
> +       fwts_framework *fw,
> +       const char *name,
> +       ACPI_OBJECT *pcc_obj,
> +       bool *failed)
> +{
> +       fwts_pcc_register_resource *pcc_rr;
> +
> +       if (pcc_obj->Type != ACPI_TYPE_BUFFER) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHElementOneNotBuffer",
> +                       "PCCH object %s returned a package with element zero "
> +                       "was not an ACPI_BUFFER. This does not conform to the "
> +                       "PCC specification.", name);
> +               *failed = true;
> +               return;
> +       }
> +
> +       if (pcc_obj->Buffer.Pointer == NULL) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHElementOneBufferNull",
> +                       "PCCH object %s returned a package with element one "
> +                       "which was an ACPI_BUFFER, however, the buffer pointer "
> +                       "was NULL. This does not conform to the PCC "
> +                       "specification.", name);
> +               *failed = true;
> +               return;
> +       }
> +
> +       if (pcc_obj->Buffer.Length < sizeof(fwts_pcc_register_resource)) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHRegisterResourceIllegalSize",
> +                       "PCCH object %s returned a PCC register resource buffer "
> +                       "which was the wrong size. Got %" PRIu32 " bytes, "
> +                       "expected %zu bytes.", name,
> +                       pcc_obj->Buffer.Length, sizeof(fwts_pcc_register_resource));
> +               *failed = true;
> +               return;
> +       }
> +
> +       pcc_rr = (fwts_pcc_register_resource *)pcc_obj->Buffer.Pointer;
> +
> +       fwts_log_info_verbatum(fw, "PCC Register Resource (Doorbell) for %s:", name);
> +       fwts_log_info_verbatum(fw, "  Descriptor:         0x%" PRIx8, pcc_rr->descriptor);
> +       fwts_log_info_verbatum(fw, "  Length:             0x%" PRIx8, pcc_rr->length);
> +       fwts_log_info_verbatum(fw, "  Space ID:           0x%" PRIx8, pcc_rr->space_id);
> +       fwts_log_info_verbatum(fw, "  Bit Width:          0x%" PRIx8, pcc_rr->bit_width);
> +       fwts_log_info_verbatum(fw, "  Bit Offset:         0x%" PRIx8, pcc_rr->bit_offset);
> +       fwts_log_info_verbatum(fw, "  Access Size:        0x%" PRIx8, pcc_rr->access_size);
> +       fwts_log_info_verbatum(fw, "  Address:            0x%" PRIx64, pcc_rr->address);
> +
> +       if (pcc_rr->space_id != ACPI_ADR_SPACE_SYSTEM_IO) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCRegisterResourceSpaceIdWrongType",
> +                       "PCC Register Resource Space ID is of the wrong type, got 0x%" PRIx8
> +                       ", expected to get type 0x%" PRIx8 " (ACPI_ADR_SPACE_SYSTEM_IO).",
> +                       pcc_rr->space_id, ACPI_ADR_SPACE_SYSTEM_IO);
> +               *failed = true;
> +       }
> +
> +       if ((pcc_rr->bit_width < 1) || (pcc_rr->bit_width > 32)) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCRegisterResourceBitWidthError",
> +                       "PCC Register Resource Bit Width is incorrect, got 0x%" PRIx8,
> +                       pcc_rr->bit_width);
> +               *failed = true;
> +       }
> +
> +       if (pcc_rr->address == 0) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCRegisterResourceAddressZero",
> +                       "PCC Register Resource Address is incorrect, got 0x%" PRIx64,
> +                       pcc_rr->address);
> +               *failed = true;
> +       }
> +
> +       fwts_log_nl(fw);
> +}
> +
> +static void pcc_check_doorbell_preserve_mask(
> +       fwts_framework *fw,
> +       const char *name,
> +       ACPI_OBJECT *pcc_obj,
> +       bool *failed)
> +{
> +       if (pcc_obj->Type != ACPI_TYPE_INTEGER) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHElementTwoNotInteger",
> +                       "PCCH object %s returned a package with element two "
> +                       "was not an ACPI_INTEGER. This does not conform to the "
> +                       "PCC specification.", name);
> +               *failed = true;
> +               return;
> +       }
> +
> +       fwts_log_info_verbatum(fw, "PCC Doorbell Preserve Mask for %s:", name);
> +       fwts_log_info_verbatum(fw, "  Preserve Mask:      0x%" PRIx64, pcc_obj->Integer.Value);
> +       fwts_log_nl(fw);
> +}
> +
> +static void pcc_check_doorbell_write_mask(
> +       fwts_framework *fw,
> +       const char *name,
> +       ACPI_OBJECT *pcc_obj,
> +       bool *failed)
> +{
> +       if (pcc_obj->Type != ACPI_TYPE_INTEGER) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHElementTwoNotInteger",
> +                       "PCCH object %s returned a package with element three "
> +                       "was not an ACPI_INTEGER. This does not conform to the "
> +                       "PCC specification.", name);
> +               *failed = true;
> +               return;
> +       }
> +
> +       fwts_log_info_verbatum(fw, "PCC Doorbell Write Mask for %s:", name);
> +       fwts_log_info_verbatum(fw, "  Write Mask:         0x%" PRIx64, pcc_obj->Integer.Value);
> +       fwts_log_nl(fw);
> +
> +       if (pcc_obj->Integer.Value == 0) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCDoorBellWriteMaskZero",
> +                       "PCC Doorbell Write Mask is incorrect, got 0x%" PRIx64,
> +                       pcc_obj->Integer.Value);
> +               *failed = true;
> +       }
> +}
> +
> +static void pcc_check_buffer(
> +       fwts_framework *fw,
> +       const char *name,
> +       ACPI_BUFFER *buf)
> +{
> +       ACPI_OBJECT *obj;
> +       bool failed = false;
> +
> +       if (buf->Pointer == NULL) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHNullPointer",
> +                       "PCCH object %s returned a NULL pointer when evaluated.", name);
> +               return;
> +       }
> +
> +       obj = buf->Pointer;
> +
> +       if (obj->Type != ACPI_TYPE_PACKAGE) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHNotPackage",
> +                       "PCCH object %s did not return an ACPI_PACKAGE when evaluated.", name);
> +               return;
> +       }
> +
> +       if (obj->Package.Count != 4) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCHExpectedTwoElements",
> +                       "PCCH object %s did not return an ACPI_PACKAGE "
> +                       "that contained two elements, got %" PRIu32 " instead.",
> +                       name, obj->Package.Count);
> +               return;
> +       }
> +
> +       pcc_check_shared_memory_region(fw, name, &obj->Package.Elements[0], &failed);
> +       pcc_check_doorbell_address(fw, name, &obj->Package.Elements[1], &failed);
> +       pcc_check_doorbell_preserve_mask(fw, name, &obj->Package.Elements[2], &failed);
> +       pcc_check_doorbell_write_mask(fw, name, &obj->Package.Elements[3], &failed);
> +
> +       if (!failed)
> +               fwts_passed(fw, "PCC pased; %s returned sane looking data structures.", name);
> +}
> +
> +static int pcc_test1(fwts_framework *fw)
> +{
> +       ACPI_BUFFER       buf;
> +       ACPI_STATUS       ret;
> +       fwts_list_link  *item;
> +       fwts_list *pccs;
> +       static char *name = "PCCH";
> +       size_t name_len = strlen(name);
> +       int count = 0;
> +
> +       fwts_log_info(fw,
> +               "This test checks the sanity of the Processor Clocking Control "
> +               "as found on some HP ProLiant machines.  Most computers do not "
> +               "use this interface to control the CPU clock frequency, so this "
> +               "test will be skipped.");
> +       fwts_log_nl(fw);
> +
> +       if ((pccs = fwts_method_get_names()) != NULL) {
> +               fwts_list_foreach(item, pccs) {
> +                       char *pcc_name = fwts_list_data(char*, item);
> +                       size_t len = strlen(pcc_name);
> +
> +                       if (strncmp(name, pcc_name + len - name_len, name_len) == 0) {
> +                               ret = fwts_method_evaluate(fw, pcc_name, NULL, &buf);
> +                               if (ACPI_FAILURE(ret) == AE_OK) {
> +                                       pcc_check_buffer(fw, pcc_name, &buf);
> +                                       count++;
> +
> +                                       if (buf.Length && buf.Pointer)
> +                                               free(buf.Pointer);
> +                               }
> +                       }
> +               }
> +       }
> +
> +       if (count > 1) {
> +               fwts_failed(fw, LOG_LEVEL_HIGH, "PCCTooManyPCCHObjects",
> +                       "The firmware contains too many PCCH objects, expected 1, got %d.", count);
> +       }
> +
> +       /* Nothing found, this is not an error */
> +       if (count == 0) {
> +               fwts_log_info(fw, "This machine does not use Processor Clocking Control (PCC).");
> +               fwts_infoonly(fw);
> +       }
> +
> +       return FWTS_OK;
> +}
> +
> +/* Just one big test */
> +static fwts_framework_minor_test pcc_tests[] = {
> +       { pcc_test1, "Check PCCH." },
> +       { NULL, NULL }
> +};
> +
> +static fwts_framework_ops pcc_ops = {
> +       .description = "Processor Clocking Control (PCC) Test.",
> +       .init        = pcc_init,
> +       .deinit      = pcc_deinit,
> +       .minor_tests = pcc_tests
> +};
> +
> +FWTS_REGISTER(pcc, &pcc_ops, FWTS_TEST_ANYTIME, FWTS_FLAG_BATCH);
> --
> 1.7.10.4
>
Acked-by: Keng-Yu Lin <kengyu at canonical.com>



More information about the fwts-devel mailing list