Unplug before sending close() message, to avoid screwing up interfaces
authorMichael Brown <mcb30@etherboot.org>
Tue, 10 Jul 2007 05:25:30 +0000 (06:25 +0100)
committerMichael Brown <mcb30@etherboot.org>
Tue, 10 Jul 2007 05:25:30 +0000 (06:25 +0100)
which respond to close with a reopen() (e.g. iSCSI).

src/core/job.c
src/core/resolv.c
src/core/xfer.c

index 00c0e6c..6c2faf3 100644 (file)
 void job_done ( struct job_interface *job, int rc ) {
        struct job_interface *dest = job_get_dest ( job );
 
-       dest->op->done ( dest, rc );
        job_unplug ( job );
+       dest->op->done ( dest, rc );
        job_put ( dest );
 }
 
 void job_kill ( struct job_interface *job ) {
        struct job_interface *dest = job_get_dest ( job );
 
-       dest->op->kill ( dest );
        job_unplug ( job );
+       dest->op->kill ( dest );
        job_put ( dest );
 }
 
index 5c25ddb..808ab65 100644 (file)
@@ -50,8 +50,8 @@ void resolv_done ( struct resolv_interface *resolv, struct sockaddr *sa,
                   int rc ) {
        struct resolv_interface *dest = resolv_get_dest ( resolv );
 
-       dest->op->done ( dest, sa, rc );
        resolv_unplug ( resolv );
+       dest->op->done ( dest, sa, rc );
        resolv_put ( dest );
 }
 
index 0689587..3add057 100644 (file)
@@ -38,8 +38,8 @@ void xfer_close ( struct xfer_interface *xfer, int rc ) {
 
        DBGC ( xfer, "XFER %p->%p close\n", xfer, dest );
 
-       dest->op->close ( dest, rc );
        xfer_unplug ( xfer );
+       dest->op->close ( dest, rc );
        xfer_put ( dest );
 }