Re: [PATCH] infiniband-diags/saquery.c: switchinfo support added

2013-02-28 Thread Ira Weiny
]], query_guidinfo_records}, + {SwitchInfoRecord, SWIR, IB_SA_ATTR_SWITCHINFORECORD, + [lid], query_switchinfo_records}, {0} }; -- 1.7.1 -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8008 wei...@llnl.gov -- To unsubscribe from this list: send the line unsubscribe linux

Re: [PATCH] infiniband-diags/saquery.c: switchinfo support added

2013-02-21 Thread Ira Weiny
, 22, 1, NULL, + switchInfo_lid (SwitchInfoRecord) + LID of switch port 0}, As above no need for the additional option. Thanks, Ira {0} }; -- 1.7.1 -- Ira Weiny Member of Technical Staff Lawrence Livermore National Lab 925-423-8008 wei...@llnl.gov -- To unsubscribe from

Re: [PATCH] infiniband-diags/saquery.c: switchinfo support added

2013-02-21 Thread Ira Weiny
On Thu, 21 Feb 2013 10:09:00 -0800 Ira Weiny wei...@llnl.gov wrote: [snip] + ib_net16_t swir_lid; We don't need this field or lid option. Other records simply have an optional parameter to specify the lid. The usage should be: SwitchRecord [lid] Sorry I meant SwitchInfoRecord

Re: [PATCH] infiniband-diags/saquery.c: switchinfo support added

2013-02-24 Thread Ira Weiny
- From: Ira Weiny wei...@llnl.gov To: Ira Weiny wei...@llnl.gov Cc: Husam Kahalah hkaha...@asaltech.com, linux-r...@vger.kernel.org, linux-kernel@vger.kernel.org Date: Thu, 21 Feb 2013 10:33:44 -0800 Subject: Re: [PATCH] infiniband-diags/saquery.c: switchinfo support added On Thu, 21 Feb 2013 10

[PATCH RFC] IB/mad: remove obsolete snoop interface

2015-09-30 Thread ira . weiny
From: Ira Weiny <ira.we...@intel.com> This interface has no current users and is obsolete. Signed-off-by: Ira Weiny <ira.we...@intel.com> --- This has undergone a medium level of testing. I have run it against mlx4, qib, and OPA hardware and it does not seem to cause any regression

Re: [PATCH 0/7] IB/hfi1: Remove write() and use ioctl() for user access

2016-04-15 Thread Ira Weiny
On Fri, Apr 15, 2016 at 07:01:26AM +0300, Leon Romanovsky wrote: > On Thu, Apr 14, 2016 at 01:48:31PM -0400, Ira Weiny wrote: > > On Thu, Apr 14, 2016 at 10:45:50AM -0600, Jason Gunthorpe wrote: > > > On Thu, Apr 14, 2016 at 08:41:35AM -0700, Dennis Dalessandro wrote: > &g

Re: [PATCH 0/7] IB/hfi1: Remove write() and use ioctl() for user access

2016-04-15 Thread Ira Weiny
On Sat, Apr 16, 2016 at 12:23:28AM +0300, Leon Romanovsky wrote: > > Intel as usual decided to do it in their way and the result is presented > on this mailing list. Excuse me, but this statement is completely unfair. We were specifically asked by Al and Linus to fix our char device with

Re: [PATCH 0/7] IB/hfi1: Remove write() and use ioctl() for user access

2016-04-18 Thread Ira Weiny
On Mon, Apr 18, 2016 at 11:24:11AM -0700, Christoph Hellwig wrote: > On Mon, Apr 18, 2016 at 11:40:47AM -0600, Jason Gunthorpe wrote: > > I wasn't arguing this should integrate into verbs in some way, only > > that the way to access the driver-specific uAPI of a RDMA device should > > be through

Re: [PATCH 0/7] IB/hfi1: Remove write() and use ioctl() for user access

2016-04-14 Thread Ira Weiny
On Thu, Apr 14, 2016 at 10:45:50AM -0600, Jason Gunthorpe wrote: > On Thu, Apr 14, 2016 at 08:41:35AM -0700, Dennis Dalessandro wrote: > > This patch series removes the write() interface for user access in favor of > > an > > ioctl() based approach. This is in response to the complaint that we

Re: [PATCH] staging/rdma/hfi1: select CRC32

2016-04-14 Thread Ira Weiny
ce to `crc32_le' > > Signed-off-by: Markus Böhme <markus.boe...@mailbox.org> Doug did you pick this up? Reviewed-by: Ira Weiny <ira.we...@intel.com> > --- > drivers/staging/rdma/hfi1/Kconfig |1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/sta