On Fri, 2009-11-06 at 08:09 -0500, Joseph J VLcek wrote:
> Alexander Eremin wrote:
> > On Fri, 2009-11-06 at 07:52 -0500, Joseph J VLcek wrote:
> >> Alexander Eremin wrote:
> >>> Changing cycle for test on:
> >>> while [[ ${loop_cnt} -lt ${umount_loop} ]]; do
> >>>
> >>>
> >>>
> >>> # umount new slice
> >>>
> >>> umount -f $s0bdev > /dev/null 2>&1
> >>>
> >>> if [[ $? -eq 0 ]]; then
> >>>
> >>> echo "UMOUNT!"
> >>>
> >>> #break;
> >>>
> >>> fi
> >>>
> >>> # if the umount failed try again in a second
> >>>
> >>> sleep 1
> >>>
> >>> echo $loop_cnt
> >>>
> >>> (( loop_cnt=${loop_cnt} + 1 ))
> >>>
> >>>
> >>>
> >>> done
> >>>
> >>> echo "Done"
> >>> exit 0
> >>>
> >>> I've got:
> >>> # ./usbcopy osol121.usb
> >>> Found the following USB devices:
> >>> 0: /dev/rdsk/c2t0d0p0 976.5 MB Multi Flash Reader
> >>> 1.00
> >>> Enter the number of your choice: 0
> >>>
> >>> WARNING: All data on your USB storage will be lost.
> >>> Are you sure you want to install to
> >>> Multi Flash Reader 1.00, 976 MB at /dev/rdsk/c2t0d0p0 ? (y/n) y
> >>> 0
> >>> 1
> >>> 2
> >>> 3
> >>> UMOUNT!
> >>> 4
> >>> UMOUNT!
> >>> 5
> >>> 6
> >>> 7
> >>> 8
> >>> 9
> >>> 10
> >>> 11
> >>> 12
> >>> 13
> >>> 14
> >>> Done
> >>>
> >>> Sometimes hal doing even three-four mounts. 15sec is enough, so if we
> >>> must have hal running, just non-breakable cycle is required.
> >>>
> >>> Cheers,
> >>> Alex
> >> Thank you for correcting my misunderstanding Alex!
> >>
> >> If HAL didn't do the three-four mounts this loop would not be necessary.
> >>
> >> Joe
> >>
> >>
> >> Joe
> > Today I've tested this again and loop works correct (only one umount).
> > So would be great if this will be checked by other testers.
> > http://cr.opensolaris.org/~alhazred/10098/raw_files/new/usr/src/cmd/install-tools/usbcopy
> >
> > Regards,
> > Alex
>
> Alex,
>
> What has changed in your testing today? Why didn't you encounter the
> extra HAL induced mounts? Is it possible that others using this version
> of usbcopy might encounter the extra HAL induced mounts?
>
> Joe
Ha-ha. For 11 test just got second umount. So, will remove break from
code.
Cheers,
Alex
