Re: [PATCH RFC 0/6] LLVMLinux: Patches to enable the kernel to be compiled with clang/LLVM

[Behan Webster]

On 09/02/14 16:01, Marcel Holtmann wrote:
> Hi Behan,
>
> > These patches remove the use of Variable Length Arrays In Structs (VLAIS) in
> > crypto related code. Presented here for comments as a whole (since they all do
> > the same thing in the same way). Once everyone is happy I will submit them
> > individually to their appropriate maintainers.
> >
> > The LLVMLinux project aims to fully build the Linux kernel using both gcc and
> > clang (the C front end for the LLVM compiler infrastructure project).
> >
> >
> > Jan-Simon Möller (4):
> >   crypto, dm: LLVMLinux: Remove VLAIS usage from dm-crypt
> >   crypto: LLVMLinux: Remove VLAIS usage from crypto/hmac.c
> >   crypto: LLVMLinux: Remove VLAIS usage from libcrc32c.c
> >   crypto: LLVMLinux: Remove VLAIS usage from crypto/testmgr.c
> >
> > Vinícius Tinti (2):
> >   apparmor: LLVMLinux: Remove VLAIS
> >   btrfs: LLVMLinux: Remove VLAIS
> >
> > crypto/hmac.c              | 27 +++++++++++++--------------
> > crypto/testmgr.c           | 16 ++++++++--------
> > drivers/md/dm-crypt.c      | 38 ++++++++++++++++++--------------------
> > fs/btrfs/hash.c            | 18 +++++++++---------
> > lib/libcrc32c.c            | 18 +++++++++---------
> > security/apparmor/crypto.c | 19 +++++++++----------
> > 6 files changed, 66 insertions(+), 70 deletions(-)
> are you sure these are all of them? I know for a fact that we are using the same construct in net/bluetooth/amp.c as well.
There have been other places where this was an issue before too (ext4, 
mac80211, USB gadget, ...). Some have already been fixed.

Hmm. Yeah, I thought we had a patch for bluetooth too. I can't find it 
now though.

Suffice it to say that similar patches are required for the other 
instances of this kind of code elsewhere as well.

Thanks Marcel,

Behan

--
Behan Webster
behanw@xxxxxxxxxxxxxxxxxx

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/