Remove extraneous sysfd2path()
authorBarret Rhoden <brho@cs.berkeley.edu>
Tue, 30 Apr 2019 00:54:33 +0000 (20:54 -0400)
committerBarret Rhoden <brho@cs.berkeley.edu>
Tue, 30 Apr 2019 00:54:33 +0000 (20:54 -0400)
We already have sys_fd2path(), and no one was using sysfd2path().  The
latter, which we are removing, is the Plan 9 implementation, I think.

Arguably, the one we are removing is a little faster, since it doesn't
invoke the snprintf() mechanisms and is a straight memcpy, but it also
doesn't handle the "no string" case yet.  If we ever want to make
sys_fd2path() faster, we certainly can.

Signed-off-by: Barret Rhoden <brho@cs.berkeley.edu>
kern/include/ns.h
kern/src/ns/sysfile.c

index 35c6933..32a51d6 100644 (file)
@@ -1074,7 +1074,6 @@ int sys_dup_to(struct proc *from_proc, unsigned int from_fd,
                struct proc *to_proc, unsigned int to_fd);
 int sysfstat(int fd, uint8_t*, int n);
 int sysfstatakaros(int fd, struct kstat *);
-char *sysfd2path(int fd);
 char *sysgetcwd(void);
 int sysfauth(int fd, char *aname);
 int sysfversion(int fd, unsigned int msize, char *vers, unsigned int arglen);
index 557f3ef..7f91293 100644 (file)
@@ -307,31 +307,6 @@ int sys_dup_to(struct proc *from_proc, unsigned int from_fd,
        return 0;
 }
 
-char *sysfd2path(int fd)
-{
-       ERRSTACK(1);
-       struct chan *c;
-       char *s;
-
-       if (waserror()) {
-               poperror();
-               return NULL;
-       }
-       c = fdtochan(&current->open_files, fd, -1, 0, 1);
-       s = NULL;
-       if (c->name != NULL) {
-               s = kzmalloc(c->name->len + 1, 0);
-               if (s == NULL) {
-                       cclose(c);
-                       error(ENOMEM, ERROR_FIXME);
-               }
-               memmove(s, c->name->s, c->name->len + 1);
-       }
-       cclose(c);
-       poperror();
-       return s;
-}
-
 char *sysgetcwd(void)
 {
        char *s = NULL;