[systemd-devel] kdbus: merge 3.17 branch into master?

Greg KH gregkh at linuxfoundation.org
Tue Aug 19 07:21:42 PDT 2014


On Tue, Aug 19, 2014 at 04:08:15PM +0200, Daniel Mack wrote:
> On 08/19/2014 03:56 PM, Greg KH wrote:
> > On Tue, Aug 19, 2014 at 03:24:39PM +0200, Daniel Mack wrote:
> >> On 08/19/2014 02:47 PM, Greg KH wrote:
> >>> In file included from /lib/modules/3.17.0-rc1+/build/include/uapi/linux/posix_types.h:4:0,
> >>>                  from /lib/modules/3.17.0-rc1+/build/include/uapi/linux/types.h:13,
> >>>                  from ../kdbus.h:23,
> >>
> >> Ok, got it now. No idea why it worked on my system, but a fix for now is
> >> to make cpp look at the system include file search pathes first, and
> >> then fall back to $(KERNELDIR)/include/uapi.
> >>
> >> Care to test again?
> > 
> > Close, I now get a bunch of these warnings:
> > 
> > 	cc1: note: obsolete option -I- used, please use -iquote instead
> > 
> > and then finally a failure of:
> > 
> > kdbus-util.c:173:16: error: ‘__NR_memfd_create’ undeclared (first use in
> > this function)
> >   ret = syscall(__NR_memfd_create, name, MFD_ALLOW_SEALING);
> 
> Ok, cpp is really confused about which headers to look at now. I guess
> we have to rely on the kernel headers being available under
> $(KERNELDIR)/usr/include.
> 
> Could you please try if temporarily reverting my top-most commit and
> then doing a 'make headers_install' in your kernel repo (before you
> build kdbus) fixes it?

Yes, that works.  And we can live with that, it's reasonable to rely on
that kernel header now.

thanks,

greg k-h


More information about the systemd-devel mailing list