Re: [PATCH 2/30] ACPI-processor: Improve two size determinations in acpi_processor_get_performance_s
From: Borislav Petkov <bp@alien8.de>
Date: 2016-09-10 11:54:35
Also in:
linux-acpi, lkml
On Sat, Sep 10, 2016 at 11:36:41AM +0200, SF Markus Elfring wrote:
quoted hunk ↗ jump to hunk
From: Markus Elfring <redacted> Date: Tue, 6 Sep 2016 11:11:12 +0200 Replace the specification of a data structure by pointer dereferences as the parameter for the operator "sizeof" to make the corresponding size determination a bit safer according to the Linux coding style convention. Signed-off-by: Markus Elfring <redacted> --- drivers/acpi/processor_perflib.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)diff --git a/drivers/acpi/processor_perflib.c b/drivers/acpi/processor_perflib.c index 78f9025..004e24c 100644 --- a/drivers/acpi/processor_perflib.c +++ b/drivers/acpi/processor_perflib.c@@ -358,7 +358,7 @@ static int acpi_processor_get_performance_states(struct acpi_processor *pr) struct acpi_processor_px *px = &(pr->performance->states[i]); - state.length = sizeof(struct acpi_processor_px); + state.length = sizeof(*px);
How is this better? Now the person reading the code has to go and lookup what px is in order to know what its size is and which goes in there. Sure, it is a couple of lines up but this is not always the case...
quoted hunk ↗ jump to hunk
state.pointer = px; ACPI_DEBUG_PRINT((ACPI_DB_INFO, "Extracting state %d\n", i));@@ -400,7 +400,8 @@ static int acpi_processor_get_performance_states(struct acpi_processor *pr) * Copy this valid entry over last_invalid entry */ memcpy(&(pr->performance->states[last_invalid]), - px, sizeof(struct acpi_processor_px)); + px, + sizeof(*px)); ++last_invalid;
... like here, for example.
In any case, I won't take this one if I were the maintainer.
--
Regards/Gruss,
Boris.
ECO tip #101: Trim your mails when you reply.