--- IO-AIO/AIO.xs 2005/07/31 19:04:45 1.19 +++ IO-AIO/AIO.xs 2005/08/17 03:52:20 1.26 @@ -1,6 +1,4 @@ -#ifndef _REENTRANT -# define _REENTRANT 1 -#endif +#define _REENTRANT 1 #include #include "EXTERN.h" @@ -33,8 +31,10 @@ REQ_QUIT, REQ_OPEN, REQ_CLOSE, REQ_READ, REQ_WRITE, REQ_READAHEAD, - REQ_STAT, REQ_LSTAT, REQ_FSTAT, REQ_UNLINK, + REQ_STAT, REQ_LSTAT, REQ_FSTAT, REQ_FSYNC, REQ_FDATASYNC, + REQ_UNLINK, REQ_RMDIR, + REQ_SYMLINK, }; typedef struct aio_cb { @@ -49,7 +49,7 @@ mode_t mode; /* open */ int errorno; SV *data, *callback, *fh; - void *dataptr; + void *dataptr, *data2ptr; STRLEN dataoffset; Stat_t *statdata; @@ -64,11 +64,29 @@ static pthread_mutex_t reslock = PTHREAD_MUTEX_INITIALIZER; static pthread_mutex_t reqlock = PTHREAD_MUTEX_INITIALIZER; +static pthread_mutex_t frklock = PTHREAD_MUTEX_INITIALIZER; static pthread_cond_t reqwait = PTHREAD_COND_INITIALIZER; static volatile aio_req reqs, reqe; /* queue start, queue end */ static volatile aio_req ress, rese; /* queue start, queue end */ +static void free_req (aio_req req) +{ + if (req->data) + SvREFCNT_dec (req->data); + + if (req->fh) + SvREFCNT_dec (req->fh); + + if (req->type == REQ_STAT || req->type == REQ_LSTAT || req->type == REQ_FSTAT) + Safefree (req->statdata); + + if (req->callback) + SvREFCNT_dec (req->callback); + + Safefree (req); +} + static void poll_wait () { @@ -87,6 +105,7 @@ { dSP; int count = 0; + int do_croak = 0; aio_req req, prv; pthread_mutex_lock (&reslock); @@ -94,7 +113,7 @@ { /* read any signals sent by the worker threads */ char buf [32]; - while (read (respipe [0], buf, 32) > 0) + while (read (respipe [0], buf, 32) == 32) ; } @@ -118,19 +137,11 @@ SvCUR_set (req->data, req->dataoffset + req->result > 0 ? req->result : 0); - if (req->data) - SvREFCNT_dec (req->data); - - if (req->fh) - SvREFCNT_dec (req->fh); - if (req->type == REQ_STAT || req->type == REQ_LSTAT || req->type == REQ_FSTAT) { PL_laststype = req->type == REQ_LSTAT ? OP_LSTAT : OP_STAT; PL_laststatval = req->result; PL_statcache = *(req->statdata); - - Safefree (req->statdata); } ENTER; @@ -160,19 +171,19 @@ } LEAVE; - - if (req->callback) - SvREFCNT_dec (req->callback); + do_croak = SvTRUE (ERRSV); + errno = errorno; count++; } prv = req; req = req->next; - Safefree (prv); + free_req (prv); - /* TODO: croak on errors? */ + if (do_croak) + croak (0); } return count; @@ -231,12 +242,93 @@ end_thread (void) { aio_req req; - New (0, req, 1, aio_cb); + Newz (0, req, 1, aio_cb); req->type = REQ_QUIT; send_req (req); } + +static void min_parallel (int nthreads) +{ + while (nthreads > started) + start_thread (); +} + +static void max_parallel (int nthreads) +{ + int cur = started; + while (cur > nthreads) + { + end_thread (); + cur--; + } + + while (started > nthreads) + { + poll_wait (); + poll_cb (); + } +} + +static void create_pipe () +{ + if (pipe (respipe)) + croak ("unable to initialize result pipe"); + + if (fcntl (respipe [0], F_SETFL, O_NONBLOCK)) + croak ("cannot set result pipe to nonblocking mode"); + + if (fcntl (respipe [1], F_SETFL, O_NONBLOCK)) + croak ("cannot set result pipe to nonblocking mode"); +} + +static void atfork_prepare (void) +{ + pthread_mutex_lock (&frklock); + pthread_mutex_lock (&reqlock); + pthread_mutex_lock (&reslock); +} + +static void atfork_parent (void) +{ + pthread_mutex_unlock (&reslock); + pthread_mutex_unlock (&reqlock); + pthread_mutex_unlock (&frklock); +} + +static void atfork_child (void) +{ + int restart = started; + started = 0; + + while (reqs) + { + free_req (reqs); + reqs = reqs->next; + } + + reqs = reqe = 0; + + while (ress) + { + free_req (ress); + ress = ress->next; + } + + ress = rese = 0; + + close (respipe [0]); + close (respipe [1]); + + create_pipe (); + + atfork_parent (); + + min_parallel (restart); +} + +/*****************************************************************************/ /* work around various missing functions */ #if !HAVE_PREADWRITE @@ -308,6 +400,8 @@ } #endif +/*****************************************************************************/ + static void * aio_proc (void *thr_arg) { @@ -354,6 +448,8 @@ case REQ_OPEN: req->result = open (req->dataptr, req->fd, req->mode); break; case REQ_CLOSE: req->result = close (req->fd); break; case REQ_UNLINK: req->result = unlink (req->dataptr); break; + case REQ_RMDIR: req->result = rmdir (req->dataptr); break; + case REQ_SYMLINK: req->result = symlink (req->data2ptr, req->dataptr); break; case REQ_FDATASYNC: req->result = fdatasync (req->fd); break; case REQ_FSYNC: req->result = fsync (req->fd); break; @@ -392,49 +488,37 @@ return 0; } +#define dREQ \ + aio_req req; \ + \ + if (SvOK (callback) && !SvROK (callback)) \ + croak ("clalback must be undef or of reference type"); \ + \ + Newz (0, req, 1, aio_cb); \ + if (!req) \ + croak ("out of memory during aio_req allocation"); \ + \ + req->callback = SvREFCNT_inc (callback); + MODULE = IO::AIO PACKAGE = IO::AIO PROTOTYPES: ENABLE BOOT: { - if (pipe (respipe)) - croak ("unable to initialize result pipe"); - - if (fcntl (respipe [0], F_SETFL, O_NONBLOCK)) - croak ("cannot set result pipe to nonblocking mode"); - - if (fcntl (respipe [1], F_SETFL, O_NONBLOCK)) - croak ("cannot set result pipe to nonblocking mode"); + create_pipe (); + pthread_atfork (atfork_prepare, atfork_parent, atfork_child); } void min_parallel(nthreads) int nthreads PROTOTYPE: $ - CODE: - while (nthreads > started) - start_thread (); void max_parallel(nthreads) int nthreads PROTOTYPE: $ - CODE: -{ - int cur = started; - while (cur > nthreads) - { - end_thread (); - cur--; - } - - while (started > nthreads) - { - poll_wait (); - poll_cb (); - } -} int max_outstanding(nreqs) @@ -453,19 +537,13 @@ PROTOTYPE: $$$;$ CODE: { - aio_req req; - - Newz (0, req, 1, aio_cb); - - if (!req) - croak ("out of memory during aio_req allocation"); + dREQ; req->type = REQ_OPEN; req->data = newSVsv (pathname); - req->dataptr = SvPV_nolen (req->data); + req->dataptr = SvPVbyte_nolen (req->data); req->fd = flags; req->mode = mode; - req->callback = SvREFCNT_inc (callback); send_req (req); } @@ -481,17 +559,11 @@ aio_fdatasync = REQ_FDATASYNC CODE: { - aio_req req; - - Newz (0, req, 1, aio_cb); - - if (!req) - croak ("out of memory during aio_req allocation"); + dREQ; req->type = ix; req->fh = newSVsv (fh); req->fd = PerlIO_fileno (IoIFP (sv_2io (fh))); - req->callback = SvREFCNT_inc (callback); send_req (req); } @@ -512,7 +584,7 @@ { aio_req req; STRLEN svlen; - char *svptr = SvPV (data, svlen); + char *svptr = SvPVbyte (data, svlen); SvUPGRADE (data, SVt_PV); SvPOK_on (data); @@ -538,22 +610,21 @@ if (length < 0) croak ("length must not be negative"); - Newz (0, req, 1, aio_cb); - - if (!req) - croak ("out of memory during aio_req allocation"); + { + dREQ; - req->type = ix; - req->fh = newSVsv (fh); - req->fd = PerlIO_fileno (ix == REQ_READ ? IoIFP (sv_2io (fh)) - : IoOFP (sv_2io (fh))); - req->offset = offset; - req->length = length; - req->data = SvREFCNT_inc (data); - req->dataptr = (char *)svptr + dataoffset; - req->callback = SvREFCNT_inc (callback); + req->type = ix; + req->fh = newSVsv (fh); + req->fd = PerlIO_fileno (ix == REQ_READ ? IoIFP (sv_2io (fh)) + : IoOFP (sv_2io (fh))); + req->offset = offset; + req->length = length; + req->data = SvREFCNT_inc (data); + req->dataptr = (char *)svptr + dataoffset; + req->callback = SvREFCNT_inc (callback); - send_req (req); + send_req (req); + } } void @@ -565,22 +636,13 @@ PROTOTYPE: $$$;$ CODE: { - aio_req req; - - if (length < 0) - croak ("length must not be negative"); - - Newz (0, req, 1, aio_cb); - - if (!req) - croak ("out of memory during aio_req allocation"); + dREQ; req->type = REQ_READAHEAD; req->fh = newSVsv (fh); req->fd = PerlIO_fileno (IoIFP (sv_2io (fh))); req->offset = offset; req->length = length; - req->callback = SvREFCNT_inc (callback); send_req (req); } @@ -594,23 +656,17 @@ aio_lstat = REQ_LSTAT CODE: { - aio_req req; - - Newz (0, req, 1, aio_cb); - - if (!req) - croak ("out of memory during aio_req allocation"); + dREQ; New (0, req->statdata, 1, Stat_t); - if (!req->statdata) - croak ("out of memory during aio_req->statdata allocation"); + croak ("out of memory during aio_req->statdata allocation (sorry, i just leaked memory, too)"); if (SvPOK (fh_or_path)) { req->type = ix; req->data = newSVsv (fh_or_path); - req->dataptr = SvPV_nolen (req->data); + req->dataptr = SvPVbyte_nolen (req->data); } else { @@ -619,8 +675,6 @@ req->fd = PerlIO_fileno (IoIFP (sv_2io (fh_or_path))); } - req->callback = SvREFCNT_inc (callback); - send_req (req); } @@ -628,19 +682,34 @@ aio_unlink(pathname,callback=&PL_sv_undef) SV * pathname SV * callback + ALIAS: + aio_unlink = REQ_UNLINK + aio_rmdir = REQ_RMDIR CODE: { - aio_req req; + dREQ; - Newz (0, req, 1, aio_cb); + req->type = ix; + req->data = newSVsv (pathname); + req->dataptr = SvPVbyte_nolen (req->data); - if (!req) - croak ("out of memory during aio_req allocation"); + send_req (req); +} + +void +aio_symlink(oldpath,newpath,callback=&PL_sv_undef) + SV * oldpath + SV * newpath + SV * callback + CODE: +{ + dREQ; - req->type = REQ_UNLINK; - req->data = newSVsv (pathname); - req->dataptr = SvPV_nolen (req->data); - req->callback = SvREFCNT_inc (callback); + req->type = REQ_SYMLINK; + req->fh = newSVsv (oldpath); + req->data2ptr = SvPVbyte_nolen (req->fh); + req->data = newSVsv (newpath); + req->dataptr = SvPVbyte_nolen (req->data); send_req (req); }