do we need to use locks on the avl client_trees in source.c and fserv.c? Found in TODO, still relevant?