* [LTP] [PATCH v3] lib: tst_module: add library functions for kernel modules
@ 2013-06-26 16:36 Alexey Kodanev
2013-06-27 14:22 ` chrubis
0 siblings, 1 reply; 3+ messages in thread
From: Alexey Kodanev @ 2013-06-26 16:36 UTC (permalink / raw)
To: ltp-list; +Cc: vasily.isaenko, Alexey Kodanev
There are new functions: tst_module_exists(), tst_module_load() and
tst_module_unload(). These functions help to load and unload kernel
modules in the tests.
Signed-off-by: Alexey Kodanev <alexey.kodanev@oracle.com>
---
include/tst_module.h | 69 ++++++++++++++++++++++++++++++++
lib/tst_module.c | 108 ++++++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 177 insertions(+), 0 deletions(-)
create mode 100644 include/tst_module.h
create mode 100644 lib/tst_module.c
diff --git a/include/tst_module.h b/include/tst_module.h
new file mode 100644
index 0000000..57aa5ca
--- /dev/null
+++ b/include/tst_module.h
@@ -0,0 +1,69 @@
+/*
+ * Copyright (c) 2013 Oracle and/or its affiliates. All Rights Reserved.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it would be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+ *
+ * Author:
+ * Alexey Kodanev <alexey.kodanev@oracle.com>
+ *
+ * These functions help to load and unload kernel modules in the tests.
+ *
+ * tst_module_load and tst_module_unload functions already include
+ * tst_module_exists function, which is checking the following possible
+ * module's locations:
+ *
+ * 1. Current working directory
+ *
+ * 2. LTP installation path (using env LTPROOT, which is usually /opt/ltp)
+ *
+ * 3. If tmp directory created, it'll look at the test start working directory
+ *
+ */
+
+#ifndef TST_MODULE
+#define TST_MODULE
+
+/*
+ * Check module existence.
+ *
+ * @mod_path: if it is not NULL, then tst_module_exists places the found
+ * module's path into the location pointed to by *mod_path. It must be freed
+ * with free() when it is no longer needed.
+ *
+ * In case of failure, test'll call cleanup_fn and exit with TCONF return value.
+ */
+void tst_module_exist(void (cleanup_fn)(void), const char *mod_name,
+ char **mod_path);
+
+/*
+ * Load a module using insmod program.
+ *
+ * @argv: an array of pointers to null-terminated strings that represent the
+ * additional parameters to the module. The array of pointers should be
+ * terminated by a NULL pointer. If argv points to NULL, it will be ignored.
+ *
+ * In case of insmod failure, test will call cleanup_fn and exit with TBROK
+ * return value.
+ */
+void tst_module_load(void (cleanup_fn)(void),
+ const char *mod_name, char *const argv[]);
+
+/*
+ * Unload a module using rmmod program. In case of failure, test will call
+ * cleanup_fn and exit with TBROK return value.
+ */
+void tst_module_unload(void (cleanup_fn)(void), const char *mod_name);
+
+#endif /* TST_MODULE */
diff --git a/lib/tst_module.c b/lib/tst_module.c
new file mode 100644
index 0000000..2538777
--- /dev/null
+++ b/lib/tst_module.c
@@ -0,0 +1,108 @@
+/*
+ * Copyright (c) 2013 Oracle and/or its affiliates. All Rights Reserved.
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License as
+ * published by the Free Software Foundation; either version 2 of
+ * the License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it would be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
+ *
+ * Author: Alexey Kodanev <alexey.kodanev@oracle.com>
+ *
+ */
+
+#define _GNU_SOURCE
+#include <stdio.h>
+#include <stdlib.h>
+#include <unistd.h>
+
+#include "test.h"
+#include "ltp_priv.h"
+#include "tst_module.h"
+
+void tst_module_exists(void (cleanup_fn)(void),
+ const char *mod_name, char **mod_path)
+{
+ /* check current working directory */
+ if (access(mod_name, F_OK) == 0) {
+ if (mod_path != NULL)
+ *mod_path = strdup(mod_name);
+ return;
+ }
+ char *buf = NULL;
+ int err = -1;
+ /* check LTP installation path */
+ const char *ltproot = getenv("LTPROOT");
+ if (ltproot != NULL) {
+ if (asprintf(&buf, "%s/testcases/bin/%s",
+ ltproot, mod_name) == -1) {
+ tst_brkm(TBROK | TERRNO, cleanup_fn,
+ "asprintf failed at %s:%d",
+ __FILE__, __LINE__);
+ }
+ err = access(buf, F_OK);
+ }
+ /* check start working directory */
+ if (err == -1 && tst_tmpdir_created()) {
+ free(buf);
+ if (asprintf(&buf, "%s/%s", tst_get_startwd(),
+ mod_name) == -1) {
+ tst_brkm(TBROK | TERRNO, cleanup_fn,
+ "asprintf failed at %s:%d",
+ __FILE__, __LINE__);
+ }
+ err = access(buf, F_OK);
+ }
+
+ if (err != 0) {
+ free(buf);
+ tst_brkm(TCONF, cleanup_fn, "Failed to find module '%s'",
+ mod_name);
+ }
+
+ if (mod_path != NULL)
+ *mod_path = buf;
+ else
+ free(buf);
+}
+
+void tst_module_load(void (cleanup_fn)(void),
+ const char *mod_name, char *const argv[])
+{
+ char *mod_path = NULL;
+ tst_module_exists(cleanup_fn, mod_name, &mod_path);
+
+ static int offset = 2; /* command name & module path */
+ int size = 0;
+ while (argv && argv[size])
+ ++size;
+ size += offset;
+ char *mod_argv[size + 1]; /* + NULL in the end */
+ mod_argv[size] = NULL;
+ mod_argv[0] = "insmod";
+ mod_argv[1] = mod_path;
+
+ int i;
+ for (i = offset; i < size; ++i)
+ mod_argv[i] = argv[i - offset];
+
+ tst_run_cmd(cleanup_fn, mod_argv);
+ free(mod_path);
+}
+
+void tst_module_unload(void (cleanup_fn)(void), const char *mod_name)
+{
+ char *mod_path = NULL;
+ tst_module_exists(cleanup_fn, mod_name, &mod_path);
+ char *const argv[] = { "rmmod", mod_path, NULL };
+ tst_run_cmd(cleanup_fn, argv);
+ free(mod_path);
+}
--
1.7.1
------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:
Build for Windows Store.
http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [LTP] [PATCH v3] lib: tst_module: add library functions for kernel modules
2013-06-26 16:36 [LTP] [PATCH v3] lib: tst_module: add library functions for kernel modules Alexey Kodanev
@ 2013-06-27 14:22 ` chrubis
[not found] ` <51CC4C53.4020602@oracle.com>
0 siblings, 1 reply; 3+ messages in thread
From: chrubis @ 2013-06-27 14:22 UTC (permalink / raw)
To: Alexey Kodanev; +Cc: vasily.isaenko, ltp-list
Hi!
> +/*
> + * Load a module using insmod program.
> + *
> + * @argv: an array of pointers to null-terminated strings that represent the
> + * additional parameters to the module. The array of pointers should be
> + * terminated by a NULL pointer. If argv points to NULL, it will be ignored.
Again should vs must.
> + * In case of insmod failure, test will call cleanup_fn and exit with TBROK
> + * return value.
> + */
> +void tst_module_load(void (cleanup_fn)(void),
> + const char *mod_name, char *const argv[]);
> +
> +/*
> + * Unload a module using rmmod program. In case of failure, test will call
> + * cleanup_fn and exit with TBROK return value.
> + */
> +void tst_module_unload(void (cleanup_fn)(void), const char *mod_name);
> +
> +#endif /* TST_MODULE */
> diff --git a/lib/tst_module.c b/lib/tst_module.c
> new file mode 100644
> index 0000000..2538777
> --- /dev/null
> +++ b/lib/tst_module.c
> @@ -0,0 +1,108 @@
> +/*
> + * Copyright (c) 2013 Oracle and/or its affiliates. All Rights Reserved.
> + *
> + * This program is free software; you can redistribute it and/or
> + * modify it under the terms of the GNU General Public License as
> + * published by the Free Software Foundation; either version 2 of
> + * the License, or (at your option) any later version.
> + *
> + * This program is distributed in the hope that it would be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
> + *
> + * You should have received a copy of the GNU General Public License
> + * along with this program; if not, write the Free Software Foundation,
> + * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
> + *
> + * Author: Alexey Kodanev <alexey.kodanev@oracle.com>
> + *
> + */
> +
> +#define _GNU_SOURCE
> +#include <stdio.h>
> +#include <stdlib.h>
> +#include <unistd.h>
> +
> +#include "test.h"
> +#include "ltp_priv.h"
> +#include "tst_module.h"
> +
> +void tst_module_exists(void (cleanup_fn)(void),
> + const char *mod_name, char **mod_path)
> +{
> + /* check current working directory */
> + if (access(mod_name, F_OK) == 0) {
> + if (mod_path != NULL)
> + *mod_path = strdup(mod_name);
> + return;
> + }
> + char *buf = NULL;
> + int err = -1;
> + /* check LTP installation path */
> + const char *ltproot = getenv("LTPROOT");
> + if (ltproot != NULL) {
> + if (asprintf(&buf, "%s/testcases/bin/%s",
> + ltproot, mod_name) == -1) {
> + tst_brkm(TBROK | TERRNO, cleanup_fn,
> + "asprintf failed at %s:%d",
> + __FILE__, __LINE__);
> + }
> + err = access(buf, F_OK);
> + }
> + /* check start working directory */
> + if (err == -1 && tst_tmpdir_created()) {
> + free(buf);
> + if (asprintf(&buf, "%s/%s", tst_get_startwd(),
> + mod_name) == -1) {
> + tst_brkm(TBROK | TERRNO, cleanup_fn,
> + "asprintf failed at %s:%d",
> + __FILE__, __LINE__);
> + }
> + err = access(buf, F_OK);
> + }
> +
> + if (err != 0) {
> + free(buf);
> + tst_brkm(TCONF, cleanup_fn, "Failed to find module '%s'",
> + mod_name);
> + }
> +
> + if (mod_path != NULL)
> + *mod_path = buf;
> + else
> + free(buf);
> +}
So you expect that the '.ko' suffix is part of the module name. It
should be written in the comment in the header. Because at least I
expect the module name to be the name that is returned by lsmod.
> +void tst_module_load(void (cleanup_fn)(void),
> + const char *mod_name, char *const argv[])
> +{
> + char *mod_path = NULL;
> + tst_module_exists(cleanup_fn, mod_name, &mod_path);
> +
> + static int offset = 2; /* command name & module path */
> + int size = 0;
> + while (argv && argv[size])
> + ++size;
> + size += offset;
> + char *mod_argv[size + 1]; /* + NULL in the end */
> + mod_argv[size] = NULL;
> + mod_argv[0] = "insmod";
> + mod_argv[1] = mod_path;
> +
> + int i;
> + for (i = offset; i < size; ++i)
> + mod_argv[i] = argv[i - offset];
> +
> + tst_run_cmd(cleanup_fn, mod_argv);
> + free(mod_path);
> +}
> +
> +void tst_module_unload(void (cleanup_fn)(void), const char *mod_name)
> +{
> + char *mod_path = NULL;
> + tst_module_exists(cleanup_fn, mod_name, &mod_path);
> + char *const argv[] = { "rmmod", mod_path, NULL };
Hmm, man rmmod suggest that it takes module name (not module path) as
parameter i.e. the string without the path and the '.ko' suffix. Does it
work with module path too?
> + tst_run_cmd(cleanup_fn, argv);
> + free(mod_path);
> +}
The rest is fine.
--
Cyril Hrubis
chrubis@suse.cz
------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:
Build for Windows Store.
http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [LTP] [PATCH v3] lib: tst_module: add library functions for kernel modules
[not found] ` <51CC4C53.4020602@oracle.com>
@ 2013-06-27 14:35 ` chrubis
0 siblings, 0 replies; 3+ messages in thread
From: chrubis @ 2013-06-27 14:35 UTC (permalink / raw)
To: alexey.kodanev; +Cc: vasily.isaenko, ltp-list
Hi!
> >> +
> >> + if (err != 0) {
> >> + free(buf);
> >> + tst_brkm(TCONF, cleanup_fn, "Failed to find module '%s'",
> >> + mod_name);
> >> + }
> >> +
> >> + if (mod_path != NULL)
> >> + *mod_path = buf;
> >> + else
> >> + free(buf);
> >> +}
> > So you expect that the '.ko' suffix is part of the module name. It
> > should be written in the comment in the header. Because at least I
> > expect the module name to be the name that is returned by lsmod.
> I will change module name to module's file name in the comments, is it OK?
Ok.
> >> +void tst_module_unload(void (cleanup_fn)(void), const char *mod_name)
> >> +{
> >> + char *mod_path = NULL;
> >> + tst_module_exists(cleanup_fn, mod_name,&mod_path);
> >> + char *const argv[] = { "rmmod", mod_path, NULL };
> > Hmm, man rmmod suggest that it takes module name (not module path) as
> > parameter i.e. the string without the path and the '.ko' suffix. Does it
> > work with module path too?
> >
> It works fine with path too, also without/with suffix .ko
Ok, then we can avoid the tst_module_exists() call and call rmmod with
just mod_name.
--
Cyril Hrubis
chrubis@suse.cz
------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:
Build for Windows Store.
http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-06-27 14:33 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-26 16:36 [LTP] [PATCH v3] lib: tst_module: add library functions for kernel modules Alexey Kodanev
2013-06-27 14:22 ` chrubis
[not found] ` <51CC4C53.4020602@oracle.com>
2013-06-27 14:35 ` chrubis
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.