<feed xmlns='http://www.w3.org/2005/Atom'>
<title>staging/chunkeey/include/image.mk, branch v19.07.3</title>
<subtitle>Staging tree of Christian Lamparter</subtitle>
<id>https://git.openwrt.org/openwrt/staging/chunkeey/atom?h=v19.07.3</id>
<link rel='self' href='https://git.openwrt.org/openwrt/staging/chunkeey/atom?h=v19.07.3'/>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/openwrt/staging/chunkeey/'/>
<updated>2019-09-08T16:50:11Z</updated>
<entry>
<title>Revert "build: remove harmful -nopad option from mksquashfs"</title>
<updated>2019-09-08T16:50:11Z</updated>
<author>
<name>Jonas Gorski</name>
</author>
<published>2019-09-06T12:55:36Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/openwrt/staging/chunkeey/commit/?id=4c8258c925a0cd6a4738b9eed7cd503c1873e88f'/>
<id>urn:sha1:4c8258c925a0cd6a4738b9eed7cd503c1873e88f</id>
<content type='text'>
This reverts commit 1c0290c5cc6258c48b8ba46b4f9c85a21de4f875.

Dropping the nopad can make the padding overflow into the next erase
block on devices using a non-aligned rootfs start. This breaks the jffs2
overlay partition with the following messages:

[   30.343877] jffs2_scan_eraseblock(): End of filesystem marker found at 0x10000
[   30.376512] jffs2: Cowardly refusing to erase blocks on filesystem with no valid JFFS2 nodes
[   30.385253] jffs2: empty_blocks 196, bad_blocks 0, c-&gt;nr_blocks 197

Signed-off-by: Jonas Gorski &lt;jonas.gorski@gmail.com&gt;
(cherry picked from commit f11d90a76b719106336b94d85b166b4ebf19dbb0)
</content>
</entry>
<entry>
<title>build: remove harmful -nopad option from mksquashfs</title>
<updated>2019-09-04T11:44:20Z</updated>
<author>
<name>Christian Lamparter</name>
</author>
<published>2019-08-30T14:52:25Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/openwrt/staging/chunkeey/commit/?id=5a102a010fe6be82d5793104acefabbbc01d6b85'/>
<id>urn:sha1:5a102a010fe6be82d5793104acefabbbc01d6b85</id>
<content type='text'>
While the -nopad option prevents mksquashfs from padding the
image to an arbitrary 4k. It does not take into consideration
that squashfs is programmed to have this 4k padding when it's
being used on on a block device... which is its main "use-case".

Now, after a week long discussion on the ML that included a
back-and-forth between some of the possible options.
But this is likely the best KISS patch to deal with the issue
right away given the limited resources.

From squashfs code point of view, be warned. The 4k padding is
not enough when dealing with devices that have a PAGE_SIZE
bigger than 4k.

if it turns out to be affecting you, then please look-up either:
"FS#2460 - kernel panic reading squashfs from ubi volume" bug
Or the discussion on the OpenWrt-Devel ML in
"amp821xx: use newly added pad-squashfs for Meraki MR24" and
"Squashfs breakage lottery with UBI..."
before making an educated guess.

Note: This will not affect the "tiny"/small flash devices as
much as it seems at first. This is because the the rootfs_data
partition that follows uses jffs2. And it requires to be aligned
to the flash block-size in order to work at all.

So either the involved FSes will meet in the middle as before,
or not at all. But in that latter case the image was already
hoping for the "undefined behaviour" gamble to turn out in its
favour and this is probably why this was unnoticed for so long.

Fixes: FS#2460
Reported-by: Russell Senior &lt;russell@personaltelco.net&gt;
Signed-off-by: Christian Lamparter &lt;chunkeey@gmail.com&gt;
(cherry picked from commit 1c0290c5cc6258c48b8ba46b4f9c85a21de4f875)
</content>
</entry>
<entry>
<title>build: allow overriding default selection state for devices</title>
<updated>2019-09-04T11:39:37Z</updated>
<author>
<name>Jonas Gorski</name>
</author>
<published>2019-08-13T10:33:47Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/openwrt/staging/chunkeey/commit/?id=f783ad5b5b105407fd47123b02217a8a09525b44'/>
<id>urn:sha1:f783ad5b5b105407fd47123b02217a8a09525b44</id>
<content type='text'>
Allow overriding the default selection state for Devices, similar to
setting a default for packages.

E.g. by setting DEFAULT to n, they won't be selected by default anymore
when enabling all device in the multi device profile.

This allows preventing images being built by the default config for
known broken devices, devices without enough RAM/flash, or devices not
working with a certain kernel versions.

This does not prevent the devices from being manually selected or images
being built by the ImageBuilder. These devices often still have worth
with a reduced package-set, or as a device for regression testing, when
no better device is available.

Signed-off-by: Jonas Gorski &lt;jonas.gorski@gmail.com&gt;
(cherry picked from commit 7546be60074e452751ba2a48eddbc13910bec708)
</content>
</entry>
<entry>
<title>build: add UBOOT_PATH to DEFAULT_DEVICE_VARS and set a default</title>
<updated>2019-05-18T14:37:30Z</updated>
<author>
<name>Christian Lamparter</name>
</author>
<published>2019-02-11T17:55:32Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/openwrt/staging/chunkeey/commit/?id=80f861ef757af33828edcefba8666dec5781c8db'/>
<id>urn:sha1:80f861ef757af33828edcefba8666dec5781c8db</id>
<content type='text'>
The helper shared Build/append-uboot in include/image-commands.mk
uses it, so include this variable in DEFAULT_DEVICE_VARS.

Signed-off-by: Christian Lamparter &lt;chunkeey@gmail.com&gt;
</content>
</entry>
<entry>
<title>build: remove sgid permission from tar</title>
<updated>2019-04-06T17:14:06Z</updated>
<author>
<name>Eneas U de Queiroz</name>
</author>
<published>2019-03-28T16:58:07Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/openwrt/staging/chunkeey/commit/?id=8252511dc0b5a71e9e64b96f233a27ad73e28b7f'/>
<id>urn:sha1:8252511dc0b5a71e9e64b96f233a27ad73e28b7f</id>
<content type='text'>
Otherwise tar will keep the sgid bit when running
from a sgid-set directory, resulting in a different
file being generated.

Signed-off-by: Eneas U de Queiroz &lt;cote2004-github@yahoo.com&gt;
Signed-off-by: Christian Lamparter &lt;chunkeey@gmail.com&gt;
[reworked commit message, removed DMARC]
</content>
</entry>
<entry>
<title>build: image: Fix off-by-one in DTC kernel version checks</title>
<updated>2019-04-01T11:53:31Z</updated>
<author>
<name>Petr Štetiar</name>
</author>
<published>2019-04-01T11:53:31Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/openwrt/staging/chunkeey/commit/?id=6027e490930a2a5ade1c7056303ebc18740ec6f1'/>
<id>urn:sha1:6027e490930a2a5ade1c7056303ebc18740ec6f1</id>
<content type='text'>
It was reported to me on IRC today, that my change causes issues with
kernel versions between 4.14 and 4.19.

It's because I've wrongly used `git describe` in order to get kernel
version where we should disable noisy DTC checks, but I should've used
`git tag --contains` instead.

Fixes: cbbef976e2b ("build: dtc: Disable noisy warnings by default")
Signed-off-by: Petr Štetiar &lt;ynezz@true.cz&gt;
</content>
</entry>
<entry>
<title>IB: include SUPPORTED_DEVICES in 'make info' output</title>
<updated>2019-03-11T04:32:29Z</updated>
<author>
<name>Daniel Golle</name>
</author>
<published>2019-03-11T04:32:29Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/openwrt/staging/chunkeey/commit/?id=d6fa04a437033d4bdfa1b0d90a4786bfe492f749'/>
<id>urn:sha1:d6fa04a437033d4bdfa1b0d90a4786bfe492f749</id>
<content type='text'>
Signed-off-by: Daniel Golle &lt;daniel@makrotopia.org&gt;
</content>
</entry>
<entry>
<title>ib: display whether profile comes with image metadata</title>
<updated>2019-03-06T13:50:42Z</updated>
<author>
<name>Daniel Golle</name>
</author>
<published>2019-03-06T13:45:15Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/openwrt/staging/chunkeey/commit/?id=13c379e5c6e37b3822bf6e4e18cc6317e5b40fbb'/>
<id>urn:sha1:13c379e5c6e37b3822bf6e4e18cc6317e5b40fbb</id>
<content type='text'>
Having image metadata (and signature) appended is a condition for
semi-automated sysupgrade, hence IB needs to be able to tell which
images will end up with metadata.

Signed-off-by: Daniel Golle &lt;daniel@makrotopia.org&gt;
</content>
</entry>
<entry>
<title>tools: migrate from squashfs4 to squashfskit4</title>
<updated>2019-02-28T00:38:46Z</updated>
<author>
<name>Alexander Couzens</name>
</author>
<published>2018-12-12T15:10:42Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/openwrt/staging/chunkeey/commit/?id=889b6423b78bcde8af81d68a3fabcdd019181de4'/>
<id>urn:sha1:889b6423b78bcde8af81d68a3fabcdd019181de4</id>
<content type='text'>
squashfskit is a fork of the squashfs-tools.
squashfskit creates reproducible filesystems and includes
many of the distro patches.

Signed-off-by: Alexander Couzens &lt;lynxis@fe80.eu&gt;
Tested-by: Paul Spooren &lt;mail@aparcar.org&gt;
</content>
</entry>
<entry>
<title>build: Accept BIN_DIR parameter for legacy-images</title>
<updated>2019-02-17T18:22:39Z</updated>
<author>
<name>Sven Eckelmann</name>
</author>
<published>2019-02-11T15:26:42Z</published>
<link rel='alternate' type='text/html' href='https://git.openwrt.org/openwrt/staging/chunkeey/commit/?id=9a5a10eb6924efa519e1d9e27b61dc254876f9ec'/>
<id>urn:sha1:9a5a10eb6924efa519e1d9e27b61dc254876f9ec</id>
<content type='text'>
BIN_DIR can be set to overwrite the output path for new images. This is an
advertised feature for the imagebuilder and is used by systems like
LibreMesh's chef.

The legacy images are build using a new sub-make which doesn't receive the
variable overwrites of the parent make process. As result, the BIN_DIR is
automatically defined to the default value from rules.mk. The images will
therefore not be placed in the output path which was selected by the user.

Providing BIN_DIR as an explicit variable override to the sub-make works
around this problem.

Fixes: 26c771452cd8 ("image.mk: add LegacyDevice wrapper to allow legacy image building code to be used for device profiles")
Reported-by: Paul Spooren &lt;mail@aparcar.org&gt;
Signed-off-by: Sven Eckelmann &lt;sven@narfation.org&gt;
</content>
</entry>
</feed>
