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 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 > > >

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 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 fi

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 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: ..

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 reg

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 p

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 2>&1 + 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 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 2>&1 > + if [ $? -eq 0 ]; then > + break > + fi > + printf "$COLOUR_ORANGE\tSleeping to wait for u

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: ./d

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 cas

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 proc

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 d

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

2014-11-07 Thread Lukasz Majewski
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 device is found by lsof(8) or fuser(1)) TX: md5sum:0