Page MenuHomeFreeBSD

umass: Remove some 20-year-old dead code
ClosedPublic

Authored by imp on Mar 10 2025, 7:19 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 3, 11:57 AM
Unknown Object (File)
Mar 30 2025, 2:22 PM
Unknown Object (File)
Mar 24 2025, 5:20 AM
Unknown Object (File)
Mar 22 2025, 1:48 AM
Unknown Object (File)
Mar 18 2025, 4:38 AM
Unknown Object (File)
Mar 17 2025, 9:21 AM
Unknown Object (File)
Mar 16 2025, 1:39 AM
Unknown Object (File)
Mar 16 2025, 12:10 AM
Subscribers
None

Details

Summary

A REQUEST SENSE CDB was just copied into the cmd buffer, so testing for
INQUIRY will always fail. Remove the dead code.

This code was added, apparently by mistake in 2003. 8541fbec7900e
merged changes from NetBSD's umass_scsipi.c 1.8 to address some BBB
bulk-in clear problems. NetBSD had fixed a problem in the
FORCE_SHORT_INQUIRY quirk code they had ported from FreeBSD that FreeBSD
also needed. That merge also included the dead code, which was not in
NetBSD.

Sponsored by: Netflix

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

imp requested review of this revision.Mar 10 2025, 7:19 PM
imp created this revision.
This revision was not accepted when it landed; it landed in state Needs Review.Mar 11 2025, 2:35 AM
This revision was automatically updated to reflect the committed changes.