cam(4): Limit search for disks in SES enclosure by single bus
ClosedPublic
Actions

Authored by mav on Oct 5 2021, 2:26 AM.

Details

Reviewers

ken
gibbs
imp

Group Reviewers

cam

Commits

rG7e321b97917d: cam(4): Limit search for disks in SES enclosure by single bus
rG730ea72c706e: cam(4): Limit search for disks in SES enclosure by single bus

Summary

At least for SAS we only support now disks are typically connected to the same bus as the enclosure. Limiting the search scope makes it faster on systems with multiple buses and thousands of disks.

Test Plan

Works best in combination with https://reviews.freebsd.org/D32304.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

mav requested review of this revision.Oct 5 2021, 2:26 AM

mav created this revision.

mav edited the summary of this revision. (Show Details)

mav added a reviewer: imp.Oct 5 2021, 3:40 AM

imp added inline comments.Oct 5 2021, 3:52 AM

sys/cam/scsi/scsi_enc_ses.c
898	This is going in after your change from MATCH_NONE == 0 to MATCH_ANY == 0? If so, then it's good.

mav added inline comments.Oct 5 2021, 4:23 AM

sys/cam/scsi/scsi_enc_ses.c
898	It is. Though not sure why have you mentioned MATCH_ANY == 0 part, not the optimization in the same review.

This revision was not accepted when it landed; it landed in state Needs Review.Oct 5 2021, 7:04 PM

Closed by commit rG730ea72c706e: cam(4): Limit search for disks in SES enclosure by single bus (authored by mav). · Explain Why

This revision was automatically updated to reflect the committed changes.

mav added a commit: rG730ea72c706e: cam(4): Limit search for disks in SES enclosure by single bus.

mav added a commit: rG7e321b97917d: cam(4): Limit search for disks in SES enclosure by single bus.Oct 19 2021, 3:07 PM

Revision Contents
Changeset List

Path

Size

sys/

cam/

scsi/

scsi_enc_ses.c

8 lines

Diff 96248

View Options

sys/cam/scsi/scsi_enc_ses.c

cam(4): Limit search for disks in SES enclosure by single busClosedPublicActions