On Mon, Mar 26 2018 at 10:12am -0400, Denis Semakin <d.semakin@xxxxxxxxxxxx> wrote: > Also, will it be usefully to add a "get...()" function for > num_secure_erase_bios? Let's say get_num_secure_erase_bios(struct dm_target &ti); > As it's done for num_discard_bios... Yes, it is required. I'll share a link to the final patch shortly, I've still attributed the work to you. BTW, please look into your text editor's settings: seems your patches are using spaces rather than tabs (and you aren't even using 8 spaces?) Mike -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel