Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/tests/fs Add basic test for umapfs functionality.



details:   https://anonhg.NetBSD.org/src/rev/f5bda9d2bccd
branches:  trunk
changeset: 753476:f5bda9d2bccd
user:      pooka <pooka%NetBSD.org@localhost>
date:      Tue Mar 30 01:05:28 2010 +0000

description:
Add basic test for umapfs functionality.

XXX: the reverse mapping case (last subsubtest in t_basic) does
not make any sense, but apparently that how umapfs works.  I'm not
familiar enough with the code to determine if this is a wanted
feature or a pure and simple bug.

diffstat:

 tests/fs/Makefile         |    4 +-
 tests/fs/umapfs/Atffile   |    6 +
 tests/fs/umapfs/Makefile  |   12 +++
 tests/fs/umapfs/t_basic.c |  162 ++++++++++++++++++++++++++++++++++++++++++++++
 4 files changed, 182 insertions(+), 2 deletions(-)

diffs (208 lines):

diff -r cf520953cc76 -r f5bda9d2bccd tests/fs/Makefile
--- a/tests/fs/Makefile Tue Mar 30 01:02:47 2010 +0000
+++ b/tests/fs/Makefile Tue Mar 30 01:05:28 2010 +0000
@@ -1,10 +1,10 @@
-# $NetBSD: Makefile,v 1.6 2010/03/30 01:02:47 pooka Exp $
+# $NetBSD: Makefile,v 1.7 2010/03/30 01:05:28 pooka Exp $
 
 .include <bsd.own.mk>
 
 TESTSDIR=      ${TESTSBASE}/fs
 
-SUBDIR+=       ffs nullfs puffs tmpfs union
+SUBDIR+=       ffs nullfs puffs tmpfs umapfs union
 
 FILES= h_funcs.subr
 FILESDIR= ${TESTSDIR}
diff -r cf520953cc76 -r f5bda9d2bccd tests/fs/umapfs/Atffile
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/fs/umapfs/Atffile   Tue Mar 30 01:05:28 2010 +0000
@@ -0,0 +1,6 @@
+Content-Type: application/X-atf-atffile; version="1"
+X-NetBSD-Id: "$NetBSD: Atffile,v 1.1 2010/03/30 01:05:28 pooka Exp $"
+
+prop: test-suite = "NetBSD"
+
+tp: t_*
diff -r cf520953cc76 -r f5bda9d2bccd tests/fs/umapfs/Makefile
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/fs/umapfs/Makefile  Tue Mar 30 01:05:28 2010 +0000
@@ -0,0 +1,12 @@
+#      $NetBSD: Makefile,v 1.1 2010/03/30 01:05:28 pooka Exp $
+#
+
+TESTSDIR=      ${TESTSBASE}/fs/union
+WARNS=         4
+
+TESTS_C=       t_basic
+
+LDADD+=        -lrumpfs_tmpfs -lrumpfs_umapfs -lrumpvfs_layerfs        # fs drivers
+LDADD+=        -lrumpvfs -lrump -lrumpuser -lpthread                   # base
+
+.include <bsd.test.mk>
diff -r cf520953cc76 -r f5bda9d2bccd tests/fs/umapfs/t_basic.c
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/fs/umapfs/t_basic.c Tue Mar 30 01:05:28 2010 +0000
@@ -0,0 +1,162 @@
+/*     $NetBSD: t_basic.c,v 1.1 2010/03/30 01:05:28 pooka Exp $        */
+
+#include <sys/types.h>
+#include <sys/mount.h>
+
+#include <atf-c.h>
+#include <err.h>
+#include <errno.h>
+#include <fcntl.h>
+#include <stdio.h>
+#include <unistd.h>
+#include <string.h>
+#include <stdlib.h>
+
+#include <rump/rump.h>
+#include <rump/rump_syscalls.h>
+#include <rump/rumpvfs_if_pub.h>
+
+#include <fs/tmpfs/tmpfs_args.h>
+#include <miscfs/umapfs/umap.h>
+
+#define USE_ATF
+#include "../../h_macros.h"
+
+#ifdef USE_ATF
+ATF_TC(basic);
+ATF_TC_HEAD(basic, tc)
+{
+       atf_tc_set_md_var(tc, "descr", "basic umapfs mapping");
+}
+#else
+#define atf_tc_fail(...) errx(1, __VA_ARGS__)
+#endif
+
+/* deal with time_t change for running this on 5.0 */
+#if __NetBSD_Prereq__(5,99,7)
+#define statfn rump_sys_stat
+#else
+#define statfn rump_pub_sys___stat30
+#endif
+
+static void
+xtouch(const char *path)
+{
+       int fd;
+
+       fd = rump_sys_open(path, O_CREAT | O_RDWR, 0777);
+       if (fd == -1)
+               atf_tc_fail_errno("create %s", path);
+       rump_sys_close(fd);
+}
+
+static void
+xchown(const char *path, uid_t uid, gid_t gid)
+{
+
+       if (rump_sys_chown(path, uid, gid) == -1)
+               atf_tc_fail_errno("chown %s failed", path);
+}
+
+static void
+testuidgid(const char *path, uid_t uid, gid_t gid)
+{
+       struct stat sb;
+
+       if (statfn(path, &sb) == -1)
+               atf_tc_fail_errno("stat %s", path);
+       if (uid != (uid_t)-1) {
+               if (sb.st_uid != uid)
+                       atf_tc_fail("%s: expected uid %d, got %d",
+                           path, uid, sb.st_uid);
+       }
+       if (gid != (gid_t)-1) {
+               if (sb.st_gid != gid)
+                       atf_tc_fail("%s: expected gid %d, got %d",
+                           path, gid, sb.st_gid);
+       }
+}
+
+#ifdef USE_ATF
+ATF_TC_BODY(basic, tc)
+#else
+int main(int argc, char *argv[])
+#endif
+{
+       struct umap_args umargs;
+       struct tmpfs_args targs;
+       u_long umaps[2][2];
+       u_long gmaps[2][2];
+
+       rump_init();
+       if (rump_sys_mkdir("/td1", 0777) == -1)
+               atf_tc_fail_errno("mp1");
+       if (rump_sys_mkdir("/td2", 0777) == -1)
+               atf_tc_fail_errno("mp1");
+
+       /* use tmpfs because rumpfs doesn't support ownership */
+       memset(&targs, 0, sizeof(targs));
+       targs.ta_version = TMPFS_ARGS_VERSION;
+       targs.ta_root_mode = 0777;
+       if (rump_sys_mount(MOUNT_TMPFS, "/td1", 0, &targs, sizeof(targs)) == -1)
+               atf_tc_fail_errno("could not mount tmpfs td1");
+
+       memset(&umargs, 0, sizeof(umargs));
+
+       /*
+        * Map td1 uid 555 to td2 uid 777 (yes, IMHO the umapfs
+        * mapping format is counter-intuitive).
+        */
+       umaps[0][0] = 777;
+       umaps[0][1] = 555;
+       umaps[1][0] = 0;
+       umaps[1][1] = 0;
+       gmaps[0][0] = 4321;
+       gmaps[0][1] = 1234;
+       gmaps[1][0] = 0;
+       gmaps[1][1] = 0;
+
+       umargs.umap_target = __UNCONST("/td1");
+       umargs.nentries = 2;
+       umargs.gnentries = 2;
+       umargs.mapdata = umaps;
+       umargs.gmapdata = gmaps;
+
+       if (rump_sys_mount(MOUNT_UMAP, "/td2", 0, &umargs,sizeof(umargs)) == -1)
+               atf_tc_fail_errno("could not mount umapfs");
+
+       xtouch("/td1/noch");
+       testuidgid("/td1/noch", 0, 0);
+       testuidgid("/td2/noch", 0, 0);
+
+       xtouch("/td1/nomap");
+       xchown("/td1/nomap", 1, 2);
+       testuidgid("/td1/nomap", 1, 2);
+       testuidgid("/td2/nomap", -1, -1);
+
+       xtouch("/td1/forwmap");
+       xchown("/td1/forwmap", 555, 1234);
+       testuidgid("/td1/forwmap", 555, 1234);
+       testuidgid("/td2/forwmap", 777, 4321);
+
+       /*
+        * this *CANNOT* be correct???
+        */
+       xtouch("/td1/revmap");
+       /*
+        * should be 777 / 4321 (?), but makes first test fail since
+        * it gets 777 / 4321, i.e. unmapped results.
+        */
+       xchown("/td2/revmap", 555, 1234);
+       testuidgid("/td1/revmap", 555, 1234);
+       testuidgid("/td2/revmap", 777, 4321);
+       
+}
+
+#ifdef USE_ATF
+ATF_TP_ADD_TCS(tp)
+{
+       ATF_TP_ADD_TC(tp, basic);
+       return 0; /*XXX?*/
+}
+#endif



Home | Main Index | Thread Index | Old Index