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 | |
| parent | tcm_fc: Convert to wake_up_process and schedule_timeout_interruptible (diff) | |
| download | linux-dev-f5fbed856931faa1e94a76caf0f0c273c67f0d32.tar.xz linux-dev-f5fbed856931faa1e94a76caf0f0c273c67f0d32.zip | |
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')
| -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); |