On 6/4/24 06:31, Christoph Hellwig wrote:
On Mon, Jun 03, 2024 at 06:43:56AM +0000, Nitesh Shetty wrote:
Also most block limits are in kb. Not that I really know why we are
doing that, but is there a good reason to deviate from that scheme?
We followed discard as a reference, but we can move to kb, if that helps
with overall readability.
I'm not really sure what is better. Does anyone remember why we did
the _kb version? Either way some amount of consistency would be nice.
If memory serves correctly we introduced the _kb versions as a
convenience to the user; exposing values in 512 bytes increments tended
to be confusing, especially when it comes to LBA values (is the size in
units of hardware sector size? 512 increments? kilobytes?)
Cheers,
Hannes
--
Dr. Hannes Reinecke Kernel Storage Architect
hare@xxxxxxx +49 911 74053 688
SUSE Software Solutions GmbH, Frankenstr. 146, 90461 Nürnberg
HRB 36809 (AG Nürnberg), GF: I. Totev, A. McDonald, W. Knoblich
