-----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.g...@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----- _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
