[PATCH] drivers: android: Restructure code in lowmemorykiller

Dan Carpenter dan.carpenter at oracle.com
Fri Feb 1 06:48:41 UTC 2013


On Fri, Feb 01, 2013 at 12:48:16AM +0200, Stratos Karafotis wrote:
> I will fix it.

I don't know the status of this driver.

But in general the patch should have been divided up into smaller
changes and sent as a patchset.  It creates a bunch of messy new
globals where it would have been cleaner to use function parameters.
Don't introduce new calls to lowmem_print() because we have better
debugging facilities already in the kernel and don't need more.

Btw, the more things you put into one patch, the more there is to
complain about and the less likely it is to be merged.

regards,
dan carpenter



More information about the devel mailing list