Plus a lot of other things.
Details
Details
- Reviewers
- None
- Commits
- rS298002: New CAM I/O scheduler for FreeBSD. The default I/O scheduler is the same
Diff Detail
Diff Detail
- Repository
- rS FreeBSD src repository - subversion
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 1718 Build 1724: arc lint + arc unit
Event Timeline
| head/sys/cam/ata/ata_da.c | ||
|---|---|---|
| 923 ↗ | (On Diff #15202) |
|
| 1300 ↗ | (On Diff #15202) | Why 16? |
| head/sys/cam/cam_ccb.h | ||
| 584 ↗ | (On Diff #15202) | Why isn't this list sorted? |
| head/sys/cam/scsi/scsi_da.c | ||
| 2817 ↗ | (On Diff #15202) | XXX comment? |
| 2906 ↗ | (On Diff #15202) | XXX comment in C++? |
| head/sys/conf/options | ||
| 332 ↗ | (On Diff #15202) | Why is this in FreeBSD (a specific NETFLIX constant?) |
| head/sys/dev/ahci/ahci.c | ||
| 2420 ↗ | (On Diff #15202) | Could this be done better (even if it breaks KBI), and if so, is there a plan to do this? |
| head/sys/cam/ata/ata_da.c | ||
|---|---|---|
| 923 ↗ | (On Diff #15202) | That's a regression. It used to be 5 seconds :) |
| 1300 ↗ | (On Diff #15202) | Because it's long enough and doesn't warrant its own #define. |
| head/sys/cam/cam_ccb.h | ||
| 584 ↗ | (On Diff #15202) | Because ABI breakage. |
| head/sys/conf/options | ||
| 332 ↗ | (On Diff #15202) | Because it is the Netflix scheduler, as opposed to the default scheduler. That's its name, not a Netflix, Inc specific define. |
| head/sys/dev/ahci/ahci.c | ||
| 2420 ↗ | (On Diff #15202) | Can't be done w/o breaking the KBI between CAM and userland. Ken Merry, Scott Long and I have been working on a more general solution because the Auxiliary register is used in a lot of the SMR commands, and it needs to have values other than 1 or 0. Since I did most of this work in 10.x, I didn't want to break the KBI. So we have this ugly kludge until the more general work is completed. |