]> git.sesse.net Git - bcachefs-tools-debian/commit
bch_bindgen: fix packed and aligned structs on i686, ppc64
authorThomas Bertschinger <tahbertschinger@gmail.com>
Sat, 17 Feb 2024 18:20:25 +0000 (11:20 -0700)
committerKent Overstreet <kent.overstreet@linux.dev>
Sat, 17 Feb 2024 18:38:09 +0000 (13:38 -0500)
commitb2824ebac134606ee1938581889c65fe3b02489b
treecb1c6053994d3f4abe8907a827183f07151aa0fa
parent46590fa31003c5e8acd4e72ef87f317fe905998f
bch_bindgen: fix packed and aligned structs on i686, ppc64

This patch addresses build issues with bch_bindgen on architectures
where the natural alignment of u64 is 4 instead of 8, such as i686 and
ppc64. The issue arises from rustc's refusing to compile structs with a
"packed" attribute that have members with an explicit "align(N)"
attribute.

rust-bindgen generally does not place "align(N)" attributes on types
when it is redundant with the type's natural alignment. There are a few
types in bcachefs with "__aligned(8)" in C where the type's natural
alignment is 8 except on architectures like those mentioned above where
it is 4. rust-bindgen places the "align(8)" attribute on these types,
for those architectures. The affected types include:

- bch_csum
- bch_sb_layout
- bch_ioctl_data_progress

bch_ioctl_data_progress, and all types that depend on it, are not used
on the Rust side currently and bindings are only generated for them
because they are covered by `.allowlist_type("bch_.*")` in
bch_bindgen/build.rs. This patch resolves the build failure for this
type by excluding it from bch_bindgen. If/when accessing this type in
Rust is needed, a decision can be made then about how to represent its
layout in Rust.

bch_csum and bch_sb_layout, and types that depend on them, are currently
used in Rust so they can't be excluded. This patch addresses these types
by stripping off the "packed" attribute in Rust on types that embed
these types, since that attribute happens to not be necessary for proper
layout.

Signed-off-by: Thomas Bertschinger <tahbertschinger@gmail.com>
Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
bch_bindgen/build.rs