a0411faab2
During device removal, we are first waiting for device to close() in a tight loop for 10 seconds. I am not sure why do we need it. First of all we come here once the umount() is successful so device should be free. For some reason of device is temporarily busy, then removeDevice() logic retries device removal logic in a loop for 10 seconds and that should cover it. Can't see why one more 10 seoncds loop is required before attempting device removal. One loop should be able to cover all the temporary device busy conditions and if condition is not temporary then 10 seconds loop is not going to help anyway. So instead of two loops of 10 seconds each, I am converting it to a single loop of 20 seconds. May be 10 second loop is good enough but for now I am keeping it 20 seconds to avoid any regressions. Signed-off-by: Vivek Goyal <vgoyal@redhat.com> Upstream-commit: f74d12012c21349b2bd51d9c395a99331ff0a9a5 Component: engine