Changeset View
Changeset View
Standalone View
Standalone View
sys/arm64/include/pmc_mdep.h
Show All 32 Lines | |||||
/* | /* | ||||
* On the ARMv8 platform we support the following PMCs. | * On the ARMv8 platform we support the following PMCs. | ||||
* | * | ||||
* ARMV8 ARM Cortex-A53/57/72 processors | * ARMV8 ARM Cortex-A53/57/72 processors | ||||
*/ | */ | ||||
#include <dev/hwpmc/hwpmc_arm64.h> | #include <dev/hwpmc/hwpmc_arm64.h> | ||||
union pmc_md_op_pmcallocate { | union pmc_md_op_pmcallocate { | ||||
struct { | |||||
uint32_t pm_md_config; | |||||
uint32_t pm_md_flags; | uint32_t pm_md_flags; | ||||
#define PM_MD_RAW_EVENT 0x1 | #define PM_MD_RAW_EVENT 0x1 | ||||
uint32_t __pad32; | }; | ||||
uint64_t __pad[3]; | uint64_t __pad[4]; | ||||
mhorne: Assuming there's no issue with using C11 anonymous structs... I found at least one other recent… | |||||
Not Done Inline ActionsIs this used by userspace? It's a union, so this would change the size. (The old code seems to assume it's a struct) andrew: Is this used by userspace? It's a union, so this would change the size. (The old code seems to… | |||||
Done Inline ActionsYes, this is visible to userspace. It was my recent change 8cc3815f02be that mistakenly treated it as a struct, so this restores the previous size. mhorne: Yes, this is visible to userspace. It was my recent change 8cc3815f02be that mistakenly treated… | |||||
}; | }; | ||||
/* Logging */ | /* Logging */ | ||||
#define PMCLOG_READADDR PMCLOG_READ64 | #define PMCLOG_READADDR PMCLOG_READ64 | ||||
#define PMCLOG_EMITADDR PMCLOG_EMIT64 | #define PMCLOG_EMITADDR PMCLOG_EMIT64 | ||||
#ifdef _KERNEL | #ifdef _KERNEL | ||||
union pmc_md_pmc { | union pmc_md_pmc { | ||||
Show All 18 Lines |
Assuming there's no issue with using C11 anonymous structs... I found at least one other recent commit doing so.