> -----Original Message-----
> From: Hal Rosenstock [mailto:h...@dev.mellanox.co.il]
> Sent: Wednesday, December 09, 2015 9:06 AM

> > This is the best effort and it should work for most common
> > configurations, but may not work for more complicated cases.
> 
> Right, there are various scenarios where it will not work. This was one of
> them but there are others I can think of.
> 
> > Any other suggestions?
> 
> Unfortunately not. It comes down to whether the out of box cases outweigh
> the debug when it's an exception case. The premise of this patch is that
> that's the case.

I would argue that it does. Without this patch, ibacm will not work on secure 
fabric out of box (where "default" is interpret as 0xffff), and it will be 
equal or more likely not to work by default in more complicated configurations, 
where debugging is required anyway. This patch enables ibacm to work properly 
in most common configurations out of box.

Kaike
N�����r��y����b�X��ǧv�^�)޺{.n�+����{��ٚ�{ay�ʇڙ�,j��f���h���z��w���
���j:+v���w�j�m��������zZ+�����ݢj"��!�i

Reply via email to