Re: [openib-general] [PATCH] osm: port to WinIB stack

2006-10-04 Thread Hal Rosenstock
Hi Eitan, On Tue, 2006-10-03 at 17:49, Hal Rosenstock wrote: Hi Eitan, Aside from the varargs handling (relative to 2 patches) and the osmtest.c question, osmtest.c patch has been applied. The question is more general as to why the casts were needed for Windows. -- Hal also pending

Re: [openib-general] [PATCH] opensm: setup function for 'null' routing engine

2006-10-04 Thread Hal Rosenstock
On Wed, 2006-10-04 at 15:18, Sasha Khapyorsky wrote: This defines setup function for 'null' routing engine. Currently there is only log message and fallback to default behavior, so the function is needed to prevent opensm crash when '-R null' is used. Signed-off-by: Sasha Khapyorsky [EMAIL

Re: [openib-general] [PATCH] opensm: verbose message about fallback to default routing engine.

2006-10-04 Thread Hal Rosenstock
On Wed, 2006-10-04 at 15:34, Sasha Khapyorsky wrote: This provides verbose message for cases then specified routing engine (with -R) was not found or this setup was failed. Signed-off-by: Sasha Khapyorsky [EMAIL PROTECTED] Thanks. Applied. -- Hal

Re: [openib-general] [PATCH] OpenSM/osm_sa_mcmember_record.c: In osm_mcmr_rcv_create_new_mgrp, fix exactly selectors in response

2006-10-03 Thread Hal Rosenstock
Hi Yevgeny, On Mon, 2006-09-25 at 08:35, Yevgeny Kliteynik wrote: Hi Hal. The patch looks ok. Thanks. Did you just look at things or did you also run any tests ? A few remarks thought: It appears that the multicast group mtu/rate selectors are actually not referenced by anyone - the

Re: [openib-general] Setting HCA LinkWidth

2006-10-03 Thread Hal Rosenstock
On Tue, 2006-10-03 at 13:34, Amit H Kumar wrote: Hi OpenIB, I am not sure if this question belongs to this list or not. But if anyone can give me some lead it will be very helpful. One of the HCA card in our cluster is running at LinkWidth 1x: as opposed to 4x on all other nodes You

Re: [openib-general] [PATCH 4/13] osm: port to WinIB stack : osmtest/osmtest.c

2006-10-03 Thread Hal Rosenstock
Hi Eitan, On Tue, 2006-10-03 at 15:05, Eitan Zahavi wrote: Hi Hal, I see this was not committed. Do you see a reason it should not? I had gotten part way through the Windows patches before the start of last week and have not got back to them. This and some others in the series are still

[openib-general] [PATCH] diags/ibportstate: Support changing LinkSpeedEnabled on any IB port

2006-10-03 Thread Hal Rosenstock
diags/ibportstate: Support changing LinkSpeedEnabled on any IB port Signed-off-by: Hal Rosenstock [EMAIL PROTECTED] Index: src/ibportstate.c === --- src/ibportstate.c (revision 9687) +++ src/ibportstate.c (working copy

Re: [openib-general] [PATCH 2/13] osm: port to WinIB stack : opensm/osm_prtn_config.c

2006-10-03 Thread Hal Rosenstock
Hi Eitan, On Sun, 2006-09-17 at 11:59, Eitan Zahavi wrote: Hi Hal 1. Avoid varargs macros not supported by win What about using __VA_ARGS__? It is C99, and MS claims that they support this (http://msdn2.microsoft.com/en-us/library/ms177415.aspx), in this way: #define MACRO(s, ...) printf(s,

Re: [openib-general] [PATCH 13/13] osm: port to WinIB stack : opensm/osm_pkey_mgr.c

2006-10-03 Thread Hal Rosenstock
On Sun, 2006-09-17 at 12:00, Eitan Zahavi wrote: Hi Hal Avoid using array initialization statements which do not compile on win. Thanks Thanks. Applied. -- Hal Eitan ___ openib-general mailing list openib-general@openib.org

Re: [openib-general] [PATCH 12/13] osm: port to WinIB stack : opensm/osm_qos.c

2006-10-03 Thread Hal Rosenstock
Hi Eitan, On Sun, 2006-09-17 at 12:00, Eitan Zahavi wrote: Hi Hal Port num is uint8_t (avoid casting by using correct size field). Added some explicit casts Thanks. Applied. -- Hal Thanks Eitan ___ openib-general mailing list

Re: [openib-general] [PATCH 7/13] osm: port to WinIB stack : opensm/osm_ucast_file.c

2006-10-03 Thread Hal Rosenstock
Hi Eitan, On Sun, 2006-09-17 at 11:59, Eitan Zahavi wrote: Hi Hal 1. Avoid varargs macros not supported by win What about using __VA_ARGS__? It is C99, and MS claims that they support this (http://msdn2.microsoft.com/en-us/library/ms177415.aspx), in this way: #define MACRO(s, ...) printf(s,

Re: [openib-general] [PATCH] osm: port to WinIB stack

2006-10-03 Thread Hal Rosenstock
Hi Eitan, Aside from the varargs handling (relative to 2 patches) and the osmtest.c question, also pending is a patch to remove the WIN defines just added in multiple places and move them to config.h for the Windows build ? Can you/when can you prepare a patch for this ? Thanks! -- Hal

Re: [openib-general] [PATCH TRIVIAL] opensm: libibumad: show open()'s errno string.

2006-09-28 Thread Hal Rosenstock
On Tue, 2006-09-26 at 16:51, Sasha Khapyorsky wrote: Show errno string then open() fails. Signed-off-by: Sasha Khapyorsky [EMAIL PROTECTED] Thanks. Applied to trunk only. -- Hal ___ openib-general mailing list openib-general@openib.org

Re: [openib-general] [PATCH 3/3] IB/iser: fix the description of iSER in Kconfig

2006-09-28 Thread Hal Rosenstock
On Wed, 2006-09-27 at 09:48, Erez Zilber wrote: fix the description of iSER in Kconfig. It is not accurate. Signed-off-by: Erez Zilber [EMAIL PROTECTED] --- drivers/infiniband/ulp/iser/Kconfig | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-)

Re: [openib-general] [PATCH] osm_vendor_mlx_sa.c - missing status on timeout SA query

2006-09-28 Thread Hal Rosenstock
On Thu, 2006-09-28 at 00:40, Eitan Zahavi wrote: Hi Hal Similar to the bug discovered by Yevgeny on the osm_vendor_ibumad_sa.c the very same bug happens on osm_vendor_mlx_sa.c which fails osmtest. The issue is that the status of the result of the query is not returned as the result of the

Re: [openib-general] [PATCH 1/2] osm: osmtest ignores error status

2006-09-28 Thread Hal Rosenstock
On Thu, 2006-09-28 at 11:16, Yevgeny Kliteynik wrote: Hi Hal. This patch takes care of several cases where osmtest ignored error status. Yevgeny Signed-off-by: Yevgeny Kliteynik [EMAIL PROTECTED] Thanks. Applied to trunk only. -- Hal ___

Re: [openib-general] [PATCH 2/2] osm: osmtest ignores error status

2006-09-28 Thread Hal Rosenstock
On Thu, 2006-09-28 at 11:20, Yevgeny Kliteynik wrote: Hi Hal. This patch takes care of several cases where osmtest ignored error status (plus some cosmetics). Yevgeny Signed-off-by: Yevgeny Kliteynik [EMAIL PROTECTED] Thanks. Applied to trunk only. -- Hal

Re: [openib-general] OpenSM - 'open /dev/infiniband/umad1 failed'

2006-09-26 Thread Hal Rosenstock
Hi, Do you have udev installed and configured ? You may want to refer to the wiki (https://openib.org/tiki/tiki-index.php) for more troubleshooting info. There's some info in the cheat sheet (https://openib.org/tiki/tiki-index.php?page=Installation+Cheat+Sheet) which may help. -- Hal

Re: [openib-general] [PATCH 3/13] osm: port to WinIB stack : include/iba/ib_types.h

2006-09-22 Thread Hal Rosenstock
On Sun, 2006-09-17 at 11:58, Eitan Zahavi wrote: Hi Hal Most are just adding OSM_API for fucntion declarations. Some minor indentations. Thanks Eitan Signed-off-by: Eitan Zahavi [EMAIL PROTECTED] Thanks. Applied (with some cosmetic changes) to trunk only. -- Hal

Re: [openib-general] [PATCHv2] osm: fixing bugs in osmtest

2006-09-21 Thread Hal Rosenstock
Hi Yevgeny, On Thu, 2006-09-21 at 03:30, Yevgeny Kliteynik wrote: Hi Hal It appears that each mailer is messing with white spaces in its own very special way... Anyway, this time it is ok for sure. Yevgeny Signed-off-by: Yevgeny Kliteynik [EMAIL PROTECTED] Yes, that's better!

Re: [openib-general] gen2_basic patch 5/10: select a valid port number

2006-09-20 Thread Hal Rosenstock
On Wed, 2006-09-20 at 01:05, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re: gen2_basic patch 5/10: select a valid port number On Tue, 2006-09-19 at 21:16, Robert Walsh wrote: Hal Rosenstock wrote: On Tue, 2006-09-19 at 20:28, Robert Walsh wrote

Re: [openib-general] [PATCH] osm: fixing bugs in osmtest

2006-09-20 Thread Hal Rosenstock
Hi Yevgeny, On Wed, 2006-09-20 at 02:36, Yevgeny Kliteynik wrote: Hi Hal I'm doing a major review of the osmtest. Good. This has been long overdue. This patch is fixing a few bugs in osmtest where failures were ignored. More precisely, osmtest was expecting error, but got IB_SUCCESS and

Re: [openib-general] OFED 1.1 and OpenSM on SLES 10 for PPC64

2006-09-20 Thread Hal Rosenstock
On Wed, 2006-09-20 at 16:42, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: RE: OFED 1.1 and OpenSM on SLES 10 for PPC64 On Wed, 2006-09-20 at 11:11, Eitan Zahavi wrote: I will try to get to that tomorrow It's not an OpenSM issue. See the latest

Re: [openib-general] OFED 1.1 and OpenSM on SLES 10 for PPC64

2006-09-20 Thread Hal Rosenstock
On Wed, 2006-09-20 at 17:38, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Short term, can we just skip saquery utility in OFED 1.1? Hal, can you approve this please? I would prefer that is not the case and this is part of OFED 1.1. So - what do you suggest

Re: [openib-general] [PATCH TRIVIAL] opensm: remove osm_switch_get_lid() prototype

2006-09-20 Thread Hal Rosenstock
On Wed, 2006-09-20 at 21:27, Sasha Khapyorsky wrote: Hi Hal, Some trivial cleanup. Sasha. opensm: remove osm_switch_get_lid() prototype Remove prototype of non-existing osm_switch_get_lid() function. Signed-off-by: Sasha Khapyorsky [EMAIL PROTECTED] Thanks. Applied to trunk

Re: [openib-general] [PATCH TRIVIAL] opensm: LOG_ENTER name fix

2006-09-20 Thread Hal Rosenstock
On Wed, 2006-09-20 at 21:50, Sasha Khapyorsky wrote: Hi Hal, Some trivial stuff. Sasha opensm: LOG_ENTER name fix In osm_pkey_get_tables() fix the name used with LOG_ENTER(). Signed-off-by: Sasha Khapyorsky [EMAIL PROTECTED] Thanks. Applied to trunk only. -- Hal

Re: [openib-general] [PATCH] osm: bug in OpenSM on broken fabrics

2006-09-19 Thread Hal Rosenstock
On Mon, 2006-09-18 at 14:52, Yevgeny Kliteynik wrote: Hi Hal This patch fixes a bug in opensm that was discovered on a 'broken' fabrics when opensm was executed with --stay_on_fatal. Replacing assert with a real check. Yevgeny Signed-off-by: Yevgeny Kliteynik [EMAIL PROTECTED]

[openib-general] [PATCH][TRIVIAL]OpenSM/osm_node_info_rcv.c: Eliminate superfluous call level

2006-09-19 Thread Hal Rosenstock
OpenSM/osm_node_info_rcv.c: Eliminate superfluous call level Signed-off-by: Hal Rosenstock [EMAIL PROTECTED] Index: opensm/osm_node_info_rcv.c === --- opensm/osm_node_info_rcv.c (revision 9536) +++ opensm/osm_node_info_rcv.c

Re: [openib-general] [PATCH 2/13] osm: port to WinIB stack : include/opensm/osm_pkey.h

2006-09-19 Thread Hal Rosenstock
On Sun, 2006-09-17 at 11:58, Eitan Zahavi wrote: Hi Hal Partition tables blocks are always 16 bits. This resolves the need to later cast back and forth. Thanks Eitan Signed-off-by: Eitan Zahavi [EMAIL PROTECTED] Thanks. Applied to trunk only in conjunction with patch 10/13 on

Re: [openib-general] [PATCH 10/13] osm: port to WinIB stack : opensm/osm_pkey.c

2006-09-19 Thread Hal Rosenstock
On Sun, 2006-09-17 at 12:00, Eitan Zahavi wrote: Hi Hal Some explicit casting required and also pkey blocks are only uint16_t . Thanks Eitan Signed-off-by: Eitan Zahavi [EMAIL PROTECTED] Thanks. Applied to trunk only in conjunction with patch 2/13 on osm_pkey.h. -- Hal

Re: [openib-general] [PATCH 2/13] osm: port to WinIB stack : opensm/osm_subnet.c

2006-09-19 Thread Hal Rosenstock
Hi Eitan, On Sun, 2006-09-17 at 11:59, Eitan Zahavi wrote: Hi Hal I think this patch is really 5/13 rather than 2/13. No need for stdio.h but do need stdlib.h ... It appears to be the other way around (stdio.h needed but stdlib.h isn't), right ? Also map snprintf to _snprintf in windows

Re: [openib-general] [PATCH 9/13] osm: port to WinIB stack : opensm/osm_prtn.c

2006-09-19 Thread Hal Rosenstock
On Sun, 2006-09-17 at 11:59, Eitan Zahavi wrote: Hi Hal Required cl_debug.h for PRIx64 Also map snprintf to _snprintf and stat to _stat Thanks Eitan Signed-off-by: Eitan Zahavi [EMAIL PROTECTED] Thanks. Applied to trunk only. -- Hal

Re: [openib-general] [PATCH 11/13] osm: port to WinIB stack : opensm/osm_log.c

2006-09-19 Thread Hal Rosenstock
On Sun, 2006-09-17 at 12:00, Eitan Zahavi wrote: Hi Hal 1. function mappings for stat, fstat and fileno 2. Currently no imp for log file truncation Thanks Eitan Signed-off-by: Eitan Zahavi [EMAIL PROTECTED] Thanks. Applied to trunk only. -- Hal

Re: [openib-general] [PATCH 8/13] osm: port to WinIB stack : opensm/osm_opensm.c

2006-09-19 Thread Hal Rosenstock
On Sun, 2006-09-17 at 11:59, Eitan Zahavi wrote: Hi Hal Explicit NULL in empty array initializer Thanks Eitan Signed-off-by: Eitan Zahavi [EMAIL PROTECTED] Thanks. Applied to trunk only. -- Hal ___ openib-general mailing list

Re: [openib-general] IB diagnostics problems (OFED-1.1-rc5)

2006-09-19 Thread Hal Rosenstock
it is not convenient for an end user. Regards, Mirko Hal Rosenstock schrieb: Hi again Mirko, On Mon, 2006-09-18 at 07:20, Mirko Benz wrote: Hi Hal, This was a default/build all OFED install. Either we should place these tools under ../ofed/sbin or make it work for every

Re: [openib-general] gen2_basic patch 5/10: select a valid port number

2006-09-19 Thread Hal Rosenstock
On Tue, 2006-09-19 at 20:28, Robert Walsh wrote: gen2_basic - select a valid port number Port numbers start at 1, not 0. True for CA and routers but not switches. Signed-off by: Robert Walsh [EMAIL PROTECTED] diff -rNu a/gen2_basic/test_poll_post.c b/gen2_basic/test_poll_post.c ---

Re: [openib-general] gen2_basic patch 5/10: select a valid port number

2006-09-19 Thread Hal Rosenstock
On Tue, 2006-09-19 at 21:16, Robert Walsh wrote: Hal Rosenstock wrote: On Tue, 2006-09-19 at 20:28, Robert Walsh wrote: gen2_basic - select a valid port number Port numbers start at 1, not 0. True for CA and routers but not switches. Yeah. Does anyone run gen2_basic on switches

Re: [openib-general] [PATCH] osm: bug in __osmv_send_sa_req

2006-09-18 Thread Hal Rosenstock
Hi Yevgeny, On Sun, 2006-09-17 at 10:20, Yevgeny Kliteynik wrote: Hi Hal This patch fixes a bug is __osmv_send_sa_req in libvendor. After sending a MAD, the status of the responce was ignored. Yevgeny Signed-off-by: Yevgeny Kliteynik [EMAIL PROTECTED] Thanks. Applied to trunk and

Re: [openib-general] [openfabrics-ewg] OpenSm on sles10 ppc64 OFED 1.0 - bug ?

2006-09-18 Thread Hal Rosenstock
www.voltaire.com -Original Message- From: Hal Rosenstock [mailto:[EMAIL PROTECTED] Sent: Sunday, September 17, 2006 5:50 PM To: Moshe Kazir Cc: openib-general@openib.org; OpenFabricsEWG; Sasha Khapyorsky Subject: Re: [openfabrics-ewg] OpenSm on sles10 ppc64 Hi Moshe

Re: [openib-general] [PATCH] osm: PathRecord prefer 1K MTU for MT23108 - not for MTU Sel=3

2006-09-18 Thread Hal Rosenstock
Hi Eitan, On Sun, 2006-09-17 at 10:52, Eitan Zahavi wrote: Hi Hal We have reviewed the patch for the above and figured out there is an issue with it: Currently when MTU_SEL=3 the quirk applies. We think this is wrong behavior as MTU_SEL=3 means max possible MTU by the IBTA spec. So if an

Re: [openib-general] [PATCH 1/13] osm: port to WinIB stack : include/opensm/osm_base.h

2006-09-18 Thread Hal Rosenstock
On Sun, 2006-09-17 at 11:58, Eitan Zahavi wrote: Hi Hal osm_base.h uses cache dir for osm-partitions.conf. Thanks Eitan Signed-off-by: Eitan Zahavi [EMAIL PROTECTED] Thanks. Applied to trunk only. -- Hal ___ openib-general mailing list

Re: [openib-general] [PATCH 2/13] osm: port to WinIB stack : include/opensm/osm_pkey.h

2006-09-18 Thread Hal Rosenstock
Hi Eitan, On Sun, 2006-09-17 at 11:58, Eitan Zahavi wrote: Hi Hal Partition tables blocks are always 16 bits. This resolves the need to later cast back and forth. Thanks Eitan Signed-off-by: Eitan Zahavi [EMAIL PROTECTED] Index: include/opensm/osm_pkey.h

Re: [openib-general] IB diagnostics problems (OFED-1.1-rc5)

2006-09-18 Thread Hal Rosenstock
Hi Mirko, On Mon, 2006-09-18 at 06:59, Mirko Benz wrote: Hello, We are testing OFED-1.1-rc5 under Scientific Linux x86-64 (RHEL 4 clone). Some IB diagnostics tools e.g. ibhosts and ibswitches (located under .../ofed/bin/) do not work with a normal user account -- no output given. It works

Re: [openib-general] IB diagnostics problems (OFED-1.1-rc5)

2006-09-18 Thread Hal Rosenstock
Regards, Mirko Hal Rosenstock schrieb: Hi Mirko, On Mon, 2006-09-18 at 06:59, Mirko Benz wrote: Hello, We are testing OFED-1.1-rc5 under Scientific Linux x86-64 (RHEL 4 clone). Some IB diagnostics tools e.g. ibhosts and ibswitches (located under .../ofed/bin/) do

Re: [openib-general] IB diagnostics problems (OFED-1.1-rc5)

2006-09-18 Thread Hal Rosenstock
On Mon, 2006-09-18 at 07:40, Michael S. Tsirkin wrote: Quoting r. Mirko Benz [EMAIL PROTECTED]: Subject: Re: IB diagnostics problems (OFED-1.1-rc5) Hi Hal, This was a default/build all OFED install. Either we should place these tools under ../ofed/sbin or make it work for every

Re: [openib-general] What does --process_mad-- exactly?

2006-09-18 Thread Hal Rosenstock
Hi Michael, On Mon, 2006-09-18 at 08:47, Michael Arndt wrote: Hi, the function ib_mad_recv_done_handler, which is called if a dr_smp packet was received, calls port_priv-device-process_mad. I analyze that function (recursive) for mthca, but I never found the point where a set or get

Re: [openib-general] [PATCH 2/13] osm: port to WinIB stack : include/opensm/osm_pkey.h

2006-09-18 Thread Hal Rosenstock
On Mon, 2006-09-18 at 16:11, Eitan Zahavi wrote: Hal Rosenstock wrote: Hi Eitan, On Sun, 2006-09-17 at 11:58, Eitan Zahavi wrote: Hi Hal Partition tables blocks are always 16 bits. This resolves the need to later cast back and forth. Thanks Eitan Signed-off-by: Eitan

Re: [openib-general] Bug in OpenSM multicast group creation?

2006-09-18 Thread Hal Rosenstock
On Mon, 2006-09-18 at 18:34, Roland Dreier wrote: Around line 1340 of osm_sa_mcmember_record.c, there is the code: /* the mcmember_record should have mtu_sel, rate_sel and pkt_lifetime_sel = 2 */ (*pp_mgrp)-mcmember_rec.mtu |= 26; /* exactly */ (*pp_mgrp)-mcmember_rec.rate |= 26; /*

[openib-general] [PATCH] OpenSM/osm_sa_mcmember_record.c: In osm_mcmr_rcv_create_new_mgrp, fix exactly selectors in response

2006-09-18 Thread Hal Rosenstock
OpenSM/osm_sa_mcmember_record.c: In osm_mcmr_rcv_create_new_mgrp, set exactly selectors after rather than before mgrp is initialized Pointed out by: Roland Dreier [EMAIL PROTECTED] Signed-off-by: Hal Rosenstock [EMAIL PROTECTED] Index: opensm/osm_sa_mcmember_record.c

Re: [openib-general] [PATCH] osm: bug in OpenSM on broken fabrics

2006-09-18 Thread Hal Rosenstock
Hi Yevgeny, On Mon, 2006-09-18 at 14:52, Yevgeny Kliteynik wrote: Hi Hal This patch fixes a bug in opensm that was discovered on a 'broken' fabrics when opensm was executed with --stay_on_fatal. Replacing assert with a real check. Yevgeny Signed-off-by: Yevgeny Kliteynik [EMAIL

Re: [openib-general] [openfabrics-ewg] OpenSm on sles10 ppc64

2006-09-17 Thread Hal Rosenstock
Hi Moshe, On Sun, 2006-09-17 at 10:41, Moshe Kazir wrote: /etc/init.d/opensm start produce an error on my JS21 ppc64 SLES10 OFED 1.0 . What error ? Should ppc64 SLES10 OFED 1.0 work ? I don't think so. Anyone tried it ? OFED 1.0 OpenSM release notes say: * PPC support: No PPC QA was

Re: [openib-general] [PATCH] osm: PathRecord prefer 1K MTU for MT23108 devices

2006-09-15 Thread Hal Rosenstock
Hi Sasha, On Fri, 2006-09-15 at 18:17, Sasha Khapyorsky wrote: Hi Eitan, Some comments about the patch. On 14:45 Fri 15 Sep , Eitan Zahavi wrote: Hi Hal The following patch solves an issue with OpenSM preferring largest MTU for PathRecord/MultiPathRecord for paths going to or

Re: [openib-general] [PATCH] osm: PathRecord prefer 1K MTU for MT23108 devices

2006-09-15 Thread Hal Rosenstock
Hi Eitan, On Fri, 2006-09-15 at 07:45, Eitan Zahavi wrote: Hi Hal The following patch solves an issue with OpenSM preferring largest MTU for PathRecord/MultiPathRecord for paths going to or from MT23108 (Tavor) devices instead of using a 1K MTU which is best for this device. Since this

Re: [openib-general] [PATCH] osm: PathRecord prefer 1K MTU for MT23108 devices

2006-09-15 Thread Hal Rosenstock
On Fri, 2006-09-15 at 20:04, Sasha Khapyorsky wrote: [snip...] To summarize the functionality change: 1. Added enable_quirks option 2. If enable_quirks is FALSE do nothing I see those quirks are SA specific. Then should this option be called 'enable_sa_quirks' instead?

Re: [openib-general] [PATCH] IB/ipoib: use appropriate path selector

2006-09-14 Thread Hal Rosenstock
On Thu, 2006-09-14 at 00:46, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re: [PATCH] IB/ipoib: use appropriate path selector On Wed, 2006-09-13 at 18:08, Michael S. Tsirkin wrote: Quoting r. Roland Dreier [EMAIL PROTECTED]: Subject: Re: [PATCH] IB

Re: [openib-general] [PATCH] IB/ipoib: use appropriate path selector

2006-09-14 Thread Hal Rosenstock
On Thu, 2006-09-14 at 07:14, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re: [PATCH] IB/ipoib: use appropriate path selector On Thu, 2006-09-14 at 00:46, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re: [PATCH

[openib-general] [PATCH] OpenSM: Change default temp directory location for non Windows platforms

2006-09-14 Thread Hal Rosenstock
OpenSM: Change default temp directory location for non Windows platforms This patch is intended for both trunk and 1.1. Signed-off-by: Hal Rosenstock [EMAIL PROTECTED] Index: ../osm/include/opensm/osm_base.h === --- ../osm/include

Re: [openib-general] [PATCH] OpenSM: Change default temp directory location for non Windows platforms

2006-09-14 Thread Hal Rosenstock
Hi Eitan, On Thu, 2006-09-14 at 09:39, Eitan Zahavi wrote: Hi Hal, Looks simple enough to get into the OFED 1.1 I assume you are going to commit it into the branch? Done (r9484). -- Hal EZ Hal Rosenstock wrote: OpenSM: Change default temp directory location for non Windows

Re: [openib-general] Fwd: IPoIB Multicast

2006-09-14 Thread Hal Rosenstock
On Thu, 2006-09-14 at 17:11, Roland Dreier wrote: Hmm, IPoIB does not seem to copy anything except the pkey. Looks like a compliance issue. Well, the only MUST attributes we seem to be missing are HopLimit and MTU (P_Key, Q_Key and SL are all copied from the broadcast group when

Re: [openib-general] Fwd: IPoIB Multicast

2006-09-14 Thread Hal Rosenstock
On Thu, 2006-09-14 at 20:35, Roland Dreier wrote: Hal Are you referring to create or join here ? The whole thing is about new groups that IPoIB creates. Currently we don't specify HopLimit, MTU or Rate, and the IPoIB RFC says we should. That indeed is true for create. However, send only

Re: [openib-general] Fwd: IPoIB Multicast

2006-09-14 Thread Hal Rosenstock
On Thu, 2006-09-14 at 20:52, Roland Dreier wrote: Hal That indeed is true for create. However, send only members Hal can never create a group (only full members can do this). Am Hal I confusing this with a different patch which went by ? Yes, I think so. Look back to the

[openib-general] [PATCH] libibmad: Add sa_rpc_call API

2006-09-13 Thread Hal Rosenstock
libibmad: Add sa_rpc_call API Signed-off-by: Hal Rosenstock [EMAIL PROTECTED] Index: libibmad/include/infiniband/mad.h === --- libibmad/include/infiniband/mad.h (revision 9425) +++ libibmad/include/infiniband/mad.h (working copy

Re: [openib-general] ibis

2006-09-13 Thread Hal Rosenstock
Hi, On Wed, 2006-09-13 at 09:05, john t wrote: Hi, In OFED there are commands like ibis, ibdmsh and ibmssh I'm adding Eitan who is the maintainer for these tools. all of these provide a shell prompt and allow some operations like new_IBFabric, delete_IBFabric etc. What are these

Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limit MTU to 1K

2006-09-13 Thread Hal Rosenstock
On Wed, 2006-09-13 at 11:57, Michael S. Tsirkin wrote: Tavor systems get better performance with 1K MTU. Since there does not seem to be any way to find out whether the remote system uses Tavor, add an option to limit the MTU globally. Can't Tavor be determined locally ? And couldn't the

Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K

2006-09-13 Thread Hal Rosenstock
On Wed, 2006-09-13 at 12:22, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K On Wed, 2006-09-13 at 11:57, Michael S. Tsirkin wrote: Tavor systems get better performance with 1K

Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K

2006-09-13 Thread Hal Rosenstock
Hi Fab, On Wed, 2006-09-13 at 12:39, Fabian Tillier wrote: On 9/13/06, Michael S. Tsirkin [EMAIL PROTECTED] wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K On Wed, 2006-09-13 at 11:57, Michael S

Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K

2006-09-13 Thread Hal Rosenstock
Hi Fab, On Wed, 2006-09-13 at 13:23, Fabian Tillier wrote: Hi Hal, On 13 Sep 2006 12:35:58 -0400, Hal Rosenstock [EMAIL PROTECTED] wrote: Hi Fab, On Wed, 2006-09-13 at 12:39, Fabian Tillier wrote: On 9/13/06, Michael S. Tsirkin [EMAIL PROTECTED] wrote: Quoting r. Hal Rosenstock

Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K

2006-09-13 Thread Hal Rosenstock
On Wed, 2006-09-13 at 15:05, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: If I understood Michael's comment properly, this will have the side effect that IPoIB won't work since IPoIB requires 2K MTUs. As far as I know, there is no way to specify whether a path

Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K

2006-09-13 Thread Hal Rosenstock
On Wed, 2006-09-13 at 15:03, Michael S. Tsirkin wrote: Quoting r. Fabian Tillier [EMAIL PROTECTED]: Subject: Re: [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K On 9/13/06, Michael S. Tsirkin [EMAIL PROTECTED] wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re

Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K

2006-09-13 Thread Hal Rosenstock
On Wed, 2006-09-13 at 15:13, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re: [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K On Wed, 2006-09-13 at 12:22, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re

Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K

2006-09-13 Thread Hal Rosenstock
On Wed, 2006-09-13 at 17:17, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: If path MTU selector in path query allows MTU 1K (e.g. best MTU) and one of the sides is Tavor, select the best MTU that is 1K and not the largest possible. How would it be identified

Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K

2006-09-13 Thread Hal Rosenstock
On Wed, 2006-09-13 at 17:09, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re: [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K On Wed, 2006-09-13 at 15:05, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: If I understood

Re: [openib-general] [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K

2006-09-13 Thread Hal Rosenstock
On Wed, 2006-09-13 at 17:49, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re: [PATCH for-2.6.18] IB/cma: option to limitMTU to 1K On Wed, 2006-09-13 at 17:09, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re

Re: [openib-general] [PATCH] IB/ipoib: use appropriate path selector

2006-09-13 Thread Hal Rosenstock
On Wed, 2006-09-13 at 18:08, Michael S. Tsirkin wrote: Quoting r. Roland Dreier [EMAIL PROTECTED]: Subject: Re: [PATCH] IB/ipoib: use appropriate path selector Michael IPoIB in linux needs 2K MTU. Therefore it must set mtu Michael selector in path record query accordingly.

Re: [openib-general] [PATCH] IB/ipoib: use appropriate path selector

2006-09-13 Thread Hal Rosenstock
On Wed, 2006-09-13 at 19:07, Roland Dreier wrote: + path-pathrec.mtu= priv-broadcast-mcmember.mtu; + path-pathrec.mtu_selector = IB_SA_GTE; Does this do anything without setting the component mask of the actual request?? As you imply (if you are asking for

[openib-general] [PATCH][TRIVIAL] OpenSM: Eliminate unused max_port_profile parameter

2006-09-12 Thread Hal Rosenstock
OpenSM: Eliminate unused max_port_profile parameter in OpenSM subnet options structure Signed-off-by: Hal Rosenstock [EMAIL PROTECTED] Index: include/opensm/osm_subnet.h === --- include/opensm/osm_subnet.h (revision 9424

Re: [openib-general] OpenSM Multiple HCA cards on the same host

2006-09-12 Thread Hal Rosenstock
Hi Michael, On Tue, 2006-09-12 at 07:20, Michael Arndt wrote: Hi, in the osm/docs Which doc ? BTW, what version of OpenSM are you using ? is mentioned that at the next release multiple HCA cards on the same host will be supported. If I understand your question correctly, OpenIB OpenSM

Re: [openib-general] RDMA question

2006-09-12 Thread Hal Rosenstock
Hi Makia, On Tue, 2006-09-12 at 09:56, Makia Minich wrote: I'm looking for some information on whether or not you can set a service level for RDMA packets What API or ULP are you planning on using ? Sounds like you are planning on using verbs directly. Is this userspace or kernel ? (as a

Re: [openib-general] [PATCH] osm: OSM bug fix with --run-once option

2006-09-11 Thread Hal Rosenstock
Hi Yevgeny, On Sun, 2006-09-10 at 02:35, Yevgeny Kliteynik wrote: Hi Hal This patch fixes the bug that was occurring when OSM was running with --run-once option (-o) and the SM port was down. In that case, OSM would be stuck in cond_wait forever (or until the port will become active), and

Re: [openib-general] [PATCH] osm: OSM bug fix with --run-once option

2006-09-10 Thread Hal Rosenstock
Hi Yevgeny, On Sun, 2006-09-10 at 02:35, Yevgeny Kliteynik wrote: Hi Hal This patch fixes the bug that was occurring when OSM was running with --run-once option (-o) and the SM port was down. In that case, OSM would be stuck in cond_wait forever (or until the port will become active), and

[openib-general] OpenSM/osm_log API: Use symbol versions rather than polluting namespace

2006-09-06 Thread Hal Rosenstock
PROTECTED] Signed-off-by: Hal Rosenstock [EMAIL PROTECTED] Index: osm/opensm/libopensm.map === --- osm/opensm/libopensm.map(revision 9253) +++ osm/opensm/libopensm.map(working copy) @@ -3,7 +3,6 @@ OPENSM_1.3

Re: [openib-general] OpenSM/osm_log API: Use symbol versions rather than polluting namespace

2006-09-06 Thread Hal Rosenstock
On Wed, 2006-09-06 at 09:42, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: OpenSM/osm_log API: Use symbol versions rather than polluting namespace OpenSM/osm_log API: Rather than polluting the namespace with needless symbols, use symbol versions

[openib-general] [PATCH] OpenSM/osm_base.h: Change OSM_DEFAULT_TMP_DIR to /var/log for Linux

2006-09-06 Thread Hal Rosenstock
OpenSM/osm_base.h: Change OSM_DEFAULT_TMP_DIR to /var/log for Linux Signed-off-by: Hal Rosenstock [EMAIL PROTECTED] Index: include/opensm/osm_base.h === --- include/opensm/osm_base.h (revision 9158) +++ include/opensm/osm_base.h

Re: [openib-general] OpenSM/osm_log API: Use symbol versions rather than polluting namespace

2006-09-06 Thread Hal Rosenstock
On Wed, 2006-09-06 at 11:16, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: Subject: Re: OpenSM/osm_log API: Use symbol versions rather than polluting namespace On Wed, 2006-09-06 at 09:42, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED

Re: [openib-general] OpenSM/osm_log API: Use symbol versions rather than polluting namespace

2006-09-06 Thread Hal Rosenstock
On Wed, 2006-09-06 at 11:27, Michael S. Tsirkin wrote: Quoting r. Doug Ledford [EMAIL PROTECTED]: Subject: Re: OpenSM/osm_log API: Use symbol versions rather than polluting namespace On Wed, 2006-09-06 at 10:14 -0400, Hal Rosenstock wrote: On Wed, 2006-09-06 at 09:42, Michael S

Re: [openib-general] OpenSM/osm_log API: Use symbol versions rather than polluting namespace

2006-09-06 Thread Hal Rosenstock
On Wed, 2006-09-06 at 12:10, Michael S. Tsirkin wrote: Quoting r. Hal Rosenstock [EMAIL PROTECTED]: (which, as I understand it, is really only an issue because opensm can log so much), which is what this entire patch series was designed to address. They are two different problem

Re: [openib-general] OpenSM - guid2lid cache file questions

2006-09-05 Thread Hal Rosenstock
Hi Leonid, On Tue, 2006-09-05 at 03:30, Leonid Arsh wrote: Hi list, I have a question regarding the guid2lid cache file. The file is read by OpenSM on the start up. OpenSM may reassign LIDs according to the LIDs saved in this file. It isn't always acceptable. Is it a right

Re: [openib-general] MPI Brodcast doubt

2006-09-05 Thread Hal Rosenstock
John, On Mon, 2006-09-04 at 08:56, john t wrote: Hi, I have 3 nodes connected via IB as shown below: node1 --- switch1 --- node2 |-- node3 If node1 sends a brodcast message to node2 and node3, I want to know if the message is delivered to the switch

Re: [openib-general] OpenSM - guid2lid cache file questions

2006-09-05 Thread Hal Rosenstock
it without touching the file. Not currently. There is the -x (--honor_guid2lid) which will do this (ignore the guid2lid file) when OpenSM is coming out of STANDBY though. -- Hal Thanks, Leonid On 05 Sep 2006 06:57:53 -0400, Hal Rosenstock [EMAIL PROTECTED] wrote: Hi Leonid, On Tue, 2006

Re: [openib-general] OpenSM - guid2lid cache file questions

2006-09-05 Thread Hal Rosenstock
Leonid, On Tue, 2006-09-05 at 09:13, Leonid Arsh wrote: Thanks, On 05 Sep 2006 08:46:22 -0400, Hal Rosenstock [EMAIL PROTECTED] wrote: I have a problem when OpenSM, being started, reads an out-if-date guid2lid file. OpenSM changes LIDs in this case. How do you know the file

Re: [openib-general] [PATCH] opensm: osm_log_init_v2() - new osm_log initializer

2006-09-05 Thread Hal Rosenstock
On Mon, 2006-09-04 at 13:20, Sasha Khapyorsky wrote: There is new osm_log initializer osm_log_init_v2(), this is wrapped by osm_log_init() in order to preserve existing API. Signed-off-by: Sasha Khapyorsky [EMAIL PROTECTED] Thanks. Applied (to trunk and 1.1). -- Hal

Re: [openib-general] libibcm can't connect/talk to libicm on other machine.

2006-09-05 Thread Hal Rosenstock
Hi Bub, On Tue, 2006-09-05 at 10:22, Bub Thomas wrote: I’m still in the process of migrating my gen1 application to gen2. Actually I CAN connect a gen2 application to a gen2 listener application on the same machine but NOT to a gen 2 listener on another machine. Any hints where to look

Re: [openib-general] Utilities for sending traffic with different SL

2006-08-30 Thread Hal Rosenstock
Hi Suri, On Thu, 2006-08-24 at 16:42, Suresh Shelvapille wrote: Folks: Is there a utility within the OFED1.0 package which can be used for generating traffic on different SL (akin to the Voltaire perf_main utility)? With OpenSM, you can configure an SL per IPoIB based partition and use any

Re: [openib-general] [PATCH] opensm: libibmad: rpc API which supports more than one ports.

2006-08-30 Thread Hal Rosenstock
Hi Sasha, On Tue, 2006-08-29 at 21:29, Sasha Khapyorsky wrote: Hi Hal, On 20:09 Tue 29 Aug , Hal Rosenstock wrote: Hi Sasha, On Fri, 2006-08-25 at 09:17, Sasha Khapyorsky wrote: This provides RPC like API which may work with several ports. I think you mean can work rather

Re: [openib-general] [PATCH] opensm: libibmad: rpc API which supports more than one ports.

2006-08-30 Thread Hal Rosenstock
On Fri, 2006-08-25 at 09:17, Sasha Khapyorsky wrote: This provides RPC like API which may work with several ports. Signed-off-by: Sasha Khapyorsky [EMAIL PROTECTED] --- libibmad/include/infiniband/mad.h |9 +++ libibmad/src/libibmad.map |4 + libibmad/src/register.c

[openib-general] [PATCH][TRIVIAL] libibmad/src/rpc.c: Validate num_classes

2006-08-30 Thread Hal Rosenstock
libibmad/src/rpc.c: Validate num_classes Signed-off-by: Hal Rosenstock [EMAIL PROTECTED] Index: libibmad/src/rpc.c === --- libibmad/src/rpc.c (revision 9192) +++ libibmad/src/rpc.c (working copy) @@ -306,6 +306,9 @@ madrpc_init

Re: [openib-general] [PATCH] libibsa: userspace SA query and multicast support

2006-08-29 Thread Hal Rosenstock
On Wed, 2006-08-23 at 17:26, Sean Hefty wrote: Roland Dreier wrote: What's the plan for how this would be used? We can't let unprivileged userspace processes talk to the SA, because they could cause problems like deleting someone else's multicast group membership. And I don't think we

Re: [openib-general] [PATCH] osm: fix memory leak in vendor ibumad

2006-08-29 Thread Hal Rosenstock
Hi Eitan, Hi Hal These are two trivial fixes for memory leaks in the ibumad vendor. Thanks Eitan Signed-off-by: Eitan Zahavi eitan at mellanox.co.il Thanks. Applied (to both trunk and 1.1). -- Hal ___ openib-general mailing list

Re: [openib-general] [PATCH] osm: OSM fails to bind - TRIVIAL message addition

2006-08-29 Thread Hal Rosenstock
On Sun, 2006-08-27 at 06:16, Yevgeny Kliteynik wrote: Hi Hal This patch just makes the error message more informative for user, since another instance of running SM is most probably the reason why osm_opensm_bind failed. Yevgeny Signed-off-by: Yevgeny Kliteynik [EMAIL PROTECTED]

Re: [openib-general] [PATCH] libibsa: userspace SA query and multicast support

2006-08-29 Thread Hal Rosenstock
On Tue, 2006-08-29 at 11:58, Sean Hefty wrote: Why SEND ? In general, couldn't it be used like SET/DELETE (in addition to being used like the GET method variants) ? Also, the SA doesn't use the SEND method. The latest version of the patch only allows GET or GET_TABLE for PathRecords

<    1   2   3   4   5   6   7   8   9   10   >