Source-Changes-HG archive

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

[src/trunk]: src Add a helper program that loads a module and returns the err...



details:   https://anonhg.NetBSD.org/src/rev/ff1b681b55f8
branches:  trunk
changeset: 778775:ff1b681b55f8
user:      jruoho <jruoho%NetBSD.org@localhost>
date:      Fri Apr 13 07:05:32 2012 +0000

description:
Add a helper program that loads a module and returns the errno from modctl(8).

XXX: As noted on tech-kern@, the syscall should be modified to fail
     consistently with a given predefined errno on non-MODULAR kernels.

diffstat:

 distrib/sets/lists/tests/module.mi  |   4 +-
 tests/modules/Makefile              |   3 +-
 tests/modules/k_helper3/Makefile    |  13 +++++
 tests/modules/k_helper3/k_helper3.c |  93 +++++++++++++++++++++++++++++++++++++
 tests/modules/t_modload.sh          |  23 +++++---
 5 files changed, 124 insertions(+), 12 deletions(-)

diffs (206 lines):

diff -r cba9be5ea095 -r ff1b681b55f8 distrib/sets/lists/tests/module.mi
--- a/distrib/sets/lists/tests/module.mi        Fri Apr 13 06:38:55 2012 +0000
+++ b/distrib/sets/lists/tests/module.mi        Fri Apr 13 07:05:32 2012 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: module.mi,v 1.6 2012/02/17 22:36:50 jmmv Exp $
+# $NetBSD: module.mi,v 1.7 2012/04/13 07:05:32 jruoho Exp $
 #
 # These are only made for ports doing modules.
 #
@@ -7,6 +7,8 @@
 ./usr/tests/modules/k_helper/k_helper.kmod     tests-sys-tests         atf
 ./usr/tests/modules/k_helper2                  tests-sys-tests         atf
 ./usr/tests/modules/k_helper2/k_helper2.kmod   tests-sys-tests         atf
+./usr/tests/modules/k_helper3                  tests-sys-tests         atf
+./usr/tests/modules/k_helper3/k_helper3                tests-sys-tests         atf
 ./usr/tests/modules/k_uvm                      tests-sys-tests         atf
 ./usr/tests/modules/k_uvm/k_uvm.kmod           tests-sys-tests         atf
 ./usr/tests/modules/t_abi_uvm                  tests-sys-tests         atf
diff -r cba9be5ea095 -r ff1b681b55f8 tests/modules/Makefile
--- a/tests/modules/Makefile    Fri Apr 13 06:38:55 2012 +0000
+++ b/tests/modules/Makefile    Fri Apr 13 07:05:32 2012 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: Makefile,v 1.9 2012/02/17 22:36:50 jmmv Exp $
+# $NetBSD: Makefile,v 1.10 2012/04/13 07:05:32 jruoho Exp $
 
 .include <bsd.own.mk>
 
@@ -19,6 +19,7 @@
 
 SUBDIR=                k_helper
 SUBDIR+=       k_helper2
+SUBDIR+=       k_helper3
 SUBDIR+=       k_uvm
 
 .include <bsd.test.mk>
diff -r cba9be5ea095 -r ff1b681b55f8 tests/modules/k_helper3/Makefile
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/modules/k_helper3/Makefile  Fri Apr 13 07:05:32 2012 +0000
@@ -0,0 +1,13 @@
+#      $NetBSD: Makefile,v 1.1 2012/04/13 07:05:32 jruoho Exp $
+
+.include <bsd.own.mk>
+
+PROG=          k_helper3
+SRCS=                  k_helper3.c
+
+LDADD+=                -lprop
+
+MAN=           # defined
+ATFFILE=       no
+
+.include <bsd.prog.mk>
diff -r cba9be5ea095 -r ff1b681b55f8 tests/modules/k_helper3/k_helper3.c
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/tests/modules/k_helper3/k_helper3.c       Fri Apr 13 07:05:32 2012 +0000
@@ -0,0 +1,93 @@
+/*     $NetBSD: k_helper3.c,v 1.1 2012/04/13 07:05:32 jruoho Exp $ */
+
+/*-
+ * Copyright (c) 2012 The NetBSD Foundation, Inc.
+ * All rights reserved.
+ *
+ * This code is derived from software contributed to The NetBSD Foundation
+ * by Jukka Ruohonen.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE NETBSD FOUNDATION, INC. AND CONTRIBUTORS
+ * ``AS IS'' AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
+ * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED.  IN NO EVENT SHALL THE FOUNDATION OR CONTRIBUTORS
+ * BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
+ * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
+ * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
+ * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
+ * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
+ * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGE.
+ */
+#include <sys/cdefs.h>
+__RCSID("$NetBSD: k_helper3.c,v 1.1 2012/04/13 07:05:32 jruoho Exp $");
+
+#include <sys/module.h>
+
+#include <assert.h>
+#include <errno.h>
+#include <stdarg.h>
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+
+#include <prop/proplib.h>
+
+static int load(const char *, ...);
+
+/*
+ * A program that loads a module and returns the errno(2) from modctl(8).
+ */
+int
+main(int argc, char *argv[])
+{
+       assert(argc == 3);
+
+       return load(argv[1], argv[2]);
+}
+
+static __printflike(1, 2) int
+load(const char *fmt, ...)
+{
+       char filename[MAXPATHLEN], *propsstr;
+       prop_dictionary_t props;
+       modctl_load_t ml;
+       int serrno, rv;
+       va_list ap;
+
+       props = prop_dictionary_create();
+       propsstr = prop_dictionary_externalize(props);
+       assert(propsstr != NULL);
+       prop_object_release(props);
+
+       va_start(ap, fmt);
+       (void)vsnprintf(filename, sizeof(filename), fmt, ap);
+       va_end(ap);
+
+       ml.ml_filename = filename;
+       ml.ml_flags = 0;
+       ml.ml_props = propsstr;
+       ml.ml_propslen = strlen(propsstr);
+
+       printf("Loading module %s\n", filename);
+       errno = serrno = 0;
+
+       rv = modctl(MODCTL_LOAD, &ml);
+       serrno = errno;
+
+       if (rv != -1)
+               (void)modctl(MODCTL_UNLOAD, filename);
+
+       free(propsstr);
+
+       return serrno;
+}
diff -r cba9be5ea095 -r ff1b681b55f8 tests/modules/t_modload.sh
--- a/tests/modules/t_modload.sh        Fri Apr 13 06:38:55 2012 +0000
+++ b/tests/modules/t_modload.sh        Fri Apr 13 07:05:32 2012 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: t_modload.sh,v 1.10 2012/03/20 05:50:11 jruoho Exp $
+# $NetBSD: t_modload.sh,v 1.11 2012/04/13 07:05:32 jruoho Exp $
 #
 # Copyright (c) 2008 The NetBSD Foundation, Inc.
 # All rights reserved.
@@ -37,11 +37,12 @@
 }
 plain_body() {
 
-       # XXX: There should be a reliable way to detect MODULAR.
+       # XXX: Adjust when modctl(8) fails consistently.
        #
-       sysctl machdep.xen > /dev/null 2>&1
+       $(atf_get_srcdir)/k_helper3/k_helper3 \
+               "%s/k_helper/k_helper.kmod" $(atf_get_srcdir)
 
-       if [ $? -eq 0 ]; then
+       if [ $? -eq 78 ]; then
                atf_skip "host does not support modules"
        fi
 
@@ -109,11 +110,12 @@
 }
 iflag_body() {
 
-       # XXX: There should be a reliable way to detect MODULAR.
+       # XXX: Adjust when modctl(8) fails consistently.
        #
-       sysctl machdep.xen > /dev/null 2>&1
+       $(atf_get_srcdir)/k_helper3/k_helper3 \
+               "%s/k_helper/k_helper.kmod" $(atf_get_srcdir)
 
-       if [ $? -eq 0 ]; then
+       if [ $? -eq 78 ]; then
                atf_skip "host does not support modules"
        fi
 
@@ -162,11 +164,12 @@
 }
 sflag_body() {
 
-       # XXX: There should be a reliable way to detect MODULAR.
+       # XXX: Adjust when modctl(8) fails consistently.
        #
-       sysctl machdep.xen > /dev/null 2>&1
+       $(atf_get_srcdir)/k_helper3/k_helper3 \
+               "%s/k_helper/k_helper.kmod" $(atf_get_srcdir)
 
-       if [ $? -eq 0 ]; then
+       if [ $? -eq 78 ]; then
                atf_skip "host does not support modules"
        fi
 



Home | Main Index | Thread Index | Old Index