Re: [PATCH] IB/core: Don't warn on no SA support in event handler

2015-06-11 Thread Michael Wang
On 06/10/2015 06:15 PM, Weiny, Ira wrote: Registering an event handler is done for a device. This device may have one RoCE port (no SA cap) and one InfiniBand port (has SA cap). Therefore, warning from the event handler about a specific port that doesn't have SA cap is correct but pollutes

Re: [PATCH] IB/core: Don't warn on no SA support in event handler

2015-06-10 Thread Michael Wang
Hi, Moni On 06/10/2015 11:13 AM, Moni Shoua wrote: Registering an event handler is done for a device. This device may have one RoCE port (no SA cap) and one InfiniBand port (has SA cap). Therefore, warning from the event handler about a specific port that doesn't have SA cap is correct but

RE: [PATCH] IB/core: Don't warn on no SA support in event handler

2015-06-10 Thread Weiny, Ira
Registering an event handler is done for a device. This device may have one RoCE port (no SA cap) and one InfiniBand port (has SA cap). Therefore, warning from the event handler about a specific port that doesn't have SA cap is correct but pollutes the kernel log without a need.

RE: [PATCH] IB/core: Don't warn on no SA support in event handler

2015-06-10 Thread Hefty, Sean
Registering an event handler is done for a device. This device may have one RoCE port (no SA cap) and one InfiniBand port (has SA cap). Therefore, warning from the event handler about a specific port that doesn't have SA cap is correct but pollutes the kernel log without a need. Maybe

Re: [PATCH] IB/core: Don't warn on no SA support in event handler

2015-06-10 Thread Doug Ledford
On Wed, 2015-06-10 at 12:13 +0300, Moni Shoua wrote: Registering an event handler is done for a device. This device may have one RoCE port (no SA cap) and one InfiniBand port (has SA cap). Therefore, warning from the event handler about a specific port that doesn't have SA cap is correct but