[OpenWrt-Devel] [PATCH 2/4] ubinize-image: Change the rootfs to a static volume

Maxime Ripard maxime.ripard at free-electrons.com
Mon Oct 27 06:16:13 EDT 2014


Hi Daniel,

On Sun, Oct 26, 2014 at 11:37:18PM +0100, Daniel Golle wrote:
> Hi Maxime,
> 
> On Thu, Oct 09, 2014 at 05:59:27PM +0200, Maxime Ripard wrote:
> > On boards with large page size, the rootfs we generate might end up using less
> > PEB than the minimum number required by UBI for a dynamic volume.
> > 
> > Change the rootfs to a static volume, which removes such a requirement, and
> > isn't changing anything, since our rootfs is in read only anyway.
> > 
> > Signed-off-by: Maxime Ripard <maxime.ripard at free-electrons.com>
> > ---
> >  scripts/ubinize-image.sh | 6 +++++-
> >  1 file changed, 5 insertions(+), 1 deletion(-)
> > 
> > diff --git a/scripts/ubinize-image.sh b/scripts/ubinize-image.sh
> > index 6762c22bc4a6..11c25ecc8ee1 100755
> > --- a/scripts/ubinize-image.sh
> > +++ b/scripts/ubinize-image.sh
> > @@ -25,13 +25,17 @@ ubivol() {
> >  	echo "[$name]"
> >  	echo "mode=ubi"
> >  	echo "vol_id=$volid"
> > -	echo "vol_type=dynamic"
> >  	echo "vol_name=$name"
> >  	if [ "$image" ]; then
> >  		echo "image=$image"
> >  	else
> >  		echo "vol_size=1MiB"
> >  	fi
> > +	if [ "$name" = "rootfs" ]; then
> > +		echo "vol_type=static"
> > +	else
> > +		echo "vol_type=dynamic"
> > +	fi
> 
> This will break things if rootfs is a read-write UBIFS volume which should
> by dynamic.

Indeed, but is this something that is usually done in OpenWRT?

> I also saw weird things happening when trying to access static
> volumes in U-Boot and thus avoided them for now.
> Please also consider that ubootenv* and kernel could be static volumes as
> well, however, that needs to be tested with a more recent U-Boot (I hope
> it'll work).

Indeed, the kernel and bootloader might be static too.

The only other solution I could think of was to force the size of the
ubifs rootfs image to something like 10MB in our case. Which looks a
bit counter-productive.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/openwrt-devel/attachments/20141027/5b3418c1/attachment.sig>
-------------- next part --------------
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel


More information about the openwrt-devel mailing list