diff options
| author | Robin Murphy <robin.murphy@arm.com> | 2024-09-02 18:52:00 +0100 |
|---|---|---|
| committer | Will Deacon <will@kernel.org> | 2024-09-04 16:04:08 +0100 |
| commit | ff436cee694ee8bc4173f2d42622ee7c17a085d3 (patch) | |
| tree | df0b3889161790fe5817be56cabae35a67ac9dc7 /drivers/perf | |
| parent | 359414b33e00bae91e4eabf3e4ef8e76024c7673 (diff) | |
| download | lwn-ff436cee694ee8bc4173f2d42622ee7c17a085d3.tar.gz lwn-ff436cee694ee8bc4173f2d42622ee7c17a085d3.zip | |
perf/arm-cmn: Improve build-time assertion
These days we can use static_assert() in the logical place rather than
jamming a BUILD_BUG_ON() into the nearest function scope.
Signed-off-by: Robin Murphy <robin.murphy@arm.com>
Link: https://lore.kernel.org/r/224ee8286f299100f1c768edb254edc898539f50.1725296395.git.robin.murphy@arm.com
Signed-off-by: Will Deacon <will@kernel.org>
Diffstat (limited to 'drivers/perf')
| -rw-r--r-- | drivers/perf/arm-cmn.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/perf/arm-cmn.c b/drivers/perf/arm-cmn.c index 48863b31ccfb..5b0edeb69394 100644 --- a/drivers/perf/arm-cmn.c +++ b/drivers/perf/arm-cmn.c @@ -581,6 +581,7 @@ struct arm_cmn_hw_event { bool wide_sel; enum cmn_filter_select filter_sel; }; +static_assert(sizeof(struct arm_cmn_hw_event) <= offsetof(struct hw_perf_event, target)); #define for_each_hw_dn(hw, dn, i) \ for (i = 0, dn = hw->dn; i < hw->num_dns; i++, dn++) @@ -591,7 +592,6 @@ struct arm_cmn_hw_event { static struct arm_cmn_hw_event *to_cmn_hw(struct perf_event *event) { - BUILD_BUG_ON(sizeof(struct arm_cmn_hw_event) > offsetof(struct hw_perf_event, target)); return (struct arm_cmn_hw_event *)&event->hw; } |