[openib-general] kdapl build error on ia64
Talpey, Thomas
Thomas.Talpey at netapp.com
Mon Aug 1 09:30:31 PDT 2005
At 12:28 PM 8/1/2005, Talpey, Thomas wrote:
>It doesn't need two printk's, the one printk just needs
>to have %lu as its format (instead of %ld), right?
I mean %llu. (long long)
>
>Tom.
>
>
>At 12:21 PM 8/1/2005, John Partridge wrote:
>>Hal,
>>
>>No the patch did not fix the warnings :-
>>
>> CC [M] drivers/infiniband/ulp/kdapl/ib/dapl_openib_qp.o
>> CC [M] drivers/infiniband/ulp/kdapl/ib/dapl_openib_util.o
>> CC [M] drivers/infiniband/ulp/kdapl/ib/dapl_openib_cm.o
>> drivers/infiniband/ulp/kdapl/ib/dapl_openib_cm.c: In function
>>`dapl_path_comp_handler':
>> drivers/infiniband/ulp/kdapl/ib/dapl_openib_cm.c:346: warning: long
>>long int format, u64 arg (arg 3)
>> drivers/infiniband/ulp/kdapl/ib/dapl_openib_cm.c: In function
>>`dapl_rt_comp_handler':
>> drivers/infiniband/ulp/kdapl/ib/dapl_openib_cm.c:416: warning: long
>>long int format, u64 arg (arg 3)
>>
>>Let me look at it and I will produce a patch that fixes the build
>>error and the warnings OK ?
>>
>>Cheers
>>John
>>
>>
>>Hal Rosenstock wrote:
>>> Hi John,
>>>
>>> My normal email sending is not working right now so I am using an
>>alternate method.
>>> Hope the formatting comes through OK.
>>>
>>> On Fri, 2005-07-29 at 17:46, John Partridge wrote:
>>>
>>>>With this fix the ia64 modules all build to completion with just a
>>>>couple of warnings :-
>>>
>>>
>>>> CC [M] drivers/infiniband/ulp/kdapl/ib/dapl_openib_cm.o
>>>>drivers/infiniband/ulp/kdapl/ib/dapl_openib_cm.c: In function
>>`dapl_path_comp_handler':
>>>>drivers/infiniband/ulp/kdapl/ib/dapl_openib_cm.c:346: warning: long
>>long int format, u64 arg (arg 3)
>>>>drivers/infiniband/ulp/kdapl/ib/dapl_openib_cm.c: In function
>>`dapl_rt_comp_handler':
>>>>drivers/infiniband/ulp/kdapl/ib/dapl_openib_cm.c:416: warning: long
>>long int format, u64 arg (arg 3)
>>>
>>>
>>> Can you try this patch and see if it removes the warnings ?
>>>
>>> Thanks.
>>>
>>> -- Hal
>>>
>>> Index: dapl_openib_cm.c
>>> ===================================================================
>>> --- dapl_openib_cm.c (revision 2935)
>>> +++ dapl_openib_cm.c (working copy)
>>> @@ -341,9 +341,15 @@ static void dapl_path_comp_handler(u64 r
>>> &cm_ctx->dapl_path, 1,
>>> &cm_ctx->dapl_comp);
>>> if (status) {
>>> +#if defined(__ia64__)
>>> + printk(KERN_ERR "dapl_path_comp_handler: "
>>> + "ib_at_paths_by_route returned %d id %ld\n",
>>> + status, cm_ctx->dapl_comp.req_id);
>>> +#else
>>> printk(KERN_ERR "dapl_path_comp_handler: "
>>> "ib_at_paths_by_route returned %d id %lld\n",
>>> status, cm_ctx->dapl_comp.req_id);
>>> +#endif
>>> event = DAT_CONNECTION_EVENT_BROKEN;
>>> goto error;
>>> }
>>> @@ -412,8 +418,13 @@ static void dapl_rt_comp_handler(u64 req
>>> status = ib_at_paths_by_route(&cm_ctx->dapl_rt, 0,
>>&cm_ctx->dapl_path, 1,
>>> &cm_ctx->dapl_comp);
>>> if (status) {
>>> +#if defined(__ia64__)
>>> + printk(KERN_ERR "dapl_rt_comp_handler:
>ib_at_paths_by_route "
>>> + "returned %d id %ld\n", status,
>>cm_ctx->dapl_comp.req_id);
>>> +#else
>>> printk(KERN_ERR "dapl_rt_comp_handler:
>ib_at_paths_by_route "
>>> "returned %d id %lld\n", status,
>>cm_ctx->dapl_comp.req_id);
>>> +#endif
>>> event = DAT_CONNECTION_EVENT_BROKEN;
>>> goto error;
>>> }
>>
>>--
>>John Partridge
>>
>>Silicon Graphics Inc
>>Tel: 651-683-3428
>>Vnet: 233-3428
>>E-Mail: johnip at sgi.com
>>_______________________________________________
>>openib-general mailing list
>>openib-general at openib.org
>>http://openib.org/mailman/listinfo/openib-general
>>
>>To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
>>
>>
>_______________________________________________
>openib-general mailing list
>openib-general at openib.org
>http://openib.org/mailman/listinfo/openib-general
>
>To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
More information about the general
mailing list