Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-15 Thread Jason Gunthorpe
On Thu, Mar 15, 2012 at 05:27:12PM -0700, Jim Foraker wrote: > for zone work. My impression is that most IB admins want to be able to > run diagnostics from anywhere on the fabric, which implies much wider > deployment. For the most part, if the keyfile ends up on every host, > it's not all that

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-15 Thread Jim Foraker
On Tue, 2012-03-13 at 09:50 -0700, Jason Gunthorpe wrote: > On Fri, Mar 09, 2012 at 05:09:07PM -0800, Jim Foraker wrote: > > > > I don't think using passwords for this sort of thing is really a good > > > idea. It has been shown again and again the extrapolating keys from > > > passwords does no

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-14 Thread Hal Rosenstock
On 3/13/2012 12:35 PM, Jason Gunthorpe wrote: > On Tue, Mar 13, 2012 at 08:31:50AM -0400, Hal Rosenstock wrote: >> On 3/9/2012 1:04 PM, Jason Gunthorpe wrote: >>> On Fri, Mar 09, 2012 at 07:59:58AM -0500, Hal Rosenstock wrote: >>> What mkey model is being proposed here ? It looks to me like it

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-13 Thread Jason Gunthorpe
On Fri, Mar 09, 2012 at 05:09:07PM -0800, Jim Foraker wrote: > > I don't think using passwords for this sort of thing is really a good > > idea. It has been shown again and again the extrapolating keys from > > passwords does not stand up. Better to design around a large true > > random key (eg ho

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-13 Thread Jason Gunthorpe
On Tue, Mar 13, 2012 at 08:31:50AM -0400, Hal Rosenstock wrote: > On 3/9/2012 1:04 PM, Jason Gunthorpe wrote: > > On Fri, Mar 09, 2012 at 07:59:58AM -0500, Hal Rosenstock wrote: > > > >> What mkey model is being proposed here ? It looks to me like it is a > >> single mkey for all ports in the subn

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-13 Thread Hal Rosenstock
On 3/9/2012 1:04 PM, Jason Gunthorpe wrote: > On Fri, Mar 09, 2012 at 07:59:58AM -0500, Hal Rosenstock wrote: > >> What mkey model is being proposed here ? It looks to me like it is a >> single mkey for all ports in the subnet which is the simplest but least >> flexible model. If so, I think we ne

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-13 Thread Hal Rosenstock
On 3/9/2012 12:09 PM, Hefty, Sean wrote: >> What mkey model is being proposed here ? It looks to me like it is a >> single mkey for all ports in the subnet which is the simplest but least >> flexible model. If so, I think we need something more flexible as IBA >> allows each port to have it's own d

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-09 Thread Jim Foraker
On Fri, 2012-03-09 at 13:01 -0800, Jason Gunthorpe wrote: > On Fri, Mar 09, 2012 at 12:32:55PM -0800, Jim Foraker wrote: > > > > On Fri, 2012-03-09 at 10:04 -0800, Jason Gunthorpe wrote: > > > On Fri, Mar 09, 2012 at 07:59:58AM -0500, Hal Rosenstock wrote: > > > > > > > What mkey model is being

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-09 Thread Jason Gunthorpe
On Fri, Mar 09, 2012 at 12:32:55PM -0800, Jim Foraker wrote: > > On Fri, 2012-03-09 at 10:04 -0800, Jason Gunthorpe wrote: > > On Fri, Mar 09, 2012 at 07:59:58AM -0500, Hal Rosenstock wrote: > > > > > What mkey model is being proposed here ? It looks to me like it is a > > > single mkey for all p

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-09 Thread Jim Foraker
On Fri, 2012-03-09 at 10:04 -0800, Jason Gunthorpe wrote: > On Fri, Mar 09, 2012 at 07:59:58AM -0500, Hal Rosenstock wrote: > > > What mkey model is being proposed here ? It looks to me like it is a > > single mkey for all ports in the subnet which is the simplest but least > > flexible model. If

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-09 Thread Ira Weiny
On Fri, 9 Mar 2012 11:04:59 -0700 Jason Gunthorpe wrote: > On Fri, Mar 09, 2012 at 07:59:58AM -0500, Hal Rosenstock wrote: > > > What mkey model is being proposed here ? It looks to me like it is a > > single mkey for all ports in the subnet which is the simplest but least > > flexible model. If

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-09 Thread Jason Gunthorpe
On Fri, Mar 09, 2012 at 07:59:58AM -0500, Hal Rosenstock wrote: > What mkey model is being proposed here ? It looks to me like it is a > single mkey for all ports in the subnet which is the simplest but least > flexible model. If so, I think we need something more flexible as IBA > allows each por

RE: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-09 Thread Hefty, Sean
> What mkey model is being proposed here ? It looks to me like it is a > single mkey for all ports in the subnet which is the simplest but least > flexible model. If so, I think we need something more flexible as IBA > allows each port to have it's own different mkey. Is something more needed than

Re: [PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-09 Thread Hal Rosenstock
On 3/6/2012 5:12 PM, Jim Foraker wrote: What mkey model is being proposed here ? It looks to me like it is a single mkey for all ports in the subnet which is the simplest but least flexible model. If so, I think we need something more flexible as IBA allows each port to have it's own different mke

[PATCH fixed] libibmad: Add MKey support to SMP requests via smp_mkey_get/set()

2012-03-06 Thread Jim Foraker
Signed-off-by: Jim Foraker --- include/infiniband/mad.h |2 ++ src/libibmad.map |2 ++ src/mad_internal.h |1 + src/smp.c| 12 4 files changed, 17 insertions(+), 0 deletions(-) diff --git a/include/infiniband/mad.h b/include/infiniband/ma