[PATCH] verbs: Use correct type for printf from struct rlimit

[PATCH] verbs: Use correct type for printf from struct rlimit

[Jason Gunthorpe]

The actual type of the members is rlim_t which only happens to be
unsigned long on the platforms we have tested so far. x32 uses something
else.

Casting to unsigned long long for printing is the safe solution.

Signed-off-by: Jason Gunthorpe <jgg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
---
 libibverbs/init.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libibverbs/init.c b/libibverbs/init.c
index ff2c60ffcbc963..cda628b9e0544d 100644
--- a/libibverbs/init.c
+++ b/libibverbs/init.c
@@ -552,9 +552,9 @@ static void check_memlock_limit(void)
 	}
 
 	if (rlim.rlim_cur <= 32768)
-		fprintf(stderr, PFX "Warning: RLIMIT_MEMLOCK is %lu bytes.\n"
+		fprintf(stderr, PFX "Warning: RLIMIT_MEMLOCK is %llu bytes.\n"
 			"    This will severely limit memory registrations.\n",
-			rlim.rlim_cur);
+			(unsigned long long)rlim.rlim_cur);
 }
 
 static int same_sysfs_dev(struct verbs_sysfs_dev *sysfs1,
-- 
2.15.1

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] verbs: Use correct type for printf from struct rlimit

[Leon Romanovsky]

[-- Attachment #1: Type: text/plain, Size: 473 bytes --]

On Tue, Jan 30, 2018 at 03:49:04PM -0700, Jason Gunthorpe wrote:
> The actual type of the members is rlim_t which only happens to be
> unsigned long on the platforms we have tested so far. x32 uses something
> else.
>
> Casting to unsigned long long for printing is the safe solution.
>
> Signed-off-by: Jason Gunthorpe <jgg-VPRAkNaXOzVWk0Htik3J/w@public.gmane.org>
> ---
>  libibverbs/init.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>

Thanks, applied.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]