mbox series

[0/2] scsi: aacraid: struct sgmap: Replace 1-element arrays with flexible arrays

Message ID 20240711212732.work.162-kees@kernel.org
Headers show
Series scsi: aacraid: struct sgmap: Replace 1-element arrays with flexible arrays | expand

Message

Kees Cook July 11, 2024, 9:57 p.m. UTC
Hi,

This replaces some of the last remaining uses in the kernel of 1-element
"fake" flexible arrays with modern C99 flexible arrays. Some refactoring
is done to ease this, and binary differences are identified. For the
on stack size changes in patch 2, the "yes, that is the source of the binary
differences" debugging patch can be found here[1].

Thanks!

-Kees

[1] https://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git/commit/?h=dev/v6.10-rc2/1-element&id=45e6226bcbc5e982541754eca7ac29f403e82f5e

Kees Cook (2):
  scsi: aacraid: Rearrange order of struct aac_srb_unit
  scsi: aacraid: struct {user,}sgmap{,64,raw}: Replace 1-element arrays
    with flexible arrays

 drivers/scsi/aacraid/aachba.c   | 26 ++++++++++++--------------
 drivers/scsi/aacraid/aacraid.h  | 17 ++++++-----------
 drivers/scsi/aacraid/commctrl.c |  4 ++--
 drivers/scsi/aacraid/comminit.c |  3 +--
 drivers/scsi/aacraid/commsup.c  |  5 +++--
 5 files changed, 24 insertions(+), 31 deletions(-)

Comments

Martin K. Petersen Aug. 3, 2024, 1:40 a.m. UTC | #1
Kees,

> This replaces some of the last remaining uses in the kernel of
> 1-element "fake" flexible arrays with modern C99 flexible arrays. Some
> refactoring is done to ease this, and binary differences are
> identified. For the on stack size changes in patch 2, the "yes, that
> is the source of the binary differences" debugging patch can be found
> here[1].

Applied to 6.12/scsi-staging, thanks!
Martin K. Petersen Aug. 5, 2024, 9:17 p.m. UTC | #2
On Thu, 11 Jul 2024 14:57:36 -0700, Kees Cook wrote:

> This replaces some of the last remaining uses in the kernel of 1-element
> "fake" flexible arrays with modern C99 flexible arrays. Some refactoring
> is done to ease this, and binary differences are identified. For the
> on stack size changes in patch 2, the "yes, that is the source of the binary
> differences" debugging patch can be found here[1].
> 
> Thanks!
> 
> [...]

Applied to 6.12/scsi-queue, thanks!

[1/2] scsi: aacraid: Rearrange order of struct aac_srb_unit
      https://git.kernel.org/mkp/scsi/c/6e5860b0ad49
[2/2] scsi: aacraid: struct {user,}sgmap{,64,raw}: Replace 1-element arrays with flexible arrays
      https://git.kernel.org/mkp/scsi/c/fdb1db6ea7f6