public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [GIT PULL v2] posix cpu timers cleanups for 3.14
@ 2013-12-09 18:24 Frederic Weisbecker
  2013-12-10 11:34 ` Ingo Molnar
  0 siblings, 1 reply; 2+ messages in thread
From: Frederic Weisbecker @ 2013-12-09 18:24 UTC (permalink / raw)
  To: Ingo Molnar, Thomas Gleixner
  Cc: LKML, Frederic Weisbecker, Peter Zijlstra, Oleg Nesterov,
	Kosaki Motohiro, Andrew Morton

Ingo, Thomas,

Please pull the timers/posix-timers-for-tip-v2 branch that can be found at:

git://git.kernel.org/pub/scm/linux/kernel/git/frederic/linux-dynticks.git
	timers/posix-timers-for-tip-v2

HEAD: 531f64fd6f46a3f2a3edb1b97ecc827c775932c5

This is the second version of the pull request that I sent you a few weeks ago:
http://lkml.kernel.org/r/1385221040-24731-1-git-send-email-fweisbec@gmail.com

Changes in this v2:

* rebase against latest tip:timers/core (there was a conflict with some dynticks
  patches)

* add a cleanup suggested by Kosaki in patch 02/10: remove a useless group
  timer sample snapshot and rearm on timer rescheduling.

Also the previous pull request lacked a high level overall description, so I'm
adding one here.

---

Posix cpu timers changes for 3.14:

* Remove dying thread/process timers caching that were complicating the code
  for no significant win.

* Remove early task reference release on dying timer sample read. Again it was
  not worth the code complication. The other timer's resources aren't released until
  timer_delete() is called anyway (or when the whole process dies).

* Remove leftover arguments in reaped target cleanup

* Consolidate some timer sampling code

* Remove use of tasklist lock

* Robustify sighand locking against exec and exit by using the safer
  lock_task_sighand() API instead of sighand raw locking.

* Convert some unnecessary BUG_ON() to WARN_ON()


Thanks,
	Frederic
---

Frederic Weisbecker (10):
      posix-timers: Remove dead thread posix cpu timers caching
      posix-timers: Remove dead process posix cpu timers caching
      posix-timers: Cleanup reaped target handling
      posix-timers: Remove dead task special case
      posix-timers: Remove useless clock sample on timers cleanup
      posix-timers: Consolidate posix_cpu_clock_get()
      posix-timers: Use sighand lock instead of tasklist_lock for task clock sample
      posix-timers: Use sighand lock instead of tasklist_lock on timer deletion
      posix-timers: Remove remaining uses of tasklist_lock
      posix-timers: Convert abuses of BUG_ON to WARN_ON


 kernel/posix-cpu-timers.c | 312 +++++++++++++++++++---------------------------
 1 file changed, 129 insertions(+), 183 deletions(-)

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [GIT PULL v2] posix cpu timers cleanups for 3.14
  2013-12-09 18:24 [GIT PULL v2] posix cpu timers cleanups for 3.14 Frederic Weisbecker
@ 2013-12-10 11:34 ` Ingo Molnar
  0 siblings, 0 replies; 2+ messages in thread
From: Ingo Molnar @ 2013-12-10 11:34 UTC (permalink / raw)
  To: Frederic Weisbecker
  Cc: Thomas Gleixner, LKML, Peter Zijlstra, Oleg Nesterov,
	Kosaki Motohiro, Andrew Morton


* Frederic Weisbecker <fweisbec@gmail.com> wrote:

> Ingo, Thomas,
> 
> Please pull the timers/posix-timers-for-tip-v2 branch that can be found at:
> 
> git://git.kernel.org/pub/scm/linux/kernel/git/frederic/linux-dynticks.git
> 	timers/posix-timers-for-tip-v2
> 
> HEAD: 531f64fd6f46a3f2a3edb1b97ecc827c775932c5
> 
> This is the second version of the pull request that I sent you a few weeks ago:
> http://lkml.kernel.org/r/1385221040-24731-1-git-send-email-fweisbec@gmail.com
> 
> Changes in this v2:
> 
> * rebase against latest tip:timers/core (there was a conflict with some dynticks
>   patches)
> 
> * add a cleanup suggested by Kosaki in patch 02/10: remove a useless group
>   timer sample snapshot and rearm on timer rescheduling.
> 
> Also the previous pull request lacked a high level overall description, so I'm
> adding one here.
> 
> ---
> 
> Posix cpu timers changes for 3.14:
> 
> * Remove dying thread/process timers caching that were complicating the code
>   for no significant win.
> 
> * Remove early task reference release on dying timer sample read. Again it was
>   not worth the code complication. The other timer's resources aren't released until
>   timer_delete() is called anyway (or when the whole process dies).
> 
> * Remove leftover arguments in reaped target cleanup
> 
> * Consolidate some timer sampling code
> 
> * Remove use of tasklist lock
> 
> * Robustify sighand locking against exec and exit by using the safer
>   lock_task_sighand() API instead of sighand raw locking.
> 
> * Convert some unnecessary BUG_ON() to WARN_ON()
> 
> 
> Thanks,
> 	Frederic
> ---
> 
> Frederic Weisbecker (10):
>       posix-timers: Remove dead thread posix cpu timers caching
>       posix-timers: Remove dead process posix cpu timers caching
>       posix-timers: Cleanup reaped target handling
>       posix-timers: Remove dead task special case
>       posix-timers: Remove useless clock sample on timers cleanup
>       posix-timers: Consolidate posix_cpu_clock_get()
>       posix-timers: Use sighand lock instead of tasklist_lock for task clock sample
>       posix-timers: Use sighand lock instead of tasklist_lock on timer deletion
>       posix-timers: Remove remaining uses of tasklist_lock
>       posix-timers: Convert abuses of BUG_ON to WARN_ON
> 
> 
>  kernel/posix-cpu-timers.c | 312 +++++++++++++++++++---------------------------
>  1 file changed, 129 insertions(+), 183 deletions(-)

Pulled, thanks a lot Frederic!

	Ingo

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2013-12-10 11:34 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-12-09 18:24 [GIT PULL v2] posix cpu timers cleanups for 3.14 Frederic Weisbecker
2013-12-10 11:34 ` Ingo Molnar

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox