All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ia32-base: Remove cpio and ext3 defaults
@ 2013-11-21 15:25 Richard Purdie
  2013-11-21 16:11 ` Hart, Darren
  0 siblings, 1 reply; 4+ messages in thread
From: Richard Purdie @ 2013-11-21 15:25 UTC (permalink / raw)
  To: openembedded-core; +Cc: meta-intel, Hart, Darren

On real IA hardware, neither the ext3 or cpio images are particularly useful
or used. cpio is legacy from initramfs and that specific image now overrides
FSTYPES accordingly. The size difference in filesystems makes ext3 as a file
format less useful, mainly being useful in the qemu case.

When needed users can still override the default FSTYPES so having
saner defaults makes sense. This improves build times and uses less
network bandwidth for builds and releases.

Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
---
diff --git a/meta/conf/machine/include/ia32-base.inc b/meta/conf/machine/include/ia32-base.inc
index 8a20bca..e15f927 100644
--- a/meta/conf/machine/include/ia32-base.inc
+++ b/meta/conf/machine/include/ia32-base.inc
@@ -10,7 +10,7 @@ MACHINE_FEATURES += "screen keyboard pci usbhost ext2 ext3 x86 \
 
 MACHINE_EXTRA_RRECOMMENDS += "kernel-modules"
 
-IMAGE_FSTYPES += "ext3 cpio.gz live"
+IMAGE_FSTYPES += "live"
 
 KERNEL_IMAGETYPE ?= "bzImage"
 




^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] ia32-base: Remove cpio and ext3 defaults
  2013-11-21 15:25 [PATCH] ia32-base: Remove cpio and ext3 defaults Richard Purdie
@ 2013-11-21 16:11 ` Hart, Darren
  2013-11-22 10:49   ` Behrens, Holger
  0 siblings, 1 reply; 4+ messages in thread
From: Hart, Darren @ 2013-11-21 16:11 UTC (permalink / raw)
  To: Richard Purdie; +Cc: meta-intel@yoctoproject.org, openembedded-core

On Thu, 2013-11-21 at 15:25 +0000, Richard Purdie wrote:
> On real IA hardware, neither the ext3 or cpio images are particularly useful
> or used. cpio is legacy from initramfs and that specific image now overrides
> FSTYPES accordingly. The size difference in filesystems makes ext3 as a file
> format less useful, mainly being useful in the qemu case.
> 
> When needed users can still override the default FSTYPES so having
> saner defaults makes sense. This improves build times and uses less
> network bandwidth for builds and releases.
> 
> Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>

I'm fine with this, although it raises the question if we ought to be
doing anything more than tar rootfs in include files like this. there
are ia32 machines where a live image is not useful (although rare). Most
x86 BSPs would add live - but then again, most x86 BSPs will be starting
to consolidate on the genericx86 BSPs anyway....

But, this is a clear improvement as is.

Acked-by: Darren Hart <dvhart@linux.intel.com>

> ---
> diff --git a/meta/conf/machine/include/ia32-base.inc b/meta/conf/machine/include/ia32-base.inc
> index 8a20bca..e15f927 100644
> --- a/meta/conf/machine/include/ia32-base.inc
> +++ b/meta/conf/machine/include/ia32-base.inc
> @@ -10,7 +10,7 @@ MACHINE_FEATURES += "screen keyboard pci usbhost ext2 ext3 x86 \
>  
>  MACHINE_EXTRA_RRECOMMENDS += "kernel-modules"
>  
> -IMAGE_FSTYPES += "ext3 cpio.gz live"
> +IMAGE_FSTYPES += "live"
>  
>  KERNEL_IMAGETYPE ?= "bzImage"
>  
> 
> 

-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel


^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ia32-base: Remove cpio and ext3 defaults
  2013-11-21 16:11 ` Hart, Darren
@ 2013-11-22 10:49   ` Behrens, Holger
  2013-11-22 10:51     ` Robert P. J. Day
  0 siblings, 1 reply; 4+ messages in thread
From: Behrens, Holger @ 2013-11-22 10:49 UTC (permalink / raw)
  To: Hart, Darren, Richard Purdie
  Cc: meta-intel@yoctoproject.org, openembedded-core

> On Thu, 2013-11-21 at 15:25 +0000, Richard Purdie wrote:
> > On real IA hardware, neither the ext3 or cpio images are particularly useful
> > or used. cpio is legacy from initramfs and that specific image now overrides
> > FSTYPES accordingly. The size difference in filesystems makes ext3 as a file
> > format less useful, mainly being useful in the qemu case.
> >
> > When needed users can still override the default FSTYPES so having
> > saner defaults makes sense. This improves build times and uses less
> > network bandwidth for builds and releases.
> >
> > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> 
> I'm fine with this, although it raises the question if we ought to be
> doing anything more than tar rootfs in include files like this. there
> are ia32 machines where a live image is not useful (although rare). Most
> x86 BSPs would add live - but then again, most x86 BSPs will be starting
> to consolidate on the genericx86 BSPs anyway....

live image does not build in case of INCOMPATIBE_LICENSE = GPLv3

Holger

> But, this is a clear improvement as is.
> 
> Acked-by: Darren Hart <dvhart@linux.intel.com>
> 
> > ---
> > diff --git a/meta/conf/machine/include/ia32-base.inc
> b/meta/conf/machine/include/ia32-base.inc
> > index 8a20bca..e15f927 100644
> > --- a/meta/conf/machine/include/ia32-base.inc
> > +++ b/meta/conf/machine/include/ia32-base.inc
> > @@ -10,7 +10,7 @@ MACHINE_FEATURES += "screen keyboard pci usbhost
> ext2 ext3 x86 \
> >
> >  MACHINE_EXTRA_RRECOMMENDS += "kernel-modules"
> >
> > -IMAGE_FSTYPES += "ext3 cpio.gz live"
> > +IMAGE_FSTYPES += "live"
> >
> >  KERNEL_IMAGETYPE ?= "bzImage"
> >
> >
> >
> 
> --
> Darren Hart
> Intel Open Source Technology Center
> Yocto Project - Linux Kernel
> 
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] ia32-base: Remove cpio and ext3 defaults
  2013-11-22 10:49   ` Behrens, Holger
@ 2013-11-22 10:51     ` Robert P. J. Day
  0 siblings, 0 replies; 4+ messages in thread
From: Robert P. J. Day @ 2013-11-22 10:51 UTC (permalink / raw)
  To: Behrens, Holger
  Cc: meta-intel@yoctoproject.org, Hart, Darren, openembedded-core

On Fri, 22 Nov 2013, Behrens, Holger wrote:

> > On Thu, 2013-11-21 at 15:25 +0000, Richard Purdie wrote:
> > > On real IA hardware, neither the ext3 or cpio images are particularly useful
> > > or used. cpio is legacy from initramfs and that specific image now overrides
> > > FSTYPES accordingly. The size difference in filesystems makes ext3 as a file
> > > format less useful, mainly being useful in the qemu case.
> > >
> > > When needed users can still override the default FSTYPES so having
> > > saner defaults makes sense. This improves build times and uses less
> > > network bandwidth for builds and releases.
> > >
> > > Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> >
> > I'm fine with this, although it raises the question if we ought to be
> > doing anything more than tar rootfs in include files like this. there
> > are ia32 machines where a live image is not useful (although rare). Most
> > x86 BSPs would add live - but then again, most x86 BSPs will be starting
> > to consolidate on the genericx86 BSPs anyway....
>
> live image does not build in case of INCOMPATIBE_LICENSE = GPLv3
                                                ^^^?
  is that just a mail list typo or a copy and paste error?

rday

-- 

========================================================================
Robert P. J. Day                                 Ottawa, Ontario, CANADA
                        http://crashcourse.ca

Twitter:                                       http://twitter.com/rpjday
LinkedIn:                               http://ca.linkedin.com/in/rpjday
========================================================================


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-11-22 10:51 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-11-21 15:25 [PATCH] ia32-base: Remove cpio and ext3 defaults Richard Purdie
2013-11-21 16:11 ` Hart, Darren
2013-11-22 10:49   ` Behrens, Holger
2013-11-22 10:51     ` Robert P. J. Day

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.