Page MenuHomeFreeBSD

nvme: Count number of alginment splits
ClosedPublic

Authored by imp on May 22 2024, 11:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 9, 5:46 AM
Unknown Object (File)
Sun, Dec 8, 9:11 AM
Unknown Object (File)
Nov 25 2024, 3:11 PM
Unknown Object (File)
Nov 23 2024, 9:52 PM
Unknown Object (File)
Nov 23 2024, 1:07 AM
Unknown Object (File)
Nov 21 2024, 7:02 PM
Unknown Object (File)
Nov 20 2024, 3:42 PM
Unknown Object (File)
Nov 18 2024, 10:53 PM
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.