ufshci: validate the CDB before allocating a request
ClosedPublic
Actions

Authored by jaeyoon on Thu, Aug 13, 8:39 AM.

Details

Reviewers

Commits

rGf3c3db35d545: ufshci: validate the CDB before allocating a request

Summary

The CDB pointer and length checks depend only on the CCB, so perform
them before allocating and initializing the request. This avoids a
wasted allocation for invalid CCBs on the I/O path and removes one
request-free error path.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

jaeyoon requested review of this revision.Thu, Aug 13, 8:39 AM

jaeyoon created this revision.

Harbormaster completed remote builds in B75553: Diff 183970.Thu, Aug 13, 8:39 AM

jaeyoon added a parent revision: D58816: ufshci: pass the queue being destroyed to the cmd descriptor teardown.Thu, Aug 13, 8:40 AM

jaeyoon added a child revision: D58818: ufshci: consolidate the device query submit/poll pattern.

imp accepted this revision.Fri, Aug 14, 9:06 PM

This revision is now accepted and ready to land.Fri, Aug 14, 9:06 PM

Closed by commit rGf3c3db35d545: ufshci: validate the CDB before allocating a request (authored by jaeyoon). · Explain WhyTue, Aug 18, 5:26 AM

This revision was automatically updated to reflect the committed changes.

jaeyoon added a commit: rGf3c3db35d545: ufshci: validate the CDB before allocating a request.

Revision Contents
Changeset List

Path

Size

sys/

dev/

ufshci/

ufshci_sim.c

22 lines

Diff 184231

View Options

sys/dev/ufshci/ufshci_sim.c

ufshci: validate the CDB before allocating a requestClosedPublicActions