Re: [PATCH v4] arm: boards: phytec-som-am335x: Update boot scripts

[Daniel Schultz <d.schultz@phytec.de>]

Hi Sascha,

Am 19.06.2017 um 09:25 schrieb Sascha Hauer:
> On Tue, Jun 13, 2017 at 03:37:08PM +0200, Daniel Schultz wrote:
>> Expand the boot scripts with EMMC and add a default file source for
>> expansions.
>>
>> Removed "rw" and "rootwait" bootargs from existing boot scripts.
>> Added "rootflags='data=journal'" bootarg to SD card boot script.
>>
>> Signed-off-by: Daniel Schultz <d.schultz@phytec.de>
>> ---
>> +#!/bin/sh
>> +
>> +path="/mnt/tftp"
>> +
>> +global.bootm.image="${path}/${global.user}-linux-${global.hostname}"
>> +
>> +oftree="${path}/${global.user}-oftree-${global.hostname}"
>> +if [ -f "${oftree}" ]; then
>> +    global.bootm.oftree="$oftree"
>> +fi
>> +
>> +nfsroot="/nfsroot/${global.hostname}"
> 
> This assumed there is one nfsroot for all users. The intention with the
> original naming was that each user can create a link in his home
> directory to wherever the nfsroot lives, which may include some globally
> available nfsroot for all users. Why change it?
> 
Seems like this was an older script. We will switch to the Barebox 
network boot script.

>> +bootargs-ip
>> +
>> +global.linux.bootargs.dyn.root="root=/dev/nfs nfsroot=$nfsroot,vers=3,udp"
> 
> Why udp? Normally tcp is preferred, no?
> 
UDP is used as default by the nfsroot kernel system and also by tftp.
Additionally, we had troubles with booting in tcp mode.

> Sascha
> 

-- 
Mit freundlichen Grüßen,
With best regards,
   Daniel Schultz

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox