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

From: Moore, Eric Dean
Date: Mon Nov 08 2004 - 18:22:24 EST


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.

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@xxxxxxxxx>
>
> ---
> 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/