* [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection
@ 2021-06-07 10:35 Priya N S
2021-06-07 10:35 ` [master PATCH 1/3] create-sdcard.sh: fix regex used for rootfs tarball Priya N S
` (3 more replies)
0 siblings, 4 replies; 6+ messages in thread
From: Priya N S @ 2021-06-07 10:35 UTC (permalink / raw)
To: meta-arago; +Cc: M V Pratap Reddy, Sekhar Nori
From: Priya N S <priya.ns@ti.com>
Hi All,
Below are the patches to fix the issues observed in the create-sdcard
script.
Issue:
1. The create-sdcard script uses 'rootfs' as a regex keyword to find
and list the filesystem tarballs which are available in the SDK
package. As in the current SDK packages, the tarball names are in the
following format,
tisdk-<IMAGE_TYPE>-image-<MACHINE>.tar.xz
Which doesn't have the rootfs keyword. This fails to list the tarballs
available in the package and crashes the script execution.
2. The create-sdcard script detects a partitioned SD card as not
partitioned because of the failure of the size check condition in the
script.
Fix:
1. The 'image' is used as the regex keyword to match the tarball names
in the SDK packages.
2. The usage of the cylinder size unit for partitioning the SD card is
removed since the cylinder size is SD card manufacturer dependent.
The parted command is replaced with fdisk since fdisk is an available
installed utility.
Priya N S (3):
create-sdcard.sh: fix regex used for rootfs tarball
create-sdcard: Fix wrong rootfs tarball selection
create-sdcard.sh: Stop using cylinder as unit for partition size
create-sdcard.sh | 75 +++++++++++++++++++++++++++++++++++++-----------
1 file changed, 58 insertions(+), 17 deletions(-)
--
2.19.1.windows.1
^ permalink raw reply [flat|nested] 6+ messages in thread
* [master PATCH 1/3] create-sdcard.sh: fix regex used for rootfs tarball
2021-06-07 10:35 [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection Priya N S
@ 2021-06-07 10:35 ` Priya N S
2021-06-07 10:35 ` [master PATCH 2/3] create-sdcard: Fix wrong rootfs tarball selection Priya N S
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Priya N S @ 2021-06-07 10:35 UTC (permalink / raw)
To: meta-arago; +Cc: M V Pratap Reddy, Sekhar Nori, Priya N S
From: Priya N S <priya.ns@ti.com>
From: Priya N S <priya.ns@ti.com>
Filesystem tarballs present in SDK are in the format
tisdk-<IMAGE_TYPE>-image-<MACHINE>.tar.xz. This is different from
earlier format where 'rootfs' was present in all the filesystem
tarball names. Update regex used for filesystem tarball search to
use "image" as the keyword.
Signed-off-by: Priya N S <priya.ns@ti.com>
Signed-off-by: Sekhar Nori <nsekhar@ti.com>
---
create-sdcard.sh | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/create-sdcard.sh b/create-sdcard.sh
index 736cd42..264095a 100644
--- a/create-sdcard.sh
+++ b/create-sdcard.sh
@@ -752,10 +752,9 @@ if [ $FILEPATHOPTION -eq 1 ] ; then
BOOTUENV=`ls $BOOTFILEPATH | grep uEnv.txt | awk {'print $1'}`
#rootfs
ROOTFILEPARTH="$PARSEPATH/filesystem"
- #ROOTFSTAR=`ls $ROOTFILEPARTH | grep tisdk.*rootfs | awk {'print $1'}`
#Make sure there is only 1 tar
- CHECKNUMOFTAR=`ls $ROOTFILEPARTH | grep "tisdk.*rootfs" | grep 'tar.xz' | grep -n '' | grep '2:' | awk {'print $1'}`
+ CHECKNUMOFTAR=`ls $ROOTFILEPARTH | grep "tisdk.*image" | grep 'tar.xz' | grep -n '' | grep '2:' | awk {'print $1'}`
if [ -n "$CHECKNUMOFTAR" ]
then
cat << EOM
@@ -767,15 +766,15 @@ cat << EOM
################################################################################
EOM
- ls --sort=size $ROOTFILEPARTH | grep "tisdk.*rootfs" | grep 'tar.xz' | grep -n '' | awk {'print " " , $1'}
+ ls --sort=size $ROOTFILEPARTH | grep "tisdk.*image" | grep 'tar.xz' | grep -n '' | awk {'print " " , $1'}
echo ""
read -p "Enter Number of rootfs Tarball: " TARNUMBER
echo " "
- FOUNDTARFILENAME=`ls --sort=size $ROOTFILEPARTH | grep "rootfs" | grep 'tar.xz' | grep -n '' | grep "${TARNUMBER}:" | cut -c3- | awk {'print$1'}`
+ FOUNDTARFILENAME=`ls --sort=size $ROOTFILEPARTH | grep "image" | grep 'tar.xz' | grep -n '' | grep "${TARNUMBER}:" | cut -c3- | awk {'print$1'}`
ROOTFSTAR=$FOUNDTARFILENAME
else
- ROOTFSTAR=`ls $ROOTFILEPARTH | grep "tisdk.*rootfs" | grep 'tar.xz' | awk {'print $1'}`
+ ROOTFSTAR=`ls $ROOTFILEPARTH | grep "tisdk.*image" | grep 'tar.xz' | awk {'print $1'}`
fi
ROOTFSUSERFILEPATH=$ROOTFILEPARTH/$ROOTFSTAR
--
2.19.1.windows.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [master PATCH 2/3] create-sdcard: Fix wrong rootfs tarball selection
2021-06-07 10:35 [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection Priya N S
2021-06-07 10:35 ` [master PATCH 1/3] create-sdcard.sh: fix regex used for rootfs tarball Priya N S
@ 2021-06-07 10:35 ` Priya N S
2021-06-07 10:35 ` [master PATCH 3/3] create-sdcard.sh: Stop using cylinder as unit for partition size Priya N S
2021-06-07 21:22 ` [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection Nishanth Menon
3 siblings, 0 replies; 6+ messages in thread
From: Priya N S @ 2021-06-07 10:35 UTC (permalink / raw)
To: meta-arago; +Cc: M V Pratap Reddy, Sekhar Nori, Priya N S
From: Priya N S <priya.ns@ti.com>
From: Priya N S <priya.ns@ti.com>
* Validate user input of rootfs tarball selection.
* This will not allow the user to proceed with wrong rootfs
tarball selection.
Signed-off-by: Priya N S <priya.ns@ti.com>
Signed-off-by: Sekhar Nori <nsekhar@ti.com>
---
create-sdcard.sh | 16 +++++++++++++---
1 file changed, 13 insertions(+), 3 deletions(-)
diff --git a/create-sdcard.sh b/create-sdcard.sh
index 264095a..8f99687 100644
--- a/create-sdcard.sh
+++ b/create-sdcard.sh
@@ -767,9 +767,19 @@ cat << EOM
EOM
ls --sort=size $ROOTFILEPARTH | grep "tisdk.*image" | grep 'tar.xz' | grep -n '' | awk {'print " " , $1'}
- echo ""
- read -p "Enter Number of rootfs Tarball: " TARNUMBER
- echo " "
+ COUNT=`ls $ROOTFILEPARTH | grep "tisdk.*image" | grep 'tar.xz' | grep -n '' | awk {'print $1'} | wc -l`
+ ENTERCORRECTLY="0"
+ while [ $ENTERCORRECTLY -ne 1 ]
+ do
+ read -p "Enter Number of rootfs Tarball: " TARNUMBER
+ echo " "
+ if [ -z "${TARNUMBER//[0-$COUNT]}" ] && [ -n "$TARNUMBER" ] ; then
+ ENTERCORRECTLY=1
+ else
+ echo "Invalid selection!"
+ fi
+ echo ""
+ done
FOUNDTARFILENAME=`ls --sort=size $ROOTFILEPARTH | grep "image" | grep 'tar.xz' | grep -n '' | grep "${TARNUMBER}:" | cut -c3- | awk {'print$1'}`
ROOTFSTAR=$FOUNDTARFILENAME
--
2.19.1.windows.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [master PATCH 3/3] create-sdcard.sh: Stop using cylinder as unit for partition size
2021-06-07 10:35 [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection Priya N S
2021-06-07 10:35 ` [master PATCH 1/3] create-sdcard.sh: fix regex used for rootfs tarball Priya N S
2021-06-07 10:35 ` [master PATCH 2/3] create-sdcard: Fix wrong rootfs tarball selection Priya N S
@ 2021-06-07 10:35 ` Priya N S
2021-06-07 21:22 ` [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection Nishanth Menon
3 siblings, 0 replies; 6+ messages in thread
From: Priya N S @ 2021-06-07 10:35 UTC (permalink / raw)
To: meta-arago; +Cc: M V Pratap Reddy, Sekhar Nori, Priya N S
From: Priya N S <priya.ns@ti.com>
From: Priya N S <priya.ns@ti.com>
* Cylinder size of sdcard is manufacturer dependent. To standardize
sdcard partitioning remove the use of cylinder (cyl) unit.
* While at it shift to using fdisk for the partitioning the sdcard.
fdisk is already used in the script and this reduces the dependencies
of this script on available installed applications.
* This fixes cases where already partitioned cards were being detected
as not partitioned because of size check elsewhere in the script
failing.
Signed-off-by: Priya N S <priya.ns@ti.com>
Signed-off-by: Sekhar Nori <nsekhar@ti.com>
---
create-sdcard.sh | 50 +++++++++++++++++++++++++++++++++++++++---------
1 file changed, 41 insertions(+), 9 deletions(-)
diff --git a/create-sdcard.sh b/create-sdcard.sh
index 8f99687..1114f70 100644
--- a/create-sdcard.sh
+++ b/create-sdcard.sh
@@ -501,11 +501,29 @@ SIZE=`fdisk -l $DRIVE | grep Disk | awk '{print $5}'`
echo DISK SIZE - $SIZE bytes
-parted -s $DRIVE mklabel msdos
-parted -s $DRIVE unit cyl mkpart primary fat32 -- 0 9
-parted -s $DRIVE set 1 boot on
-parted -s $DRIVE unit cyl mkpart primary ext2 -- 9 310
-parted -s $DRIVE unit cyl mkpart primary ext2 -- 310 -2
+cat << END | fdisk $DRIVE
+n
+p
+1
+
++75M
+n
+p
+2
+
++2.4G
+n
+p
+3
+
+
+t
+1
+c
+a
+1
+w
+END
cat << EOM
@@ -559,10 +577,24 @@ SIZE=`fdisk -l $DRIVE | grep Disk | awk '{print $5}'`
echo DISK SIZE - $SIZE bytes
-parted -s $DRIVE mklabel msdos
-parted -s $DRIVE unit cyl mkpart primary fat32 -- 0 9
-parted -s $DRIVE set 1 boot on
-parted -s $DRIVE unit cyl mkpart primary ext2 -- 9 -2
+cat << END | fdisk $DRIVE
+n
+p
+1
+
++75M
+n
+p
+2
+
+
+t
+1
+c
+a
+1
+w
+END
cat << EOM
--
2.19.1.windows.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection
2021-06-07 10:35 [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection Priya N S
` (2 preceding siblings ...)
2021-06-07 10:35 ` [master PATCH 3/3] create-sdcard.sh: Stop using cylinder as unit for partition size Priya N S
@ 2021-06-07 21:22 ` Nishanth Menon
2021-06-22 13:44 ` Vigna Raja Priya N S
3 siblings, 1 reply; 6+ messages in thread
From: Nishanth Menon @ 2021-06-07 21:22 UTC (permalink / raw)
To: Priya N S; +Cc: meta-arago, Sekhar Nori
On 16:05-20210607, Priya N S wrote:
> From: Priya N S <priya.ns@ti.com>
>
> Hi All,
> Below are the patches to fix the issues observed in the create-sdcard
> script.
a) Do these apply to meta-arago?
b) why not use the wic and bmap files instead?
>
> Issue:
> 1. The create-sdcard script uses 'rootfs' as a regex keyword to find
> and list the filesystem tarballs which are available in the SDK
> package. As in the current SDK packages, the tarball names are in the
> following format,
> tisdk-<IMAGE_TYPE>-image-<MACHINE>.tar.xz
> Which doesn't have the rootfs keyword. This fails to list the tarballs
> available in the package and crashes the script execution.
> 2. The create-sdcard script detects a partitioned SD card as not
> partitioned because of the failure of the size check condition in the
> script.
>
> Fix:
> 1. The 'image' is used as the regex keyword to match the tarball names
> in the SDK packages.
> 2. The usage of the cylinder size unit for partitioning the SD card is
> removed since the cylinder size is SD card manufacturer dependent.
> The parted command is replaced with fdisk since fdisk is an available
> installed utility.
>
> Priya N S (3):
> create-sdcard.sh: fix regex used for rootfs tarball
> create-sdcard: Fix wrong rootfs tarball selection
> create-sdcard.sh: Stop using cylinder as unit for partition size
>
> create-sdcard.sh | 75 +++++++++++++++++++++++++++++++++++++-----------
> 1 file changed, 58 insertions(+), 17 deletions(-)
>
> --
> 2.19.1.windows.1
>
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection
2021-06-07 21:22 ` [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection Nishanth Menon
@ 2021-06-22 13:44 ` Vigna Raja Priya N S
0 siblings, 0 replies; 6+ messages in thread
From: Vigna Raja Priya N S @ 2021-06-22 13:44 UTC (permalink / raw)
To: Nishanth Menon; +Cc: meta-arago, Sekhar Nori
On Tue, Jun 8, 2021 at 2:52 AM Nishanth Menon <nm@ti.com> wrote:
>
> On 16:05-20210607, Priya N S wrote:
> > From: Priya N S <priya.ns@ti.com>
> >
> > Hi All,
> > Below are the patches to fix the issues observed in the create-sdcard
> > script.
>
> a) Do these apply to meta-arago?
These patches need to be applied to the tisdk-setup-scripts repo.
I will update the subject line of the patches with the repo name in V2.
> b) why not use the wic and bmap files instead?
SD create scripts are supported in the SDK 7.03 release. Need to align
if we need to remove the support for SD create scripts and migrate to
wic files completely in the future release.
>
> >
> > Issue:
> > 1. The create-sdcard script uses 'rootfs' as a regex keyword to find
> > and list the filesystem tarballs which are available in the SDK
> > package. As in the current SDK packages, the tarball names are in the
> > following format,
> > tisdk-<IMAGE_TYPE>-image-<MACHINE>.tar.xz
> > Which doesn't have the rootfs keyword. This fails to list the tarballs
> > available in the package and crashes the script execution.
> > 2. The create-sdcard script detects a partitioned SD card as not
> > partitioned because of the failure of the size check condition in the
> > script.
> >
> > Fix:
> > 1. The 'image' is used as the regex keyword to match the tarball names
> > in the SDK packages.
> > 2. The usage of the cylinder size unit for partitioning the SD card is
> > removed since the cylinder size is SD card manufacturer dependent.
> > The parted command is replaced with fdisk since fdisk is an available
> > installed utility.
> >
> > Priya N S (3):
> > create-sdcard.sh: fix regex used for rootfs tarball
> > create-sdcard: Fix wrong rootfs tarball selection
> > create-sdcard.sh: Stop using cylinder as unit for partition size
> >
> > create-sdcard.sh | 75 +++++++++++++++++++++++++++++++++++++-----------
> > 1 file changed, 58 insertions(+), 17 deletions(-)
> >
> > --
> > 2.19.1.windows.1
> >
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>
> --
> Regards,
> Nishanth Menon
> Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-06-22 13:44 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-07 10:35 [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection Priya N S
2021-06-07 10:35 ` [master PATCH 1/3] create-sdcard.sh: fix regex used for rootfs tarball Priya N S
2021-06-07 10:35 ` [master PATCH 2/3] create-sdcard: Fix wrong rootfs tarball selection Priya N S
2021-06-07 10:35 ` [master PATCH 3/3] create-sdcard.sh: Stop using cylinder as unit for partition size Priya N S
2021-06-07 21:22 ` [master PATCH 0/3] create-sdcard: Fix rootfs tarball search and selection Nishanth Menon
2021-06-22 13:44 ` Vigna Raja Priya N S
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.