[PATCH i-g-t 6/6] lib/igt_pm: Convert suspended_time from int to uint64_t
Kamil Konieczny
kamil.konieczny at linux.intel.com
Tue May 21 13:04:28 UTC 2024
Hi Rodrigo,
On 2024-05-20 at 15:03:47 -0400, Rodrigo Vivi wrote:
> Suspended time is in miliseconds printed from u64 variable in kernel.
> Convert to a more appropriate variable type.
>
One more nit about printing, see below.
> Cc: Swati Sharma <swati2.sharma at intel.com>
> Suggested-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi at intel.com>
> ---
> lib/igt_pm.c | 9 +++++----
> lib/igt_pm.h | 2 +-
> tests/intel/kms_pm_dc.c | 9 +++++----
> 3 files changed, 11 insertions(+), 9 deletions(-)
>
> diff --git a/lib/igt_pm.c b/lib/igt_pm.c
> index d54549d11..057634203 100644
> --- a/lib/igt_pm.c
> +++ b/lib/igt_pm.c
> @@ -1392,16 +1392,17 @@ bool i915_is_slpc_enabled(int drm_fd)
> *
> * Return: The total time that the device has been suspended.
> */
> -int igt_pm_get_runtime_suspended_time(struct pci_device *pci_dev)
> +uint64_t igt_pm_get_runtime_suspended_time(struct pci_device *pci_dev)
> {
> char time_str[64];
> - int time, time_fd;
> + int time_fd;
> + uint64_t time;
>
> time_fd = igt_pm_get_power_attr_fd_rdonly(pci_dev, "runtime_suspended_time");
> if (igt_pm_read_power_attr(time_fd, time_str, 64, false)) {
> - igt_assert(sscanf(time_str, "%d", &time) > 0);
> + igt_assert(sscanf(time_str, "%ld", &time) > 0);
>
> - igt_debug("runtime suspend time for PCI '%04x:%02x:%02x.%01x' = %d\n",
> + igt_debug("runtime suspend time for PCI '%04x:%02x:%02x.%01x' = %ld\n",
----------------------------------------------------------------------- ^^^
Instead of %ld use %" PRId64:
igt_debug("runtime suspend time for PCI '%04x:%02x:%02x.%01x' = %" PRId64 "\n",
Regards,
Kamil
> pci_dev->domain, pci_dev->bus, pci_dev->dev, pci_dev->func, time);
>
> return time;
> diff --git a/lib/igt_pm.h b/lib/igt_pm.h
> index 15e301533..6b428f53e 100644
> --- a/lib/igt_pm.h
> +++ b/lib/igt_pm.h
> @@ -93,7 +93,7 @@ void igt_pm_restore_pci_card_runtime_pm(void);
> void igt_pm_print_pci_card_runtime_status(void);
> bool i915_is_slpc_enabled_gt(int drm_fd, int gt);
> bool i915_is_slpc_enabled(int drm_fd);
> -int igt_pm_get_runtime_suspended_time(struct pci_device *pci_dev);
> +uint64_t igt_pm_get_runtime_suspended_time(struct pci_device *pci_dev);
> uint64_t igt_pm_get_runtime_active_time(struct pci_device *pci_dev);
> int igt_pm_get_runtime_usage(struct pci_device *pci_dev);
> void igt_pm_ignore_slpc_efficient_freq(int i915, int gtfd, bool val);
> diff --git a/tests/intel/kms_pm_dc.c b/tests/intel/kms_pm_dc.c
> index 176f7a04b..7766d34d7 100644
> --- a/tests/intel/kms_pm_dc.c
> +++ b/tests/intel/kms_pm_dc.c
> @@ -488,10 +488,10 @@ static bool support_dc6(int debugfs_fd)
> return strstr(buf, "DC5 -> DC6 count");
> }
>
> -static int read_runtime_suspended_time(int drm_fd)
> +static uint64_t read_runtime_suspended_time(int drm_fd)
> {
> struct pci_device *i915;
> - int ret;
> + uint64_t ret;
>
> i915 = igt_device_get_pci_device(drm_fd);
> ret = igt_pm_get_runtime_suspended_time(i915);
> @@ -500,7 +500,7 @@ static int read_runtime_suspended_time(int drm_fd)
> return ret;
> }
>
> -static bool dc9_wait_entry(data_t *data, int dc_target, int prev_dc, int prev_rpm, int msecs)
> +static bool dc9_wait_entry(data_t *data, int dc_target, int prev_dc, uint64_t prev_rpm, int msecs)
> {
> /*
> * Runtime suspended residency should increment once DC9 is achieved;
> @@ -522,7 +522,8 @@ static void check_dc9(data_t *data, int dc_target, int prev_dc, int prev_rpm)
>
> static void setup_dc9_dpms(data_t *data, int dc_target)
> {
> - int prev_dc = 0, prev_rpm, sysfs_fd;
> + uint64_t prev_rpm;
> + int prev_dc = 0, sysfs_fd;
>
> igt_require((sysfs_fd = open(KMS_HELPER, O_RDONLY)) >= 0);
> __igt_sysfs_get_boolean(sysfs_fd, "poll", &kms_poll_saved_state);
> --
> 2.44.0
>
More information about the igt-dev
mailing list