All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michael Tokarev <mjt@tls.msk.ru>
To: Ravi Anand <ravi.anand@qlogic.com>
Cc: Linux-SCSI Mailing List <linux-scsi@vger.kernel.org>,
	open-iscsi <open-iscsi@googlegroups.com>
Subject: Re: [RFC] [PATCH 0/7] requested for qla4xxx!!!
Date: Sat, 01 Jul 2006 23:27:56 +0400	[thread overview]
Message-ID: <44A6CCBC.7000603@tls.msk.ru> (raw)
In-Reply-To: <20060629064812.GA7220@ranandlinuxbox.oc.cox.net>

[Sorry for repost: was typo in linux-scsi@vger
 email, and open-iscsi@ didn't accept email
 either, as it says "i'm not subscribed" (which
 isn't _entirely_ true ;)]

Ravi Anand wrote:
> All,
> 
> Today in the following email thread , Mike Christie outlined the 
> modifications that has been done to qla4xx driver from the last submission: 
> 
>> http://marc.theaimsgroup.com/?l=linux-scsi&m=115151443814051&w=2
[...]
> All the patches has been uploaded at the following URL:
>
> 	ftp://ftp.qlogic.com/outgoing/linux/iSCSI/upstream/5.00.05b6-k/
>
> Complete driver package and the diff from previous submission for review
> has also been uploaded at the above mentioned link.
[...]

Hi.

I'm not of much help in reviewing the code and/or making code
suggestions.  Instead, I'm trying to build the driver as we've
several new servers with ISP4010 adapters, which should work
somehow... ;)

Except of the obvious prob wrt ql4_netlink.h file which you
already fixed by subsequent patch.. well....

....Ok.  I'm new in this area (just joined open-iscsi list).
And my first question is: which [i]scsi implementation you
were used as a base for this driver?

I've tried against curren open-iscsi svn repo, against
scsi-misc kernel git tree, and against open-iscsi-1.0-485
tarball.  Whch all are mostly similar (in API anyway).
ql4 driver does not compile - even after "fixing" some
"missing" #includes (like iscsi_if.h defining most of
the ISCSI_* constants used in ql4_os.c), there are alot
of functions missing (iscsi_{add,remove,free}_session()
etc), undeclared structures used (*dscvr*) etc.

What was used as a base for the work?

And also -- how ql3 driver is related to this ql4 one?
Does it (ql4) require ql3 to be compiled/loaded as well?
(The question is because, as it seems, the card represents
as two PCI (sub)devices - one is ethernet controller and
another is "network controller", like this:

04:04.0 Ethernet controller: QLogic Corp. QLA3010 Network Adapter (rev 05)
04:04.1 Network controller: QLogic Corp. QLA4010 iSCSI TOE Adapter (rev 05)
)

Thanks.

/mjt

  reply	other threads:[~2006-07-01 19:28 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-06-29  6:48 [RFC] [PATCH 0/7] requested for qla4xxx!!! Ravi Anand
2006-07-01 19:27 ` Michael Tokarev [this message]
2006-07-05 17:30   ` Mike Christie
2006-07-05 19:06     ` Michael Tokarev
2006-07-06  5:10     ` Or Gerlitz
2006-07-06 15:15       ` Mike Christie

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=44A6CCBC.7000603@tls.msk.ru \
    --to=mjt@tls.msk.ru \
    --cc=linux-scsi@vger.kernel.org \
    --cc=open-iscsi@googlegroups.com \
    --cc=ravi.anand@qlogic.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.