Bug 167556 - StatSN value is not checked for the 3.x and 4.x drivers
StatSN value is not checked for the 3.x and 4.x drivers
Status: CLOSED WONTFIX
Product: Red Hat Enterprise Linux 4
Classification: Red Hat
Component: kernel (Show other bugs)
4.0
All Linux
medium Severity medium
: ---
: ---
Assigned To: Mike Christie
Brock Organ
: FutureFeature
Depends On:
Blocks: 170416
  Show dependency treegraph
 
Reported: 2005-09-05 08:05 EDT by Smitha Narayanaswamy
Modified: 2007-11-30 17:07 EST (History)
5 users (show)

See Also:
Fixed In Version:
Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2006-07-21 12:24:39 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---


Attachments (Terms of Use)
The 3.x patch fix for checking the StatSN value (12.71 KB, patch)
2005-09-05 08:05 EDT, Smitha Narayanaswamy
no flags Details | Diff
The 4.x patch fix for checking the StatSN value (8.42 KB, patch)
2005-09-05 08:06 EDT, Smitha Narayanaswamy
no flags Details | Diff
statsn checking addition patch from cisco (16.37 KB, patch)
2005-10-24 12:15 EDT, Mike Christie
no flags Details | Diff

  None (edit)
Description Smitha Narayanaswamy 2005-09-05 08:05:29 EDT
Description of problem:
The driver does not look into the StatSN values and report an error incase of a
mismatch

Version-Release number of selected component (if applicable):
3.x and 4.x

How reproducible:
Always


Expected results:
StatSN value needs to be checked by the driver

Additional info:
This fix is important for Cisco's customer.
Comment 1 Smitha Narayanaswamy 2005-09-05 08:05:30 EDT
Created attachment 118463 [details]
The 3.x patch fix for checking the StatSN value
Comment 2 Smitha Narayanaswamy 2005-09-05 08:06:19 EDT
Created attachment 118464 [details]
The 4.x patch fix for checking the StatSN value
Comment 9 Mike Christie 2005-09-26 17:20:22 EDT
The patch needs work. Both need to be made so we can configre the old behaivior
back in.

For the RHEL4 version do not do this

 	case ISCSI_OP_REJECT:
-	        handle_reject(session, hdr, data);
+	        rc = handle_reject(session, hdr, data);
 		break;

Just do

if handle_reject
 		goto fail

like the existing code.

Also use 0 for success and some -Exxx value for error.
Comment 10 Mike Christie 2005-09-26 17:27:51 EDT
could you also put it in a function.
Comment 11 Mike Christie 2005-09-26 17:54:24 EDT
Also why is the check for a difference of one?
Comment 14 Mike Christie 2005-10-24 12:10:41 EDT
Changing Component to kernel. In the future please try to select
iscsi-initiator-utils for userspace problems and kernel for driver problems. If
there is a kernel and userspace change required then make two :(. iSCSI is
legacy Component from when it was all bundled in one rpm. Thanks.
Comment 15 Mike Christie 2005-10-24 12:11:42 EDT
Could you also make seperate bugs for RHEL3 and RHEL4 bugs - thanks.
Comment 16 Mike Christie 2005-10-24 12:15:50 EDT
Created attachment 120309 [details]
statsn checking addition patch from cisco

Patch to add statsn checking from cisco
Comment 20 RHEL Product and Program Management 2006-07-21 12:24:40 EDT
Development Management has reviewed and declined this request.  You may appeal this decision by reopening this request.

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