OpenVZ Forum: Devel » [PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls

Home » Mailing lists » Devel » [PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls

Show: Today's Messages :: Show Polls :: Message Navigator
E-mail to friend

Re: [PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls [message #44571 is a reply to message #44570]

Thu, 15 December 2011 06:48

Glauber Costa
Messages: 916
Registered: October 2011

Senior Member

On 12/15/2011 09:48 AM, David Miller wrote:
> From: KAMEZAWA Hiroyuki<kamezawa.hiroyu@jp.fujitsu.com>
> Date: Thu, 15 Dec 2011 14:40:19 +0900
>
>> I met this bug at _1st_ run. Please enable _all_ debug options!.
>
> Plus the CONFIG_NET=n and other build failures.
>
> This patch series was seriously rushed, and very poorly handled.
>
> Yet I kept getting so much pressure to review, comment upon, and
> ultimately apply these patches. Never, ever, do this to me ever
> again.
>
> If I don't feel your patches are high priority enough or ready enough
> for me to review, then TOO BAD. Don't ask people to pressure me or
> get my attention. Instead, ask others for help and do testing before
> wasting MY time and crapping up MY tree.
>
> I should have noticed a red flag when I have James Bottomly asking me
> to look at these patches, I should have pushed back. Instead, I
> relented, and now I'm very seriously regretting it.
>
> All the regressions in the net-next tree over the past several days
> have been due to this patch set, and this patch set alone.
>
> This code wasn't ready and needed, at a minimum, several more weeks of
> work before being put in.
>
> Instead, we're going to bandaid patch it up after the fact, rather
> than just letting these changes mature naturally during the review
> process.
Hi Dave,

You are right about all points. I will admit to it, face it, and
apologize it.
I guess the best I can do right now is fix whatever you guys point me to
and not repeat it in the future.

Thanks

Report message to a moderator

[Message index]

		[PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls By: Glauber Costa on Mon, 12 December 2011 07:47
		[PATCH v9 1/9] Basic kernel memory functionality for the Memory Controller By: Glauber Costa on Mon, 12 December 2011 07:47
		Re: [PATCH v9 1/9] Basic kernel memory functionality for the Memory Controller By: Glauber Costa on Thu, 15 December 2011 12:29
		Re: [PATCH v9 1/9] Basic kernel memory functionality for the Memory Controller By: Michal Hocko on Fri, 16 December 2011 12:32
		Re: [PATCH v9 1/9] Basic kernel memory functionality for the Memory Controller By: Greg Thelen on Fri, 16 December 2011 06:20
		Re: [PATCH v9 1/9] Basic kernel memory functionality for the Memory Controller By: Glauber Costa on Fri, 16 December 2011 13:02
		Re: [PATCH v9 1/9] Basic kernel memory functionality for the Memory Controller By: Michal Hocko on Fri, 16 December 2011 13:30
		Re: [PATCH v9 1/9] Basic kernel memory functionality for the Memory Controller By: Michal Hocko on Wed, 14 December 2011 17:04
		[PATCH v9 2/9] foundations of per-cgroup memory pressure controlling. By: Glauber Costa on Mon, 12 December 2011 07:47
		[PATCH v9 4/9] tcp memory pressure controls By: Glauber Costa on Mon, 12 December 2011 07:47
		[PATCH v9 3/9] socket: initial cgroup code. By: Glauber Costa on Mon, 12 December 2011 07:47
		Re: [PATCH v9 3/9] socket: initial cgroup code. By: Glauber Costa on Fri, 23 December 2011 08:57
		Re: [PATCH v9 3/9] socket: initial cgroup code. By: Jason Baron on Thu, 22 December 2011 21:10
		[PATCH v9 5/9] per-netns ipv4 sysctl_tcp_mem By: Glauber Costa on Mon, 12 December 2011 07:47
		[PATCH v9 7/9] Display current tcp memory allocation in kmem cgroup By: Glauber Costa on Mon, 12 December 2011 07:47
		[PATCH v9 6/9] tcp buffer limitation: per-cgroup limit By: Glauber Costa on Mon, 12 December 2011 07:47
		[PATCH v9 8/9] Display current tcp failcnt in kmem cgroup By: Glauber Costa on Mon, 12 December 2011 07:47
		[PATCH v9 9/9] Display maximum tcp memory allocation in kmem cgroup By: Glauber Costa on Mon, 12 December 2011 07:47
		Re: [PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls By: davem on Tue, 13 December 2011 00:07
		Re: [PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls By: Christoph Paasch on Tue, 13 December 2011 13:49
		Re: [PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls By: Eric Dumazet on Tue, 13 December 2011 13:59
		Re: [PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls By: davem on Tue, 13 December 2011 18:45
		Re: [PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls By: Glauber Costa on Tue, 13 December 2011 20:11
		Re: [PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls By: KAMEZAWA Hiroyuki on Thu, 15 December 2011 05:40
		Re: [PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls By: davem on Thu, 15 December 2011 05:48
		Re: [PATCH v9 0/9] Request for inclusion: per-cgroup tcp memory pressure controls By: Glauber Costa on Thu, 15 December 2011 06:48

Previous Topic:	[RFC] special case cpuacct cpuusage when cpu cgroup is comounted
Next Topic:	Re: [PATCH] move get_idle_time , get_iowait_time to sched/core.c

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

] [

]

Current Time: Tue Aug 05 06:40:06 GMT 2025

Total time taken to generate the page: 1.25882 seconds