Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-14 Thread Lukasz Majewski
Hi Lukasz, On Wed, 12 Nov 2014 11:11:33 -0700 Stephen Warren swar...@wwwdotorg.org wrote: On 11/12/2014 09:45 AM, Wolfgang Denk wrote: Dear Lukasz, In message 20141112152949.1fe6ce3c@amdc2363 you wrote: But this is a bug. Papering over is not a good idea. It should be

Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-12 Thread Lukasz Majewski
Hi Wolfgang, Dear Stephen, In message 545d40e1.2030...@wwwdotorg.org you wrote: My gut feeling is that there might be some USB driver error involved here. Where I've seen this is writing to an SD card in a USB-based SD card reader. I have a fairly regular amd64 machine

Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-12 Thread Wolfgang Denk
Dear Lukasz, In message 20141112152949.1fe6ce3c@amdc2363 you wrote: But this is a bug. Papering over is not a good idea. It should be analyzed, reported, and finally fixed. I've debugged the script with strace. Thanks a lot for that! The problem is with umount() syscall: ... I need

Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-12 Thread Stephen Warren
On 11/12/2014 09:45 AM, Wolfgang Denk wrote: Dear Lukasz, In message 20141112152949.1fe6ce3c@amdc2363 you wrote: But this is a bug. Papering over is not a good idea. It should be analyzed, reported, and finally fixed. I've debugged the script with strace. Thanks a lot for that! The

Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-12 Thread Lukasz Majewski
On Wed, 12 Nov 2014 11:11:33 -0700 Stephen Warren swar...@wwwdotorg.org wrote: On 11/12/2014 09:45 AM, Wolfgang Denk wrote: Dear Lukasz, In message 20141112152949.1fe6ce3c@amdc2363 you wrote: But this is a bug. Papering over is not a good idea. It should be analyzed, reported, and

Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-07 Thread Stephen Warren
On 11/07/2014 06:05 AM, Lukasz Majewski wrote: This change helps to run script on machines with quite long uptime. Without this the following error emerges: File: ./dat_14M.img umount: /mnt/tmp-ums-test: device is busy. (In some cases useful info about processes that use the

Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-07 Thread Marek Vasut
On Friday, November 07, 2014 at 02:05:55 PM, Lukasz Majewski wrote: This change helps to run script on machines with quite long uptime. Without this the following error emerges: File: ./dat_14M.img umount: /mnt/tmp-ums-test: device is busy. (In some cases useful info about processes

Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-07 Thread Lukasz Majewski
Hi Marek, On Friday, November 07, 2014 at 02:05:55 PM, Lukasz Majewski wrote: This change helps to run script on machines with quite long uptime. Without this the following error emerges: File: ./dat_14M.img umount: /mnt/tmp-ums-test: device is busy. (In some cases useful

Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-07 Thread Marek Vasut
On Friday, November 07, 2014 at 05:52:27 PM, Lukasz Majewski wrote: Hi Marek, On Friday, November 07, 2014 at 02:05:55 PM, Lukasz Majewski wrote: This change helps to run script on machines with quite long uptime. Without this the following error emerges: File: ./dat_14M.img

Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-07 Thread Wolfgang Denk
Dear Lukasz, In message 141536-9790-1-git-send-email-l.majew...@samsung.com you wrote: -umount $MNT_DIR +while true; do + umount $MNT_DIR /dev/null 21 + if [ $? -eq 0 ]; then + break + fi + printf $COLOUR_ORANGE\tSleeping to wait for

Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-07 Thread Stephen Warren
On 11/07/2014 02:54 PM, Wolfgang Denk wrote: Dear Lukasz, In message 141536-9790-1-git-send-email-l.majew...@samsung.com you wrote: -umount $MNT_DIR +while true; do + umount $MNT_DIR /dev/null 21 + if [ $? -eq 0 ]; then + break + fi + printf

Re: [U-Boot] [PATCH v2] test: ums: Add sleep before unmount directory

2014-11-07 Thread Wolfgang Denk
Dear Stephen, In message 545d40e1.2030...@wwwdotorg.org you wrote: My gut feeling is that there might be some USB driver error involved here. Where I've seen this is writing to an SD card in a USB-based SD card reader. I have a fairly regular amd64 machine running Ubuntu. I put the