On Tue, Aug 23, 2016 at 1:30 AM, David Miller <da...@davemloft.net> wrote: > From: Mikko Rapeli <mikko.rap...@iki.fi> > Date: Mon, 22 Aug 2016 20:32:44 +0200 > >> Fixes userspace compiler error: >> >> error: ‘IFNAMSIZ’ undeclared here (not in a function) >> >> Suggested by Frans Klaver <franskla...@gmail.com> on lkml message >> <20150530195223.ga15...@bugger.home>. >> >> Signed-off-by: Mikko Rapeli <mikko.rap...@iki.fi> > > IFNAMSIZ has to be in linux/if.h, you aren't explaining why you have > to move it to the hdlc header instead of having the hdlc header > include linux/if.h
Circular references. linux/if.h includes hdlc/ioctl.h, and has to define IFNAMSIZ before doing so. If IFNAMSIZ is moved to hdlc/ioctl.h, it is still pulled in if one includes linux/if.h. What we gain is that you can include hdlc/ioctl.h directly (which is what one of the tests is already doing, iirc). But yes, it should be explained in the commit message. Thanks, Frans
