Re: [PATCH 06/34] vmscan: add shrink_slab tracepoints

From: Mel Gorman
Date: Fri Jul 20 2012 - 06:06:22 EST


On Thu, Jul 19, 2012 at 05:07:21PM -0500, Jonathan Nieder wrote:
> Mel Gorman wrote:
> > On Thu, Jul 19, 2012 at 03:30:17PM -0500, Jonathan Nieder wrote:
>
> >> It doesn't sound like it fixes a serious issue.
> >
> > You're right, it doesn't. There are a few patches in this series that
> > were applied because they made other patches easier to apply and this is
> > one of them. I should have noted this properly. Unlike other patches of
> > this type in the series, this particular one would have been easy to work
> > around. How about this as an updated note or would you prefer it was
> > dropped entirely?
>
> Some of the other patches of this type made sense, but I'd personally
> prefer if this one was dropped, yes. Though I am just a nobody that
> reads patches rather than one of the relevant people. ;-)
>

It's a valid point but I'm going to leave it in for now and see what the
general opinion is. I'm happy to go either way with this patch. I'll
repost shortly with the "stable note" updates and the linux-mm included
as I managed to screw up its mail address in the sending script.

Thanks.

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