* [PATCH 0/2] Fix compilation errors when not using GNU libc @ 2025-03-04 12:21 Luis Henriques 2025-03-04 12:21 ` [PATCH 1/2] src: include libgen.h for function basename() if " Luis Henriques 2025-03-04 12:21 ` [PATCH 2/2] src/locktest: add missing struct cast with syscall getsockname() Luis Henriques 0 siblings, 2 replies; 8+ messages in thread From: Luis Henriques @ 2025-03-04 12:21 UTC (permalink / raw) To: Zorro Lang; +Cc: fstests, Luis Henriques Hi! Following this patch I'm sending 2 simple patches that fix build failures when trying to compile fstests in a system that does not have GNU libc. Luis Henriques (2): src: include libgen.h for function basename() if not using GNU libc src/locktest: add missing struct cast with syscall getsockname() src/aio-dio-regress/aio-dio-append-write-read-race.c | 3 +++ src/locktest.c | 2 +- src/splice-test.c | 3 +++ src/t_ext4_dax_inline_corruption.c | 3 +++ src/t_ext4_dax_journal_corruption.c | 3 +++ src/t_mmap_collision.c | 3 +++ src/t_mmap_dio.c | 3 +++ 7 files changed, 19 insertions(+), 1 deletion(-) ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 1/2] src: include libgen.h for function basename() if not using GNU libc 2025-03-04 12:21 [PATCH 0/2] Fix compilation errors when not using GNU libc Luis Henriques @ 2025-03-04 12:21 ` Luis Henriques 2025-03-04 14:05 ` Zorro Lang 2025-03-04 12:21 ` [PATCH 2/2] src/locktest: add missing struct cast with syscall getsockname() Luis Henriques 1 sibling, 1 reply; 8+ messages in thread From: Luis Henriques @ 2025-03-04 12:21 UTC (permalink / raw) To: Zorro Lang; +Cc: fstests, Luis Henriques Compiling fstests on a system without the GNU libc fails because the basename() is defined in libgen.h by POSIX and not in string.h. Fix that by including the libgen.h header if __USE_GNU isn't defined. Signed-off-by: Luis Henriques <luis@igalia.com> --- src/aio-dio-regress/aio-dio-append-write-read-race.c | 3 +++ src/splice-test.c | 3 +++ src/t_ext4_dax_inline_corruption.c | 3 +++ src/t_ext4_dax_journal_corruption.c | 3 +++ src/t_mmap_collision.c | 3 +++ src/t_mmap_dio.c | 3 +++ 6 files changed, 18 insertions(+) diff --git a/src/aio-dio-regress/aio-dio-append-write-read-race.c b/src/aio-dio-regress/aio-dio-append-write-read-race.c index d9f8982f007c..1a0c8e99c61f 100644 --- a/src/aio-dio-regress/aio-dio-append-write-read-race.c +++ b/src/aio-dio-regress/aio-dio-append-write-read-race.c @@ -19,6 +19,9 @@ #include <sys/stat.h> #include <fcntl.h> #include <errno.h> +#ifndef __USE_GNU +#include <libgen.h> +#endif #include <libaio.h> #include <pthread.h> diff --git a/src/splice-test.c b/src/splice-test.c index eb8636738064..e6753a15dc47 100644 --- a/src/splice-test.c +++ b/src/splice-test.c @@ -18,6 +18,9 @@ #include <string.h> #include <errno.h> #include <malloc.h> +#ifndef __USE_GNU +#include <libgen.h> +#endif unsigned int sector_size; unsigned int buffer_size; diff --git a/src/t_ext4_dax_inline_corruption.c b/src/t_ext4_dax_inline_corruption.c index e1a39a6c1e46..a6cb768512fc 100644 --- a/src/t_ext4_dax_inline_corruption.c +++ b/src/t_ext4_dax_inline_corruption.c @@ -10,6 +10,9 @@ #include <sys/types.h> #include <time.h> #include <unistd.h> +#ifndef __USE_GNU +#include <libgen.h> +#endif #define PAGE(a) ((a)*0x1000) #define STRLEN 256 diff --git a/src/t_ext4_dax_journal_corruption.c b/src/t_ext4_dax_journal_corruption.c index ba7a96e43559..4e5762d77058 100644 --- a/src/t_ext4_dax_journal_corruption.c +++ b/src/t_ext4_dax_journal_corruption.c @@ -10,6 +10,9 @@ #include <sys/types.h> #include <time.h> #include <unistd.h> +#ifndef __USE_GNU +#include <libgen.h> +#endif #define PAGE(a) ((a)*0x1000) #define STRLEN 256 diff --git a/src/t_mmap_collision.c b/src/t_mmap_collision.c index c872f4e26940..638424d2e619 100644 --- a/src/t_mmap_collision.c +++ b/src/t_mmap_collision.c @@ -24,6 +24,9 @@ #include <sys/stat.h> #include <sys/types.h> #include <unistd.h> +#ifndef __USE_GNU +#include <libgen.h> +#endif void *dax_data; int nodax_fd; diff --git a/src/t_mmap_dio.c b/src/t_mmap_dio.c index 6c8ca1a39181..342e2c6deb4b 100644 --- a/src/t_mmap_dio.c +++ b/src/t_mmap_dio.c @@ -14,6 +14,9 @@ #include <libaio.h> #include <errno.h> #include <sys/time.h> +#ifndef __USE_GNU +#include <libgen.h> +#endif void usage(char *prog) { ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] src: include libgen.h for function basename() if not using GNU libc 2025-03-04 12:21 ` [PATCH 1/2] src: include libgen.h for function basename() if " Luis Henriques @ 2025-03-04 14:05 ` Zorro Lang 2025-03-04 14:31 ` Luis Henriques 0 siblings, 1 reply; 8+ messages in thread From: Zorro Lang @ 2025-03-04 14:05 UTC (permalink / raw) To: Luis Henriques; +Cc: fstests On Tue, Mar 04, 2025 at 12:21:18PM +0000, Luis Henriques wrote: > Compiling fstests on a system without the GNU libc fails because the > basename() is defined in libgen.h by POSIX and not in string.h. > > Fix that by including the libgen.h header if __USE_GNU isn't defined. > > Signed-off-by: Luis Henriques <luis@igalia.com> > --- > src/aio-dio-regress/aio-dio-append-write-read-race.c | 3 +++ > src/splice-test.c | 3 +++ > src/t_ext4_dax_inline_corruption.c | 3 +++ > src/t_ext4_dax_journal_corruption.c | 3 +++ > src/t_mmap_collision.c | 3 +++ > src/t_mmap_dio.c | 3 +++ > 6 files changed, 18 insertions(+) > > diff --git a/src/aio-dio-regress/aio-dio-append-write-read-race.c b/src/aio-dio-regress/aio-dio-append-write-read-race.c > index d9f8982f007c..1a0c8e99c61f 100644 > --- a/src/aio-dio-regress/aio-dio-append-write-read-race.c > +++ b/src/aio-dio-regress/aio-dio-append-write-read-race.c > @@ -19,6 +19,9 @@ > #include <sys/stat.h> > #include <fcntl.h> > #include <errno.h> > +#ifndef __USE_GNU > +#include <libgen.h> > +#endif Do you use clang and musl to build fstests? I'm even not sure if fstests can be built with them :) I don't have experience on this part, but I don't mind letting fstests to be supported by other libc. The patch looks safe for current users. If anyone has this experience, please feel free to review. I'll think about merging this patch if no objection from others. Thanks, Zorro > > #include <libaio.h> > #include <pthread.h> > diff --git a/src/splice-test.c b/src/splice-test.c > index eb8636738064..e6753a15dc47 100644 > --- a/src/splice-test.c > +++ b/src/splice-test.c > @@ -18,6 +18,9 @@ > #include <string.h> > #include <errno.h> > #include <malloc.h> > +#ifndef __USE_GNU > +#include <libgen.h> > +#endif > > unsigned int sector_size; > unsigned int buffer_size; > diff --git a/src/t_ext4_dax_inline_corruption.c b/src/t_ext4_dax_inline_corruption.c > index e1a39a6c1e46..a6cb768512fc 100644 > --- a/src/t_ext4_dax_inline_corruption.c > +++ b/src/t_ext4_dax_inline_corruption.c > @@ -10,6 +10,9 @@ > #include <sys/types.h> > #include <time.h> > #include <unistd.h> > +#ifndef __USE_GNU > +#include <libgen.h> > +#endif > > #define PAGE(a) ((a)*0x1000) > #define STRLEN 256 > diff --git a/src/t_ext4_dax_journal_corruption.c b/src/t_ext4_dax_journal_corruption.c > index ba7a96e43559..4e5762d77058 100644 > --- a/src/t_ext4_dax_journal_corruption.c > +++ b/src/t_ext4_dax_journal_corruption.c > @@ -10,6 +10,9 @@ > #include <sys/types.h> > #include <time.h> > #include <unistd.h> > +#ifndef __USE_GNU > +#include <libgen.h> > +#endif > > #define PAGE(a) ((a)*0x1000) > #define STRLEN 256 > diff --git a/src/t_mmap_collision.c b/src/t_mmap_collision.c > index c872f4e26940..638424d2e619 100644 > --- a/src/t_mmap_collision.c > +++ b/src/t_mmap_collision.c > @@ -24,6 +24,9 @@ > #include <sys/stat.h> > #include <sys/types.h> > #include <unistd.h> > +#ifndef __USE_GNU > +#include <libgen.h> > +#endif > > void *dax_data; > int nodax_fd; > diff --git a/src/t_mmap_dio.c b/src/t_mmap_dio.c > index 6c8ca1a39181..342e2c6deb4b 100644 > --- a/src/t_mmap_dio.c > +++ b/src/t_mmap_dio.c > @@ -14,6 +14,9 @@ > #include <libaio.h> > #include <errno.h> > #include <sys/time.h> > +#ifndef __USE_GNU > +#include <libgen.h> > +#endif > > void usage(char *prog) > { > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] src: include libgen.h for function basename() if not using GNU libc 2025-03-04 14:05 ` Zorro Lang @ 2025-03-04 14:31 ` Luis Henriques 2025-03-08 23:21 ` Luis Henriques 0 siblings, 1 reply; 8+ messages in thread From: Luis Henriques @ 2025-03-04 14:31 UTC (permalink / raw) To: Zorro Lang; +Cc: fstests On Tue, Mar 04 2025, Zorro Lang wrote: > On Tue, Mar 04, 2025 at 12:21:18PM +0000, Luis Henriques wrote: >> Compiling fstests on a system without the GNU libc fails because the >> basename() is defined in libgen.h by POSIX and not in string.h. >> >> Fix that by including the libgen.h header if __USE_GNU isn't defined. >> >> Signed-off-by: Luis Henriques <luis@igalia.com> >> --- >> src/aio-dio-regress/aio-dio-append-write-read-race.c | 3 +++ >> src/splice-test.c | 3 +++ >> src/t_ext4_dax_inline_corruption.c | 3 +++ >> src/t_ext4_dax_journal_corruption.c | 3 +++ >> src/t_mmap_collision.c | 3 +++ >> src/t_mmap_dio.c | 3 +++ >> 6 files changed, 18 insertions(+) >> >> diff --git a/src/aio-dio-regress/aio-dio-append-write-read-race.c b/src/aio-dio-regress/aio-dio-append-write-read-race.c >> index d9f8982f007c..1a0c8e99c61f 100644 >> --- a/src/aio-dio-regress/aio-dio-append-write-read-race.c >> +++ b/src/aio-dio-regress/aio-dio-append-write-read-race.c >> @@ -19,6 +19,9 @@ >> #include <sys/stat.h> >> #include <fcntl.h> >> #include <errno.h> >> +#ifndef __USE_GNU >> +#include <libgen.h> >> +#endif > > Do you use clang and musl to build fstests? I'm even not sure if fstests > can be built with them :) Yes, I'm able to compile and run fstests using gcc (not clang) and musl. Though there's a hack I still need to use to compile, which is to define CFLAGS="-D_LARGEFILE64_SOURCE". Since that doesn't require changing anything, I haven't yet investigated how that could be fixed. > I don't have experience on this part, but I don't mind letting fstests > to be supported by other libc. The patch looks safe for current users. > > If anyone has this experience, please feel free to review. I'll think > about merging this patch if no objection from others. The only links I can provide you with some extra info are [1] (section about basename) and the basename(3) man page. Looking at it again makes me think that including libgen.h unconditionally should also be OK, forcing the POSIX implementation to be used instead. But using this #ifndef seems to be used in other projects. And thanks, Zorro, for your review! [1] https://www.gnu.org/software/libc/manual/html_node/Finding-Tokens-in-a-String.html Cheers, -- Luís > > Thanks, > Zorro > >> >> #include <libaio.h> >> #include <pthread.h> >> diff --git a/src/splice-test.c b/src/splice-test.c >> index eb8636738064..e6753a15dc47 100644 >> --- a/src/splice-test.c >> +++ b/src/splice-test.c >> @@ -18,6 +18,9 @@ >> #include <string.h> >> #include <errno.h> >> #include <malloc.h> >> +#ifndef __USE_GNU >> +#include <libgen.h> >> +#endif >> >> unsigned int sector_size; >> unsigned int buffer_size; >> diff --git a/src/t_ext4_dax_inline_corruption.c b/src/t_ext4_dax_inline_corruption.c >> index e1a39a6c1e46..a6cb768512fc 100644 >> --- a/src/t_ext4_dax_inline_corruption.c >> +++ b/src/t_ext4_dax_inline_corruption.c >> @@ -10,6 +10,9 @@ >> #include <sys/types.h> >> #include <time.h> >> #include <unistd.h> >> +#ifndef __USE_GNU >> +#include <libgen.h> >> +#endif >> >> #define PAGE(a) ((a)*0x1000) >> #define STRLEN 256 >> diff --git a/src/t_ext4_dax_journal_corruption.c b/src/t_ext4_dax_journal_corruption.c >> index ba7a96e43559..4e5762d77058 100644 >> --- a/src/t_ext4_dax_journal_corruption.c >> +++ b/src/t_ext4_dax_journal_corruption.c >> @@ -10,6 +10,9 @@ >> #include <sys/types.h> >> #include <time.h> >> #include <unistd.h> >> +#ifndef __USE_GNU >> +#include <libgen.h> >> +#endif >> >> #define PAGE(a) ((a)*0x1000) >> #define STRLEN 256 >> diff --git a/src/t_mmap_collision.c b/src/t_mmap_collision.c >> index c872f4e26940..638424d2e619 100644 >> --- a/src/t_mmap_collision.c >> +++ b/src/t_mmap_collision.c >> @@ -24,6 +24,9 @@ >> #include <sys/stat.h> >> #include <sys/types.h> >> #include <unistd.h> >> +#ifndef __USE_GNU >> +#include <libgen.h> >> +#endif >> >> void *dax_data; >> int nodax_fd; >> diff --git a/src/t_mmap_dio.c b/src/t_mmap_dio.c >> index 6c8ca1a39181..342e2c6deb4b 100644 >> --- a/src/t_mmap_dio.c >> +++ b/src/t_mmap_dio.c >> @@ -14,6 +14,9 @@ >> #include <libaio.h> >> #include <errno.h> >> #include <sys/time.h> >> +#ifndef __USE_GNU >> +#include <libgen.h> >> +#endif >> >> void usage(char *prog) >> { >> > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] src: include libgen.h for function basename() if not using GNU libc 2025-03-04 14:31 ` Luis Henriques @ 2025-03-08 23:21 ` Luis Henriques 2025-03-09 12:57 ` Zorro Lang 0 siblings, 1 reply; 8+ messages in thread From: Luis Henriques @ 2025-03-08 23:21 UTC (permalink / raw) To: Zorro Lang; +Cc: fstests Hi Zorro, On Tue, Mar 04 2025, Luis Henriques wrote: > On Tue, Mar 04 2025, Zorro Lang wrote: > >> On Tue, Mar 04, 2025 at 12:21:18PM +0000, Luis Henriques wrote: >>> Compiling fstests on a system without the GNU libc fails because the >>> basename() is defined in libgen.h by POSIX and not in string.h. >>> >>> Fix that by including the libgen.h header if __USE_GNU isn't defined. >>> >>> Signed-off-by: Luis Henriques <luis@igalia.com> >>> --- >>> src/aio-dio-regress/aio-dio-append-write-read-race.c | 3 +++ >>> src/splice-test.c | 3 +++ >>> src/t_ext4_dax_inline_corruption.c | 3 +++ >>> src/t_ext4_dax_journal_corruption.c | 3 +++ >>> src/t_mmap_collision.c | 3 +++ >>> src/t_mmap_dio.c | 3 +++ >>> 6 files changed, 18 insertions(+) >>> >>> diff --git a/src/aio-dio-regress/aio-dio-append-write-read-race.c b/src/aio-dio-regress/aio-dio-append-write-read-race.c >>> index d9f8982f007c..1a0c8e99c61f 100644 >>> --- a/src/aio-dio-regress/aio-dio-append-write-read-race.c >>> +++ b/src/aio-dio-regress/aio-dio-append-write-read-race.c >>> @@ -19,6 +19,9 @@ >>> #include <sys/stat.h> >>> #include <fcntl.h> >>> #include <errno.h> >>> +#ifndef __USE_GNU >>> +#include <libgen.h> >>> +#endif >> >> Do you use clang and musl to build fstests? I'm even not sure if fstests >> can be built with them :) > > Yes, I'm able to compile and run fstests using gcc (not clang) and musl. > Though there's a hack I still need to use to compile, which is to define > CFLAGS="-D_LARGEFILE64_SOURCE". Since that doesn't require changing > anything, I haven't yet investigated how that could be fixed. > >> I don't have experience on this part, but I don't mind letting fstests >> to be supported by other libc. The patch looks safe for current users. >> >> If anyone has this experience, please feel free to review. I'll think >> about merging this patch if no objection from others. > > The only links I can provide you with some extra info are [1] (section > about basename) and the basename(3) man page. Looking at it again makes > me think that including libgen.h unconditionally should also be OK, > forcing the POSIX implementation to be used instead. But using this > #ifndef seems to be used in other projects. > Please hold on this patch as I think some of the changes are broken. The POSIX basename() is a bit different from the GNU version, and needs to be handled with a bit more care. I'll resend this later in a few days. (But feel free to pick the second patch, as I believe it's still a valid fix.) Cheers, -- Luís > And thanks, Zorro, for your review! > > [1] https://www.gnu.org/software/libc/manual/html_node/Finding-Tokens-in-a-String.html > > Cheers, > -- > Luís > >> >> Thanks, >> Zorro >> >>> >>> #include <libaio.h> >>> #include <pthread.h> >>> diff --git a/src/splice-test.c b/src/splice-test.c >>> index eb8636738064..e6753a15dc47 100644 >>> --- a/src/splice-test.c >>> +++ b/src/splice-test.c >>> @@ -18,6 +18,9 @@ >>> #include <string.h> >>> #include <errno.h> >>> #include <malloc.h> >>> +#ifndef __USE_GNU >>> +#include <libgen.h> >>> +#endif >>> >>> unsigned int sector_size; >>> unsigned int buffer_size; >>> diff --git a/src/t_ext4_dax_inline_corruption.c b/src/t_ext4_dax_inline_corruption.c >>> index e1a39a6c1e46..a6cb768512fc 100644 >>> --- a/src/t_ext4_dax_inline_corruption.c >>> +++ b/src/t_ext4_dax_inline_corruption.c >>> @@ -10,6 +10,9 @@ >>> #include <sys/types.h> >>> #include <time.h> >>> #include <unistd.h> >>> +#ifndef __USE_GNU >>> +#include <libgen.h> >>> +#endif >>> >>> #define PAGE(a) ((a)*0x1000) >>> #define STRLEN 256 >>> diff --git a/src/t_ext4_dax_journal_corruption.c b/src/t_ext4_dax_journal_corruption.c >>> index ba7a96e43559..4e5762d77058 100644 >>> --- a/src/t_ext4_dax_journal_corruption.c >>> +++ b/src/t_ext4_dax_journal_corruption.c >>> @@ -10,6 +10,9 @@ >>> #include <sys/types.h> >>> #include <time.h> >>> #include <unistd.h> >>> +#ifndef __USE_GNU >>> +#include <libgen.h> >>> +#endif >>> >>> #define PAGE(a) ((a)*0x1000) >>> #define STRLEN 256 >>> diff --git a/src/t_mmap_collision.c b/src/t_mmap_collision.c >>> index c872f4e26940..638424d2e619 100644 >>> --- a/src/t_mmap_collision.c >>> +++ b/src/t_mmap_collision.c >>> @@ -24,6 +24,9 @@ >>> #include <sys/stat.h> >>> #include <sys/types.h> >>> #include <unistd.h> >>> +#ifndef __USE_GNU >>> +#include <libgen.h> >>> +#endif >>> >>> void *dax_data; >>> int nodax_fd; >>> diff --git a/src/t_mmap_dio.c b/src/t_mmap_dio.c >>> index 6c8ca1a39181..342e2c6deb4b 100644 >>> --- a/src/t_mmap_dio.c >>> +++ b/src/t_mmap_dio.c >>> @@ -14,6 +14,9 @@ >>> #include <libaio.h> >>> #include <errno.h> >>> #include <sys/time.h> >>> +#ifndef __USE_GNU >>> +#include <libgen.h> >>> +#endif >>> >>> void usage(char *prog) >>> { >>> >> > ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] src: include libgen.h for function basename() if not using GNU libc 2025-03-08 23:21 ` Luis Henriques @ 2025-03-09 12:57 ` Zorro Lang 0 siblings, 0 replies; 8+ messages in thread From: Zorro Lang @ 2025-03-09 12:57 UTC (permalink / raw) To: Luis Henriques; +Cc: fstests On Sat, Mar 08, 2025 at 11:21:19PM +0000, Luis Henriques wrote: > Hi Zorro, > > On Tue, Mar 04 2025, Luis Henriques wrote: > > > On Tue, Mar 04 2025, Zorro Lang wrote: > > > >> On Tue, Mar 04, 2025 at 12:21:18PM +0000, Luis Henriques wrote: > >>> Compiling fstests on a system without the GNU libc fails because the > >>> basename() is defined in libgen.h by POSIX and not in string.h. > >>> > >>> Fix that by including the libgen.h header if __USE_GNU isn't defined. > >>> > >>> Signed-off-by: Luis Henriques <luis@igalia.com> > >>> --- > >>> src/aio-dio-regress/aio-dio-append-write-read-race.c | 3 +++ > >>> src/splice-test.c | 3 +++ > >>> src/t_ext4_dax_inline_corruption.c | 3 +++ > >>> src/t_ext4_dax_journal_corruption.c | 3 +++ > >>> src/t_mmap_collision.c | 3 +++ > >>> src/t_mmap_dio.c | 3 +++ > >>> 6 files changed, 18 insertions(+) > >>> > >>> diff --git a/src/aio-dio-regress/aio-dio-append-write-read-race.c b/src/aio-dio-regress/aio-dio-append-write-read-race.c > >>> index d9f8982f007c..1a0c8e99c61f 100644 > >>> --- a/src/aio-dio-regress/aio-dio-append-write-read-race.c > >>> +++ b/src/aio-dio-regress/aio-dio-append-write-read-race.c > >>> @@ -19,6 +19,9 @@ > >>> #include <sys/stat.h> > >>> #include <fcntl.h> > >>> #include <errno.h> > >>> +#ifndef __USE_GNU > >>> +#include <libgen.h> > >>> +#endif > >> > >> Do you use clang and musl to build fstests? I'm even not sure if fstests > >> can be built with them :) > > > > Yes, I'm able to compile and run fstests using gcc (not clang) and musl. > > Though there's a hack I still need to use to compile, which is to define > > CFLAGS="-D_LARGEFILE64_SOURCE". Since that doesn't require changing > > anything, I haven't yet investigated how that could be fixed. > > > >> I don't have experience on this part, but I don't mind letting fstests > >> to be supported by other libc. The patch looks safe for current users. > >> > >> If anyone has this experience, please feel free to review. I'll think > >> about merging this patch if no objection from others. > > > > The only links I can provide you with some extra info are [1] (section > > about basename) and the basename(3) man page. Looking at it again makes > > me think that including libgen.h unconditionally should also be OK, > > forcing the POSIX implementation to be used instead. But using this > > #ifndef seems to be used in other projects. > > > > Please hold on this patch as I think some of the changes are broken. The > POSIX basename() is a bit different from the GNU version, and needs to be > handled with a bit more care. I'll resend this later in a few days. Thanks Luis, breaking the building with musl is not a big issue, but please never break the current glibc build :) > > (But feel free to pick the second patch, as I believe it's still a valid > fix.) Today I'll make a fstests release, your second patch will be in the next release (after today). You can find it in patches-in-queue branch soon. Feel free to resend the whole patchset or resend this single one. Thanks, Zorro > > Cheers, > -- > Luís > > > And thanks, Zorro, for your review! > > > > [1] https://www.gnu.org/software/libc/manual/html_node/Finding-Tokens-in-a-String.html > > > > Cheers, > > -- > > Luís > > > >> > >> Thanks, > >> Zorro > >> > >>> > >>> #include <libaio.h> > >>> #include <pthread.h> > >>> diff --git a/src/splice-test.c b/src/splice-test.c > >>> index eb8636738064..e6753a15dc47 100644 > >>> --- a/src/splice-test.c > >>> +++ b/src/splice-test.c > >>> @@ -18,6 +18,9 @@ > >>> #include <string.h> > >>> #include <errno.h> > >>> #include <malloc.h> > >>> +#ifndef __USE_GNU > >>> +#include <libgen.h> > >>> +#endif > >>> > >>> unsigned int sector_size; > >>> unsigned int buffer_size; > >>> diff --git a/src/t_ext4_dax_inline_corruption.c b/src/t_ext4_dax_inline_corruption.c > >>> index e1a39a6c1e46..a6cb768512fc 100644 > >>> --- a/src/t_ext4_dax_inline_corruption.c > >>> +++ b/src/t_ext4_dax_inline_corruption.c > >>> @@ -10,6 +10,9 @@ > >>> #include <sys/types.h> > >>> #include <time.h> > >>> #include <unistd.h> > >>> +#ifndef __USE_GNU > >>> +#include <libgen.h> > >>> +#endif > >>> > >>> #define PAGE(a) ((a)*0x1000) > >>> #define STRLEN 256 > >>> diff --git a/src/t_ext4_dax_journal_corruption.c b/src/t_ext4_dax_journal_corruption.c > >>> index ba7a96e43559..4e5762d77058 100644 > >>> --- a/src/t_ext4_dax_journal_corruption.c > >>> +++ b/src/t_ext4_dax_journal_corruption.c > >>> @@ -10,6 +10,9 @@ > >>> #include <sys/types.h> > >>> #include <time.h> > >>> #include <unistd.h> > >>> +#ifndef __USE_GNU > >>> +#include <libgen.h> > >>> +#endif > >>> > >>> #define PAGE(a) ((a)*0x1000) > >>> #define STRLEN 256 > >>> diff --git a/src/t_mmap_collision.c b/src/t_mmap_collision.c > >>> index c872f4e26940..638424d2e619 100644 > >>> --- a/src/t_mmap_collision.c > >>> +++ b/src/t_mmap_collision.c > >>> @@ -24,6 +24,9 @@ > >>> #include <sys/stat.h> > >>> #include <sys/types.h> > >>> #include <unistd.h> > >>> +#ifndef __USE_GNU > >>> +#include <libgen.h> > >>> +#endif > >>> > >>> void *dax_data; > >>> int nodax_fd; > >>> diff --git a/src/t_mmap_dio.c b/src/t_mmap_dio.c > >>> index 6c8ca1a39181..342e2c6deb4b 100644 > >>> --- a/src/t_mmap_dio.c > >>> +++ b/src/t_mmap_dio.c > >>> @@ -14,6 +14,9 @@ > >>> #include <libaio.h> > >>> #include <errno.h> > >>> #include <sys/time.h> > >>> +#ifndef __USE_GNU > >>> +#include <libgen.h> > >>> +#endif > >>> > >>> void usage(char *prog) > >>> { > >>> > >> > > > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/2] src/locktest: add missing struct cast with syscall getsockname() 2025-03-04 12:21 [PATCH 0/2] Fix compilation errors when not using GNU libc Luis Henriques 2025-03-04 12:21 ` [PATCH 1/2] src: include libgen.h for function basename() if " Luis Henriques @ 2025-03-04 12:21 ` Luis Henriques 2025-03-04 13:48 ` Zorro Lang 1 sibling, 1 reply; 8+ messages in thread From: Luis Henriques @ 2025-03-04 12:21 UTC (permalink / raw) To: Zorro Lang; +Cc: fstests, Luis Henriques The usage of variable 'myAddr' (struct sockaddr_in) is being casted into a 'struct sockaddr' when used with bind() and with connect(). This patch adds that cast when used with getsockname() as well, otherwise we'll get the following error: locktest.c: In function 'main': locktest.c:1155:39: error: passing argument 2 of 'getsockname' from incompatible pointer type [-Wincompatible-pointer-types] 1155 | if (getsockname(s_fd, &myAddr, &addr_len)) { | ^~~~~~~ | | | struct sockaddr_in * In file included from /usr/include/fortify/sys/socket.h:23, from locktest.c:19: /usr/include/sys/socket.h:391:23: note: expected 'struct sockaddr * restrict' but argument is of type 'struct sockaddr_in *' 391 | int getsockname (int, struct sockaddr *__restrict, socklen_t *__restrict); | ^ Signed-off-by: Luis Henriques <luis@igalia.com> --- src/locktest.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/locktest.c b/src/locktest.c index 0e7c3008d1d0..a6cf3b1d5a99 100644 --- a/src/locktest.c +++ b/src/locktest.c @@ -1152,7 +1152,7 @@ main(int argc, char *argv[]) if (port == 0) { socklen_t addr_len = sizeof(myAddr); - if (getsockname(s_fd, &myAddr, &addr_len)) { + if (getsockname(s_fd, (struct sockaddr *)&myAddr, &addr_len)) { perror("getsockname"); exit(1); } ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] src/locktest: add missing struct cast with syscall getsockname() 2025-03-04 12:21 ` [PATCH 2/2] src/locktest: add missing struct cast with syscall getsockname() Luis Henriques @ 2025-03-04 13:48 ` Zorro Lang 0 siblings, 0 replies; 8+ messages in thread From: Zorro Lang @ 2025-03-04 13:48 UTC (permalink / raw) To: Luis Henriques; +Cc: fstests On Tue, Mar 04, 2025 at 12:21:19PM +0000, Luis Henriques wrote: > The usage of variable 'myAddr' (struct sockaddr_in) is being casted into a > 'struct sockaddr' when used with bind() and with connect(). This patch > adds that cast when used with getsockname() as well, otherwise we'll get the > following error: > > locktest.c: In function 'main': > locktest.c:1155:39: error: passing argument 2 of 'getsockname' from incompatible pointer type [-Wincompatible-pointer-types] > 1155 | if (getsockname(s_fd, &myAddr, &addr_len)) { > | ^~~~~~~ > | | > | struct sockaddr_in * > In file included from /usr/include/fortify/sys/socket.h:23, > from locktest.c:19: > /usr/include/sys/socket.h:391:23: note: expected 'struct sockaddr * restrict' but argument is of type 'struct sockaddr_in *' > 391 | int getsockname (int, struct sockaddr *__restrict, socklen_t *__restrict); > | ^ > > Signed-off-by: Luis Henriques <luis@igalia.com> > --- From the manual of getsockname: int getsockname(int sockfd, struct sockaddr *restrict addr, socklen_t *restrict addrlen); looks like the 2nd argument is "struct sockaddr *" in GNU libc too. So I think this patch is good to me. Reviewed-by: Zorro Lang <zlang@redhat.com> > src/locktest.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/locktest.c b/src/locktest.c > index 0e7c3008d1d0..a6cf3b1d5a99 100644 > --- a/src/locktest.c > +++ b/src/locktest.c > @@ -1152,7 +1152,7 @@ main(int argc, char *argv[]) > if (port == 0) { > socklen_t addr_len = sizeof(myAddr); > > - if (getsockname(s_fd, &myAddr, &addr_len)) { > + if (getsockname(s_fd, (struct sockaddr *)&myAddr, &addr_len)) { > perror("getsockname"); > exit(1); > } > ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2025-03-09 12:57 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2025-03-04 12:21 [PATCH 0/2] Fix compilation errors when not using GNU libc Luis Henriques 2025-03-04 12:21 ` [PATCH 1/2] src: include libgen.h for function basename() if " Luis Henriques 2025-03-04 14:05 ` Zorro Lang 2025-03-04 14:31 ` Luis Henriques 2025-03-08 23:21 ` Luis Henriques 2025-03-09 12:57 ` Zorro Lang 2025-03-04 12:21 ` [PATCH 2/2] src/locktest: add missing struct cast with syscall getsockname() Luis Henriques 2025-03-04 13:48 ` Zorro Lang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox