linux-kernel - Re: [SCSI] tcm_fc: Adding FC_FC4 provider (tcm_fc) for FCoE target (TCM

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <4DEFE0C9.10206@intel.com>
Date:	Wed, 08 Jun 2011 13:51:21 -0700
From:	Kiran Patil <kiran.patil@...el.com>
To:	Geert Uytterhoeven <geert@...ux-m68k.org>
CC:	Joe Eykholt <jeykholt@...co.com>,
	Nicholas Bellinger <nab@...ux-iscsi.org>,
	"James E.J. Bottomley" <JBottomley@...allels.com>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	scsi <linux-scsi@...r.kernel.org>
Subject: Re: [SCSI] tcm_fc: Adding FC_FC4 provider (tcm_fc) for FCoE target
 (TCM - target core) support

Hi Geert,

Response in-line.

On 6/8/2011 1:10 PM, Geert Uytterhoeven wrote:
>>     [SCSI] tcm_fc: Adding FC_FC4 provider (tcm_fc) for FCoE target (TCM - target core) support
>>
>>     This is a comprehensive patch for FC-FC4 provider. tcm_fc is a FC-FC4
>>     provider which glues target core (TCM) with Fiber channel library
>>     (libfc). tcm_fc uses existing FC4 provider hooks from Fiber channel
>>     library. This Fiber channel library is used by FCoE (transport - FC
>>     over Ethernet) protocol driver as well.
>> --- /dev/null
>> +++ b/drivers/target/tcm_fc/tfc_conf.c
>> +static int __init ft_init(void)
>> +{
>> +       if (ft_register_configfs())
>> +               return -1;
>> +       if (fc_fc4_register_provider(FC_TYPE_FCP,&ft_prov)) {
>> +               ft_deregister_configfs();
>> +               return -1;
>> +       }
>> +       blocking_notifier_chain_register(&fc_lport_notifier_head,&ft_notifier);
>> +       fc_lport_iterate(ft_lport_add, NULL);
>> +       return 0;
>> +}
>> +
>> +static void __exit ft_exit(void)
>> +{
>> +       blocking_notifier_chain_unregister(&fc_lport_notifier_head,
>> +&ft_notifier);
>> +       fc_fc4_deregister_provider(FC_TYPE_FCP,&ft_prov);
>> +       fc_lport_iterate(ft_lport_del, NULL);
>> +       ft_deregister_configfs();
>> +       synchronize_rcu();
>> +}
>> +
>> +#ifdef MODULE
>> +MODULE_DESCRIPTION("FC TCM fabric driver " FT_VERSION);
>> +MODULE_LICENSE("GPL");
>> +module_init(ft_init);
>> +module_exit(ft_exit);
>> +#endif /* MODULE */
> If CONFIG_TCM_FC=y, this gives:
>
> | drivers/target/tcm_fc/tfc_conf.c:642: warning: ‘ft_init’ defined but not used
>
> Shouldn't the #ifdef MODULE and #endif just be removed?
Agree. Will get it fixed.
> Who else initializes this module if it's builtin?
> E.g. ft_register_configfs() is global, but not used outside this source file?
>
> Gr{oetje,eeting}s,
>
>                          Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
>                                  -- Linus Torvalds
Thanks
-- KIran P.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/