[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAM_iQpWw4UMKZcdZfpp5D-tDfj954fbptyXJUzydgFCero6xNw@mail.gmail.com>
Date: Thu, 6 Nov 2014 10:05:41 -0800
From: Cong Wang <xiyou.wangcong@...il.com>
To: Eric Dumazet <eric.dumazet@...il.com>
Cc: Linux Kernel Network Developers <netdev@...r.kernel.org>,
Jamal Hadi Salim <jhs@...atatu.com>
Subject: Re: [PATCH 00/13] net_sched: misc cleanups and improvements
On Tue, Nov 4, 2014 at 5:47 PM, Eric Dumazet <eric.dumazet@...il.com> wrote:
> On Tue, 2014-11-04 at 17:25 -0800, Cong Wang wrote:
>
>> Seriously, think about why it should when it's just cleanup's, be practical.
>
> I seriously ask you to not do cleanups then.
Apparently you didn't say this when the following commits got accepted:
commit 436f7c206860729d543a457aca5887e52039a5f4
Author: Fabian Frederick <fabf@...net.be>
Date: Tue Nov 4 20:52:14 2014 +0100
igmp: remove camel case definitions
use standard uppercase for definitions
Signed-off-by: Fabian Frederick <fabf@...net.be>
Signed-off-by: David S. Miller <davem@...emloft.net>
commit c18450a52a10a5c4cea3dc426c40447a7152290f
Author: Fabian Frederick <fabf@...net.be>
Date: Tue Nov 4 20:48:41 2014 +0100
udp: remove else after return
commit aa1f731e52807077e9e13a86c0cad12d442c8fd4
Author: Fabian Frederick <fabf@...net.be>
Date: Tue Nov 4 20:44:04 2014 +0100
inet: frags: remove inline on static in c file
remove __inline__ / inline and let compiler decide what to do
with static functions
Inspired-by: "David S. Miller" <davem@...emloft.net>
Signed-off-by: Fabian Frederick <fabf@...net.be>
Signed-off-by: David S. Miller <davem@...emloft.net>
>
> Some people are working adding real stuff here, this code changing every
> month is slowing them a lot.
>
Who works on what? Does he/she at least announce it on netdev?
(If you meant John, I already waited for his rcu stuffs in the last
merge window,
I assumed his works is almost done therefore sent this patchset.)
Since when it becomes a rule that we should yield to something not merged,
not even announced? If so, why not adding it to netdev-FAQ?
--
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