On Fri, Jan 06, 2023 at 06:04:22PM -0800, Brian Norris wrote: > We're assuming all root= arguments are /dev/ paths, but many targets > utilize root=PARTUUID=<xxx> strategies. At least allow them to fall back > to scanning all block devices. > > Signed-off-by: Brian Norris <computersforpe...@gmail.com>
Can you elaborate this a bit more? This is dependent of the google based devices with ipq806x but why? In theory we should have other device with sd card/eMMC that use uuid to select the partition... Also can't we just ignore the device bootargs and provide a custom one? > --- > > Changes in v2: > * fstools, not fsutils (sorry for the noise) > > libfstools/partname.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/libfstools/partname.c b/libfstools/partname.c > index f59c52eb8f3c..9c27015643ad 100644 > --- a/libfstools/partname.c > +++ b/libfstools/partname.c > @@ -128,12 +128,12 @@ static struct volume *partname_volume_find(char *name) > return NULL; > } > > - if (get_var_from_file("/proc/cmdline", "root", rootparam, > sizeof(rootparam))) { > + if (get_var_from_file("/proc/cmdline", "root", rootparam, > sizeof(rootparam)) && rootparam[0] == '/') { > rootdev = rootdevname(rootparam); > /* find partition on same device as rootfs */ > snprintf(ueventgstr, sizeof(ueventgstr), "%s/%s/*/uevent", > block_dir_name, rootdev); > } else { > - /* no 'root=' kernel cmdline parameter, find on any block > device */ > + /* no useful 'root=' kernel cmdline parameter, find on any > block device */ > snprintf(ueventgstr, sizeof(ueventgstr), "%s/*/uevent", > block_dir_name); > } > > -- > 2.39.0 > > > _______________________________________________ > openwrt-devel mailing list > openwrt-devel@lists.openwrt.org > https://lists.openwrt.org/mailman/listinfo/openwrt-devel -- Ansuel _______________________________________________ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/mailman/listinfo/openwrt-devel