Re: [PATCH net] sctp: fix pr_warn max_data argument type mismatch

2018-12-06 Thread Jakub Audykowicz 
On 2018-12-06 09:03, David Miller wrote:

> From: Jakub Audykowicz 
> Date: Thu,  6 Dec 2018 08:58:37 +0100
>
>> My previous patch introduced a compilation warning regarding a type
>> mismatch (int vs size_t). This is a one-letter fix for good housekeeping.
>>
>> Signed-off-by: Jakub Audykowicz 
> Still wrong and I fixed it when I applied your patch.
>
> You need to use the 'Z' prefix for size_t, so %Zu in this case.

Right, I just realized that as well, thanks!

Re: [PATCH net] sctp: fix pr_warn max_data argument type mismatch

2018-12-06 Thread David Miller 
From: Jakub Audykowicz 
Date: Thu,  6 Dec 2018 08:58:37 +0100

> My previous patch introduced a compilation warning regarding a type
> mismatch (int vs size_t). This is a one-letter fix for good housekeeping.
> 
> Signed-off-by: Jakub Audykowicz 

Still wrong and I fixed it when I applied your patch.

You need to use the 'Z' prefix for size_t, so %Zu in this case.

[PATCH net] sctp: fix pr_warn max_data argument type mismatch

2018-12-05 Thread Jakub Audykowicz 
My previous patch introduced a compilation warning regarding a type
mismatch (int vs size_t). This is a one-letter fix for good housekeeping.

Signed-off-by: Jakub Audykowicz 
---
 net/sctp/chunk.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/sctp/chunk.c b/net/sctp/chunk.c
index d5b91bc8a377..ee5638358ad5 100644
--- a/net/sctp/chunk.c
+++ b/net/sctp/chunk.c
@@ -194,7 +194,7 @@ struct sctp_datamsg *sctp_datamsg_from_user(struct 
sctp_association *asoc,
if (unlikely(!max_data)) {
max_data = sctp_min_frag_point(sctp_sk(asoc->base.sk),
   sctp_datachk_len(>stream));
-   pr_warn_ratelimited("%s: asoc:%p frag_point is zero, forcing 
max_data to default minimum (%d)",
+   pr_warn_ratelimited("%s: asoc:%p frag_point is zero, forcing 
max_data to default minimum (%ld)",
__func__, asoc, max_data);
}
 
-- 
2.17.1