Bug 112826 - LTC5732 - MMIO alignment error when inserting the olympic TR module.
Summary: LTC5732 - MMIO alignment error when inserting the olympic TR module.
Alias: None
Product: Red Hat Enterprise Linux 2.1
Classification: Red Hat
Component: kernel   
(Show other bugs)
Version: 2.1
Hardware: powerpc
OS: Linux
Target Milestone: ---
Assignee: Julie DeWandel
QA Contact: Brian Brock
Depends On:
TreeView+ depends on / blocked
Reported: 2004-01-02 23:08 UTC by Mike Ranweiler
Modified: 2007-11-30 22:06 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2004-05-12 01:08:10 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
patch from bug 1011 (725 bytes, patch)
2004-01-02 23:10 UTC, Mike Ranweiler
no flags Details | Diff

External Trackers
Tracker ID Priority Status Summary Last Updated
Red Hat Product Errata RHSA-2004:188 normal SHIPPED_LIVE Important: Updated kernel packages available for Red Hat Enterprise Linux 3 Update 2 2004-05-11 04:00:00 UTC

Description Mike Ranweiler 2004-01-02 23:08:48 UTC
Hardware Environment: iSeries 64bit Regatta

Software Environment: RHEL 3 QU1 Beta 1

Steps to Reproduce:
1. Boot a partition with native TR.
2. Remove (rmmod) the olympic module by hand
3. Reinsert (insmod) the olympic module by hand.
4. May require one or two repetitions.

Actual Results:
A kernel panic is caused:  

Expected Results:
The kernel panic should not happen.

Additional Information:
This problem was previously solved.  It is documented in bug 1011.  I
am not
sure why the fix has been lost from the kernel.  Here is an
explanation of the
change that was proposed:
-------- Begin snip from bug 1011
I made a change to the ./drivers/net/tokenring/olympic.c code to
handle the 
case where the skb does not happen to be aligned on a word boundary. 
generally happens on a error path where the initial call to
olympic_open failed 
and it is doing a second try. 

The change was to replace the 3 lines of code that clear out the srb
field with "memset_io(init_srb,0,SRB_COMMAND_SIZE); "

This change is checked into the 2.4.19 cvs tree. 
-------- End snip

Comment 1 Mike Ranweiler 2004-01-02 23:10:23 UTC
Created attachment 96748 [details]
patch from bug 1011

------- Additional Comment #1 From Michael Ranweiler  2004-01-02 17:55 -------

patch from bug 1011

This is Al's patch from the 2.4.19 kernel - can see if it fixes...

Comment 2 IBM Bug Proxy 2004-01-05 22:36:54 UTC
------ Additional Comments From jsbryant@us.ibm.com  2004-01-05 17:26 -------
Mike, I have tested this patch in a kernel and verified that the kernel no
longer panics on Regatta. 

Comment 3 IBM Bug Proxy 2004-01-05 23:02:35 UTC
------ Additional Comments From mranweil@us.ibm.com  2004-01-05 17:50 -------
Julie/Mark - I don't know what you thought of that patch (was grabbed out of an
old bug we had) but it makes olympic on power4 not panic, which is probably
useful.  ;)  Any chance on the refresh of QU1? 

Comment 4 Julie DeWandel 2004-01-06 12:28:39 UTC
Mike, it is unlikely that this will be taken in the U1 refresh at this
late date since it doesn't fix a data corruptor or security problem.
How big an issue is this? Do you have many customers with native token
ring doing this type of operation (rmmod and then insmod)? We would
prefer to wait until Update 2 for this fix.

Comment 5 IBM Bug Proxy 2004-01-06 16:09:22 UTC
------ Additional Comments From mranweil@us.ibm.com  2004-01-06 10:53 -------
Well, it's not completely critical - it only happens after the rmmod, and on
power 4.  But it'd be nice... (it is a panic). 

Comment 6 IBM Bug Proxy 2004-01-09 21:16:46 UTC
------ Additional Comments From mranweil@us.ibm.com  2004-01-09 16:05 -------
Setting target to qu2. 

Comment 7 IBM Bug Proxy 2004-01-13 16:01:58 UTC
----- Additional Comments From mranweil@us.ibm.com  2004-01-13 10:53 -------
Added Jay Vosburgh to the cc - hopefully he'll have comments on the patch... 
Need to get something for this problem into mainline. 

Comment 8 IBM Bug Proxy 2004-01-19 21:43:03 UTC
----- Additional Comments From mranweil@us.ibm.com  2004-01-19 16:31 -------
Jay - you have a chance to look at the patch here? 

Comment 9 IBM Bug Proxy 2004-01-26 04:02:42 UTC
----- Additional Comments From khoa@us.ibm.com  2004-01-25 19:19 -------
Patch available, so move this bug into FixedAwaitingTest state.
Jay - please review the patch above.  Thanks. 

Comment 10 IBM Bug Proxy 2004-01-26 04:03:18 UTC
----- Additional Comments From khoa@us.ibm.com  2004-01-25 19:20 -------
This is targeted for RHEL3 QU2. 

Comment 11 IBM Bug Proxy 2004-01-26 04:03:58 UTC
----- Additional Comments From khoa@us.ibm.com  2004-01-25 19:21 -------
Jay (Bryant) already tested this patch and found it not panic anymore, so
set the state to Tested. 

Comment 12 Julie DeWandel 2004-02-05 12:14:36 UTC
Fix has been submitted to our tree and will appear in Update 2.

Comment 13 IBM Bug Proxy 2004-02-09 20:38:41 UTC
----- Additional Comments From mranweil@us.ibm.com  2004-02-09 15:40 -------
Jay V checked for this - it's in 2.6.2 from Jeff Garzik - thanks for pushing.

Closing bug. 

Comment 14 John Flanagan 2004-05-12 01:08:10 UTC
An errata 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.


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