[openib-general] [patch] misc trivia nitpicks 01

Bernhard Fischer blist at aon.at
Thu May 5 18:14:26 PDT 2005


On Thu, May 05, 2005 at 06:08:28PM -0700, Libor Michalek wrote:
>On Fri, May 06, 2005 at 02:49:05AM +0200, Bernhard Fischer wrote:
>> On Thu, May 05, 2005 at 05:32:57PM -0700, Libor Michalek wrote:
>> >On Thu, May 05, 2005 at 07:09:56PM +0200, Bernhard Fischer wrote:
>> >> Hi,
>> >> 
>> >> please consider attached.
>> >> 
>> >> diffstat ./gen2.2257.cow-20050505-1902.diff
>> >>  linux-kernel/infiniband/ulp/sdp/sdp_actv.c |   27 +--
>> >>  userspace/libmthca/src/mthca.c |    4
>> >>  userspace/management/osm/include/opensm/osm_subnet.h |   79 +++++-----
>> >>  userspace/management/osm/include/vendor/osm_vendor_mtl_transaction_mgr.h |   43 ++---
>> >>  userspace/management/osm/opensm/osm_sa_service_record.c |    8 -
>> >>  userspace/management/osm/opensm/osm_sm_state_mgr.c |   38 ++--
>> >>  userspace/management/osm/opensm/osm_sw_info_rcv.c |    9 -
>> >>  userspace/management/osm/opensm/osm_trap_rcv.c |   62 +++----
>> >>  8 files changed, 134 insertions(+), 136 deletions(-)
[]
>> At any rate, i'll try to send them as separate patches per vendor in the
>> future..
[]
>  I was just suggesting breaking it up by module/program which would
>avoid a loaded value judgment like this yet still achieves the desired

Yes, i see. Thanks in advance for applying the parts you feel
comfortable with :)

cheers,

>result. The only reason I suggested it was that I saw SDP and OpenSM
>in the same patch and I haven't even built OpenSM in a while, so I
>didn't want to check in changes to it.
>
>-Libor
>     
>



More information about the general mailing list