Re: [PATCH] some kmalloc/memset ->kzalloc (tree wide)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 6 Jul 2007 18:07:20 -0400 (EDT)
James Morris <[email protected]> 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.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux