Messages in this thread | | | From | Avri Altman <> | Subject | RE: [PATCH 2/2] scsi: ufs: Return a bsg request immediatley if eh-in-progress | Date | Mon, 8 Nov 2021 18:00:36 +0000 |
| |
> On 11/8/21 9:24 AM, Avri Altman wrote: > > I am not sure. I would expect a retry / polling / other, if any, to be > > done in user-space and not in the kernel. e.g. a common practice in > > the code that send SG_IO or other ioctls is to retry on EBUSY. Not > > sure that this is the case in ufs-utils though. > Shouldn't we aim to make sure that user space code does not have to use > busy waiting? I don't know. Waiting in the kernel seems like an unnecessary complication. If you find it useless, better to just drop it.
I looked it up in ufs-utils public repository (https://github.com/westerndigitalcorporation/ufs-utils), and it looks like that:
while (((ret = ioctl(fd, SG_IO, &io_hdr_v4)) < 0) && ((errno == EINTR) || (errno == EAGAIN))) ; Thanks, Avri
> > Thanks, > > Bart.
| |