Retrying write(2) if it returns a short non-negative write count

This commit is contained in:
Thomas Schmitt 2014-04-14 10:33:58 +00:00
parent 511d23f574
commit 856081559c
2 changed files with 26 additions and 10 deletions

View File

@ -1 +1 @@
#define Cdrskin_timestamP "2014.04.13.120906" #define Cdrskin_timestamP "2014.04.14.103311"

View File

@ -86,6 +86,10 @@ extern struct libdax_msgs *libdax_messenger;
*/ */
#define Libburn_bd_streamed_obS (64 * 1024) #define Libburn_bd_streamed_obS (64 * 1024)
/* The number of retries if write(2) returns a short, non-negative write count.
*/
#define Libburn_stdio_write_retrieS 16
static int type_to_ctrl(int mode) static int type_to_ctrl(int mode)
{ {
@ -2575,29 +2579,41 @@ int burn_stdio_read_source(struct burn_source *source, char *buf, int bufsize,
int burn_stdio_write(int fd, char *buf, int count, struct burn_drive *d, int burn_stdio_write(int fd, char *buf, int count, struct burn_drive *d,
int flag) int flag)
{ {
int ret; int ret = 0;
char *msg = NULL; char *msg = NULL;
int todo, done, retries;
if (d->cancel) if (d->cancel || count <= 0)
return 0; return 0;
todo = count;
done = 0;
for (retries = 0; todo > 0 && retries <= Libburn_stdio_write_retrieS;
retries++) {
/* /*
fprintf(stderr, "libburn_DEBUG: write(%d, %lX, %d)\n", fprintf(stderr, "libburn_DEBUG: write(%d, %lX, %d)\n",
fd, (unsigned long) buf, count); fd, (unsigned long) buf, count);
*/ */
ret = write(fd, buf + done, todo);
ret = write(fd, buf, count); if (ret < 0)
if (ret != count) { break;
done += ret;
todo -= ret;
}
if (done != count) {
BURN_ALLOC_MEM(msg, char, 160); BURN_ALLOC_MEM(msg, char, 160);
sprintf(msg, sprintf(msg, "Cannot write desired amount of %d bytes.", count);
"Cannot write desired amount of data. write(2) returned %d.", if (retries > 1)
ret); sprintf(msg + strlen(msg), " Did %d retries. Last",
retries - 1);
sprintf(msg + strlen(msg), " write(2) returned %d.", ret);
libdax_msgs_submit(libdax_messenger, d->global_index, libdax_msgs_submit(libdax_messenger, d->global_index,
0x00020148, 0x00020148,
LIBDAX_MSGS_SEV_SORRY, LIBDAX_MSGS_PRIO_HIGH, LIBDAX_MSGS_SEV_SORRY, LIBDAX_MSGS_PRIO_HIGH,
msg, errno, 0); msg, errno, 0);
d->cancel = 1; d->cancel = 1;
return 0; ret = 0; goto ex;
} }
ex:; ex:;
BURN_FREE_MEM(msg); BURN_FREE_MEM(msg);