> Quoting Steve Wise <[EMAIL PROTECTED]>:
> Subject: problems with ofed-1.2.c chelsio low level driver
>
> Michael,
>
> I've discovered that I was really testing the ofed-1.2 cxgb3.ko module
> in my testing 1.2.c regression test. Upon getting a complete 1.2.c
> installed, I'm seeing crashes wi
> Quoting Tang, Changqing <[EMAIL PROTECTED]>:
> Subject: RE: RFCv2: SRC API
>
>
> >
> > > Is there any performance worry to let j2(the first job on a
> > node) to
> > > do all the "work" ?
> >
> > How do you mean?
>
> I mean that j2 has all the QP connections with all other remote jobs, so
>
> > Is there any performance worry to let j2(the first job on a
> node) to
> > do all the "work" ?
>
> How do you mean?
I mean that j2 has all the QP connections with all other remote jobs, so
j2 does not save anything compared with non-SRC case, can this affect
j2's performance ?
I have a
From: Andi Kleen <[EMAIL PROTECTED]>
Date: 08 Aug 2007 14:38:10 +0200
> Jeff Garzik <[EMAIL PROTECTED]> writes:
> > > + val, reg_index, addr, addr+4); */
> > > + writel(cpu_to_le32(reg_index), addr);
> > > + writel(cpu_to_le32(val),(u8 *)addr + 4);
> >
> > wrong -- endian conversi
> +static struct ib_mw *nes_alloc_mw(struct ib_pd *ibpd) {
> +get_random_bytes(&next_stag_index, sizeof(next_stag_index));
Could this use up a lot of entropy? Is random32() sufficient?
> +stag_key = (u8)next_stag_index;
I don't think this cast is needed.
> +if (ret) {
> +
Michael Buesch wrote:
On Wednesday 08 August 2007 18:59:08 Jeff Garzik wrote:
Michael Buesch wrote:
writel doesn't guarantee flushing either.
readl does.
Not quite -- there are multiple kinds of flushing. You're thinking
about flushing across PCI bridges, which is correct, but you also have
Hi Vlad,
Please pull recent ofed_1_2 branch of ~sashak/management.git.
In addition to already pulled:
commit 067fc2c8d91b405546d1d5629540880681c45df1
Author: Sasha Khapyorsky <[EMAIL PROTECTED]>
opensm: descrease default HoQLife value for leaf ports.
commit 2705d36f473ea80f571a91cde47e44d9
> Quoting Steve Wise <[EMAIL PROTECTED]>:
> Subject: problems with ofed-1.2.c chelsio low level driver
>
> Michael,
>
> I've discovered that I was really testing the ofed-1.2 cxgb3.ko module
> in my testing 1.2.c regression test. Upon getting a complete 1.2.c
> installed, I'm seeing crashes wi
Michael,
I've discovered that I was really testing the ofed-1.2 cxgb3.ko module
in my testing 1.2.c regression test. Upon getting a complete 1.2.c
installed, I'm seeing crashes with the chelsio driver. I think there
are some patches that went into the ofed-1.2 branch, that did not get
merge
Hi. I created most of the requested directory/owner pairs in
/var/www/openfabrics.org/downloads. I left out the various MPI
directories, figuring the appropriate web pages will be linked from
somewhere (possibly the downloads web page). I gave Stan Smith an
account. Stan, please contact me to get t
On Wednesday 08 August 2007 18:59:08 Jeff Garzik wrote:
> Michael Buesch wrote:
> > writel doesn't guarantee flushing either.
> > readl does.
>
>
> Not quite -- there are multiple kinds of flushing. You're thinking
> about flushing across PCI bridges, which is correct, but you also have
> CPU
> git.openfabrics.org/~glenn/libnes.git
> git.openfabrics.org/~glenn/ofed_1_2.git
> git.openfabrics.org/~glenn/ofascripts.git
> git.openfabrics.org/~glenn/ofed_1_2_scripts.git
these aren't actually git URLs. prepending git:// seems to work though.
_
> +nesupd->udoorbell = mmap(NULL, page_size, PROT_WRITE | PROT_READ,
> MAP_SHARED,
> +context->cmd_fd, nesupd->db_index * page_size);
> +
> +if (((void *)-1) == nesupd->udoorbell) {
probably cleaner to test against MAP_FAILED here.
> +asm __v
> +ibv_driver_init;
This is unnecessary -- I realize you copied it from libmthca or
libmlx4 probably, but there it is a relic of older code.
I'll delete it from the libmthca and libmlx4 linker script to avoid
confusing anyone else.
___
ewg
Michael Buesch wrote:
writel doesn't guarantee flushing either.
readl does.
Not quite -- there are multiple kinds of flushing. You're thinking
about flushing across PCI bridges, which is correct, but you also have
CPU write posting and CPU write ordering and such.
Without taking all that
On Wed, Aug 08, 2007 at 09:46:16AM -0700, Roland Dreier wrote:
>
> Not mmiowb() -- that is for ordering between CPUs, eg on systems like
> Altix where PCI transactions might get reordered in the system fabric
> before reaching the PCI bus.
>
Yes, that's right. This is a continual source of
> The barrier/ordering issue however might be a critical thing,
> when using __raw_XXX. So one must always mmiowb() after such a write.
Not mmiowb() -- that is for ordering between CPUs, eg on systems like
Altix where PCI transactions might get reordered in the system fabric
before reaching the
On Wednesday 08 August 2007 18:33:24 Jeff Garzik wrote:
> Michael Buesch wrote:
> > On Wednesday 08 August 2007 18:18:31 Roland Dreier wrote:
> >> > But there are indeed a few cases that look wrong.
> >>
> >> yes...
> >>
> >> > arch/x86_64/kernel/pci-calgary.c: writel(cpu_to_be32(val),
> >
> Most likely the __raw_writel variant is portable, but I am not
> sure. Anybody sure?
Yes, it should be fine. I use that construct in a few IB drivers.
- R.
___
ewg mailing list
ewg@lists.openfabrics.org
http://lists.openfabrics.org/cgi-bin/mailman
Michael Buesch wrote:
On Wednesday 08 August 2007 18:18:31 Roland Dreier wrote:
> But there are indeed a few cases that look wrong.
yes...
> arch/x86_64/kernel/pci-calgary.c: writel(cpu_to_be32(val), target);
eg this almost certainly wants to be
writel(swab32(val), target);
On Wednesday 08 August 2007 18:18:31 Roland Dreier wrote:
> > But there are indeed a few cases that look wrong.
>
> yes...
>
> > arch/x86_64/kernel/pci-calgary.c: writel(cpu_to_be32(val), target);
>
> eg this almost certainly wants to be
>
> writel(swab32(val), target);
>
> or so
Roland Dreier wrote:
> But there are indeed a few cases that look wrong.
yes...
> arch/x86_64/kernel/pci-calgary.c: writel(cpu_to_be32(val), target);
eg this almost certainly wants to be
writel(swab32(val), target);
or something equivalent like
__raw_writel(cpu_to_be
Andi Kleen wrote:
Jeff Garzik <[EMAIL PROTECTED]> writes:
+ val, reg_index, addr, addr+4); */
+ writel(cpu_to_le32(reg_index), addr);
+ writel(cpu_to_le32(val),(u8 *)addr + 4);
wrong -- endian conversion macros not needed with writel()
Are you sure?
Yes.
r
> But there are indeed a few cases that look wrong.
yes...
> arch/x86_64/kernel/pci-calgary.c: writel(cpu_to_be32(val), target);
eg this almost certainly wants to be
writel(swab32(val), target);
or something equivalent like
__raw_writel(cpu_to_be32(val), target);
This code is far from a TCP stack. It's main purpose is to exchange
RDMA MPA request/response messages that are required by the iWarp specs
and therefore needed by our hardware. All RNIC hardware vendors need
this MPA message exchange, including those already accepted into
kernel.org. Do you hav
Erez Zilber wrote:
Tziporet,
Can you add the following as a known issue:
"open-iscsi over TCP does not support usage of data digest on RedHat EL/AS 4
up3,
up4 & up5."
Note that this comment should be added to OFED release notes, not to
iSER release notes. This is not an iSER bug (it's a bug
Tziporet,
Can we make the change to 1.2.5? This should probably include:
- change builds/connectx to builds/ofed-1.2.5 (or just add a link)
- change build names from 1.2.c to 1.2.5
Scott Weitzenkamp (sweitzen) wrote:
Cisco has been testing 1.2.c-10 IPoIB/SDP/MPI successfully on a 32-node
Jeff,
Thanks for the input. I'll take your suggestions into account for the
patch v2 posting.
Glenn.
-Original Message-
From: Jeff Garzik [mailto:[EMAIL PROTECTED]
Sent: Tuesday, August 07, 2007 8:58 PM
To: Glenn Grundstrom
Cc: [EMAIL PROTECTED]; ewg@lists.openfabrics.org; [EMAIL PROTE
On Wednesday 08 August 2007 15:48:25 Michael Buesch wrote:
> > However if that's true then there are quite some drivers wrong:
> >
> > % grep -r write[bwl]\(cpu_to * | wc -l
> > 57
>
> Yeah, seems so.
Most of them seem to be
__raw_writew(cpu_toXX(...
which I think might be valid.
But there a
On Wednesday 08 August 2007 15:38:50 Andi Kleen wrote:
> On Wed, Aug 08, 2007 at 03:28:33PM +0200, Michael Buesch wrote:
> > On Wednesday 08 August 2007 15:08:28 Andi Kleen wrote:
> > > On Wed, Aug 08, 2007 at 03:02:35PM +0200, Michael Buesch wrote:
> > > > On Wednesday 08 August 2007 14:55:11 Andi
Tziporet,
Can you add the following as a known issue:
"open-iscsi over TCP does not support usage of data digest on RedHat EL/AS 4
up3,
up4 & up5."
Note that this comment should be added to OFED release notes, not to
iSER release notes. This is not an iSER bug (it's a bug in the backport
for
On Wed, Aug 08, 2007 at 03:28:33PM +0200, Michael Buesch wrote:
> On Wednesday 08 August 2007 15:08:28 Andi Kleen wrote:
> > On Wed, Aug 08, 2007 at 03:02:35PM +0200, Michael Buesch wrote:
> > > On Wednesday 08 August 2007 14:55:11 Andi Kleen wrote:
> > > > On Wed, Aug 08, 2007 at 01:50:35PM +0200,
On Wednesday 08 August 2007 15:08:28 Andi Kleen wrote:
> On Wed, Aug 08, 2007 at 03:02:35PM +0200, Michael Buesch wrote:
> > On Wednesday 08 August 2007 14:55:11 Andi Kleen wrote:
> > > On Wed, Aug 08, 2007 at 01:50:35PM +0200, Michael Buesch wrote:
> > > > On Wednesday 08 August 2007 14:38:10 Andi
On Wed, Aug 08, 2007 at 03:02:35PM +0200, Michael Buesch wrote:
> On Wednesday 08 August 2007 14:55:11 Andi Kleen wrote:
> > On Wed, Aug 08, 2007 at 01:50:35PM +0200, Michael Buesch wrote:
> > > On Wednesday 08 August 2007 14:38:10 Andi Kleen wrote:
> > > > Jeff Garzik <[EMAIL PROTECTED]> writes:
>
On Wednesday 08 August 2007 14:55:11 Andi Kleen wrote:
> On Wed, Aug 08, 2007 at 01:50:35PM +0200, Michael Buesch wrote:
> > On Wednesday 08 August 2007 14:38:10 Andi Kleen wrote:
> > > Jeff Garzik <[EMAIL PROTECTED]> writes:
> > > > > + val, reg_index, addr, addr+4); */
> > > >
Hi All,
OFED 1.2.c-11 is available now on the OFA server under:
http://www.openfabrics.org/builds/connectx/release/
Note: This release supports FW 2.1.000 that is available on Mellanox web
site for download
and FW 2.2.0 that will be released soon
Supported Platforms and Operating Systems
=
On Wed, Aug 08, 2007 at 01:50:35PM +0200, Michael Buesch wrote:
> On Wednesday 08 August 2007 14:38:10 Andi Kleen wrote:
> > Jeff Garzik <[EMAIL PROTECTED]> writes:
> > > > + val, reg_index, addr, addr+4); */
> > > > + writel(cpu_to_le32(reg_index), addr);
> > > > +
Vlad,
This patch is already merged in upstream kernel.
Please add it also to kernel patches of OFED-1.2.5
thanks
MoniS
--
ib_find_pkey() is used as a replacement for ib_find_cached_pkey(), and
the original function ignored the membership bit when searching for a
P_Key, so ib_find_pkey() sh
On Wednesday 08 August 2007 14:38:10 Andi Kleen wrote:
> Jeff Garzik <[EMAIL PROTECTED]> writes:
> > > + val, reg_index, addr, addr+4); */
> > > + writel(cpu_to_le32(reg_index), addr);
> > > + writel(cpu_to_le32(val),(u8 *)addr + 4);
> >
> > wrong -- endian conversion macros not ne
[EMAIL PROTECTED] writes:
> NetEffect connection manager routines.
This seems more like a new TCP stack. I don't think such code
is appropiate, since Linux already has one.
-Andi
___
ewg mailing list
ewg@lists.openfabrics.org
http://lists.openfabrics.o
Jeff Garzik <[EMAIL PROTECTED]> writes:
> > + val, reg_index, addr, addr+4); */
> > + writel(cpu_to_le32(reg_index), addr);
> > + writel(cpu_to_le32(val),(u8 *)addr + 4);
>
> wrong -- endian conversion macros not needed with writel()
Are you sure? I don't think that's true.
Scott Weitzenkamp (sweitzen) wrote:
Cisco has been testing 1.2.c-10 IPoIB/SDP/MPI successfully on a
32-node cluster. We are still working on tvflash, though.
We can add tvflash at beginning of next week
Tziporet
___
ewg mailing list
ewg@lists.ope
Is this patch relevant for 1.2.5?
If yes pls feel free to send it to Vlad.
--
MST
___
ewg mailing list
ewg@lists.openfabrics.org
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ewg
43 matches
Mail list logo