Bug 474451 - [Intel 5.4 FEAT] Request to Support Intel Modular Server in scsi_dh
Summary: [Intel 5.4 FEAT] Request to Support Intel Modular Server in scsi_dh
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: device-mapper-multipath
Version: 5.4
Hardware: other
OS: Linux
high
high
Target Milestone: beta
: 5.4
Assignee: Ben Marzinski
QA Contact: Cluster QE
URL:
Whiteboard:
Depends On:
Blocks: 452016 483784 502000
TreeView+ depends on / blocked
 
Reported: 2008-12-03 21:37 UTC by joseph.r.gruher
Modified: 2018-10-19 19:19 UTC (History)
27 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 488325 (view as bug list)
Environment:
Last Closed: 2009-09-02 11:46:58 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
patch-multipath-tools (25.96 KB, patch)
2008-12-03 21:38 UTC, joseph.r.gruher
no flags Details | Diff
scsi_dh-patch (1.21 KB, patch)
2008-12-03 21:38 UTC, joseph.r.gruher
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
Red Hat Product Errata RHEA-2009:1377 0 normal SHIPPED_LIVE device-mapper-multipath bug-fix and enhancement update 2009-09-01 12:41:23 UTC

Description joseph.r.gruher 2008-12-03 21:37:23 UTC
We would like to add a patch to RHEL5.3 to support the dual storage controllers in the Intel Modular Server being used with the new scsi_dh framework.  This will allow users to setup multipath on the system without loading patches and having to rebuild scsi_dh.

The patch has two parts:

1) Change in the scsi-dh framework to accept Intel devices.  This is a single line code change to add Intel the Multiflex device name to the supported devices list.  This code is in attachment "scsi_dh-patch".

2) User-space tool to give priority of a path depending on the preferred bit of the controller.  This tool is based on mpath_prio_alua.  For correct failover/failback operations we need the correct priority of the path, this is done by this second tool.  This code is in attachment "patch-multipath-tools".

Comment 1 joseph.r.gruher 2008-12-03 21:38:10 UTC
Created attachment 325601 [details]
patch-multipath-tools

Comment 2 joseph.r.gruher 2008-12-03 21:38:37 UTC
Created attachment 325602 [details]
scsi_dh-patch

Comment 3 John Villalovos 2009-01-19 19:45:29 UTC
Joseph,

It is to late for RHEL 5.3.  But we need code submitted by 29-Jan-2009 for acceptance into RHEL 5.4.

Are the attached patches the latest versions?

Comment 4 joseph.r.gruher 2009-01-20 20:01:02 UTC
John,
Yes, attached patches are still the latest and greatest versions, no known issues at this time.

Thanks,
Joe

Comment 5 Mike Christie 2009-01-30 19:42:40 UTC
Ben, I am going to take the kernel patch in comment #2 and push upstream and then put it in RHEL. I have a bugzilla for the kernel stuff already.

You can use this bz for all the multipath tools fun :)

Comment 6 Mike Christie 2009-01-30 19:45:13 UTC
(In reply to comment #1)
> Created an attachment (id=325601) [details]
> patch-multipath-tools

Promise/Intel developers,

What is up with this patch? It has IBM copyrights in the header, but then prints out a intel copyright in the print version.

Can the default path_priority/pp_alua callout be used instead of adding your own or can the default one be used?

Comment 7 Mike Christie 2009-01-30 19:45:59 UTC
(In reply to comment #6)
> Can the default path_priority/pp_alua callout be used instead of adding your
> own or can the default one be used?

I meant to write modified to support whatever extra you need?

Comment 8 joseph.r.gruher 2009-01-31 22:35:37 UTC
With regard to the question about copyright Promise tells us this is a minor change from the original pp_alua code so we must sustain copyright information in compliance with GPL requirements.

With regard to the question about modifying pp_alua this is the response from Promise:
We already tested with calling pp_alua, but got a problem which did not get exact priority from device.  So it failed to get exact path.

Comment 9 Mike Christie 2009-02-02 01:25:46 UTC
(In reply to comment #8)
> 
> With regard to the question about modifying pp_alua this is the response from
> Promise:
> We already tested with calling pp_alua, but got a problem which did not get
> exact priority from device.  So it failed to get exact path.


Ok. I am adding some more info since Ben does multipath tools, but in the mail Ilgu sent there was some more detail which would help him know what is going on.

Ilgu from Promise also said:

       We will check and try to find minimize patch size for pp_intel.



Intel/Promise guys if you send mail about multipath tools could you make sure to  cc Benjamin Marzinski <bmarzins>. I do not do any work with multipath tools. I have no idea what is going on :) He will be able to help you guys with those type of issues.

Comment 10 joseph.r.gruher 2009-02-17 22:08:17 UTC
Note: This BZ corresponds to IT 267306.

Comment 12 Keve Gabbert 2009-05-04 23:24:07 UTC
what's the status on this BZ?
Is it ever going to get moved from NEW to ASSIGNED?

Comment 13 Ronald Pacheco 2009-05-05 12:50:09 UTC
This is on the list of BZs approved for RHEL 5.4.

Comment 14 Ben Marzinski 2009-05-15 23:13:25 UTC
Patch applied.

Comment 16 Chris Ward 2009-07-03 18:14:58 UTC
~~ Attention - RHEL 5.4 Beta Released! ~~

RHEL 5.4 Beta has been released! There should be a fix present in the Beta release that addresses this particular request. Please test and report back results here, at your earliest convenience. RHEL 5.4 General Availability release is just around the corner!

If you encounter any issues while testing Beta, please describe the issues you have encountered and set the bug into NEED_INFO. If you encounter new issues, please clone this bug to open a new issue and request it be reviewed for inclusion in RHEL 5.4 or a later update, if it is not of urgent severity.

Please do not flip the bug status to VERIFIED. Only post your verification results, and if available, update Verified field with the appropriate value.

Questions can be posted to this bug or your customer or partner representative.

Comment 17 Chris Ward 2009-07-10 19:07:03 UTC
~~ Attention Partners - RHEL 5.4 Snapshot 1 Released! ~~

RHEL 5.4 Snapshot 1 has been released on partners.redhat.com. If you have already reported your test results, you can safely ignore this request. Otherwise, please notice that there should be a fix available now that addresses this particular request. Please test and report back your results here, at your earliest convenience. The RHEL 5.4 exception freeze is quickly approaching.

If you encounter any issues while testing Beta, please describe the issues you have encountered and set the bug into NEED_INFO. If you encounter new issues, please clone this bug to open a new issue and request it be reviewed for inclusion in RHEL 5.4 or a later update, if it is not of urgent severity.

Do not flip the bug status to VERIFIED. Instead, please set your Partner ID in the Verified field above if you have successfully verified the resolution of this issue. 

Further questions can be directed to your Red Hat Partner Manager or other appropriate customer representative.

Comment 18 Chris Ward 2009-07-21 14:36:37 UTC
Keve - Intel, what's the testing status of this feature? It should be included in the latest snapshots.

Comment 19 Chris Ward 2009-07-28 13:11:14 UTC
Intel, Ping?

Comment 20 joseph.r.gruher 2009-07-28 21:03:32 UTC
Testing is in progress.

Comment 22 Chris Ward 2009-08-04 08:26:24 UTC
Thanks Intel. Please update us with test results or test status as soon as possible.

Comment 23 ilgu hong 2009-08-04 21:18:29 UTC
Hi,all

We tested RHEL 5.4 alpa, beta, and snapshot 1 and we found that this version support our device in multipath tools and scsi_dh_alua.


we did FO/FB test and we did not find "scsi id issue" which we found in RHEL 5.3 version.

So RHEL 5.4 work fine with clearbay.

Thanks.

Comment 24 Chris Ward 2009-08-11 07:01:59 UTC
Intel, any update on testing?

Comment 25 joseph.r.gruher 2009-08-11 16:33:58 UTC
Chris, comment #23 reports the results of our testing, we consider this verified and closed from our perspective.  Thanks!

Comment 26 errata-xmlrpc 2009-09-02 11:46:58 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/RHEA-2009-1377.html

Comment 29 Keve Gabbert 2014-06-23 14:50:59 UTC
closed


Note You need to log in before you can comment on or make changes to this bug.