On Mon, Feb 03, 2025 at 06:57:13PM +0530, Kanchan Joshi wrote: > But, patches do exactly that i.e., hardware cusm support. And posted > numbers [*] are also when hardware is checksumming the data blocks. I'm still not sure why you think the series implements hardware csum support. The buf mode is just a duplicate implementation of the block layer automatic PI. The no buf means PRACT which let's the device auto generate and strip PI. Especially the latter one (which is the one that was benchmarked) literally provides no additional protection over what the device would already do. It's the "trust me, bro" of data integrity :) Which to be fair will work pretty well as devices that support PI are the creme de la creme of storage devices and will have very good internal data protection internally. But the point of data checksums is to not trust the storage device and not trust layers between the checksum generation and the storage device. IFF using PRACT is an acceptable level of protection just running NODATASUM and disabling PI generation/verification in the block layer using the current sysfs attributes (or an in-kernel interface for that) to force the driver to set PRACT will do exactly the same thing.
