diff --git a/wcfs/client/wcfs.cpp b/wcfs/client/wcfs.cpp
index 3003df8c88be021e6444d813ef0731ac656cb64a..0a96e830e829f488511f0fda7c7f5467ec072300 100644
--- a/wcfs/client/wcfs.cpp
+++ b/wcfs/client/wcfs.cpp
@@ -820,6 +820,9 @@ string WCFS::String() const {
 // NOTE String must be called with Conn.atMu locked.
 string _Conn::String() const {
     const _Conn& wconn = *this;
+    // XXX don't include wcfs as prefix here?
+    // (e.g. to use Conn.String in tracing without wcfs prefix)
+    // (if yes -> go and correct all xerr::Contextf calls)
     return fmt::sprintf("%s: conn%d @%s", v(wconn._wc), wconn._wlink->fd(), v(wconn.at));
 }
 
diff --git a/wcfs/client/wcfs_watchlink.cpp b/wcfs/client/wcfs_watchlink.cpp
index adc7cb2caaf1a768b7aa9b2153d5f8acbbfce10f..562e9d0fe0eca49b787ecb5617d53629371e5d13 100644
--- a/wcfs/client/wcfs_watchlink.cpp
+++ b/wcfs/client/wcfs_watchlink.cpp
@@ -479,6 +479,7 @@ void _WatchLink::decref() {
 
 string _WatchLink::String() const {
     const _WatchLink& wlink = *this;
+    // XXX don't include wcfs as prefix here? (see Conn.String for details)
     return fmt::sprintf("%s: wlink%d", v(wlink._wc), wlink._f->fd());
 }