All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jason Gunthorpe <jgunthorpe-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
To: Todd Rimmer <todd.rimmer-h88ZbnxC6KDQT0dZR+AlfA@public.gmane.org>
Cc: Mike Heinz
	<michael.heinz-h88ZbnxC6KDQT0dZR+AlfA@public.gmane.org>,
	"linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Roland Dreier <rdreier-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org>,
	"ewg-G2znmakfqn7U1rindQTSdQ@public.gmane.org"
	<ewg-G2znmakfqn7U1rindQTSdQ@public.gmane.org>
Subject: Re: [ewg] [PATCH] pkey fix for ipoib - resubmission
Date: Fri, 18 Jun 2010 15:46:01 -0600	[thread overview]
Message-ID: <20100618214601.GN4630@obsidianresearch.com> (raw)
In-Reply-To: <5AEC2602AE03EB46BFC16C6B9B200DA816B7530901-e4KNYiSEog6Xx9kJd3VG2h2eb7JE58TQ@public.gmane.org>

On Fri, Jun 18, 2010 at 01:38:13PM -0500, Todd Rimmer wrote:
> Jason,
> 
> > Fixing the maddr table, and dealing with race issues with ongoing
> > joins my not be straightforward, but, IMHO, necessary for this patch
> > to be acceptable.

> So you are pointing out an existing bug, which is no worse with this
> patch than without it.  Namely that pkey changes with IPv6 may have
> issues.  IPv4 pkey changes should be ok.

No.. It is my understanding, that today, the pkey of an IPOIB
interface can never change once it is established.

This patch changes that invariant without actually addressing all
the complexity of making it work.

> In at least the HPC world, PKey changes are extremely rare, however
> having server's reliably boot is important.

Various IPv6 multicast joins occur as soon as the interface is up'd,
using the pkey in-place at that time. Changing the pkey while the
interface is down might work OK, but changing it while it is up'd will
break all multicast joins that were started before, ie IPv6.

Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2010-06-18 21:46 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-16 13:59 [PATCH] pkey fix for ipoib - resubmission Mike Heinz
     [not found] ` <4C2744E8AD2982428C5BFE523DF8CDCB49D09E7A43-amwN6d8PyQWXx9kJd3VG2h2eb7JE58TQ@public.gmane.org>
2010-06-17 16:36   ` Eli Cohen
     [not found]     ` <AANLkTimLfiuWMpiUFGIvotG0aWiebXC5jv5kfgKUmG4a-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-06-17 16:39       ` Mike Heinz
2010-06-18 17:09   ` Mike Heinz
     [not found]     ` <4C2744E8AD2982428C5BFE523DF8CDCB49D09E7C6D-amwN6d8PyQWXx9kJd3VG2h2eb7JE58TQ@public.gmane.org>
2010-06-18 17:22       ` Jason Gunthorpe
     [not found]         ` <20100618172216.GK4630-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2010-06-18 17:32           ` [ewg] " Todd Rimmer
     [not found]             ` <5AEC2602AE03EB46BFC16C6B9B200DA816B75308EF-e4KNYiSEog6Xx9kJd3VG2h2eb7JE58TQ@public.gmane.org>
2010-06-18 17:46               ` Jason Gunthorpe
     [not found]                 ` <20100618174650.GL4630-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2010-06-18 17:56                   ` Todd Rimmer
     [not found]                     ` <5AEC2602AE03EB46BFC16C6B9B200DA816B75308F7-e4KNYiSEog6Xx9kJd3VG2h2eb7JE58TQ@public.gmane.org>
2010-06-18 18:19                       ` Jason Gunthorpe
     [not found]                         ` <20100618181921.GM4630-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2010-06-18 18:38                           ` Todd Rimmer
     [not found]                             ` <5AEC2602AE03EB46BFC16C6B9B200DA816B7530901-e4KNYiSEog6Xx9kJd3VG2h2eb7JE58TQ@public.gmane.org>
2010-06-18 21:46                               ` Jason Gunthorpe [this message]
2010-06-22  6:09                           ` Or Gerlitz
     [not found]                             ` <4C20537D.5080903-hKgKHo2Ms0FWk0Htik3J/w@public.gmane.org>
2010-06-22 14:46                               ` Jason Gunthorpe
     [not found]                                 ` <20100622144607.GA7767-ePGOBjL8dl3ta4EC/59zMFaTQe2KTcn/@public.gmane.org>
2010-06-23  6:58                                   ` Or Gerlitz

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20100618214601.GN4630@obsidianresearch.com \
    --to=jgunthorpe-epgobjl8dl3ta4ec/59zmfatqe2ktcn/@public.gmane.org \
    --cc=ewg-G2znmakfqn7U1rindQTSdQ@public.gmane.org \
    --cc=linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=michael.heinz-h88ZbnxC6KDQT0dZR+AlfA@public.gmane.org \
    --cc=rdreier-FYB4Gu1CFyUAvxtiuMwx3w@public.gmane.org \
    --cc=todd.rimmer-h88ZbnxC6KDQT0dZR+AlfA@public.gmane.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.