All of lore.kernel.org
 help / color / mirror / Atom feed
From: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
To: Peter Zijlstra <peterz@infradead.org>, Oleg Nesterov <oleg@redhat.com>
Cc: "Gautham R. Shenoy" <ego@linux.vnet.ibm.com>,
	Intel graphics driver community testing & development
	<intel-gfx@lists.freedesktop.org>,
	Linux kernel development <linux-kernel@vger.kernel.org>,
	David Hildenbrand <dahi@linux.vnet.ibm.com>,
	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>,
	Ingo Molnar <mingo@kernel.org>
Subject: Re: [PATCH] [RFC] kernel/cpu: Use lockref for online CPU reference counting
Date: Wed, 17 Feb 2016 14:47:31 +0200	[thread overview]
Message-ID: <1455713251.5622.9.camel@linux.intel.com> (raw)
In-Reply-To: <20160216110732.GU6357@twins.programming.kicks-ass.net>

On ti, 2016-02-16 at 12:07 +0100, Peter Zijlstra wrote:
> On Tue, Feb 16, 2016 at 12:51:03PM +0200, Joonas Lahtinen wrote:
> > Quoting my original patch;
> > 
> > "See the Bugzilla link for more details.
> 
> If its not in the Changelog it doesn't exist. Patches should be self
> contained and not refer to external sources for critical information.

The exact locking case in CPUfreq drivers causing a splat is described
in the patch. Details were already included, that's why term "more
details" was used.

This is not exactly taking us closer to a fix, so I'd rather like to
discuss about the possibility of adding one more member to task_struct.
I added Oleg as CC to catch his attention to re-evaluate based on the
previous discussion in this thread.

Regards, Joonas

-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

WARNING: multiple messages have this Message-ID (diff)
From: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
To: Peter Zijlstra <peterz@infradead.org>, Oleg Nesterov <oleg@redhat.com>
Cc: Intel graphics driver community testing & development 
	<intel-gfx@lists.freedesktop.org>,
	Linux kernel development <linux-kernel@vger.kernel.org>,
	Ingo Molnar <mingo@kernel.org>,
	David Hildenbrand <dahi@linux.vnet.ibm.com>,
	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>,
	"Gautham R. Shenoy" <ego@linux.vnet.ibm.com>,
	Chris Wilson <chris@chris-wilson.co.uk>,
	Daniel Vetter <daniel@ffwll.ch>
Subject: Re: [PATCH] [RFC] kernel/cpu: Use lockref for online CPU reference counting
Date: Wed, 17 Feb 2016 14:47:31 +0200	[thread overview]
Message-ID: <1455713251.5622.9.camel@linux.intel.com> (raw)
In-Reply-To: <20160216110732.GU6357@twins.programming.kicks-ass.net>

On ti, 2016-02-16 at 12:07 +0100, Peter Zijlstra wrote:
> On Tue, Feb 16, 2016 at 12:51:03PM +0200, Joonas Lahtinen wrote:
> > Quoting my original patch;
> > 
> > "See the Bugzilla link for more details.
> 
> If its not in the Changelog it doesn't exist. Patches should be self
> contained and not refer to external sources for critical information.

The exact locking case in CPUfreq drivers causing a splat is described
in the patch. Details were already included, that's why term "more
details" was used.

This is not exactly taking us closer to a fix, so I'd rather like to
discuss about the possibility of adding one more member to task_struct.
I added Oleg as CC to catch his attention to re-evaluate based on the
previous discussion in this thread.

Regards, Joonas

-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation

  reply	other threads:[~2016-02-17 12:47 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-15 12:36 [PATCH] [RFC] kernel/cpu: Use lockref for online CPU reference counting Joonas Lahtinen
2016-02-15 12:36 ` Joonas Lahtinen
2016-02-15 14:17 ` Peter Zijlstra
2016-02-15 14:17   ` Peter Zijlstra
2016-02-15 17:06   ` Peter Zijlstra
2016-02-15 17:06     ` Peter Zijlstra
2016-02-16  8:49     ` Joonas Lahtinen
2016-02-16  8:49       ` Joonas Lahtinen
2016-02-16  9:14       ` Peter Zijlstra
2016-02-16  9:14         ` Peter Zijlstra
2016-02-16 10:51         ` Joonas Lahtinen
2016-02-16 10:51           ` Joonas Lahtinen
2016-02-16 11:07           ` Peter Zijlstra
2016-02-16 11:07             ` Peter Zijlstra
2016-02-17 12:47             ` Joonas Lahtinen [this message]
2016-02-17 12:47               ` Joonas Lahtinen
2016-02-17 14:20               ` Peter Zijlstra
2016-02-17 14:20                 ` Peter Zijlstra
2016-02-17 16:13                 ` Daniel Vetter
2016-02-17 16:13                   ` Daniel Vetter
2016-02-17 16:14                   ` Peter Zijlstra
2016-02-17 16:14                     ` Peter Zijlstra
2016-02-17 16:33                     ` [Intel-gfx] " Daniel Vetter
2016-02-17 16:37                       ` Peter Zijlstra
2016-02-17 16:37                         ` [Intel-gfx] " Peter Zijlstra
2016-02-18 10:39                         ` Joonas Lahtinen
2016-02-18 10:39                           ` [Intel-gfx] " Joonas Lahtinen
2016-02-18 10:54     ` Joonas Lahtinen
2016-02-18 10:54       ` Joonas Lahtinen
2016-02-15 17:18   ` Daniel Vetter
2016-02-15 17:18     ` [Intel-gfx] " Daniel Vetter

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1455713251.5622.9.camel@linux.intel.com \
    --to=joonas.lahtinen@linux.intel.com \
    --cc=dahi@linux.vnet.ibm.com \
    --cc=ego@linux.vnet.ibm.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@kernel.org \
    --cc=oleg@redhat.com \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=peterz@infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.