Bug 430662
Summary: | backport hptiop driver updates from upstream kernel | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Red Hat Enterprise Linux 5 | Reporter: | HighPoint Linux Team <linux> | ||||||||||
Component: | kernel | Assignee: | Chip Coldwell <coldwell> | ||||||||||
Status: | CLOSED ERRATA | QA Contact: | Martin Jenner <mjenner> | ||||||||||
Severity: | medium | Docs Contact: | |||||||||||
Priority: | medium | ||||||||||||
Version: | 5.3 | CC: | coughlan | ||||||||||
Target Milestone: | rc | ||||||||||||
Target Release: | --- | ||||||||||||
Hardware: | All | ||||||||||||
OS: | Linux | ||||||||||||
Whiteboard: | |||||||||||||
Fixed In Version: | RHBA-2008-0314 | Doc Type: | Bug Fix | ||||||||||
Doc Text: | Story Points: | --- | |||||||||||
Clone Of: | Environment: | ||||||||||||
Last Closed: | 2008-05-21 15:08:15 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: | |||||||||||||
Attachments: |
|
Description
HighPoint Linux Team
2008-01-29 11:34:01 UTC
The code freeze for 5.2 has passed. Normally, this driver update would be queued for 5.3. The only exception may be for very low risk, critical, patches that we can take during 5.2. beta (e.g. add more PCI device IDs?). You will need to agree to do all the needed testing. If there are any such patches, plese post them here. If not, we will defer this to 5.3. For the longer term, we would also like to have a sample of the hardware here in our lab for testing and support. Created attachment 294750 [details]
add more pci device ids
add more PCI device IDs. these adapters belong to the same product family and
share the same driver interface.
Created attachment 294751 [details]
hptiop: avoid buffer overflow when returning sense data
avoid buffer overflow when returning sense data.
new adapter firmware may return sense data larger than 96 bytes, causing
overflow on scp->sense_buffer and a kernel crash.
This fix should be backported to earlier kernels.
Created attachment 294752 [details]
hptiop: fix resource releasing bug in hptiop_probe()
put pci_release_regions() after scsi_host_put() or scsi_host_put() will get
a null pointer when scsi_host_alloc() fails in hptiop_probe().
Created attachment 294753 [details]
hptiop: update scsi_cmd.resid
update scsi_cmnd.resid when finishing a request. otherwise the upper layer can
not detect buffer underrun condition.
sorry for late response. the above 4 patches are small and obvious updates to the existing code; please review them and check if they can be taken into 5.2. we are willing to run all tests required. meanwhile we'll prepare a sample for your testing. where should the hardware be shipped to? (In reply to comment #6) > sorry for late response. the above 4 patches are small and obvious updates to > the existing code; please review them and check if they can be taken into 5.2. I'm afraid the 5.2 beta deadline has passed; we'll queue these up for 5.3 > we are willing to run all tests required. meanwhile we'll prepare a sample for > your testing. where should the hardware be shipped to? Chip Coldwell Red Hat 10 Technology Park Drive Westford, MA 01886 Thanks, Chip I will request an exception, allowing us to get the four specific patches into 5.2 during beta. Justification: Two of these are system crashers, one may impact data integrity. The fourth just adds PCI IDs to a list, significantly improving customer satisfaction. The risk is low, and very well-contained to this driver. The vendor agrees to do the testing of RHEL 5.2 beta on this hardware. This request was evaluated by Red Hat Product Management for inclusion in a Red Hat Enterprise Linux maintenance release. Product Management has requested further review of this request by Red Hat Engineering, for potential inclusion in a Red Hat Enterprise Linux Update release for currently deployed products. This request is not yet committed for inclusion in an Update release. (In reply to comment #9) how about the review process? will these patches get into 5.2? (In reply to comment #10) > > how about the review process? will these patches get into 5.2? I've gone over the patches and everything looks good. I think they're all clearly things we want to have in the 5.2 kernel, and because the changes are small and have no impact outside the driver, I think it's very likely that they will go in. Thanks, Chip in kernel-$NEW_VER You can download this test kernel from http://people.redhat.com/dzickus/el5 in kernel-2.6.18-85.el5 You can download this test kernel from http://people.redhat.com/dzickus/el5 (In reply to comment #15) > in kernel-2.6.18-85.el5 > You can download this test kernel from http://people.redhat.com/dzickus/el5 Tested the new kernel on i686 and x86_64. The driver updates have been verified. 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 the 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/RHBA-2008-0314.html |