Re: linux/atm_zatm.h not really usable in userspace since cf00713a655d3019be7faa184402f16c43a0fed3
On 15 October 2016 at 16:10, Mikko Rapeli wrote: > On Sat, Oct 15, 2016 at 03:33:22PM +0100, Pascal Terjan wrote: >> On 15 October 2016 at 15:09, Mikko Rapeli wrote: >> > On Sat, Oct 15, 2016 at 01:05:10PM +0100, Pascal Terjan wrote: >> >> It is no longer possible to include + userspace >> >> headers using time, for example , this broke for example >> >> the build of linux-atm. >> >> >> >> Reproducer: >> >> >> >> $ cat test.c >> >> #include >> >> #include >> > >> > If possible, please reverse the order of includes to first include glibc >> > headers and then Linux kernel uapi ones. >> >> That was what I tried first but this didn't help: >> >> In file included from /usr/include/linux/atm_zatm.h:17:0, >> from test.c:2: >> /usr/include/linux/time.h:9:8: error: redefinition of 'struct timespec' >> struct timespec { >> ^ >> In file included from /usr/include/sys/select.h:43:0, >> from /usr/include/sys/types.h:219, >> from /usr/include/stdlib.h:314, >> from test.c:1: >> /usr/include/time.h:120:8: note: originally defined here >> struct timespec >> ^ >> In file included from /usr/include/linux/atm_zatm.h:17:0,> >> from test.c:2: >> from test.c:2: >> /usr/include/linux/time.h:15:8: error: redefinition of 'struct timeval' >> struct timeval { >> ^ >> In file included from /usr/include/sys/select.h:45:0, >> from /usr/include/sys/types.h:219, >> from /usr/include/stdlib.h:314, >> from test.c:1: >> /usr/include/bits/time.h:30:8: note: originally defined here >> struct timeval >> ^ >> > Kernel uapi headers did not declare their header file dependencies >> > correctly >> > and I've been fixing them. I have also tried to fix compatibility issues >> > with glibc headers, but unfortunately they only work when glibc headers >> > are included before kernel headers. Userspace which has been relying on >> > the magic include order for various uapi headers is now unfortunately >> > affected. Sorry about that. >> >> In this case no order works, it seems the kernel doesn't handle it in >> time.h unlike many other headers > > Ok, then https://patchwork.kernel.org/patch/9294305/ hasn't been applied yet. > You can apply that or revert cf00713a655d3019be7faa184402f16c43a0fed3 > for the time being. Ah thanks, I'll take that patch :) > It's a bit tricky to push through changes touching uapi headers for various > kernel sub systems since they may get applied at different order and time. Yeah I can imagine, thanks for doing it
Re: linux/atm_zatm.h not really usable in userspace since cf00713a655d3019be7faa184402f16c43a0fed3
On Sat, Oct 15, 2016 at 03:33:22PM +0100, Pascal Terjan wrote: > On 15 October 2016 at 15:09, Mikko Rapeli wrote: > > On Sat, Oct 15, 2016 at 01:05:10PM +0100, Pascal Terjan wrote: > >> It is no longer possible to include + userspace > >> headers using time, for example , this broke for example > >> the build of linux-atm. > >> > >> Reproducer: > >> > >> $ cat test.c > >> #include > >> #include > > > > If possible, please reverse the order of includes to first include glibc > > headers and then Linux kernel uapi ones. > > That was what I tried first but this didn't help: > > In file included from /usr/include/linux/atm_zatm.h:17:0, > from test.c:2: > /usr/include/linux/time.h:9:8: error: redefinition of 'struct timespec' > struct timespec { > ^ > In file included from /usr/include/sys/select.h:43:0, > from /usr/include/sys/types.h:219, > from /usr/include/stdlib.h:314, > from test.c:1: > /usr/include/time.h:120:8: note: originally defined here > struct timespec > ^ > In file included from /usr/include/linux/atm_zatm.h:17:0,> > from test.c:2: > from test.c:2: > /usr/include/linux/time.h:15:8: error: redefinition of 'struct timeval' > struct timeval { > ^ > In file included from /usr/include/sys/select.h:45:0, > from /usr/include/sys/types.h:219, > from /usr/include/stdlib.h:314, > from test.c:1: > /usr/include/bits/time.h:30:8: note: originally defined here > struct timeval > ^ > > Kernel uapi headers did not declare their header file dependencies correctly > > and I've been fixing them. I have also tried to fix compatibility issues > > with glibc headers, but unfortunately they only work when glibc headers > > are included before kernel headers. Userspace which has been relying on > > the magic include order for various uapi headers is now unfortunately > > affected. Sorry about that. > > In this case no order works, it seems the kernel doesn't handle it in > time.h unlike many other headers Ok, then https://patchwork.kernel.org/patch/9294305/ hasn't been applied yet. You can apply that or revert cf00713a655d3019be7faa184402f16c43a0fed3 for the time being. It's a bit tricky to push through changes touching uapi headers for various kernel sub systems since they may get applied at different order and time. -Mikko
Re: linux/atm_zatm.h not really usable in userspace since cf00713a655d3019be7faa184402f16c43a0fed3
On 15 October 2016 at 15:09, Mikko Rapeli wrote: > On Sat, Oct 15, 2016 at 01:05:10PM +0100, Pascal Terjan wrote: >> It is no longer possible to include + userspace >> headers using time, for example , this broke for example >> the build of linux-atm. >> >> Reproducer: >> >> $ cat test.c >> #include >> #include > > If possible, please reverse the order of includes to first include glibc > headers and then Linux kernel uapi ones. That was what I tried first but this didn't help: In file included from /usr/include/linux/atm_zatm.h:17:0, from test.c:2: /usr/include/linux/time.h:9:8: error: redefinition of 'struct timespec' struct timespec { ^ In file included from /usr/include/sys/select.h:43:0, from /usr/include/sys/types.h:219, from /usr/include/stdlib.h:314, from test.c:1: /usr/include/time.h:120:8: note: originally defined here struct timespec ^ In file included from /usr/include/linux/atm_zatm.h:17:0, from test.c:2: /usr/include/linux/time.h:15:8: error: redefinition of 'struct timeval' struct timeval { ^ In file included from /usr/include/sys/select.h:45:0, from /usr/include/sys/types.h:219, from /usr/include/stdlib.h:314, from test.c:1: /usr/include/bits/time.h:30:8: note: originally defined here struct timeval ^ > Kernel uapi headers did not declare their header file dependencies correctly > and I've been fixing them. I have also tried to fix compatibility issues > with glibc headers, but unfortunately they only work when glibc headers > are included before kernel headers. Userspace which has been relying on > the magic include order for various uapi headers is now unfortunately > affected. Sorry about that. In this case no order works, it seems the kernel doesn't handle it in time.h unlike many other headers
Re: linux/atm_zatm.h not really usable in userspace since cf00713a655d3019be7faa184402f16c43a0fed3
On Sat, Oct 15, 2016 at 01:05:10PM +0100, Pascal Terjan wrote: > It is no longer possible to include + userspace > headers using time, for example , this broke for example > the build of linux-atm. > > Reproducer: > > $ cat test.c > #include > #include If possible, please reverse the order of includes to first include glibc headers and then Linux kernel uapi ones. Kernel uapi headers did not declare their header file dependencies correctly and I've been fixing them. I have also tried to fix compatibility issues with glibc headers, but unfortunately they only work when glibc headers are included before kernel headers. Userspace which has been relying on the magic include order for various uapi headers is now unfortunately affected. Sorry about that. -Mikko > $ gcc -c test.c > In file included from /usr/include/sys/select.h:43:0, > from /usr/include/sys/types.h:219, > from /usr/include/stdlib.h:314, > from test.c:2: > /usr/include/time.h:120:8: error: redefinition of 'struct timespec' > struct timespec > ^ > In file included from /usr/include/linux/atm_zatm.h:17:0, > from test.c:1: > /usr/include/linux/time.h:9:8: note: originally defined here > struct timespec { > ^ > In file included from /usr/include/sys/select.h:45:0, > from /usr/include/sys/types.h:219, > from /usr/include/stdlib.h:314, > from test.c:2: > /usr/include/bits/time.h:30:8: error: redefinition of 'struct timeval' > struct timeval > ^ > In file included from /usr/include/linux/atm_zatm.h:17:0, > from test.c:1: > /usr/include/linux/time.h:15:8: note: originally defined here > struct timeval { > ^