Page MenuHomeFreeBSD

nvme: Count number of alginment splits
ClosedPublic

Authored by imp on May 22 2024, 11:08 PM.
Tags
None
Referenced Files
F131987826: D45311.id138952.diff
Sun, Oct 12, 6:02 PM
F131986397: D45311.id139050.diff
Sun, Oct 12, 5:46 PM
Unknown Object (File)
Tue, Oct 7, 9:44 PM
Unknown Object (File)
Sun, Oct 5, 11:37 PM
Unknown Object (File)
Fri, Oct 3, 3:46 AM
Unknown Object (File)
Wed, Oct 1, 10:23 AM
Unknown Object (File)
Wed, Oct 1, 7:32 AM
Unknown Object (File)
Wed, Oct 1, 4:44 AM
Subscribers

Details

Summary

When possible, we split up I/Os to NVMe drives that advertise a
preferred alignment. Add a counter for this.

Sponsored by: Netflix

Diff Detail

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

Event Timeline

gallatin added inline comments.
sys/dev/nvme/nvme_ns.c
431

You should be using the counter api, not atomics, for something like this.

sys/dev/nvme/nvme_private.h
323

Or if you really prefer an atomic, at least put it on a cacheline by itself.

sys/dev/nvme/nvme_ns.c
431

Yea. This counter should be totally cold... but if it's not, we don't want it to mess thing up...

Move to counters for this.

sys/dev/nvme/nvme_sysctl.c
426

The first couple of times I read the description, it sounded like this variable tracked how many splits were in an I/O. Would "Number of I/Os split to get better alignment" convey the right meaning of this sysctl or am I still confused?

Regardless of how this gets worded, maybe drop "namespace" as the counter is per controller.

This revision is now accepted and ready to land.May 23 2024, 1:49 PM
sys/dev/nvme/nvme_ctrlr.c
1445

I have feeling there should be counter_u64_free() somewhere.

Update per review comments: description and cleanup.

This revision now requires review to proceed.May 23 2024, 8:28 PM
This revision is now accepted and ready to land.May 23 2024, 9:36 PM
This revision was automatically updated to reflect the committed changes.