[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAE4R7bCw3m-ircfA9FiD5+VR3=5vj=cZ7YLTobkXjLoSvYUq_w@mail.gmail.com>
Date: Wed, 1 Apr 2015 11:02:48 -0700
From: Scott Feldman <sfeldma@...il.com>
To: Jiri Pirko <jiri@...nulli.us>
Cc: Netdev <netdev@...r.kernel.org>,
Roopa Prabhu <roopa@...ulusnetworks.com>,
Guenter Roeck <linux@...ck-us.net>,
Florian Fainelli <f.fainelli@...il.com>,
"Samudrala, Sridhar" <sridhar.samudrala@...el.com>,
"Arad, Ronen" <ronen.arad@...el.com>
Subject: Re: [PATCH net-next v2 00/26] switchdev: spring cleanup
On Wed, Apr 1, 2015 at 4:40 AM, Jiri Pirko <jiri@...nulli.us> wrote:
> Wed, Apr 01, 2015 at 12:07:36PM CEST, sfeldma@...il.com wrote:
>>From: Scott Feldman <sfeldma@...il.com>
>>
>>v2:
>>
>>Address review comments:
>>
>> - [Jiri] squash a few related patches
>> - [Roopa] don't remove NETIF_F_HW_SWITCH_OFFLOAD
>> - [Roopa] address VLAN setlink/dellink
>> - [Ronen] print warning is attr set revert fails
>>
>>Not address:
>>
>> - Using something other than "swdev_" prefix
>
> I do not understand why this was not addresses. Can you please provide some
> justification?
1) It was 3AM in the morning, and I didn't want to fight a rename that
touched 26 patches in the set.
2) I like "swdev_" prefix because it's short and sweet and I don't
like long-winded symbols when a shorter one will do, so I had little
motivation for 1)
3) This is one of those labeling exercises where we can't get
consensus on the name. So I'm ignoring to get the important stuff in,
and someone else can sponsor a name change.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists