Re: 2.6: unused code under drivers/message/fusion/

[Vladislav Bolkhovitin <vst@vlnb.net>]

Moore, Eric Dean wrote:
> We need to hold off on this change. Yes, there are 
> customers of LSI Logic using mptstm.c, as
> part of the target-mode drivers.  
> 
> The proposed generic target mode drivers proposal is yet part
> of the kernel.  
> http://scst.sourceforge.net/
> We are looking into supporting this once its available.

Well, SCST is already available, stable and useful. People use it 
without considerable problems, except with inconvenient LUNs management, 
which we are going to fix in the next version. I don't expect it will be 
considering for the kernel inclusion at least until 2.7. So, you can 
start supporting it right now :-).

Vlad

> Eric Moore
> LSI Logic
> 
> 
> On Sunday, November 07, 2004 1:18 PM, Adrian Bunk wrote:
> 
>>I know this patch is wrong, but I'd like to note that it actually 
>>changes nothing since all code removed or made static 
>>currently has no 
>>in-kernel users (there seems to be a mptstm.c missing?).
>>
>>Please comment on how to correctly handle this.
>>
>>
>>diffstat output:
>> drivers/message/fusion/mptbase.c  |    7 ++-----
>> drivers/message/fusion/mptbase.h  |    2 --
>> drivers/message/fusion/mptscsih.c |    2 +-
>> 3 files changed, 3 insertions(+), 8 deletions(-)
>>
>>
>>Signed-off-by: Adrian Bunk <bunk@stusta.de>
>>
>>--- 
>>linux-2.6.10-rc1-mm3-full/drivers/message/fusion/mptscsi
>>h.c.old	2004-11-07 20:56:13.000000000 +0100
>>+++ 
>>linux-2.6.10-rc1-mm3-full/drivers/message/fusion/mptscsih.c	
>>2004-11-07 20:56:48.000000000 +0100
>>@@ -98,7 +98,7 @@
>> 
>> /* Set string for command line args from insmod */
>> #ifdef MODULE
>>-char *mptscsih = NULL;
>>+static char *mptscsih = NULL;
>> module_param(mptscsih, charp, 0);
>> #endif
>> 
>>--- 
>>linux-2.6.10-rc1-mm3-full/drivers/message/fusion/mptbas
>>e.h.old	2004-11-07 20:57:21.000000000 +0100
>>+++ 
>>linux-2.6.10-rc1-mm3-full/drivers/message/fusion/mptbase.h	
>>2004-11-07 20:57:37.000000000 +0100
>>@@ -984,10 +984,8 @@
>>  *  Public data decl's...
>>  */
>> extern struct list_head	  ioc_list;
>>-extern struct proc_dir_entry	*mpt_proc_root_dir;
>> 
>> extern int		  mpt_lan_index;	/* needed by mptlan.c */
>>-extern int		  mpt_stm_index;	/* needed by mptstm.c */
>> 
>> 
>>/*=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
>>=-=-=-=-=-=-=-=*/
>> #endif		/* } __KERNEL__ */
>>--- 
>>linux-2.6.10-rc1-mm3-full/drivers/message/fusion/mptbas
>>e.c.old	2004-11-07 20:57:46.000000000 +0100
>>+++ 
>>linux-2.6.10-rc1-mm3-full/drivers/message/fusion/mptbase.c	
>>2004-11-07 20:58:54.000000000 +0100
>>@@ -133,9 +133,8 @@
>>  *  Public data...
>>  */
>> int mpt_lan_index = -1;
>>-int mpt_stm_index = -1;
>> 
>>-struct proc_dir_entry *mpt_proc_root_dir;
>>+static struct proc_dir_entry *mpt_proc_root_dir;
>> 
>> #define WHOINIT_UNKNOWN		0xAA
>> 
>>@@ -355,7 +354,7 @@
>> 			dmfprintk((MYIOC_s_INFO_FMT "Got TURBO 
>>reply req_idx=%08x\n", ioc->name, pa));
>> 			type = (pa >> MPI_CONTEXT_REPLY_TYPE_SHIFT);
>> 			if (type == 
>>MPI_CONTEXT_REPLY_TYPE_SCSI_TARGET) {
>>-				cb_idx = mpt_stm_index;
>>+				cb_idx = -1;
>> 				mf = NULL;
>> 				mr = (MPT_FRAME_HDR *) 
>>CAST_U32_TO_PTR(pa);
>> 			} else if (type == MPI_CONTEXT_REPLY_TYPE_LAN) {
>>@@ -5868,7 +5867,6 @@
>> 
>> 
>>/*=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
>>=-=-=-=-=-=-=-=*/
>> EXPORT_SYMBOL(ioc_list);
>>-EXPORT_SYMBOL(mpt_proc_root_dir);
>> EXPORT_SYMBOL(mpt_register);
>> EXPORT_SYMBOL(mpt_deregister);
>> EXPORT_SYMBOL(mpt_event_register);
>>@@ -5886,7 +5884,6 @@
>> EXPORT_SYMBOL(mpt_GetIocState);
>> EXPORT_SYMBOL(mpt_print_ioc_summary);
>> EXPORT_SYMBOL(mpt_lan_index);
>>-EXPORT_SYMBOL(mpt_stm_index);
>> EXPORT_SYMBOL(mpt_HardResetHandler);
>> EXPORT_SYMBOL(mpt_config);
>> EXPORT_SYMBOL(mpt_toolbox);
>>
>>
>>
> 
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 
>