All of lore.kernel.org
 help / color / mirror / Atom feed
From: Scott Feldman <sfeldma@gmail.com>
To: Sridhar Samudrala <sridhar.samudrala@intel.com>
Cc: Scott Feldman <sfeldman@gmail.com>, Netdev <netdev@vger.kernel.org>
Subject: Re: [RFC PATCH net-next] switchdev: fdb add/del via swdev obj.
Date: Mon, 6 Apr 2015 22:16:49 -0700	[thread overview]
Message-ID: <CAE4R7bBjPNs40cQfYGEueHVnP-_gT7gX41i=NjZmFt7R_XhLSg@mail.gmail.com> (raw)
In-Reply-To: <1428362710-20815-1-git-send-email-sridhar.samudrala@intel.com>

On Mon, Apr 6, 2015 at 4:25 PM, Sridhar Samudrala
<sridhar.samudrala@intel.com> wrote:
> This is a patch on top of Scott's v3 version of switchdev spring
> cleanup patches.
>
> Signed-off-by: Sridhar Samudrala <sridhar.samudrala@intel.com>

Sridhar,

You patch is spot-on.  We can't apply it yet until my stuff gets
in...I'm working on it.  I extended your patch for fdb dump as well,
which I'll send tomorrow.  For dump, I've added swdev_port_obj_dump()
op and moved all the netlink skb building code to code swdev code.
The driver implements very little and is unaware of the netlink msg
being built.

With your patch, we can finally do the fdb flushes and fdb ageing on
the port without having to synthesis netlink msgs inside the kernel.
I'm not sure if that's what you're working on, but it's the next
logical piece to plumb.  (Hopefully you are working on that :).

(btw, you have my email addr wrong).

Acked-in-spirit-by: Scott Feldman <sfeldma@gmail.com>

-scott

      reply	other threads:[~2015-04-07  5:16 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-06 23:25 [RFC PATCH net-next] switchdev: fdb add/del via swdev obj Sridhar Samudrala
2015-04-07  5:16 ` Scott Feldman [this message]

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='CAE4R7bBjPNs40cQfYGEueHVnP-_gT7gX41i=NjZmFt7R_XhLSg@mail.gmail.com' \
    --to=sfeldma@gmail.com \
    --cc=netdev@vger.kernel.org \
    --cc=sfeldman@gmail.com \
    --cc=sridhar.samudrala@intel.com \
    /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.