From: Roopa Prabhu <roopa@cumulusnetworks.com>
To: Scott Feldman <sfeldma@gmail.com>
Cc: "Jiří Pírko" <jiri@resnulli.us>,
"Jamal Hadi Salim" <jhs@mojatatu.com>,
"Benjamin LaHaise" <bcrl@kvack.org>,
"Thomas Graf" <tgraf@suug.ch>,
"john fastabend" <john.fastabend@gmail.com>,
"stephen@networkplumber.org" <stephen@networkplumber.org>,
"John Linville" <linville@tuxdriver.com>,
"vyasevic@redhat.com" <vyasevic@redhat.com>,
Netdev <netdev@vger.kernel.org>,
"David S. Miller" <davem@davemloft.net>,
"shm@cumulusnetworks.com" <shm@cumulusnetworks.com>,
"Andy Gospodarek" <gospo@cumulusnetworks.com>
Subject: Re: [PATCH net-next v5 0/3] remove bridge mode BRIDGE_MODE_SWDEV
Date: Mon, 08 Dec 2014 15:00:01 -0800 [thread overview]
Message-ID: <54862D71.6060703@cumulusnetworks.com> (raw)
In-Reply-To: <CAE4R7bBUDQztn-Pk-xRxVJbGbnjE2_CnyDn3-k3PWPUu7mFJsw@mail.gmail.com>
On 12/8/14, 2:49 PM, Scott Feldman wrote:
> Roopa, thanks for fixing this. I'm a little lost on status of the
> iproute2 patches related to this change and earlier changes. Would
> you and Jiri respin one set of iproute2 patches that gets us the final
> answer? It was getting a little messy with patches on top of
> yet-to-be-accepted patches. Thanks.
scott, the iproute2 v3 patch was the last (hopefully final) patch and
jiri and you signed-off on that
Its here:
http://patchwork.ozlabs.org/patch/418350/
Thanks for the review.
>
> On Mon, Dec 8, 2014 at 2:04 PM, <roopa@cumulusnetworks.com> wrote:
>> From: Roopa Prabhu <roopa@cumulusnetworks.com>
>>
>> BRIDGE_MODE_SWDEV was introduced to indicate switchdev offloads
>> for bridging from user space (In other words to call into the hw switch
>> port driver directly). But user can use existing BRIDGE_FLAGS_SELF
>> to call into the hw switch port driver today. swdev mode is not required
>> anymore. So, this patch removes it.
>>
>> v4 - v5
>> incorporate comments
>> - Define BRIDGE_MODE_UNDEF to handle cases where mode is not defined
>> - reverse the order of patches
>> - include patch comments in all patches
>>
>> Signed-off-by: Roopa Prabhu <roopa@cumulusnetworks.com>
>>
>> Roopa Prabhu (3):
>> bridge: new mode flag to indicate mode 'undefined'
>> rocker: remove swdev mode
>> bridge: remove mode 'swdev'
>>
>> drivers/net/ethernet/rocker/rocker.c | 18 +-----------------
>> include/uapi/linux/if_bridge.h | 2 +-
>> net/core/rtnetlink.c | 10 ++++++++--
>> 3 files changed, 10 insertions(+), 20 deletions(-)
>>
>> --
>> 1.7.10.4
>>
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2014-12-08 23:00 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-12-08 22:04 [PATCH net-next v5 0/3] remove bridge mode BRIDGE_MODE_SWDEV roopa
2014-12-08 22:49 ` Scott Feldman
2014-12-08 23:00 ` Roopa Prabhu [this message]
2014-12-09 23:25 ` David Miller
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=54862D71.6060703@cumulusnetworks.com \
--to=roopa@cumulusnetworks.com \
--cc=bcrl@kvack.org \
--cc=davem@davemloft.net \
--cc=gospo@cumulusnetworks.com \
--cc=jhs@mojatatu.com \
--cc=jiri@resnulli.us \
--cc=john.fastabend@gmail.com \
--cc=linville@tuxdriver.com \
--cc=netdev@vger.kernel.org \
--cc=sfeldma@gmail.com \
--cc=shm@cumulusnetworks.com \
--cc=stephen@networkplumber.org \
--cc=tgraf@suug.ch \
--cc=vyasevic@redhat.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.