Re: [PATCH] Fix: rseq uapi: Adapt header includes to follow glibc header changes

From: Steven Rostedt
Date: Wed Nov 01 2023 - 16:44:21 EST


On Wed, 1 Nov 2023 16:10:04 -0400
Mathieu Desnoyers <mathieu.desnoyers@xxxxxxxxxxxx> wrote:

> So what is the minimal change required to make things work on your
> setup? I just tested with a Debian "testing" chroot (with libc 2.37-12)
> and I cannot reproduce your issue.
>
> Should I just submit a patch that removes "#include <asm/byteorder.h>" ?
> I am really unsure which environments are affected though.
>

I guess you can drop it :-p

When I tried to reproduce it with hand writing 'gcc', I couldn't. But when
I did:

$ make foo

It gave me the error. I was confused for a bit. Then I looked at what my
Makefile was doing and what I was doing. The only difference was that the
make included:

-I.

Removing that from the Makefile worked!

My Makefile added to the CFLAGS "-I." and I forgot that this directory has
a "linux/" directory in it that I used years ago to test kernel functions.
The git history shows it was last touched in 2016 (when I was still at Red Hat)

Removing -I. now makes everything work.

I have no idea why it suddenly stopped working just a few months ago. Maybe
something was moved out of the gcc headers so my local headers no longer
see it. That is, perhaps the glibc headers moved something out and added a
#include to it, where my local headers did not have that change. I don't
know and I don't care.

Well, at least now I know why I was getting errors on my build, but
couldn't find anything on the internet showing why others were not!

Sorry for the noise. :-/

-- Steve