.

wcfs/client/wcfs.cpp

@@ -589,7 +589,8 @@ error _Conn::resync(zodb::Tid at) {
         //FileH    f    = fit.second;
 
         // XXX need to lock f.mu because wconn.atMu is only R now.
-        // XXX need to coordinate with e.g. FileH.close
+        // XXX need to coordinate with e.g. FileH.close -> "if f.state != CLOSING" ?
+        // XXX need to coordinate with e.g. Conn.open   -> "if f.state != OPENING" ?
         string ack;
         // XXX f._watchMu.lock() + unlock()
         // XXX + recheck status before sending the watch?
@@ -632,6 +633,7 @@ pair<FileH, error> _Conn::open(zodb::Oid foid) {
         f->_nopen++; // XXX lock by f.mu ?
         wconn._mu.Unlock();
 
+        // wait for: "opening" -> opened
         f->_openReady.recv();
         if (f->_openErr != nil) {
             // don't care about f->_nopen-- since f is not returned anywhere
@@ -805,6 +807,8 @@ pair<Mapping, error> _FileH::mmap(int64_t blk_start, int64_t blk_len, VMA *vma)
 
     xerr::Contextf E("%s: mmap f<%s> [blk%ld +blk%ld)", v(f.wconn), v(f.foid), blk_start, blk_len);
 
+    // XXX closed -> return E(os::ErrClosed)
+
     error err;
 
     if (blk_start < 0)

wcfs/client/wcfs_misc.cpp

@@ -37,6 +37,9 @@ using namespace golang;
 // os::
 namespace os {
 
+global<error> ErrClosed = errors::New("file already closed");
+
+
 // XXX -> make public
 static error _pathError(const char *op, const string &path, int syserr);
 

wcfs/client/wcfs_misc.h

@@ -58,6 +58,9 @@ using std::vector;
 // os::
 namespace os {
 
+extern global<error> ErrClosed;
+
+
 // os::File mimics os.File from Go.
 // its operations return error with full file context.
 typedef refptr<class _File> File;