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

Groups > linux.kernel > #1307220

Re: [STABLE] kernel oops which can be fixed by peterz's patches

Show all headers | View raw

Hi Peter,

On Tue, Jan 05, 2016 at 10:14:44AM +0100, Peter Zijlstra wrote:
> On Tue, Jan 05, 2016 at 05:52:11PM +0900, Byungchul Park wrote:
> > 
> > Upstream commits to be applied
> > ==============================
> > 
> > e3fca9e: sched: Replace post_schedule with a balance callback list
> > 4c9a4bc: sched: Allow balance callbacks for check_class_changed()
> > 8046d68: sched,rt: Remove return value from pull_rt_task()
> > fd7a4be: sched, rt: Convert switched_{from, to}_rt() / prio_changed_rt() to balance callbacks
> > 0ea60c2: sched,dl: Remove return value from pull_dl_task()
> > 9916e21: sched, dl: Convert switched_{from, to}_dl() / prio_changed_dl() to balance callbacks
> > 
> > The reason why these should be applied
> > ======================================
> > 
> > Our products developed using 3.16 kernel, faced a kernel oops which can
> > be fixed with above upstreamed patches. The oops is caused by "Unable
> > to handle kernel NULL pointer dereference at virtual address 000000xx"
> > in the call path,
> > 
> > __sched_setscheduler()
> > 	check_class_changed()
> > 		switched_to_fair()
> > 			check_preempt_curr()
> > 				check_preempt_wakeup()
> > 					find_matching_se()
> > 						is_same_group()
> > 
> > by "if (se->cfs_rq == pse->cfs_rq) // se, pse == NULL" condition.
> 
> So the reason I didn't mark them for stable is that they were non
> trivial, however they've been in for a while now and nothing broke, so I
> suppose backporting them isn't a problem.

I didn't check the code, but for older kernels, can't we simply get rid
of the issue by adding an extra test on se/pse before dereferencing it,
even if that implies a suboptimal fix which is always better than an oops ?
I must confess I don't feel at ease with backporting so many sensitive
changes into 2.6.32!

Thanks,
Willy

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

Thread

[STABLE] kernel oops which can be fixed by peterz's patches Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:00 +0100
  [PATCH for v3.14.58 7/7] sched, dl: Convert switched_{from, to}_dl() / prio_changed_dl() to balance callbacks Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:20 +0100
  Re: [STABLE] kernel oops which can be fixed by peterz's patches Peter Zijlstra <peterz@infradead.org> - 2016-01-05 10:20 +0100
    Re: [STABLE] kernel oops which can be fixed by peterz's patches Byungchul Park <byungchul.park@lge.com> - 2016-01-12 09:50 +0100
    Re: [STABLE] kernel oops which can be fixed by peterz's patches Willy Tarreau <w@1wt.eu> - 2016-01-12 11:30 +0100
  [PATCH for v3.14.58 6/7] sched,dl: Remove return value from pull_dl_task() Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:20 +0100
  [PATCH for v3.14.58 1/7] sched: Clean up idle task SMP logic Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:20 +0100
    [PATCH for v3.14.58 4/7] sched,rt: Remove return value from pull_rt_task() Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:20 +0100
    [PATCH for v3.14.58 2/7] sched: Replace post_schedule with a balance callback list Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:20 +0100
    [PATCH for v3.14.58 3/7] sched: Allow balance callbacks for check_class_changed() Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:20 +0100
    [PATCH for v3.14.58 5/7] sched, rt: Convert switched_{from, to}_rt() / prio_changed_rt() to balance callbacks Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:20 +0100
  [PATCH for v3.18.25 1/6] sched: Replace post_schedule with a balance callback list Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:30 +0100
    [PATCH for v3.18.25 3/6] sched,rt: Remove return value from pull_rt_task() Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:30 +0100
    [PATCH for v3.18.25 4/6] sched, rt: Convert switched_{from, to}_rt() / prio_changed_rt() to balance callbacks Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:30 +0100
    [PATCH for v3.18.25 6/6] sched, dl: Convert switched_{from, to}_dl() / prio_changed_dl() to balance callbacks Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:30 +0100
    [PATCH for v3.18.25 5/6] sched,dl: Remove return value from pull_dl_task() Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:30 +0100
    [PATCH for v3.18.25 2/6] sched: Allow balance callbacks for check_class_changed() Byungchul Park <byungchul.park@lge.com> - 2016-01-05 10:30 +0100

csiph-web