[U-Boot] [PULL] efi patch queue 2018-07-25

[Heinrich Schuchardt]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

On 07/28/2018 08:33 PM, Tom Rini wrote:
> On Sat, Jul 28, 2018 at 07:10:39PM +0200, Heinrich Schuchardt
> wrote:
>> -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512
>> 
>> On 07/28/2018 06:32 PM, Tom Rini wrote:
>>> On Sat, Jul 28, 2018 at 06:21:58PM +0200, Heinrich Schuchardt 
>>> wrote:
>>>> -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512
>>>> 
>>>> On 07/28/2018 06:13 PM, Tom Rini wrote:
>>>>> On Sat, Jul 28, 2018 at 06:07:20PM +0200, Heinrich
>>>>> Schuchardt wrote:
>>>>> 
>>>>>> -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512
>>>>>> 
>>>>>> On 07/28/2018 05:55 PM, Tom Rini wrote:
>>>>>>> On Wed, Jul 25, 2018 at 03:04:27PM +0200, Alexander
>>>>>>> Graf wrote:
>>>>>>> 
>>>>>>>> Hi Tom,
>>>>>>>> 
>>>>>>>> This is my current patch queue for efi.  Please
>>>>>>>> pull.
>>>>>>>> 
>>>>>>>> Alex
>>>>>>>> 
>>>>>>>> 
>>>>>>>> The following changes since commit 
>>>>>>>> 323a73adc9a1bf2de43fe03bdd9c3038ce7c2784:
>>>>>>>> 
>>>>>>>> mtd: nand: add new enum for storing ECC algorithm 
>>>>>>>> (2018-07-23 14:33:21 -0400)
>>>>>>>> 
>>>>>>>> are available in the git repository at:
>>>>>>>> 
>>>>>>>> git://github.com/agraf/u-boot.git
>>>>>>>> tags/signed-efi-next
>>>>>>>> 
>>>>>>>> for you to fetch changes up to 
>>>>>>>> 0b8a88ab6aa24de0ef2bf1e8109409f71e770a8e:
>>>>>>>> 
>>>>>>>> MAINTAINERS: assign lib/charset.c (2018-07-25
>>>>>>>> 15:00:24 +0200)
>>>>>>>> 
>>>>>>> 
>>>>>>> NAK, this breaks one of the filesystem tests. 
>>>>>>> Specifically: commit 
>>>>>>> 0dc1bfb7302d220a48364263d5632d6d572b069b Author:
>>>>>>> Heinrich Schuchardt <xypron.glpk at gmx.de> Date:   Mon
>>>>>>> Jul 2 02:41:23 2018 +0200
>>>>>>> 
>>>>>>> fs: fat: cannot write to subdirectories
>>>>>>> 
>>>>>>> Breaks TC13: 1MB write to ./1MB.file.w2
>>>>>>> 
>>>>>> 
>>>>>> Hello Tom,
>>>>>> 
>>>>>> please, provide the link to the Travis log with the
>>>>>> failure.
>>>>> 
>>>>> It's actually not in travis.  Running test/fs/fs-test.sh is
>>>>>  annoying to automate: FSTST=`./test/fs/fs-test.sh 2>&1 |
>>>>> tail -n 3 | head -n 1` echo $FSTST | grep -q "TOTAL PASS:
>>>>> 204 TOTAL FAIL: 12" && exit 0 || exit 1
>>>>> 
>>>>> but I should see if I can get that into .travis.yml.
>>>>> 
>>>> 
>>>> ./test/fs/fs-test.sh Missing mkfs binary. Exiting!
>>>> 
>>>> You wouldn't run tests as root? Is this test meant to be run 
>>>> with fakeroot?
>>> 
>>> It requires sudo to work along with various utilities to make
>>> the various filesystems.
>>> 
>> 
>> Tom please, have a look at the files created by the tests w/o my
>> patch.
>> 
>> This is what the find command returns:
>> 
>> sandbox/test/fs/mnt sandbox/test/fs/mnt/SUBDIR 
>> sandbox/test/fs/mnt/2.5GB.file sandbox/test/fs/mnt/1MB.file 
>> sandbox/test/fs/mnt/1MB.file.w sandbox/test/fs/mnt/1MB.file.w2 
>> sandbox/test/fs/mnt/./1MB.file.w2
>> 
>> You observe that the last file has an illegal file name (yes,
>> the filename itself is "./1MB.file.w2". It should never have been
>> created.
>> 
>> Without my patch this illegal file is not created.
>> 
>> Why should this be a reason to dismiss my patch?
> 
> Ah, OK, thanks for looking.  Please submit a patch that updates
> the tests.
> 

With Takahiro's patch series

fs: fat: extend FAT write operations
https://patchwork.ozlabs.org/project/uboot/list/?series=56580
https://lists.denx.de/pipermail/u-boot/2018-July/335683.html

the FAT driver will finally correctly support paths with subdirectories.

With that patch series the created files are:

sandbox/test/fs/mnt
sandbox/test/fs/mnt/SUBDIR
sandbox/test/fs/mnt/2.5GB.file
sandbox/test/fs/mnt/1MB.file
sandbox/test/fs/mnt/1MB.file.w
sandbox/test/fs/mnt/1MB.file.w2

There is nothing wrong with the TC13 test. After writing it tries to
do the verification with (b) and without (c) a relative path. If both
subtests are passed the file system is working as expected. And as you
already will have observed TC13b and TC13c are not passed without
Takahiro's patch series.

Best regards

Heinrich
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEEbcT5xx8ppvoGt20zxIHbvCwFGsQFAltc4P8ACgkQxIHbvCwF
GsSfXg/9G6RqPbMGb3wl3SOZSnPM2VzTv9tvnnpqZq8Bmaao4I7sgnI3w/Ts6r/k
iBU0c4iOrcytOiVWiQU9FPJ3L8zKOMqfK7vE9186NUZFh1u10sJrmLfESV6pbKF8
+pxKy6xM2y8F7AWE0VhgAf7lxFbA2tnhFAQPbP687mJaFKYKG48CQ0r7GwXO/iEl
YvE5KflSOEIuTJ6VD7sVuOdEWw8v9yXZTevxcjYSMQppvGg2JIpILtUc89aMlNNC
rFTesESdkqcm5V8NUaaMLv404QiW1z93ya+za7IwoUiXAlxYFuXCWJtOqWXxz4Mu
EbGOXTYXUq0xXqNJDZDjtwzOQilUANvFGAwXUJc1F6q3F0XJNLOStC7Vr6SrBjij
ibiJAj1XOgEbNip6m1yDB0ycyCjKjASr+l6RrfWzFli1AiQon4Fk6P38Rb27RAiU
DGTMKp6gqhj1tTLaNQTEzwdgcr19GvRfJQpfDYkZX0ujESETwMCyO+1i4VuM7bJo
v2h7C+1m7uk7qVDbwJAtTS0YiL7k7ZqyFv55AmGtRH0ZNVNIgIFByfXYchukrHqo
q89l1mYtMba05ZZ4JVp/21AcEFH94WbtlPvkizXhpcxeH2ZpGR1GhmZEXNJ/a9uP
d9eDngVapglWzJCya+ojtMh+RZi4Rn01Y3Qv45PfRptmzxJn2CI=
=aYo7
-----END PGP SIGNATURE-----