* [Buildroot] [PATCH v2] fs/common: Create initial console device @ 2012-11-01 10:28 Markos Chandras 2012-11-01 22:21 ` Arnout Vandecappelle 0 siblings, 1 reply; 5+ messages in thread From: Markos Chandras @ 2012-11-01 10:28 UTC (permalink / raw) To: buildroot From: Markos Chandras <markos.chandras@imgtec.com> A /dev/console node must be present in rootfs when the Linux kernel boots otherwise the kernel will print the following warning: "Warning: unable to open an initial console" This is because when we use an initramfs the /dev directory is not populated at this point. This can cause problems when a program (e.g ldso with early debugging enabled) opens a standard file descriptor for read/write before these descriptors are actually created by the init process later on. Signed-off-by: Markos Chandras <markos.chandras@imgtec.com> --- fs/cpio/cpio.mk | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/fs/cpio/cpio.mk b/fs/cpio/cpio.mk index aa20b41..3293bc9 100644 --- a/fs/cpio/cpio.mk +++ b/fs/cpio/cpio.mk @@ -21,6 +21,8 @@ define ROOTFS_CPIO_ADD_INIT fi endef +PACKAGES_PERMISSIONS_TABLE += /dev/console c 622 0 0 5 1 + endif # BR2_ROOTFS_DEVICE_CREATION_STATIC ROOTFS_CPIO_PRE_GEN_HOOKS += ROOTFS_CPIO_ADD_INIT -- 1.7.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v2] fs/common: Create initial console device 2012-11-01 10:28 [Buildroot] [PATCH v2] fs/common: Create initial console device Markos Chandras @ 2012-11-01 22:21 ` Arnout Vandecappelle 2012-11-02 11:32 ` Markos Chandras 0 siblings, 1 reply; 5+ messages in thread From: Arnout Vandecappelle @ 2012-11-01 22:21 UTC (permalink / raw) To: buildroot On 11/01/12 11:28, Markos Chandras wrote: > From: Markos Chandras<markos.chandras@imgtec.com> > > A /dev/console node must be present in rootfs when the Linux kernel > boots otherwise the kernel will print the following warning: > "Warning: unable to open an initial console" > > This is because when we use an initramfs the /dev directory is not > populated at this point. This can cause problems when a program > (e.g ldso with early debugging enabled) opens a standard file > descriptor for read/write before these descriptors are actually > created by the init process later on. > > Signed-off-by: Markos Chandras<markos.chandras@imgtec.com> > --- > fs/cpio/cpio.mk | 2 ++ > 1 files changed, 2 insertions(+), 0 deletions(-) > > diff --git a/fs/cpio/cpio.mk b/fs/cpio/cpio.mk > index aa20b41..3293bc9 100644 > --- a/fs/cpio/cpio.mk > +++ b/fs/cpio/cpio.mk > @@ -21,6 +21,8 @@ define ROOTFS_CPIO_ADD_INIT > fi > endef > > +PACKAGES_PERMISSIONS_TABLE += /dev/console c 622 0 0 5 1 The last three entries are missing (start, inc, count). Also, there should be a $(sep) at the end of the line; otherwise, any subsequent entries are appended at the end of the line. (This will probably be the last line in the table anyway, because the fs/ tree is included at the end of Makefile, but we shouldn't rely on that). Regards, Arnout > + > endif # BR2_ROOTFS_DEVICE_CREATION_STATIC > > ROOTFS_CPIO_PRE_GEN_HOOKS += ROOTFS_CPIO_ADD_INIT -- Arnout Vandecappelle arnout at mind be Senior Embedded Software Architect +32-16-286540 Essensium/Mind http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v2] fs/common: Create initial console device 2012-11-01 22:21 ` Arnout Vandecappelle @ 2012-11-02 11:32 ` Markos Chandras 2012-11-02 11:35 ` Arnout Vandecappelle 0 siblings, 1 reply; 5+ messages in thread From: Markos Chandras @ 2012-11-02 11:32 UTC (permalink / raw) To: buildroot On Thu, Nov 1, 2012 at 10:21 PM, Arnout Vandecappelle <arnout@mind.be> wrote: > On 11/01/12 11:28, Markos Chandras wrote: >> >> From: Markos Chandras<markos.chandras@imgtec.com> >> >> A /dev/console node must be present in rootfs when the Linux kernel >> boots otherwise the kernel will print the following warning: >> "Warning: unable to open an initial console" >> >> This is because when we use an initramfs the /dev directory is not >> populated at this point. This can cause problems when a program >> (e.g ldso with early debugging enabled) opens a standard file >> descriptor for read/write before these descriptors are actually >> created by the init process later on. >> >> Signed-off-by: Markos Chandras<markos.chandras@imgtec.com> >> --- >> fs/cpio/cpio.mk | 2 ++ >> 1 files changed, 2 insertions(+), 0 deletions(-) >> >> diff --git a/fs/cpio/cpio.mk b/fs/cpio/cpio.mk >> index aa20b41..3293bc9 100644 >> --- a/fs/cpio/cpio.mk >> +++ b/fs/cpio/cpio.mk >> @@ -21,6 +21,8 @@ define ROOTFS_CPIO_ADD_INIT >> fi >> endef >> >> +PACKAGES_PERMISSIONS_TABLE += /dev/console c 622 0 0 5 1 > > > The last three entries are missing (start, inc, count). Also, there should > be a $(sep) at the end of the line; otherwise, any subsequent entries are > appended at the end of the line. (This will probably be the last line in > the table anyway, because the fs/ tree is included at the end of Makefile, > but we shouldn't rely on that). > > > Regards, > Arnout > Hi Arnout, Do I really need these 3 numbers (start, inc, count)? The static file in target/generic/device_table_dev.txt is not using them for the console device. -- Regards, Markos ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v2] fs/common: Create initial console device 2012-11-02 11:32 ` Markos Chandras @ 2012-11-02 11:35 ` Arnout Vandecappelle 2012-11-02 11:45 ` Markos Chandras 0 siblings, 1 reply; 5+ messages in thread From: Arnout Vandecappelle @ 2012-11-02 11:35 UTC (permalink / raw) To: buildroot On 11/02/12 12:32, Markos Chandras wrote: >>> >> +PACKAGES_PERMISSIONS_TABLE += /dev/console c 622 0 0 5 1 >> > >> > >> > The last three entries are missing (start, inc, count). Also, there should >> > be a $(sep) at the end of the line; otherwise, any subsequent entries are >> > appended at the end of the line. (This will probably be the last line in >> > the table anyway, because the fs/ tree is included at the end of Makefile, >> > but we shouldn't rely on that). >> > > Do I really need these 3 numbers (start, inc, count)? The static file > in target/generic/device_table_dev.txt > is not using them for the console device. It is: /dev/console c 666 0 0 5 1 - - - Regards, Arnout -- Arnout Vandecappelle arnout at mind be Senior Embedded Software Architect +32-16-286540 Essensium/Mind http://www.mind.be G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle GPG fingerprint: 7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F ^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v2] fs/common: Create initial console device 2012-11-02 11:35 ` Arnout Vandecappelle @ 2012-11-02 11:45 ` Markos Chandras 0 siblings, 0 replies; 5+ messages in thread From: Markos Chandras @ 2012-11-02 11:45 UTC (permalink / raw) To: buildroot On Fri, Nov 2, 2012 at 11:35 AM, Arnout Vandecappelle <arnout@mind.be> wrote: > On 11/02/12 12:32, Markos Chandras wrote: >>>> >>>> >> +PACKAGES_PERMISSIONS_TABLE += /dev/console c 622 0 0 5 1 >>> >>> > >>> > >>> > The last three entries are missing (start, inc, count). Also, there >>> > should >>> > be a $(sep) at the end of the line; otherwise, any subsequent entries >>> > are >>> > appended at the end of the line. (This will probably be the last line >>> > in >>> > the table anyway, because the fs/ tree is included at the end of >>> > Makefile, >>> > but we shouldn't rely on that). >>> > >> >> Do I really need these 3 numbers (start, inc, count)? The static file >> >> in target/generic/device_table_dev.txt >> is not using them for the console device. > > > It is: > /dev/console c 666 0 0 5 1 - - > - HI Arnout, Hmm I thought that the hyphens there meant "no value is needed". I will prepare a new patch. -- Regards, Markos ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-11-02 11:45 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-11-01 10:28 [Buildroot] [PATCH v2] fs/common: Create initial console device Markos Chandras 2012-11-01 22:21 ` Arnout Vandecappelle 2012-11-02 11:32 ` Markos Chandras 2012-11-02 11:35 ` Arnout Vandecappelle 2012-11-02 11:45 ` Markos Chandras
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox