EDAC/device: Get rid of the silly one-shot memory allocation in edac_device_alloc_ctl...
authorBorislav Petkov <bp@suse.de>
Tue, 8 Mar 2022 13:16:17 +0000 (14:16 +0100)
committerBorislav Petkov <bp@suse.de>
Mon, 11 Apr 2022 09:43:26 +0000 (11:43 +0200)
commit9fb9ce392aae0c6654efc42c80e2f6bab88d5fe3
tree936ad0f870760d221b85e65aadff185ab3312a32
parentfb8cd45ca39bdb216e358e36063d8b9962242107
EDAC/device: Get rid of the silly one-shot memory allocation in edac_device_alloc_ctl_info()

Use boring kzalloc() instead. Add pointers to the different allocated
members in struct edac_device_ctl_info for easier freeing later.

One of the reasons, perhaps, why it was done this way is to be able to
do a single kfree(ctl_info) without having to kfree() the other parts of
the struct too but that is not nearly a sensible reason as to why there
should be this obscure pointer alignment.

There should be no functional changes resulting from this.

Signed-off-by: Borislav Petkov <bp@suse.de>
Link: https://lore.kernel.org/r/20220310095254.1510-4-bp@alien8.de
drivers/edac/edac_device.c
drivers/edac/edac_device.h
drivers/edac/edac_device_sysfs.c