Bug 460195

Summary: Need SCSI transport and LLD netlink support.
Product: Red Hat Enterprise Linux 5 Reporter: Alex Iannicelli <Alex.Iannicelli>
Component: kernelAssignee: Tom Coughlan <coughlan>
Status: CLOSED ERRATA QA Contact: Martin Jenner <mjenner>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.3CC: andriusb, coughlan, cward, james.smart, jamie.wellnitz, laurie.barry, mchristi, riek
Target Milestone: betaKeywords: FutureFeature, OtherQA
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2009-01-20 20:09:37 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 431464    
Attachments:
Description Flags
Patch to give transport and LLDs access to SCSI netlink.
none
Reworked version of scsi_host_lookup() patch
none
Reworked version of scsi_netlink patch none

Description Alex Iannicelli 2008-08-26 18:19:23 UTC
Created attachment 315023 [details]
Patch to give transport and LLDs access to SCSI netlink. 

Description of problem:
The SCSI LLDs need to have access to the SCSI netlink socket. The LLDs need this support to be able to send and receive messages to applications running in user space for management purposes. The specific use case is for the lpfc driver to add DH-CHAP authentication support. The lpfc driver uses a user mode application to perform the authentication calculations.

Access the proposed patch to the fc_transport here: http://marc.info/?l=linux-scsi&m=121831854713677&w=2

Emulex will update https://bugzilla.redhat.com/show_bug.cgi?id=441746 with changes to use this patch.

Comment 2 James Smart 2008-08-27 15:27:16 UTC
The scsi netlink patch set (not fc_transport) that was referenced really needs the associated bugfix for scsi_host_lookup.  See
http://marc.info/?l=linux-scsi&m=121822338501336&w=2

This has been pulled into Bottomley's scsi-misc-2.6 tree.

Comment 4 Mike Christie 2008-08-27 16:39:34 UTC
James,

Do you have any idea on when James might merge the netlink code into scsi-misc?

Comment 5 James Smart 2008-08-27 16:46:53 UTC
Nope. Given Q, the original requester/poster sees it as workable, I don't think there's any issue. I'll ping him.

Comment 6 Tom Coughlan 2008-08-28 22:04:08 UTC
Neither of these patches applies properly to RHEL 5.2. Please port them to 5.2 and post them here. Thanks.

Comment 9 James Smart 2008-08-29 19:02:42 UTC
I've recut the patches for the 5.2 kernel. We're busy testing them now, but as it's late in the day, we may not have results until Tuesday.  I can post the un-tested patches if desired.

Comment 10 Tom Coughlan 2008-08-29 19:13:04 UTC
Yes, please post them.

Comment 11 James Smart 2008-08-29 19:23:43 UTC
Created attachment 315394 [details]
Reworked version of scsi_host_lookup() patch

Comment 12 James Smart 2008-08-29 19:24:26 UTC
Created attachment 315395 [details]
Reworked version of scsi_netlink patch

Comment 13 Andrius Benokraitis 2008-09-02 17:27:05 UTC
Any status on testing of the patches, Emulex?

Comment 14 Alex Iannicelli 2008-09-02 17:47:51 UTC
Testing has shown that the patch is working as expected. I was able to get the lpfc driver loaded and communicating across the SCSI netlink socket without any changes to the submitted patch.

Comment 15 James Smart 2008-09-04 18:20:52 UTC
Both of the patches, corresponding to comments #11 and #12, have been pulled by James Bottomley into the scsi-misc-2.6 git tree.

Comment 17 Don Zickus 2008-09-11 19:44:11 UTC
in kernel-2.6.18-111.el5
You can download this test kernel from http://people.redhat.com/dzickus/el5

Comment 20 Chris Ward 2008-11-18 18:13:51 UTC
~~ Snapshot 3 is now available ~~ 

Snapshot 3 is now available for Partner Testing, which should contain a fix that resolves this bug. ISO's available as usual at ftp://partners.redhat.com. Your testing feedback is vital! Please let us know if you encounter any NEW issues (file a new bug) or if you have VERIFIED the fix is present and functioning as expected (add PartnerVerified Keyword).

Ping your Partner Manager with any additional questions. Thanks!

Comment 21 Chris Ward 2008-11-28 06:45:24 UTC
~~ Attention ~~ Snapshot 4 is now available for testing @ partners.redhat.com ~~

Partners, it is vital that we get your testing feedback on this important bug fix / feature request. If you are unable to test, please clearly indicate this in a comment to this bug or directly with your partner manager. If we do not receive your test feedback, this bug is at risk from being dropped from the release.

If you have VERIFIED the fix, please add PartnerVerified to the Bugzilla Keywords field, along with a description of the test results. 

If you encounter a new bug, CLONE this bug and request from your Partner manager to review. We are no longer excepting new bugs into the release, bar critical regressions.

Comment 22 Chris Ward 2008-12-04 10:18:18 UTC
Emulex, what is the current status of this bug fix? The fix should be present in the latest RHEL5.3 Snapshot. Please test and send feedback ASAP.

Comment 23 Alex Iannicelli 2008-12-04 14:09:50 UTC
Yes, it looks good.

Comment 25 errata-xmlrpc 2009-01-20 20:09:37 UTC
An advisory has been issued which should help the problem
described in this bug report. This report is therefore being
closed with a resolution of ERRATA. For more information
on therefore solution and/or where to find the updated files,
please follow the link below. You may reopen this bug report
if the solution does not work for you.

http://rhn.redhat.com/errata/RHSA-2009-0225.html