* rm_work and building kernel modules @ 2009-01-26 18:11 Cliff Brake 2009-01-26 18:34 ` Koen Kooi 0 siblings, 1 reply; 4+ messages in thread From: Cliff Brake @ 2009-01-26 18:11 UTC (permalink / raw) To: openembedded-devel I'm noticed a little glitch with rm_work and building kernel modules with OE. Typically, kernel module builds require the kernel source to be intact, which gets removed if rm_work is enabled. I'm wondering if we should make a way to suppress rm_work for kernel recipes, or is this just one of those things people need to live with? If so, any thoughts on the best way to implement? A hackish way would be to check a variable in do_rm_work() and not remove directories if INHIBIT_RM_WORK is set. Thanks, Cliff -- ======================= Cliff Brake http://bec-systems.com ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: rm_work and building kernel modules 2009-01-26 18:11 rm_work and building kernel modules Cliff Brake @ 2009-01-26 18:34 ` Koen Kooi 2009-01-26 19:24 ` Cliff Brake 0 siblings, 1 reply; 4+ messages in thread From: Koen Kooi @ 2009-01-26 18:34 UTC (permalink / raw) To: openembedded-devel On 26-01-09 19:11, Cliff Brake wrote: > I'm noticed a little glitch with rm_work and building kernel modules > with OE. Typically, kernel module builds require the kernel source to > be intact, which gets removed if rm_work is enabled. Which is why it's getting staged to STAGING_KERNEL_DIR, see module.bbclass. regards, Koen ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: rm_work and building kernel modules 2009-01-26 18:34 ` Koen Kooi @ 2009-01-26 19:24 ` Cliff Brake 2009-01-26 20:06 ` Koen Kooi 0 siblings, 1 reply; 4+ messages in thread From: Cliff Brake @ 2009-01-26 19:24 UTC (permalink / raw) To: openembedded-devel; +Cc: openembedded-devel On Mon, Jan 26, 2009 at 1:34 PM, Koen Kooi <k.kooi@student.utwente.nl> wrote: > On 26-01-09 19:11, Cliff Brake wrote: >> >> I'm noticed a little glitch with rm_work and building kernel modules >> with OE. Typically, kernel module builds require the kernel source to >> be intact, which gets removed if rm_work is enabled. > > Which is why it's getting staged to STAGING_KERNEL_DIR, see module.bbclass. ok, thanks. After digging a little more, it appears I developed this recipe on an older OE snapshot that did not have support for the new location of the asm/arch headers in 2.6.27. Everything seems to work now. Would anyone be offended if I commit the below patch? There may be some false positives, but would at least hopefully cut down on people copying the use of KERNEL_SOURCE for new module recipes. Thanks, Cliff diff --git a/packages/ark3116/ark3116_0.4.1.bb b/packages/ark3116/ark3116_0.4.1.bb index 629c50a..33962ff 100644 --- a/packages/ark3116/ark3116_0.4.1.bb +++ b/packages/ark3116/ark3116_0.4.1.bb @@ -1,3 +1,7 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see module.bbclass + + DESCRIPTION = "Driver for usb serial cables based upon ArkMicroChips 3116s chip" HOMEPAGE = "http://avr.auctionant.de/ark3116_linux_driver/" PRIORITY = "optional" diff --git a/packages/char-driver/char-driver.bb b/packages/char-driver/char-driver.bb index e6458b5..25245af 100644 --- a/packages/char-driver/char-driver.bb +++ b/packages/char-driver/char-driver.bb @@ -1,3 +1,7 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see module.bbclass + + DESCRIPTION = "char-driver and userspace program" PRIORITY = "optional" SECTION = "base" diff --git a/packages/dri/drm-kernel_cvs.bb b/packages/dri/drm-kernel_cvs.bb index 7e0bddf..ee68bac 100644 --- a/packages/dri/drm-kernel_cvs.bb +++ b/packages/dri/drm-kernel_cvs.bb @@ -1,3 +1,6 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see modules.bbclass + SECTION = "x11/base" PR = "r3" LICENSE = "MIT" diff --git a/packages/geode-drivers/geode-modules.inc b/packages/geode-drivers/geode-modules.inc index adaf885..9e78556 100644 --- a/packages/geode-drivers/geode-modules.inc +++ b/packages/geode-drivers/geode-modules.inc @@ -1,3 +1,7 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see module.bbclass + + # Linux kernel modules OE build files for the AMD Geode GX5535/LX5536 # Copyright (C) 2005-2006, Advanced Micro Devices, Inc. All Rights Reserved # Released under the MIT license (see packages/COPYING) diff --git a/packages/gpio-event/gpio-event.bb b/packages/gpio-event/gpio-event.bb index afb2a77..52a5414 100644 --- a/packages/gpio-event/gpio-event.bb +++ b/packages/gpio-event/gpio-event.bb @@ -1,3 +1,7 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see module.bbclass + + DESCRIPTION = "gpio-event driver and userspace program" PRIORITY = "optional" SECTION = "base" diff --git a/packages/pcmcia-cs/pcmcia-cs_3.2.8.bb b/packages/pcmcia-cs/pcmcia-cs_3.2.8.bb index 018f335..18a40c7 100644 --- a/packages/pcmcia-cs/pcmcia-cs_3.2.8.bb +++ b/packages/pcmcia-cs/pcmcia-cs_3.2.8.bb @@ -1,3 +1,7 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see module.bbclass + + DESCRIPTION = "Utilities and system configuration files for the Linux PCMCIA card services" SECTION = "base" PRIORITY = "required" diff --git a/packages/pvrusb2-mci/pvrusb2-mci_20060903.bb b/packages/pvrusb2-mci/pvrusb2-mci_20060903.bb index 5bd4ea2..af877e2 100644 --- a/packages/pvrusb2-mci/pvrusb2-mci_20060903.bb +++ b/packages/pvrusb2-mci/pvrusb2-mci_20060903.bb @@ -1,3 +1,7 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see module.bbclass + + require pvrusb2-mci2.inc RRECOMMENDS = "kernel-module-tveeprom \ diff --git a/packages/pvrusb2-mci/pvrusb2-mci_20080210.bb b/packages/pvrusb2-mci/pvrusb2-mci_20080210.bb index 5bd4ea2..af877e2 100644 --- a/packages/pvrusb2-mci/pvrusb2-mci_20080210.bb +++ b/packages/pvrusb2-mci/pvrusb2-mci_20080210.bb @@ -1,3 +1,7 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see module.bbclass + + require pvrusb2-mci2.inc RRECOMMENDS = "kernel-module-tveeprom \ diff --git a/packages/pwc/pwc_10.0.12-rc1.bb b/packages/pwc/pwc_10.0.12-rc1.bb index af1d5f5..85e6ade 100644 --- a/packages/pwc/pwc_10.0.12-rc1.bb +++ b/packages/pwc/pwc_10.0.12-rc1.bb @@ -1,3 +1,7 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see module.bbclass + + DESCRIPTION = "Philips USB Webcam driver for Linux that supports VGA resolution" HOMEPAGE = "http://www.saillard.org/linux/pwc" PRIORITY = "optional" diff --git a/packages/wifistix/wifistix-modules_5.0.16.p0.bb b/packages/wifistix/wifistix-modules_5.0.16.p0.bb index 0eb6877..f9e8503 100644 --- a/packages/wifistix/wifistix-modules_5.0.16.p0.bb +++ b/packages/wifistix/wifistix-modules_5.0.16.p0.bb @@ -1,3 +1,6 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see module.bbclass + DESCRIPTION = "Linux Driver for Marvel 88W8385 802.11b/g Wifi Module used in Gumstix daughtercards" SECTION = "base" PRIORITY = "optional" diff --git a/packages/wlan-ng/wlan-ng-modules.inc b/packages/wlan-ng/wlan-ng-modules.inc index 6dbadce..55fd026 100644 --- a/packages/wlan-ng/wlan-ng-modules.inc +++ b/packages/wlan-ng/wlan-ng-modules.inc @@ -1,3 +1,7 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see module.bbclass + + DESCRIPTION = "linux-wlan-ng (prism2.x, prism3, pcmcia, pci, usb) driver for 11Mbps wireless lan cards" HOMEPAGE = "http://www.linux-wlan.org" SECTION = "kernel/modules" diff --git a/packages/wlan-ng/wlan-ng-utils.inc b/packages/wlan-ng/wlan-ng-utils.inc index 72976ce..92f25dc 100644 --- a/packages/wlan-ng/wlan-ng-utils.inc +++ b/packages/wlan-ng/wlan-ng-utils.inc @@ -1,3 +1,7 @@ +# FIXME, consider using kernel staging directory instead of KERNEL_SOURCE which is +# located in the work directory. see module.bbclass + + DESCRIPTION = "linux-wlan-ng userland utilities" HOMEPAGE = "http://www.linux-wlan.org" SECTION = "kernel/userland" -- ======================= Cliff Brake http://bec-systems.com ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: rm_work and building kernel modules 2009-01-26 19:24 ` Cliff Brake @ 2009-01-26 20:06 ` Koen Kooi 0 siblings, 0 replies; 4+ messages in thread From: Koen Kooi @ 2009-01-26 20:06 UTC (permalink / raw) To: openembedded-devel On 26-01-09 20:24, Cliff Brake wrote: > On Mon, Jan 26, 2009 at 1:34 PM, Koen Kooi<k.kooi@student.utwente.nl> wrote: >> On 26-01-09 19:11, Cliff Brake wrote: >>> I'm noticed a little glitch with rm_work and building kernel modules >>> with OE. Typically, kernel module builds require the kernel source to >>> be intact, which gets removed if rm_work is enabled. >> Which is why it's getting staged to STAGING_KERNEL_DIR, see module.bbclass. > > ok, thanks. After digging a little more, it appears I developed this > recipe on an older OE snapshot that did not have support for the new > location of the asm/arch headers in 2.6.27. Everything seems to work > now. > > Would anyone be offended if I commit the below patch? There may be > some false positives, but would at least hopefully cut down on people > copying the use of KERNEL_SOURCE for new module recipes. Looks OK to me. regards, Koen > > Thanks, > Cliff > > diff --git a/packages/ark3116/ark3116_0.4.1.bb > b/packages/ark3116/ark3116_0.4.1.bb > index 629c50a..33962ff 100644 > --- a/packages/ark3116/ark3116_0.4.1.bb > +++ b/packages/ark3116/ark3116_0.4.1.bb > @@ -1,3 +1,7 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see module.bbclass > + > + > DESCRIPTION = "Driver for usb serial cables based upon ArkMicroChips > 3116s chip" > HOMEPAGE = "http://avr.auctionant.de/ark3116_linux_driver/" > PRIORITY = "optional" > diff --git a/packages/char-driver/char-driver.bb > b/packages/char-driver/char-driver.bb > index e6458b5..25245af 100644 > --- a/packages/char-driver/char-driver.bb > +++ b/packages/char-driver/char-driver.bb > @@ -1,3 +1,7 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see module.bbclass > + > + > DESCRIPTION = "char-driver and userspace program" > PRIORITY = "optional" > SECTION = "base" > diff --git a/packages/dri/drm-kernel_cvs.bb b/packages/dri/drm-kernel_cvs.bb > index 7e0bddf..ee68bac 100644 > --- a/packages/dri/drm-kernel_cvs.bb > +++ b/packages/dri/drm-kernel_cvs.bb > @@ -1,3 +1,6 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see modules.bbclass > + > SECTION = "x11/base" > PR = "r3" > LICENSE = "MIT" > diff --git a/packages/geode-drivers/geode-modules.inc > b/packages/geode-drivers/geode-modules.inc > index adaf885..9e78556 100644 > --- a/packages/geode-drivers/geode-modules.inc > +++ b/packages/geode-drivers/geode-modules.inc > @@ -1,3 +1,7 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see module.bbclass > + > + > # Linux kernel modules OE build files for the AMD Geode GX5535/LX5536 > # Copyright (C) 2005-2006, Advanced Micro Devices, Inc. All Rights Reserved > # Released under the MIT license (see packages/COPYING) > diff --git a/packages/gpio-event/gpio-event.bb > b/packages/gpio-event/gpio-event.bb > index afb2a77..52a5414 100644 > --- a/packages/gpio-event/gpio-event.bb > +++ b/packages/gpio-event/gpio-event.bb > @@ -1,3 +1,7 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see module.bbclass > + > + > DESCRIPTION = "gpio-event driver and userspace program" > PRIORITY = "optional" > SECTION = "base" > diff --git a/packages/pcmcia-cs/pcmcia-cs_3.2.8.bb > b/packages/pcmcia-cs/pcmcia-cs_3.2.8.bb > index 018f335..18a40c7 100644 > --- a/packages/pcmcia-cs/pcmcia-cs_3.2.8.bb > +++ b/packages/pcmcia-cs/pcmcia-cs_3.2.8.bb > @@ -1,3 +1,7 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see module.bbclass > + > + > DESCRIPTION = "Utilities and system configuration files for the Linux > PCMCIA card services" > SECTION = "base" > PRIORITY = "required" > diff --git a/packages/pvrusb2-mci/pvrusb2-mci_20060903.bb > b/packages/pvrusb2-mci/pvrusb2-mci_20060903.bb > index 5bd4ea2..af877e2 100644 > --- a/packages/pvrusb2-mci/pvrusb2-mci_20060903.bb > +++ b/packages/pvrusb2-mci/pvrusb2-mci_20060903.bb > @@ -1,3 +1,7 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see module.bbclass > + > + > require pvrusb2-mci2.inc > > RRECOMMENDS = "kernel-module-tveeprom \ > diff --git a/packages/pvrusb2-mci/pvrusb2-mci_20080210.bb > b/packages/pvrusb2-mci/pvrusb2-mci_20080210.bb > index 5bd4ea2..af877e2 100644 > --- a/packages/pvrusb2-mci/pvrusb2-mci_20080210.bb > +++ b/packages/pvrusb2-mci/pvrusb2-mci_20080210.bb > @@ -1,3 +1,7 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see module.bbclass > + > + > require pvrusb2-mci2.inc > > RRECOMMENDS = "kernel-module-tveeprom \ > diff --git a/packages/pwc/pwc_10.0.12-rc1.bb b/packages/pwc/pwc_10.0.12-rc1.bb > index af1d5f5..85e6ade 100644 > --- a/packages/pwc/pwc_10.0.12-rc1.bb > +++ b/packages/pwc/pwc_10.0.12-rc1.bb > @@ -1,3 +1,7 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see module.bbclass > + > + > DESCRIPTION = "Philips USB Webcam driver for Linux that supports VGA > resolution" > HOMEPAGE = "http://www.saillard.org/linux/pwc" > PRIORITY = "optional" > diff --git a/packages/wifistix/wifistix-modules_5.0.16.p0.bb > b/packages/wifistix/wifistix-modules_5.0.16.p0.bb > index 0eb6877..f9e8503 100644 > --- a/packages/wifistix/wifistix-modules_5.0.16.p0.bb > +++ b/packages/wifistix/wifistix-modules_5.0.16.p0.bb > @@ -1,3 +1,6 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see module.bbclass > + > DESCRIPTION = "Linux Driver for Marvel 88W8385 802.11b/g Wifi Module > used in Gumstix daughtercards" > SECTION = "base" > PRIORITY = "optional" > diff --git a/packages/wlan-ng/wlan-ng-modules.inc > b/packages/wlan-ng/wlan-ng-modules.inc > index 6dbadce..55fd026 100644 > --- a/packages/wlan-ng/wlan-ng-modules.inc > +++ b/packages/wlan-ng/wlan-ng-modules.inc > @@ -1,3 +1,7 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see module.bbclass > + > + > DESCRIPTION = "linux-wlan-ng (prism2.x, prism3, pcmcia, pci, usb) > driver for 11Mbps wireless lan cards" > HOMEPAGE = "http://www.linux-wlan.org" > SECTION = "kernel/modules" > diff --git a/packages/wlan-ng/wlan-ng-utils.inc > b/packages/wlan-ng/wlan-ng-utils.inc > index 72976ce..92f25dc 100644 > --- a/packages/wlan-ng/wlan-ng-utils.inc > +++ b/packages/wlan-ng/wlan-ng-utils.inc > @@ -1,3 +1,7 @@ > +# FIXME, consider using kernel staging directory instead of > KERNEL_SOURCE which is > +# located in the work directory. see module.bbclass > + > + > DESCRIPTION = "linux-wlan-ng userland utilities" > HOMEPAGE = "http://www.linux-wlan.org" > SECTION = "kernel/userland" > > ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2009-01-26 20:13 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2009-01-26 18:11 rm_work and building kernel modules Cliff Brake 2009-01-26 18:34 ` Koen Kooi 2009-01-26 19:24 ` Cliff Brake 2009-01-26 20:06 ` Koen Kooi
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.