diff options
author | Roland Dreier <roland@purestorage.com> | 2011-07-05 13:34:50 -0700 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2011-07-22 09:37:47 +0000 |
commit | f5fbed856931faa1e94a76caf0f0c273c67f0d32 (patch) | |
tree | 477499feebf9a32714ad25993f577a43eeb1e3d2 /drivers/target | |
parent | 2be18149d9d7ebfde7c3081d60f7c9d8d3b6f8c7 (diff) | |
download | linux-f5fbed856931faa1e94a76caf0f0c273c67f0d32.tar.bz2 |
target: Allow for built-in target modules
In target_fabric_configfs_init(), we should allow fabric_mod to be NULL,
since THIS_MODULE is NULL for built-in modules. The main method of
using the target code may be as modules, but having everything built-in
is useful eg to be able to do quick testing with "qemu -kernel".
In any case, we shouldn't bomb out fabric registration for a perfectly
valid configuration, so simply drop the check of fabric_mod.
Signed-off-by: Roland Dreier <roland@purestorage.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target')
-rw-r--r-- | drivers/target/target_core_configfs.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/drivers/target/target_core_configfs.c b/drivers/target/target_core_configfs.c index 63cba1e243ef..8d2aba51fc8b 100644 --- a/drivers/target/target_core_configfs.c +++ b/drivers/target/target_core_configfs.c @@ -306,10 +306,6 @@ struct target_fabric_configfs *target_fabric_configfs_init( { struct target_fabric_configfs *tf; - if (!(fabric_mod)) { - printk(KERN_ERR "Missing struct module *fabric_mod pointer\n"); - return ERR_PTR(-EINVAL); - } if (!(name)) { printk(KERN_ERR "Unable to locate passed fabric name\n"); return ERR_PTR(-EINVAL); |