Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1577379

Re: [PATCH 05/10] sched/core: Remove the tsk_cpus_allowed() wrapper

From Peter Zijlstra <peterz@infradead.org>
Newsgroups linux.kernel
Subject Re: [PATCH 05/10] sched/core: Remove the tsk_cpus_allowed() wrapper
Date 2017-02-09 10:00 +0100
Message-ID <t8SK6-3XY-3@gated-at.bofh.it> (permalink)
References <t8FjQ-3Oi-11@gated-at.bofh.it> <t8FjQ-3Oi-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Wed, Feb 08, 2017 at 07:34:18PM +0100, Ingo Molnar wrote:
> So the original intention of tsk_cpus_allowed() was to 'future-proof'
> the field - but it's pretty ineffectual at that, because half of
> the code uses ->cpus_allowed directly ...
> 
> Also, the wrapper makes the code longer than the original expression!

I still object to taking this out without replacement. This leaves RT
stranded.

If you want to implement it differently do that, but don't blindly wreck
stuff for them.

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 05/10] sched/core: Remove the tsk_cpus_allowed() wrapper Ingo Molnar <mingo@kernel.org> - 2017-02-08 19:40 +0100
  Re: [PATCH 05/10] sched/core: Remove the tsk_cpus_allowed() wrapper Peter Zijlstra <peterz@infradead.org> - 2017-02-09 10:00 +0100
    Re: [PATCH 05/10] sched/core: Remove the tsk_cpus_allowed() wrapper Ingo Molnar <mingo@kernel.org> - 2017-02-09 10:10 +0100
      Re: [PATCH 05/10] sched/core: Remove the tsk_cpus_allowed()  wrapper Thomas Gleixner <tglx@linutronix.de> - 2017-02-09 12:50 +0100
        Re: [PATCH 05/10] sched/core: Remove the tsk_cpus_allowed() wrapper Ingo Molnar <mingo@kernel.org> - 2017-02-09 21:30 +0100

csiph-web