Page MenuHomeFreeBSD
Differential D58654

ufshci: fail attribute reads on a non-zero config result code
ClosedPublic
Actions

Authored by jaeyoon on Aug 5 2026, 7:55 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Sep 9, 8:08 AM
Unknown Object (File)
Wed, Sep 9, 6:08 AM
Unknown Object (File)
Tue, Sep 8, 7:45 PM
Unknown Object (File)
Mon, Sep 7, 7:24 PM
Unknown Object (File)
Mon, Sep 7, 3:24 PM
Unknown Object (File)
Sun, Sep 6, 9:01 PM
Unknown Object (File)
Sat, Sep 5, 10:14 PM
Unknown Object (File)
Thu, Sep 3, 11:57 AM
View All 34 Files
Subscribers
None

Details

Reviewers
imp
Commits
rG154e335cd635: ufshci: fail attribute reads on a non-zero config result code
Summary

ufshci_uic_send_cmd() only logged the error code and returned success,
so a failed DME_GET gave its caller a stale value as if it were valid.
The gear and lane settings could then be programmed from that garbage.

Return ENXIO for reads instead. Writes keep logging and continuing,
because a device may reject an optional attribute and that must not
fail bring-up.

Sponsored by: Samsung Electronics

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 75373
Build 72256: arc lint + arc unit

Revision Contents
Changeset List

PathSize
sys/
dev/
ufshci/
8 lines

Diff 183417

View Options

sys/dev/ufshci/ufshci_uic_cmd.c

Loading...