From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753273AbXGFXRE (ORCPT ); Fri, 6 Jul 2007 19:17:04 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751686AbXGFXQx (ORCPT ); Fri, 6 Jul 2007 19:16:53 -0400 Received: from smtp2.linux-foundation.org ([207.189.120.14]:50062 "EHLO smtp2.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751650AbXGFXQw (ORCPT ); Fri, 6 Jul 2007 19:16:52 -0400 Date: Fri, 6 Jul 2007 16:16:17 -0700 From: Andrew Morton To: James Morris Cc: Heikki Orsila , Yoann Padioleau , kernel-janitors@lists.osdl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] some kmalloc/memset ->kzalloc (tree wide) Message-Id: <20070706161617.072f7eea.akpm@linux-foundation.org> In-Reply-To: References: <878x9ta320.fsf@wanadoo.fr> <20070706175552.GF3400@zakalwe.fi> X-Mailer: Sylpheed version 2.2.7 (GTK+ 2.8.6; i686-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 6 Jul 2007 18:07:20 -0400 (EDT) James Morris wrote: > On Fri, 6 Jul 2007, Heikki Orsila wrote: > > > > Transform some calls to kmalloc/memset to a single kzalloc (or > > > kcalloc). > > > > I looked all the files through. They looked good to me, except > > one case: > > I checked the drivers/net and net/ changes and they looked ok (aside from > the above). > > OK, thanks for checking it, guys. Yoann, the "tree wide" thing really is a problem. Your patch here is touching ten or more maintainers' code and it really does want to be split into ten separate patches and fed into the tree via the appropriate routes. I can take care of that routing if I have the ten patches, but this time I really can't be bothered splitting the patch up (it's rather boring, and gets error-prone). For this patch, I tossed out the ten or so hunks which conflicted with other pending patches in the various trees and I shall just grit the teeth and maintain and merge it as a single whopping patch. (At least this way I'll actually get some scsi stuff merged). Anything which conflicts with anyone's outstanding work will be dropped. For any future patches, _please_ split them up along some sane lines. Generally you'll find that one patch per second-level directory works pretty well: drivers/net, drivers/scsi, drivers/usb, fs/ext2, drivers/char, etc. Thanks.