Re: [LTP] [PATCH v4] lib: tst_module: add library functions for kernel modules

Mike Frysinger
2013-06-28T05:35:51Z

[-- Attachment #1.1: Type: Text/Plain, Size: 352 bytes --]

On Thursday 27 June 2013 11:09:22 Alexey Kodanev wrote:
\> +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 */

why static ? i would use const instead.
-mike

[-- Attachment #1.2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 184 bytes --]

------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev

[-- Attachment #3: Type: text/plain, Size: 155 bytes --]