Re: [PATCH] rust: Kbuild: Skip -fmin-function-alignment in bindgen flags

[Zehui Xu]

On 31.7.2024 2.39, Miguel Ojeda wrote:
> On Wed, Jul 31, 2024 at 12:21 AM Zehui Xu <zehuixu@xxxxxxxxxx> wrote:
>>
>> GCC recently added the -fmin-function-alignment option, which will appear in GCC 14. However, this flag can cause issues when passed to the Rust Makefile and affect the bindgen process. Bindgen relies on libclang to parse C code, and currently does not support the -fmin-function-alignment flag, leading to compilation failures when GCC 14 is used.
>>
>> This patch addresses the issue by adding -fmin-function-alignment to the bindgen_skip_c_flags in rust/Makefile, ensuring it is skipped during the bindgen process. This prevents the flag from causing compilation issues and maintains compatibility with the upcoming GCC 14.
>
> Thanks for the patch!
>
>> This is my first patch to the kernel, if there are any issues or improvements needed, please let me know. ;)
>
> It looks OK, a couple nits are that you would normally not put this
> line here (in the commit message), but below the --- line (which
> means it would not go into the repository).
>
> In addition, normally you would: wrap your commit messages to a
> reasonable length, use scripts/checkpatch.pl to check for this and
> other issues, use the "Link" tag instead of "Reference:", avoid
> leaving empty lines between tags and ideally use git format-patch
> --base to specify the base of your patch (not really needed in many
> cases, but it is always nice to have around).
>
> If you can try that and send a v2, that would be nice.
>
> But apart from those nits, it seems fine -- welcome!
>
> Cheers,
> Miguel

Thanks for your nits, Miguel! I have applied your suggestions and submitted a v2 patch:
https://lore.kernel.org/all/20240731034112.6060-1-zehuixu@xxxxxxxxxx/

---
Cheers,
Zehui