[Buildroot] [PATCH v2] fs/common: Create initial console device

[Markos Chandras]

On Thu, Nov 1, 2012 at 10:21 PM, Arnout Vandecappelle <arnout at mind.be> wrote:
> On 11/01/12 11:28, Markos Chandras wrote:
>>
>> From: Markos Chandras<markos.chandras at 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 at 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