* [meta-arago][master][PATCH] base-files: Remove fstab override @ 2024-03-05 19:11 Andrew Davis 2024-03-05 20:49 ` Denys Dmytriyenko 0 siblings, 1 reply; 6+ messages in thread From: Andrew Davis @ 2024-03-05 19:11 UTC (permalink / raw) To: Denys Dmytriyenko, Ryan Eatmon, meta-arago; +Cc: Andrew Davis There is already a version of this file provided by oe-core that is kept up to date. Do not override it. Signed-off-by: Andrew Davis <afd@ti.com> --- .../recipes-core/base-files/base-files/fstab | 13 ------------- .../recipes-core/base-files/base-files_%.bbappend | 2 +- 2 files changed, 1 insertion(+), 14 deletions(-) delete mode 100644 meta-arago-distro/recipes-core/base-files/base-files/fstab diff --git a/meta-arago-distro/recipes-core/base-files/base-files/fstab b/meta-arago-distro/recipes-core/base-files/base-files/fstab deleted file mode 100644 index 8ddadd98..00000000 --- a/meta-arago-distro/recipes-core/base-files/base-files/fstab +++ /dev/null @@ -1,13 +0,0 @@ -# stock fstab - you probably want to override this with a machine specific one - -/dev/root / auto defaults 1 1 -proc /proc proc defaults 0 0 -devpts /dev/pts devpts mode=0620,gid=5 0 0 -usbdevfs /proc/bus/usb usbdevfs noauto 0 0 -tmpfs /run tmpfs mode=0755,nodev,nosuid,strictatime 0 0 -tmpfs /var/volatile tmpfs defaults,size=50M 0 0 -tmpfs /media/ram tmpfs defaults,size=16M 0 0 - -# uncomment this if your device has a SD/MMC/Transflash slot -#/dev/mmcblk0p1 /media/card auto defaults,sync,noauto 0 0 - diff --git a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend index e5c53e80..7d1f6751 100644 --- a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend +++ b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend @@ -1,4 +1,4 @@ # look for files in this layer first FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:" -PR:append = ".arago4" +PR:append = ".arago5" -- 2.39.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [meta-arago][master][PATCH] base-files: Remove fstab override 2024-03-05 19:11 [meta-arago][master][PATCH] base-files: Remove fstab override Andrew Davis @ 2024-03-05 20:49 ` Denys Dmytriyenko 2024-03-19 15:18 ` Andrew Davis 0 siblings, 1 reply; 6+ messages in thread From: Denys Dmytriyenko @ 2024-03-05 20:49 UTC (permalink / raw) To: afd; +Cc: Denys Dmytriyenko, Ryan Eatmon, meta-arago On Tue, Mar 05, 2024 at 01:11:45PM -0600, Andrew Davis via lists.yoctoproject.org wrote: > There is already a version of this file provided by oe-core that > is kept up to date. Do not override it. > > Signed-off-by: Andrew Davis <afd@ti.com> > --- > .../recipes-core/base-files/base-files/fstab | 13 ------------- > .../recipes-core/base-files/base-files_%.bbappend | 2 +- > 2 files changed, 1 insertion(+), 14 deletions(-) > delete mode 100644 meta-arago-distro/recipes-core/base-files/base-files/fstab > > diff --git a/meta-arago-distro/recipes-core/base-files/base-files/fstab b/meta-arago-distro/recipes-core/base-files/base-files/fstab > deleted file mode 100644 > index 8ddadd98..00000000 > --- a/meta-arago-distro/recipes-core/base-files/base-files/fstab > +++ /dev/null > @@ -1,13 +0,0 @@ > -# stock fstab - you probably want to override this with a machine specific one > - > -/dev/root / auto defaults 1 1 > -proc /proc proc defaults 0 0 > -devpts /dev/pts devpts mode=0620,gid=5 0 0 > -usbdevfs /proc/bus/usb usbdevfs noauto 0 0 > -tmpfs /run tmpfs mode=0755,nodev,nosuid,strictatime 0 0 > -tmpfs /var/volatile tmpfs defaults,size=50M 0 0 > -tmpfs /media/ram tmpfs defaults,size=16M 0 0 The major difference is these 2 tmpfs entries - upstream doesn't have /media/ram, as that was something TI testing framework was using. Plus upstream doesn't set the size parameter - by default up to 50% of RAM can get used by tmpfs and there were issues with some low-RAM legacy platforms in the past. If neither of these differences matter any longer, this can be removed. > -# uncomment this if your device has a SD/MMC/Transflash slot > -#/dev/mmcblk0p1 /media/card auto defaults,sync,noauto 0 0 > - > diff --git a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend > index e5c53e80..7d1f6751 100644 > --- a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend > +++ b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend > @@ -1,4 +1,4 @@ > # look for files in this layer first > FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:" > > -PR:append = ".arago4" > +PR:append = ".arago5" > -- > 2.39.2 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [meta-arago][master][PATCH] base-files: Remove fstab override 2024-03-05 20:49 ` Denys Dmytriyenko @ 2024-03-19 15:18 ` Andrew Davis 2024-03-19 15:24 ` Ryan Eatmon [not found] ` <17BE33D39135657C.24580@lists.yoctoproject.org> 0 siblings, 2 replies; 6+ messages in thread From: Andrew Davis @ 2024-03-19 15:18 UTC (permalink / raw) To: Denys Dmytriyenko; +Cc: Denys Dmytriyenko, Ryan Eatmon, meta-arago On 3/5/24 2:49 PM, Denys Dmytriyenko wrote: > On Tue, Mar 05, 2024 at 01:11:45PM -0600, Andrew Davis via lists.yoctoproject.org wrote: >> There is already a version of this file provided by oe-core that >> is kept up to date. Do not override it. >> >> Signed-off-by: Andrew Davis <afd@ti.com> >> --- >> .../recipes-core/base-files/base-files/fstab | 13 ------------- >> .../recipes-core/base-files/base-files_%.bbappend | 2 +- >> 2 files changed, 1 insertion(+), 14 deletions(-) >> delete mode 100644 meta-arago-distro/recipes-core/base-files/base-files/fstab >> >> diff --git a/meta-arago-distro/recipes-core/base-files/base-files/fstab b/meta-arago-distro/recipes-core/base-files/base-files/fstab >> deleted file mode 100644 >> index 8ddadd98..00000000 >> --- a/meta-arago-distro/recipes-core/base-files/base-files/fstab >> +++ /dev/null >> @@ -1,13 +0,0 @@ >> -# stock fstab - you probably want to override this with a machine specific one >> - >> -/dev/root / auto defaults 1 1 >> -proc /proc proc defaults 0 0 >> -devpts /dev/pts devpts mode=0620,gid=5 0 0 >> -usbdevfs /proc/bus/usb usbdevfs noauto 0 0 >> -tmpfs /run tmpfs mode=0755,nodev,nosuid,strictatime 0 0 > >> -tmpfs /var/volatile tmpfs defaults,size=50M 0 0 >> -tmpfs /media/ram tmpfs defaults,size=16M 0 0 > > The major difference is these 2 tmpfs entries - upstream doesn't have > /media/ram, as that was something TI testing framework was using. Plus > upstream doesn't set the size parameter - by default up to 50% of RAM can > get used by tmpfs and there were issues with some low-RAM legacy platforms > in the past. > > If neither of these differences matter any longer, this can be removed. > If the tests do need this, then they should mount the needed tmpfs as part of the test setup, not just assume it to be setup by the distro. Simple as: # mount -t tmpfs -o size=50M tmpfs /media/ram The only way to push the test framework to do the right thing is to stop doing it for it. Andrew > >> -# uncomment this if your device has a SD/MMC/Transflash slot >> -#/dev/mmcblk0p1 /media/card auto defaults,sync,noauto 0 0 >> - >> diff --git a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >> index e5c53e80..7d1f6751 100644 >> --- a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >> +++ b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >> @@ -1,4 +1,4 @@ >> # look for files in this layer first >> FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:" >> >> -PR:append = ".arago4" >> +PR:append = ".arago5" >> -- >> 2.39.2 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [meta-arago][master][PATCH] base-files: Remove fstab override 2024-03-19 15:18 ` Andrew Davis @ 2024-03-19 15:24 ` Ryan Eatmon [not found] ` <17BE33D39135657C.24580@lists.yoctoproject.org> 1 sibling, 0 replies; 6+ messages in thread From: Ryan Eatmon @ 2024-03-19 15:24 UTC (permalink / raw) To: Andrew Davis, Denys Dmytriyenko; +Cc: Denys Dmytriyenko, meta-arago On 3/19/2024 10:18 AM, Andrew Davis wrote: > On 3/5/24 2:49 PM, Denys Dmytriyenko wrote: >> On Tue, Mar 05, 2024 at 01:11:45PM -0600, Andrew Davis via >> lists.yoctoproject.org wrote: >>> There is already a version of this file provided by oe-core that >>> is kept up to date. Do not override it. >>> >>> Signed-off-by: Andrew Davis <afd@ti.com> >>> --- >>> .../recipes-core/base-files/base-files/fstab | 13 ------------- >>> .../recipes-core/base-files/base-files_%.bbappend | 2 +- >>> 2 files changed, 1 insertion(+), 14 deletions(-) >>> delete mode 100644 >>> meta-arago-distro/recipes-core/base-files/base-files/fstab >>> >>> diff --git >>> a/meta-arago-distro/recipes-core/base-files/base-files/fstab >>> b/meta-arago-distro/recipes-core/base-files/base-files/fstab >>> deleted file mode 100644 >>> index 8ddadd98..00000000 >>> --- a/meta-arago-distro/recipes-core/base-files/base-files/fstab >>> +++ /dev/null >>> @@ -1,13 +0,0 @@ >>> -# stock fstab - you probably want to override this with a machine >>> specific one >>> - >>> -/dev/root / auto >>> defaults 1 1 >>> -proc /proc proc >>> defaults 0 0 >>> -devpts /dev/pts devpts >>> mode=0620,gid=5 0 0 >>> -usbdevfs /proc/bus/usb usbdevfs >>> noauto 0 0 >>> -tmpfs /run tmpfs >>> mode=0755,nodev,nosuid,strictatime 0 0 >> >>> -tmpfs /var/volatile tmpfs >>> defaults,size=50M 0 0 >>> -tmpfs /media/ram tmpfs >>> defaults,size=16M 0 0 >> >> The major difference is these 2 tmpfs entries - upstream doesn't have >> /media/ram, as that was something TI testing framework was using. Plus >> upstream doesn't set the size parameter - by default up to 50% of RAM can >> get used by tmpfs and there were issues with some low-RAM legacy >> platforms >> in the past. >> >> If neither of these differences matter any longer, this can be removed. >> > > If the tests do need this, then they should mount the needed tmpfs as part > of the test setup, not just assume it to be setup by the distro. Simple as: > > # mount -t tmpfs -o size=50M tmpfs /media/ram > > The only way to push the test framework to do the right thing is to > stop doing it for it. In general I agree, but that is easier said than done. We need to coordinate this change with the test team, not just dump it on them. I'm not rejecting this patch, just waiting until test has confirmed we no longer need it. > Andrew > >> >>> -# uncomment this if your device has a SD/MMC/Transflash slot >>> -#/dev/mmcblk0p1 /media/card auto >>> defaults,sync,noauto 0 0 >>> - >>> diff --git >>> a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >>> b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >>> index e5c53e80..7d1f6751 100644 >>> --- a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >>> +++ b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >>> @@ -1,4 +1,4 @@ >>> # look for files in this layer first >>> FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:" >>> -PR:append = ".arago4" >>> +PR:append = ".arago5" >>> -- >>> 2.39.2 -- Ryan Eatmon reatmon@ti.com ----------------------------------------- Texas Instruments, Inc. - LCPD - MGTS ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <17BE33D39135657C.24580@lists.yoctoproject.org>]
* Re: [meta-arago][master][PATCH] base-files: Remove fstab override [not found] ` <17BE33D39135657C.24580@lists.yoctoproject.org> @ 2024-04-08 15:01 ` Ryan Eatmon 2024-04-10 15:22 ` Andrew Davis 0 siblings, 1 reply; 6+ messages in thread From: Ryan Eatmon @ 2024-04-08 15:01 UTC (permalink / raw) To: Andrew Davis, Denys Dmytriyenko; +Cc: Denys Dmytriyenko, meta-arago On 3/19/2024 10:24 AM, Ryan Eatmon via lists.yoctoproject.org wrote: > > > On 3/19/2024 10:18 AM, Andrew Davis wrote: >> On 3/5/24 2:49 PM, Denys Dmytriyenko wrote: >>> On Tue, Mar 05, 2024 at 01:11:45PM -0600, Andrew Davis via >>> lists.yoctoproject.org wrote: >>>> There is already a version of this file provided by oe-core that >>>> is kept up to date. Do not override it. >>>> >>>> Signed-off-by: Andrew Davis <afd@ti.com> >>>> --- >>>> .../recipes-core/base-files/base-files/fstab | 13 >>>> ------------- >>>> .../recipes-core/base-files/base-files_%.bbappend | 2 +- >>>> 2 files changed, 1 insertion(+), 14 deletions(-) >>>> delete mode 100644 >>>> meta-arago-distro/recipes-core/base-files/base-files/fstab >>>> >>>> diff --git >>>> a/meta-arago-distro/recipes-core/base-files/base-files/fstab >>>> b/meta-arago-distro/recipes-core/base-files/base-files/fstab >>>> deleted file mode 100644 >>>> index 8ddadd98..00000000 >>>> --- a/meta-arago-distro/recipes-core/base-files/base-files/fstab >>>> +++ /dev/null >>>> @@ -1,13 +0,0 @@ >>>> -# stock fstab - you probably want to override this with a machine >>>> specific one >>>> - >>>> -/dev/root / auto >>>> defaults 1 1 >>>> -proc /proc proc >>>> defaults 0 0 >>>> -devpts /dev/pts devpts >>>> mode=0620,gid=5 0 0 >>>> -usbdevfs /proc/bus/usb usbdevfs >>>> noauto 0 0 >>>> -tmpfs /run tmpfs >>>> mode=0755,nodev,nosuid,strictatime 0 0 >>> >>>> -tmpfs /var/volatile tmpfs >>>> defaults,size=50M 0 0 >>>> -tmpfs /media/ram tmpfs >>>> defaults,size=16M 0 0 >>> >>> The major difference is these 2 tmpfs entries - upstream doesn't have >>> /media/ram, as that was something TI testing framework was using. Plus >>> upstream doesn't set the size parameter - by default up to 50% of RAM >>> can >>> get used by tmpfs and there were issues with some low-RAM legacy >>> platforms >>> in the past. >>> >>> If neither of these differences matter any longer, this can be removed. >>> >> >> If the tests do need this, then they should mount the needed tmpfs as >> part >> of the test setup, not just assume it to be setup by the distro. >> Simple as: >> >> # mount -t tmpfs -o size=50M tmpfs /media/ram >> >> The only way to push the test framework to do the right thing is to >> stop doing it for it. > > In general I agree, but that is easier said than done. We need to > coordinate this change with the test team, not just dump it on them. I'm > not rejecting this patch, just waiting until test has confirmed we no > longer need it. Good news. We ran a special set of RC tests with this change in place and everything looks like fine. I'll accept this patch now for scarthgap/master. > >> Andrew >> >>> >>>> -# uncomment this if your device has a SD/MMC/Transflash slot >>>> -#/dev/mmcblk0p1 /media/card auto >>>> defaults,sync,noauto 0 0 >>>> - >>>> diff --git >>>> a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >>>> b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >>>> index e5c53e80..7d1f6751 100644 >>>> --- a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >>>> +++ b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >>>> @@ -1,4 +1,4 @@ >>>> # look for files in this layer first >>>> FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:" >>>> -PR:append = ".arago4" >>>> +PR:append = ".arago5" >>>> -- >>>> 2.39.2 > -- Ryan Eatmon reatmon@ti.com ----------------------------------------- Texas Instruments, Inc. - LCPD - MGTS ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [meta-arago][master][PATCH] base-files: Remove fstab override 2024-04-08 15:01 ` Ryan Eatmon @ 2024-04-10 15:22 ` Andrew Davis 0 siblings, 0 replies; 6+ messages in thread From: Andrew Davis @ 2024-04-10 15:22 UTC (permalink / raw) To: Ryan Eatmon, Denys Dmytriyenko; +Cc: Denys Dmytriyenko, meta-arago On 4/8/24 10:01 AM, Ryan Eatmon wrote: > > > On 3/19/2024 10:24 AM, Ryan Eatmon via lists.yoctoproject.org wrote: >> >> >> On 3/19/2024 10:18 AM, Andrew Davis wrote: >>> On 3/5/24 2:49 PM, Denys Dmytriyenko wrote: >>>> On Tue, Mar 05, 2024 at 01:11:45PM -0600, Andrew Davis via lists.yoctoproject.org wrote: >>>>> There is already a version of this file provided by oe-core that >>>>> is kept up to date. Do not override it. >>>>> >>>>> Signed-off-by: Andrew Davis <afd@ti.com> >>>>> --- >>>>> .../recipes-core/base-files/base-files/fstab | 13 ------------- >>>>> .../recipes-core/base-files/base-files_%.bbappend | 2 +- >>>>> 2 files changed, 1 insertion(+), 14 deletions(-) >>>>> delete mode 100644 meta-arago-distro/recipes-core/base-files/base-files/fstab >>>>> >>>>> diff --git a/meta-arago-distro/recipes-core/base-files/base-files/fstab b/meta-arago-distro/recipes-core/base-files/base-files/fstab >>>>> deleted file mode 100644 >>>>> index 8ddadd98..00000000 >>>>> --- a/meta-arago-distro/recipes-core/base-files/base-files/fstab >>>>> +++ /dev/null >>>>> @@ -1,13 +0,0 @@ >>>>> -# stock fstab - you probably want to override this with a machine specific one >>>>> - >>>>> -/dev/root / auto defaults 1 1 >>>>> -proc /proc proc defaults 0 0 >>>>> -devpts /dev/pts devpts mode=0620,gid=5 0 0 >>>>> -usbdevfs /proc/bus/usb usbdevfs noauto 0 0 >>>>> -tmpfs /run tmpfs mode=0755,nodev,nosuid,strictatime 0 0 >>>> >>>>> -tmpfs /var/volatile tmpfs defaults,size=50M 0 0 >>>>> -tmpfs /media/ram tmpfs defaults,size=16M 0 0 >>>> >>>> The major difference is these 2 tmpfs entries - upstream doesn't have >>>> /media/ram, as that was something TI testing framework was using. Plus >>>> upstream doesn't set the size parameter - by default up to 50% of RAM can >>>> get used by tmpfs and there were issues with some low-RAM legacy platforms >>>> in the past. >>>> >>>> If neither of these differences matter any longer, this can be removed. >>>> >>> >>> If the tests do need this, then they should mount the needed tmpfs as part >>> of the test setup, not just assume it to be setup by the distro. Simple as: >>> >>> # mount -t tmpfs -o size=50M tmpfs /media/ram >>> >>> The only way to push the test framework to do the right thing is to >>> stop doing it for it. >> >> In general I agree, but that is easier said than done. We need to coordinate this change with the test team, not just dump it on them. I'm not rejecting this patch, just waiting until test has confirmed we no longer need it. > > Good news. We ran a special set of RC tests with this change in place and everything looks like fine. I'll accept this patch now for scarthgap/master. > Great! Thanks for setting up and running the extra tests for this Andrew > >> >>> Andrew >>> >>>> >>>>> -# uncomment this if your device has a SD/MMC/Transflash slot >>>>> -#/dev/mmcblk0p1 /media/card auto defaults,sync,noauto 0 0 >>>>> - >>>>> diff --git a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >>>>> index e5c53e80..7d1f6751 100644 >>>>> --- a/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >>>>> +++ b/meta-arago-distro/recipes-core/base-files/base-files_%.bbappend >>>>> @@ -1,4 +1,4 @@ >>>>> # look for files in this layer first >>>>> FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:" >>>>> -PR:append = ".arago4" >>>>> +PR:append = ".arago5" >>>>> -- >>>>> 2.39.2 >> > ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-04-10 15:22 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-05 19:11 [meta-arago][master][PATCH] base-files: Remove fstab override Andrew Davis
2024-03-05 20:49 ` Denys Dmytriyenko
2024-03-19 15:18 ` Andrew Davis
2024-03-19 15:24 ` Ryan Eatmon
[not found] ` <17BE33D39135657C.24580@lists.yoctoproject.org>
2024-04-08 15:01 ` Ryan Eatmon
2024-04-10 15:22 ` Andrew Davis
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.