Bug 187831 - [mfi] Logic issue in mfi_tbolt.c
Summary: [mfi] Logic issue in mfi_tbolt.c
Status: Open
Alias: None
Product: Base System
Classification: Unclassified
Component: kern (show other bugs)
Version: Unspecified
Hardware: Any Any
: Normal Affects Only Me
Assignee: freebsd-bugs (Nobody)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-03-21 21:10 UTC by swildner
Modified: 2018-07-05 20:11 UTC (History)
3 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description swildner 2014-03-21 21:10:00 UTC 
In sys/dev/mfi/mfi_tbolt.c, line #1111, the "if (cdb[0] != 0x28 || cdb[0] != 0x2A)" will be always true.
Comment 1 David Binderman 2016-02-08 09:08:20 UTC 
Still broken, nearly two years later.
Comment 2 Ed Maste freebsd_committer freebsd_triage 2016-02-11 19:46:09 UTC 
Introduced in r227068.

On a quick inspection it's probably supposed to be
if (cdb[0] != 0x28 && cdb[0] != 0x2A)
but I'm not completely familiar with this driver and don't have hardware to test -- adding abrisko@ to CC as he may be able to take care of this.
Comment 3 Doug Ambrisko freebsd_committer freebsd_triage 2016-03-21 20:59:42 UTC 
This is more complicated and apparently something that I wasn't sure of from
the LSI code.  It could help fix some other syspd type bugs with newer
controllers.  Adding code to deal with this didn't just work in my first
attempt so I'm going to have to debug this more.  My guess is that this
needs to handle the various size SCSI I/O as well and not just READ10/WRITE10.
Comment 4 Eitan Adler freebsd_committer freebsd_triage 2018-05-20 23:53:45 UTC 
For bugs matching the following conditions:
- Status == In Progress
- Assignee == "bugs@FreeBSD.org"
- Last Modified Year <= 2017

Do
- Set Status to "Open"
Comment 5 David Binderman 2018-07-05 20:11:36 UTC 
Still broken a couple of years later.