Re: [PATCH3/5] tick: Simplify tick-internal.h

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

 



On Wed, May 13, 2015 at 05:20:23PM +0530, Preeti U Murthy wrote:
> From: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> 
> commit b7475eb599ddb2e8cab2dc86ff38a9507463ad6b upstream
> 
> tick-internal.h is pretty confusing as a lot of the stub inlines
> are there several times.
> 
> Distangle the maze and make clear functional sections.
> 
> Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> [ rjw: Subject ]
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> Cc: Peter Zijlstra <peterz@xxxxxxxxxxxxx>
> Link: http://lkml.kernel.org/r/16068264.vcNp79HLaT@xxxxxxxxxxxxxx
> Signed-off-by: Ingo Molnar <mingo@xxxxxxxxxx>
> Signed-off-by: Preeti U Murthy <preeti@xxxxxxxxxxxxxxxxxx>

You don't mention that you changed this function from what it is
upstream, which isn't ok to do without a good reason.

I'm really not liking this series, it's very intrusive for almost no
gain.  In other words, you have a very specific platform, with limited
users, and a limited use case, yet are asking for core timer functions
to be made for a kernel that I'm going to stop maintaining after the
next release.

In other words, these patches are going to only have a lifespan of 1
more week, max.  By then, all your users should be on 4.1, so 4.0
doesn't matter anymore.

So I'm just going to drop this whole series, sorry.  If you have a
distro that "has to have these", then please work with them, otherwise,
just use 4.1, as all is good, and most importantly, properly tested,
there.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]