linux-kernel - Re: [build bug, 2.6.24-rc1] CONFIG_VIDEO_DEV=m & CONFIG_VIDEO_SAA7146

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20071101105738.GB28701@elte.hu>
Date:	Thu, 1 Nov 2007 11:57:38 +0100
From:	Ingo Molnar <mingo@...e.hu>
To:	Trent Piepho <xyzzy@...akeasy.org>
Cc:	Mauro Carvalho Chehab <mchehab@...radead.org>,
	Linux Kernel Mailing list <linux-kernel@...r.kernel.org>,
	Randy Dunlap <randy.dunlap@...cle.com>,
	Johannes Stezenbach <js@...uxtv.org>,
	Sam Ravnborg <sam@...nborg.org>
Subject: Re: [build bug, 2.6.24-rc1] CONFIG_VIDEO_DEV=m &
	CONFIG_VIDEO_SAA7146_VV=y


* Trent Piepho <xyzzy@...akeasy.org> wrote:

> All the rest of the selectors of VIDEO_SAA7146_VV are in the V4L menu, 
> which contains the necessary dependency on VIDEO_DEV.
> 
> Of course the real fix here would be have kbuild not force us to 
> manually add all the recursive dependencies of a selected module to 
> the dependencies of the module that selects it.

very much agreed. We should not be forced to put in redundant 
information that Kconfig could figure out by itself. (The only 
constraint is that we must not create circular dependencies - but 
Kconfig is checking that already.)

> On Thu, 1 Nov 2007, Ingo Molnar wrote:
> > * Trent Piepho <xyzzy@...akeasy.org> wrote:
> >
> > > On Fri, 26 Oct 2007, Ingo Molnar wrote:
> > > > the attached config (generated via make randconfig) fails to build due
> > > > to the combination of these config entries:
> > > >
> > > >   CONFIG_VIDEO_DEV=m
> > > >   CONFIG_VIDEO_SAA7146_VV=y
> > > >
> > > > i found no obvious Kconfig way to force VIDEO_SAA7146_VV to be modular
> > > > when VIDEO_DEV is modular - is there a good solution for this?
> > >
> > > I guess you didn't like my reply to your first message about this?
> > >
> > > http://lkml.org/lkml/2007/10/7/145
> > >
> > > kbuild support for modules used as libraries is really poor.  The only
> > > way to do is use select on the library module, and then have all the
> > > dependencies of the selectee be included in the selector, which must
> > > be done recursively so that any modules selected by the selectee have
> > > their dependencies added to the selector.
> >
> > any patch for me to try? This bug is blocking automated randconfig
> > build-testing quite frequently so it's quite a showstopper.
> 
> For some time now I've thought the whole ttpci config/makefile setup 
> sucked.  I've finally gone though and redone it and fixed this problem 
> too.
> 
> Here is the patch: http://linuxtv.org/hg/v4l-dvb/rev/5320c2571183

the drivers/media/dvb/ttpci/Kconfig bits do not apply:

 $ q push
 Applying patch patches/dvb-fix-5320c2571183.patch
 patching file drivers/media/common/Kconfig
 patching file drivers/media/dvb/ttpci/Kconfig
 Hunk #1 FAILED at 1.
 Hunk #2 FAILED at 63.
 Hunk #4 FAILED at 99.
 Hunk #5 FAILED at 120.
 Hunk #6 FAILED at 142.
 5 out of 6 hunks FAILED -- rejects in file 
 drivers/media/dvb/ttpci/Kconfig
 patching file drivers/media/dvb/ttpci/Makefile
 Patch patches/dvb-fix-5320c2571183.patch does not apply (enforce with -f)

got a link to the dependent patch that i'm apparently missing?

	Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/