* Re: [PATCH 3/3] cxgb4i_v4.3 : main driver files
[not found] ` <1275973167-8640-4-git-send-email-rakesh@chelsio.com>
@ 2010-06-16 22:19 ` Mike Christie
0 siblings, 0 replies; only message in thread
From: Mike Christie @ 2010-06-16 22:19 UTC (permalink / raw)
To: open-iscsi
Cc: Rakesh Ranjan, LK-NetDev, LK-SCSIDev, LKML, Karen Xie,
David Miller, James Bottomley, Anish Bhatt
On 06/07/2010 11:59 PM, Rakesh Ranjan wrote:
> From: Rakesh Ranjan<rakesh@chelsio.com>
>
>
> Signed-off-by: Rakesh Ranjan<rakesh@chelsio.com>
> ---
> drivers/scsi/cxgbi/cxgb4i.h | 175 +++++
> drivers/scsi/cxgbi/cxgb4i_ddp.c | 653 ++++++++++++++++
> drivers/scsi/cxgbi/cxgb4i_init.c | 317 ++++++++
> drivers/scsi/cxgbi/cxgb4i_offload.c | 1409 +++++++++++++++++++++++++++++++++++
> 4 files changed, 2554 insertions(+), 0 deletions(-)
> create mode 100644 drivers/scsi/cxgbi/cxgb4i.h
> create mode 100644 drivers/scsi/cxgbi/cxgb4i_ddp.c
> create mode 100644 drivers/scsi/cxgbi/cxgb4i_init.c
> create mode 100644 drivers/scsi/cxgbi/cxgb4i_offload.c
>
Do you think you can look through this patch and identify the places
where you basically just copied a cxgb3i function or copied it but
renamed some stuff and then move these type of things to the lib patch?
A lot of the scatter list stuff for example looks the same in each driver.
ddp_find_unused_entries - cxgb4i_ddp_find_unused_entries
ddp_unmark_entries - cxgb4i_ddp_unmark_entries
cxgb3i_ddp_find_page_index - cxgb4i_ddp_find_page_index
cxgb3i_ddp_adjust_page_table - cxgb4i_ddp_adjust_page_table
..... (lots of the ddp code looks duplicated).
The *_offload.c code looks the same.
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2010-06-16 22:16 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1275973167-8640-1-git-send-email-rakesh@chelsio.com>
[not found] ` <1275973167-8640-2-git-send-email-rakesh@chelsio.com>
[not found] ` <1275973167-8640-3-git-send-email-rakesh@chelsio.com>
[not found] ` <1275973167-8640-4-git-send-email-rakesh@chelsio.com>
2010-06-16 22:19 ` [PATCH 3/3] cxgb4i_v4.3 : main driver files Mike Christie
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®