| Message ID | 20200311171105.3558036-4-fabian@blaese.de |
|---|---|
| State | Superseded |
| Headers | show |
diff --git a/bsp/ar71xx/.config b/bsp/ar71xx/.config index eaf5ea4..e453386 100644 --- a/bsp/ar71xx/.config +++ b/bsp/ar71xx/.config @@ -101,4 +101,3 @@ CONFIG_PACKAGE_hostapd-mini=y CONFIG_PACKAGE_opkg=m CONFIG_PACKAGE_wpad-mini=m CONFIG_STRIP_KERNEL_EXPORTS=y -CONFIG_TARGET_SQUASHFS_BLOCK_SIZE=512
Hi, > -----Original Message----- > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of > Fabian Bläse > Sent: Mittwoch, 11. März 2020 18:11 > To: franken-dev@freifunk.net > Subject: [PATCH 3/5] Use OpenWRT default for squashfs blocksize > > Previously, the squashfs blocksize has been increased to 512 bytes, > to increase compression efficiency, because the OpenWRT default blocksize > has been only 256 bytes. > > Since OpenWRT 18.07 the blocksize has been increased to 1024 bytes for > devices with a small flash. Entweder 18.06 oder 19.07. Ansonsten Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> Muss das für die anderen configs auch entfernt werden, wenn ja, dann bitte auf einmal. Grüße Adrian > > Because increased blocksize gives us additional headroom for devices > with only 4 MiB flash, our own 512 byte override is removed. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > bsp/ar71xx/.config | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/bsp/ar71xx/.config b/bsp/ar71xx/.config > index eaf5ea4..e453386 100644 > --- a/bsp/ar71xx/.config > +++ b/bsp/ar71xx/.config > @@ -101,4 +101,3 @@ CONFIG_PACKAGE_hostapd-mini=y > CONFIG_PACKAGE_opkg=m > CONFIG_PACKAGE_wpad-mini=m > CONFIG_STRIP_KERNEL_EXPORTS=y > -CONFIG_TARGET_SQUASHFS_BLOCK_SIZE=512 > -- > 2.25.1
On 11.03.20 19:05, Adrian Schmutzler wrote: > Hi, > >> -----Original Message----- >> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of >> Fabian Bläse >> Sent: Mittwoch, 11. März 2020 18:11 >> To: franken-dev@freifunk.net >> Subject: [PATCH 3/5] Use OpenWRT default for squashfs blocksize >> >> Previously, the squashfs blocksize has been increased to 512 bytes, >> to increase compression efficiency, because the OpenWRT default blocksize >> has been only 256 bytes. >> >> Since OpenWRT 18.07 the blocksize has been increased to 1024 bytes for >> devices with a small flash. > > Entweder 18.06 oder 19.07. Verdammt. 18.06 natürlich. Search & Replace war wohl keine gute Idee. Wenn das das einzige bleibt, bitte beim applien fixen. > > Ansonsten > > Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > Muss das für die anderen configs auch entfernt werden, wenn ja, dann bitte auf einmal. Die anderen sind keine small_flash Geräte, da ist der Default 256, also weniger. Siehe: https://github.com/openwrt/openwrt/blob/7d7aa2fd924c27829ec25f825481554dd81bce97/config/Config-images.in#L154 Deswegen wollte ich das hier jetzt nicht anfassen. Man könnte aber durchaus darüber diskutieren, ob man da auf den OpenWRT default zurück möchte. Gruß Fabian
Hi, aus deiner Commit Beschreibung geht nicht hervor, dass es sich hier um eine ar71xx-spezifische Änderung handelt. Ja, ich weiß, das wird von small_flash getriggert, und wir haben z.Zt. nur ein Target mit small_flash, aber da muss man schon recht viel wissen, um das schlussfolgern zu können. Also entweder ein "ar71xx:" vor den Commit Title und ein entsprechender Hinweis in der Message, oder aber gleich alles auf OpenWrt zurück. Ich würde letzteres bevorzugen. Grüße Adrian > -----Original Message----- > From: Fabian Bläse [mailto:fabian@blaese.de] > Sent: Mittwoch, 11. März 2020 19:31 > To: Adrian Schmutzler <mail@adrianschmutzler.de>; franken-dev@freifunk.net > Subject: Re: [PATCH 3/5] Use OpenWRT default for squashfs blocksize > > > > On 11.03.20 19:05, Adrian Schmutzler wrote: > > Hi, > > > >> -----Original Message----- > >> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of > >> Fabian Bläse > >> Sent: Mittwoch, 11. März 2020 18:11 > >> To: franken-dev@freifunk.net > >> Subject: [PATCH 3/5] Use OpenWRT default for squashfs blocksize > >> > >> Previously, the squashfs blocksize has been increased to 512 bytes, > >> to increase compression efficiency, because the OpenWRT default blocksize > >> has been only 256 bytes. > >> > >> Since OpenWRT 18.07 the blocksize has been increased to 1024 bytes for > >> devices with a small flash. > > > > Entweder 18.06 oder 19.07. > Verdammt. 18.06 natürlich. Search & Replace war wohl keine gute Idee. > Wenn das das einzige bleibt, bitte beim applien fixen. > > > > > Ansonsten > > > > Reviewed-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > > > Muss das für die anderen configs auch entfernt werden, wenn ja, dann bitte > auf einmal. > Die anderen sind keine small_flash Geräte, da ist der Default 256, also weniger. > Siehe: > https://github.com/openwrt/openwrt/blob/7d7aa2fd924c27829ec25f825481554d > d81bce97/config/Config-images.in#L154 > > Deswegen wollte ich das hier jetzt nicht anfassen. > Man könnte aber durchaus darüber diskutieren, ob man da auf den OpenWRT > default zurück möchte. > > Gruß > Fabian
Reviewed-by: Robert Langhammer <rlanghammer@web.de> Am 11.03.20 um 18:11 schrieb Fabian Bläse: > Previously, the squashfs blocksize has been increased to 512 bytes, > to increase compression efficiency, because the OpenWRT default blocksize > has been only 256 bytes. > > Since OpenWRT 18.07 the blocksize has been increased to 1024 bytes for > devices with a small flash. > > Because increased blocksize gives us additional headroom for devices > with only 4 MiB flash, our own 512 byte override is removed. > > Signed-off-by: Fabian Bläse <fabian@blaese.de> > --- > bsp/ar71xx/.config | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/bsp/ar71xx/.config b/bsp/ar71xx/.config > index eaf5ea4..e453386 100644 > --- a/bsp/ar71xx/.config > +++ b/bsp/ar71xx/.config > @@ -101,4 +101,3 @@ CONFIG_PACKAGE_hostapd-mini=y > CONFIG_PACKAGE_opkg=m > CONFIG_PACKAGE_wpad-mini=m > CONFIG_STRIP_KERNEL_EXPORTS=y > -CONFIG_TARGET_SQUASHFS_BLOCK_SIZE=512
Previously, the squashfs blocksize has been increased to 512 bytes, to increase compression efficiency, because the OpenWRT default blocksize has been only 256 bytes. Since OpenWRT 18.07 the blocksize has been increased to 1024 bytes for devices with a small flash. Because increased blocksize gives us additional headroom for devices with only 4 MiB flash, our own 512 byte override is removed. Signed-off-by: Fabian Bläse <fabian@blaese.de> --- bsp/ar71xx/.config | 1 - 1 file changed, 1 deletion(-)