lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.64.0712221140330.7460@blonde.wat.veritas.com>
Date:	Sat, 22 Dec 2007 11:52:17 +0000 (GMT)
From:	Hugh Dickins <hugh@...itas.com>
To:	Balbir Singh <balbir@...ux.vnet.ibm.com>
cc:	Andrew Morton <akpm@...ux-foundation.org>,
	LKML <linux-kernel@...r.kernel.org>,
	KAMEZAWA Hiroyuki <kamezawa.hiroyu@...fujitsu.com>
Subject: Re: [RFC] [PATCH] Memory controller remove control_type feature

On Sat, 22 Dec 2007, Balbir Singh wrote:
> Andrew Morton wrote:
> > On Fri, 21 Dec 2007 00:23:58 +0530 Balbir Singh <balbir@...ux.vnet.ibm.com> wrote:
> > 
> >> Based on the discussion at http://lkml.org/lkml/2007/12/20/383, it was
> >> felt that control_type might not be a good thing to implement right away.
> >> We can add this flexibility at a later point when required.

Not studied closely, but your patch looks both too much and too
little to me, Balbir.

Too much in that it appears to bundle in some significant little
locking changes without any mention in the commment.

Too little in that it leaves behind lots of junk relating to the
different control_types: the enums, the different kinds of call
that needn't now be different, no change to the various callsites.
Needs more cleanup, I'd say.  Of course, that could be yet another
separate patch.

> > Gee the memory controller patchset is turning into a mess.

A mess indeed.

> Yes, the patchset has expanded and we have several useful bug fixes and
> cleanups and some new features.

Hah, a career in politics beckons ;)

> > Hopefully it'll look a bit better once I do a big patch-folding but we
> > still have patches interesecting everywhere and now we have patches which
> > add a feature and later ones which take it away again.
> > 
> 
> I think folding will help. I understand your concern w.r.t getting the
> correct set of patches with a good changelog.
> 
> > But I don't think it's worth the time and risk of a huge
> > rip-up-and-refactor.
> > 
> 
> I agree, given the proximity of the new merge window for 2.6.25.  We
> could review the patches after folding them and see how to consolidate
> further in case the patches continue to look messy.

Personally, I think it could benefit a lot from a rip-up-and-refactor.
But if we're rushing headlong for 2.6.25, yes, I agree it's too late.
And I'm afraid it's not something I can volunteer for at this time.

Hugh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ