search

linux-nvme / nvme-cli

NVMe management command line interface.
https://nvmexpress.org
GNU General Public License v2.0
1.45k stars 650 forks source link

plugins/virtium: use time_t for time_stamp values #2436

Closed hrasiq closed 1 month ago

hrasiq commented 1 month ago

On recent Ubuntu Oracular (24.10) builders, -Wincompatible-pointer-types will cause build errors (see nvme-cli 2.9.1-1ubuntu1 build failure on armhf.

In the vtview_log structs, time_t should be used for any time_stamp values instead of long. This ensures correct builds on 32-bit architectures even with 64-bit time_t (e.g. armhf on Ubuntu).

igaw commented 1 month ago

Thanks!