Page MenuHomeFreeBSD

nvme: Count number of alginment splits
ClosedPublic

Authored by imp on May 22 2024, 11:08 PM.
Tags
None
Referenced Files
F165066662: D45311.id138952.diff
Wed, Aug 5, 5:35 PM
F165057227: D45311.id139050.diff
Wed, Aug 5, 3:30 PM
F165053161: D45311.id139014.diff
Wed, Aug 5, 2:36 PM
Unknown Object (File)
Tue, Aug 4, 6:54 AM
Unknown Object (File)
Tue, Aug 4, 6:34 AM
Unknown Object (File)
Tue, Aug 4, 5:00 AM
Unknown Object (File)
Tue, Aug 4, 3:29 AM
Unknown Object (File)
Mon, Aug 3, 8:11 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.