devproc: change user from char* to struct username
[akaros.git] / kern / drivers / dev / root.c
index 32c361f..12f35e5 100644 (file)
@@ -54,7 +54,6 @@ int rootmaxq = MAXFILE;
 int inumber = 13;
 
 /* TODO:
- *  - fix DOTDOT in gen.
  *  - synchronization!  what needs protection from concurrent use, etc.
  *     - clean up documentation and whatnot
  *     - does remove, mkdir, rmdir work?
@@ -76,14 +75,17 @@ int inumber = 13;
  * dotdot is .., ptr is data (for files)
  * size is # elements (for dirs)
  * *sizep is a pointer for reasons not understood.
- * next is the sibling. For a dir, it's the first element after '.'.
+ * child is the qid.path of the first child of a directory.
+ * Possibly 0 == no child.
+ * To find the next sibling (in a directory), look at roottab[i].qid.vers.
+ *
  *     int     dotdot;
- *      int     child; 
+ *      int     child;
  *     void    *ptr;
  *     int     size;
  *     int     *sizep;
- * 
- * entry is empty if type is 0. We look in roottab to determine that. 
+ *
+ * entry is empty if type is 0. We look in roottab to determine that.
 */
 /* we pack the qid as follows: path is the index, vers is ., and type is type */
 
@@ -95,7 +97,19 @@ int inumber = 13;
  * If you want to add new entries, add it to the roottab such that the linked
  * list of indexes is a cycle (change the last current one), then add an entry
  * to rootdata, and then change the first rootdata entry to have another entry.
- * Yeah, it's a pain in the ass. */
+ * Yeah, it's a pain in the ass.
+ *
+ * To add subdirectories, or any child of a directory, the files (e.g. env_dir1)
+ * go in roottab.  Children of a parent are linked with their vers (note
+ * env_dir1 points to env_dir2), and the last item's vers = 0.  These files need
+ * their dotdot set in rootdata to the qid of their parent.  The directory that
+ * has children needs its child pointer set to the first qid in the list, and
+ * its data pointer must point to the roottab entry for the child.  This also
+ * means that all child entries in roottab for a parent must be contiguous.
+ *
+ * Yeah, it's a pain in the ass.  And, given this structure, it probably can't
+ * grow dynamically (I think we assume roottab[i] = entry for qid.path all over
+ * the place - imagine what happens if we wanted to squeeze in a new entry). */
 struct dirtab roottab[MAXFILE] = {
        {"", {0, 0, QTDIR}, 0, DMDIR | 0777},
        {"chan", {1, 2, QTDIR}, 0, DMDIR | 0777},
@@ -111,6 +125,8 @@ struct dirtab roottab[MAXFILE] = {
        {"srv", {11, 12, QTDIR}, 0, DMDIR | 0777},
        {"mnt", {12, 13, QTDIR}, 0, DMDIR | 0777},
        {"proc", {13, 0, QTDIR}, 0, DMDIR | 0777},
+       {"env_dir1", {14, 15, QTDIR}, 0, DMDIR | 0777},
+       {"env_dir2", {15, 0, QTDIR}, 0, DMDIR | 0777},
 };
 
 struct rootdata {
@@ -131,11 +147,13 @@ struct rootdata rootdata[MAXFILE] = {
        {0,     0,       NULL,   0,      NULL},
        {0,     0,       NULL,   0,      NULL},
        {0,     0,       NULL,   0,      NULL},
+       {0,     14,      &roottab[14],   2,      NULL},
        {0,     0,       NULL,   0,      NULL},
        {0,     0,       NULL,   0,      NULL},
        {0,     0,       NULL,   0,      NULL},
        {0,     0,       NULL,   0,      NULL},
-       {0,     0,       NULL,   0,      NULL},
+       {9,     0,       NULL,   0,      NULL},
+       {9,     0,       NULL,   0,      NULL},
 };
 
 /* this is super useful */
@@ -230,7 +248,7 @@ static struct chan *rootattach(char *spec)
 {
        struct chan *c;
        if (*spec)
-               error(EINVAL, NULL);
+               error(EINVAL, ERROR_FIXME);
        c = devattach(devname(), spec);
        mkqid(&c->qid, roottab[0].qid.path, roottab[0].qid.vers, QTDIR);
        return c;
@@ -247,26 +265,10 @@ rootgen(struct chan *c, char *name,
               tab, nd, s, name);
 
        if (s == DEVDOTDOT) {
-               panic("this is busted");
                p = rootdata[c->qid.path].dotdot;
-               // XXX we need to set the vers too.  equiv to mkqid(&c->qid, ...)
-               c->qid.path = p;
-               c->qid.type = QTDIR;
-               name = devname();
-               if (p != 0) {
-                       /* TODO: what is this doing?  do we want to walk the entire table,
-                        * or are we just walking the siblings of our parent? */
-                       for (i = p;;) {
-                               if (roottab[i].qid.path == c->qid.path) {
-                                       name = roottab[i].name;
-                                       break;
-                               }
-                               i = roottab[i].qid.vers;
-                               if (!i)
-                                       break;
-                       }
-               }
-               devdir(c, c->qid, name, 0, eve, 0777, dp);
+               c->qid = roottab[p].qid;
+               name = roottab[p].name;
+               devdir(c, c->qid, name, 0, eve.name, 0777, dp);
                return 1;
        }
 
@@ -275,7 +277,7 @@ rootgen(struct chan *c, char *name,
                if (s)
                        return -1;
                tab = &roottab[c->qid.path];
-               devdir(c, tab->qid, tab->name, tab->length, eve, tab->perm, dp);
+               devdir(c, tab->qid, tab->name, tab->length, eve.name, tab->perm, dp);
                return 1;
        }
 
@@ -287,7 +289,8 @@ rootgen(struct chan *c, char *name,
                for (iter = 0, i = rootdata[path].child; /* break */; iter++) {
                        if (strncmp(tab->name, name, KNAMELEN) == 0) {
                                printd("Rootgen returns 1 for %s\n", name);
-                               devdir(c, tab->qid, tab->name, tab->length, eve, tab->perm, dp);
+                               devdir(c, tab->qid, tab->name, tab->length, eve.name, tab->perm,
+                                      dp);
                                printd("return 1 with [%d, %d, %d]\n", dp->qid.path,
                                       dp->qid.vers, dp->qid.type);
                                return 1;
@@ -324,7 +327,7 @@ rootgen(struct chan *c, char *name,
                return -1;
        }
        printd("root scan find returns path %p name %s\n", tab->qid.path, tab->name);
-       devdir(c, tab->qid, tab->name, tab->length, eve, tab->perm, dp);
+       devdir(c, tab->qid, tab->name, tab->length, eve.name, tab->perm, dp);
        return 1;
 }
 
@@ -367,7 +370,7 @@ static void rootcreate(struct chan *c, char *name, int omode, uint32_t perm)
        /* need to filter openmode so that it gets only the access-type bits */
        omode = openmode(omode);
        c->mode = openmode(omode);
-       printd("rootcreate: c %p, name %s, omode %o, perm %x\n", 
+       printd("rootcreate: c %p, name %s, omode %o, perm %x\n",
               c, name, omode, perm);
        /* find an empty slot */
        int path = c->qid.path;
@@ -429,7 +432,7 @@ static long rootwrite(struct chan *c, void *a, long n, int64_t off)
 
        if (off + n > rd->size){
                void *p;
-               p = krealloc(rd->ptr, off + n, KMALLOC_WAIT);
+               p = krealloc(rd->ptr, off + n, MEM_WAIT);
                if (! p)
                        error(EFAIL, "rootwrite: could not grow the file to %d bytes",
                                  off + n);
@@ -454,11 +457,11 @@ static int rootwstat(struct chan *c, uint8_t *m_buf, int m_buf_sz)
        /* common trick in wstats.  we want the dir and any strings in the M.  the
         * strings are smaller than entire M (strings plus other M).  the strings
         * will be placed right after the dir (dir[1]) */
-       dir = kzmalloc(sizeof(struct dir) + m_buf_sz, KMALLOC_WAIT);
+       dir = kzmalloc(sizeof(struct dir) + m_buf_sz, MEM_WAIT);
        m_sz = convM2D(m_buf, m_buf_sz, &dir[0], (char*)&dir[1]);
        if (!m_sz) {
                kfree(dir);
-               error(ENODATA, NULL);
+               error(ENODATA, ERROR_FIXME);
        }
        /* TODO: handle more things than just the mode */
        if (!emptystr(dir->name))