Re: [PATCH v2] ibsim: allocate mft according to number of switch ports

2011-01-04 Thread Eli Dorfman (Voltaire)
Sasha, It seems that you've missed this patch. Please apply. Thanks, Eli Eli Dorfman (Voltaire) wrote: allocate mft according to number of switch ports calculate number of port masks according to number of switch ports and allocate MFT accordingly Signed-off-by: Eli Dorfman e

Re: ibnetdiscover issue

2010-12-15 Thread Eli Dorfman (Voltaire)
think that would make it go faster. I want to make sure we are using OFED as distributed from OFA. Hi Tom, You should take the following patches: commit 5b034c90d72f9fbf382df414a1569e0a9f7ac679 Author: Eli Dorfman (Voltaire) dorfman@gmail.com Date: Sun Sep 5 11:30:37 2010 +0300

[PATCH] ibnetdiscover: fix error when discovery started on node with more that one port

2010-10-04 Thread Eli Dorfman (Voltaire)
When discovery is started on a node with more than one active port it tries to send mads from second Hca port and fails. ibnetdiscover should not try and extend_path from Hca except for starting point. Signed-off-by: Eli Dorfman e...@voltaire.com ---

Re: [PATCH v2] ibsim: allocate mft according to number of switch ports

2010-09-15 Thread Eli Dorfman (Voltaire)
allocate mft according to number of switch ports calculate number of port masks according to number of switch ports and allocate MFT accordingly Signed-off-by: Eli Dorfman e...@voltaire.com --- ibsim/sim.h |5 ++--- ibsim/sim_mad.c |5 +++-- ibsim/sim_net.c |5 +++-- 3 files

[PATCH] ibsim: support MFT for switches with up to 48 ports

2010-09-13 Thread Eli Dorfman (Voltaire)
support MFT for switches with up to 48 ports Signed-off-by: Eli Dorfman e...@voltaire.com --- ibsim/sim.h |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ibsim/sim.h b/ibsim/sim.h index 5a8a92f..0de7fdc 100644 --- a/ibsim/sim.h +++ b/ibsim/sim.h @@ -45,8 +45,8 @@

[PATCH] ibsim: parse width and speed from ibnetdiscover output

2010-09-02 Thread Eli Dorfman (Voltaire)
Search for the pattern '[0-9]*x[SDQ]DR' Validate that width value is valid. Signed-off-by: Eli Dorfman e...@voltaire.com --- ibsim/sim_net.c | 48 1 files changed, 48 insertions(+), 0 deletions(-) diff --git a/ibsim/sim_net.c b/ibsim/sim_net.c

Re: [PATCH] ibnetdiscover: add '-f' flag to show full information (ports' speed and width).

2010-08-31 Thread Eli Dorfman (Voltaire)
Sasha Khapyorsky wrote: On 09:45 Tue 24 Aug , Hal Rosenstock wrote: What about the flag? do we still need it if we pass the output after the comment? I wouldn't think so. I also think we've made commentary changes to the ibnetdiscover output format like this before. If we wanted to be

Re: [PATCH] opensm: Fix sl2vl configuration

2010-08-26 Thread Eli Dorfman (Voltaire)
Hi Hal, Hal Rosenstock wrote: On Wed, Aug 25, 2010 at 4:43 AM, Eli Dorfman (Voltaire) dorfman@gmail.com wrote: Hal Rosenstock wrote: Hi Eli, On Mon, Aug 23, 2010 at 3:37 PM, Eli Dorfman dorfman@gmail.com wrote: Hi Hal, On Mon, Aug 23, 2010 at 5:25 PM, Hal Rosenstock hal.rosenst

Re: [PATCH] opensm: Fix sl2vl configuration

2010-08-25 Thread Eli Dorfman (Voltaire)
Hal Rosenstock wrote: Hi Eli, On Mon, Aug 23, 2010 at 3:37 PM, Eli Dorfman dorfman@gmail.com wrote: Hi Hal, On Mon, Aug 23, 2010 at 5:25 PM, Hal Rosenstock hal.rosenst...@gmail.com wrote: On Wed, Jul 28, 2010 at 12:26 PM, Eli Dorfman (Voltaire) dorfman@gmail.com wrote: Subject

Re: [PATCH] opensm: Added option for IPv4 MGID multiplexing.

2010-08-05 Thread Eli Dorfman (Voltaire)
This patch implements IPv4 MGID to MLID mapping in the same scheme as multicast IP to MAC mapping. Users may choose multicast IPs to use the same MLID or not. Slava Strebkov wrote: When option is enabled, same mlid may be assigned to multicast groups created by IPoIB IPv4. If there is no

Re: [PATCH v2] opensm: Fix sl2vl configuration

2010-08-01 Thread Eli Dorfman (Voltaire)
Fix the patch according to Jim's comments Changed i,j to in,out to make code readable For non-optimal sl2vl configuration in and out ports were reversed. For optimal sl2vl added override of default ALL settting with port's sl2vl when operational VL was other than the default port. Signed-off-by:

[PATCH] opensm: Fix sl2vl configuration

2010-07-28 Thread Eli Dorfman (Voltaire)
Subject: [PATCH] Fix sl2vl configuration For non-optimized sl2vl configuration in and out ports were reversed. For optimal sl2vl added override of default ALL settting with port's sl2vl when operational VL was other than the default port. Signed-off-by: Eli Dorfman e...@voltaire.com ---

[PATCH] opensm: Modify OSM_LOG_SYS messages

2010-06-29 Thread Eli Dorfman (Voltaire)
Modify OSM_LOG_SYS messages Signed-off-by: Eli Dorfman e...@voltaire.com --- opensm/opensm/osm_node_info_rcv.c |7 ++- opensm/opensm/osm_sa.c|2 +- opensm/opensm/osm_ucast_file.c|4 ++-- 3 files changed, 5 insertions(+), 8 deletions(-) diff --git

[PATCH] opensm: Fix wrong messages in MC delete flow

2010-06-20 Thread Eli Dorfman (Voltaire)
Fix wrong messages in MC delete and update flows. The requester GID was wrong. Signed-off-by: Eli Dorfman e...@voltaire.com --- opensm/opensm/osm_sa_mcmember_record.c | 11 --- 1 files changed, 4 insertions(+), 7 deletions(-) diff --git a/opensm/opensm/osm_sa_mcmember_record.c

Re: [PATCH v2] opensm: Modify connect_roots to allow connectivity between all switches

2010-06-15 Thread Eli Dorfman (Voltaire)
Yevgeny Kliteynik wrote: Eli, On 13-Jun-10 6:10 PM, Eli Dorfman (Voltaire) wrote: After a second thought and in order not to break current configuration, I send this modified patch that does not change connect_roots option but changes its functionality in up-down (I think that in fat-tree

Re: [PATCH v2] opensm: Modify connect_roots to allow connectivity between all switches

2010-06-15 Thread Eli Dorfman (Voltaire)
Yevgeny Kliteynik wrote: On 15-Jun-10 10:09 AM, Eli Dorfman (Voltaire) wrote: Yevgeny Kliteynik wrote: Eli, On 13-Jun-10 6:10 PM, Eli Dorfman (Voltaire) wrote: After a second thought and in order not to break current configuration, I send this modified patch that does not change

[PATCH] opensm: Allow connectivity between switches in up down and fat tree

2010-06-13 Thread Eli Dorfman (Voltaire)
Modify option connect_roots to connect_switches and allow connectivity between all switches (not only root nodes) in up down and fat tree routing algorithms Signed-off-by: Eli Dorfman e...@voltaire.com --- opensm/include/opensm/osm_subnet.h |6 +++--- opensm/man/opensm.8.in |

[PATCH v2] opensm: Modify connect_roots to allow connectivity between all switches

2010-06-13 Thread Eli Dorfman (Voltaire)
After a second thought and in order not to break current configuration, I send this modified patch that does not change connect_roots option but changes its functionality in up-down (I think that in fat-tree it is already implemented) Modify connect_roots option to allow connectivity between all

[PATCH] return no path when path does not exist

2010-06-10 Thread Eli Dorfman (Voltaire)
return OSM_NO_PATH (instead of port num) when path does not exists. this will also be reported as error in the log. Signed-off-by: Eli Dorfman e...@voltaire.com --- opensm/opensm/osm_switch.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/opensm/opensm/osm_switch.c

[PATCH] opensm: Always set enabled speed and width

2010-04-25 Thread Eli Dorfman (Voltaire)
Initialize default link speed and width to No State Change and always set in SubnSet(PortInfo) Signed-off-by: Eli Dorfman e...@voltaire.com --- infiniband-diags/src/ibportstate.c | 17 - 1 files changed, 8 insertions(+), 9 deletions(-) diff --git

Re: [PATCH v3] opensm: support routing engine update

2010-02-14 Thread Eli Dorfman (Voltaire)
Slava Strebkov wrote: HI! That was my misunderstanding - upon heavy sweep SM will try to load routing engines as defined in the SM.conf file (ftree updn - in that order). So ftree will be loaded when switch comes back from reboot. Slava -Original Message- From:

Re: [PATCH v2] opensm: bug in trap report for MC create(66) and delete(67) traps

2010-02-08 Thread Eli Dorfman (Voltaire)
Hal Rosenstock wrote: On Sun, Feb 7, 2010 at 4:47 AM, Eli Dorfman (Voltaire) dorfman@gmail.com wrote: Hal Rosenstock wrote: On Fri, Feb 5, 2010 at 9:18 AM, Eli Dorfman dorfman@gmail.com wrote: On Thu, Feb 4, 2010 at 10:52 PM, Hal Rosenstock hal.rosenst...@gmail.com wrote: On Thu

Re: [PATCH v2] opensm: bug in trap report for MC create(66) and delete(67) traps

2010-02-07 Thread Eli Dorfman (Voltaire)
Hal Rosenstock wrote: On Fri, Feb 5, 2010 at 9:18 AM, Eli Dorfman dorfman@gmail.com wrote: On Thu, Feb 4, 2010 at 10:52 PM, Hal Rosenstock hal.rosenst...@gmail.com wrote: On Thu, Feb 4, 2010 at 12:43 PM, Eli Dorfman (Voltaire) dorfman@gmail.com wrote: Subject: [PATCH] Wrong handling

[PATCH v2] opensm: bug in trap report for MC create(66) and delete(67) traps

2010-02-04 Thread Eli Dorfman (Voltaire)
Subject: [PATCH] Wrong handling of MC create and delete traps For these traps the GID in the data details is the MGID and not the source port gid. So the SM should check that subscriber port has the pkey of the MC group. There was also an error in comparing the subnet prefix and guid due to

opensm: suggestion to define new common port_groups file

2010-02-03 Thread Eli Dorfman (Voltaire)
Currently there is a duplication of port group definition in OpenSM configuration files. Using a single configuration file for port groups will make it easier to manage and will allow using group name as keyword in other configuration files (i.e. partitions.conf, qos-policy.conf and

[PATCH] opensm: bug in trap report for MC create(66) and delete(67) traps

2010-02-02 Thread Eli Dorfman (Voltaire)
In this case the GID in the data details is the MGID and not the source gid. So the SM gid should be taken as the source gid. There was also an error in comparing the subnet prefix and guid due to host/network order mismatch. Signed-off-by: Eli Dorfman e...@voltaire.com ---

[PATCH v2] opensm: Add update_desc command to opensm console

2010-01-13 Thread Eli Dorfman (Voltaire)
Sometimes nodes are discovered by the opensm with their default name before real host name was resolves and updated in the HCA. Since Mellanox ConnectX HCA do not support NodeDescription change trap (as part of trap 144) we need an option to refresh NodeDescription from console. Signed-off-by:

[PATCH] opensm: Add update_desc command to opensm console

2010-01-12 Thread Eli Dorfman (Voltaire)
Add update_desc command to opensm console Sometimes nodes are discovered by the opensm with their default name before real host name was resolves and updated in the HCA. Since Mellanox ConnectX HCA do not support NodeDescription change trap (as part of trap 144) we need an option to refresh

Re: [PATCH] opensm/osm_state_mgr.c: force heavy sweep when fabric consists of single switch

2009-11-11 Thread Eli Dorfman (Voltaire)
Yevgeny Kliteynik wrote: Eli Dorfman (Voltaire) wrote: Yevgeny Kliteynik wrote: Eli Dorfman (Voltaire) wrote: Yevgeny Kliteynik wrote: Eli Dorfman (Voltaire) wrote: Yevgeny Kliteynik wrote: Yevgeny Kliteynik wrote: Line Holen wrote: On 11/ 4/09 04:54 PM, Yevgeny Kliteynik wrote: Line

Re: [PATCH] opensm/osm_state_mgr.c: force heavy sweep when fabric consists of single switch

2009-11-09 Thread Eli Dorfman (Voltaire)
Yevgeny Kliteynik wrote: Eli Dorfman (Voltaire) wrote: Yevgeny Kliteynik wrote: Yevgeny Kliteynik wrote: Line Holen wrote: On 11/ 4/09 04:54 PM, Yevgeny Kliteynik wrote: Line Holen wrote: On 11/ 4/09 10:47 AM, Yevgeny Kliteynik wrote: Sasha Khapyorsky wrote: On 12:26 Tue 03 Nov

Re: [PATCH] opensm/osm_state_mgr.c: force heavy sweep when fabric consists of single switch

2009-11-08 Thread Eli Dorfman (Voltaire)
Yevgeny Kliteynik wrote: Yevgeny Kliteynik wrote: Line Holen wrote: On 11/ 4/09 04:54 PM, Yevgeny Kliteynik wrote: Line Holen wrote: On 11/ 4/09 10:47 AM, Yevgeny Kliteynik wrote: Sasha Khapyorsky wrote: On 12:26 Tue 03 Nov , Yevgeny Kliteynik wrote: Always do heavy sweep when there is

Re: [PATCH] infiniband-diags/saquery: Report SA MAD Class specific status.

2009-11-02 Thread Eli Dorfman (Voltaire)
Sasha Khapyorsky wrote: On 10:09 Sun 01 Nov , Eli Dorfman (Voltaire) wrote: Report SA MAD Class specific status. Fixes wrong error report for SA query status. I agree with patch, but one comment is below. Signed-off-by: Eli Dorfman e...@voltaire.com --- infiniband-diags/src

Re: [PATCH v2] infiniband-diags/saquery: Report SA MAD Class specific status.

2009-11-02 Thread Eli Dorfman (Voltaire)
Report SA MAD Class specific status. In addition to SM status. Signed-off-by: Eli Dorfman e...@voltaire.com --- infiniband-diags/src/saquery.c | 45 --- 1 files changed, 41 insertions(+), 4 deletions(-) diff --git a/infiniband-diags/src/saquery.c

[PATCH] infiniband-diags/saquery: Report SA MAD Class specific status.

2009-11-01 Thread Eli Dorfman (Voltaire)
Report SA MAD Class specific status. Fixes wrong error report for SA query status. Signed-off-by: Eli Dorfman e...@voltaire.com --- infiniband-diags/src/saquery.c | 41 --- 1 files changed, 37 insertions(+), 4 deletions(-) diff --git

hop weighted routing

2009-10-12 Thread Eli Dorfman (Voltaire)
Hi, I have tried to use hop weighted routing and noticed that it is supported only for minhop. Is there a reason that this was not implemented for up-down as well? Thanks, Eli -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to

Default force_link_speed value in opensm configuration

2009-10-06 Thread Eli Dorfman (Voltaire)
Hi, Is there a reason for setting default force_link_speed to 15 (set enabled as supported) and not to 0 (don't modify PortInfo:LinkSpeedEnabled )? Thanks, Eli -- To unsubscribe from this list: send the line unsubscribe linux-rdma in the body of a message to majord...@vger.kernel.org More

Re: [ofa-general] Re: [PATCH v2] infiniband-diags/scripts: Add 'ibcheckspeed' and 'ibcheckportspeed' to scripts

2009-09-15 Thread Eli Dorfman (Voltaire)
Hal Rosenstock wrote: On Mon, Sep 14, 2009 at 2:02 PM, Ira Weiny wei...@llnl.gov mailto:wei...@llnl.gov wrote: On Fri, 11 Sep 2009 09:32:39 +0530 Keshetti Mahesh keshetti.mah...@gmail.com mailto:keshetti.mah...@gmail.com wrote: My badness. I have not used

Re: [ofa-general] Re: [PATCH v2] infiniband-diags/scripts: Add 'ibcheckspeed' and 'ibcheckportspeed' to scripts

2009-09-15 Thread Eli Dorfman (Voltaire)
Keshetti Mahesh wrote: Also it would be useful to find all ports in Disable state. iblinkinfo already does this with '-d' option. It shows all the port that are in Down state - either cable disconnected or port disabled (by ibportstate). Eli -- To unsubscribe from this list: send the line