From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [RFC -v3 PATCH 2/3] sched: add yield_to function Date: Wed, 05 Jan 2011 19:19:30 +0200 Message-ID: <4D24A822.4000009@redhat.com> References: <20110103162637.29f23c40@annuminas.surriel.com> <20110103162918.577a9620@annuminas.surriel.com> <1294164289.2016.186.camel@laptop> <4D24A5F4.3000508@redhat.com> <1294247718.2016.269.camel@laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Rik van Riel , kvm@vger.kernel.org, linux-kernel@vger.kernel.org, Srivatsa Vaddagiri , Mike Galbraith , Chris Wright To: Peter Zijlstra Return-path: Received: from mx1.redhat.com ([209.132.183.28]:62466 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751761Ab1AERTt (ORCPT ); Wed, 5 Jan 2011 12:19:49 -0500 In-Reply-To: <1294247718.2016.269.camel@laptop> Sender: kvm-owner@vger.kernel.org List-ID: On 01/05/2011 07:15 PM, Peter Zijlstra wrote: > On Wed, 2011-01-05 at 19:10 +0200, Avi Kivity wrote: > > On 01/04/2011 08:04 PM, Peter Zijlstra wrote: > > > This definitely wants to be EXPORT_SYMBOL_GPL() and if it were possible > > > I'd make it so only kvm.o could use it. It really sucks that kvm is a > > > module. > > > > Why does it suck? I mean apart from the "virtualization is crap" song. > > Because it needs hooks all over the core kernel, like this yield_to() > stuff. Exporting this might lead to others wanting to use it too. Well, it's very convenient for development (modprobe vs. reboot). What hooks do you object to? mmu notifiers are useful for some drivers, sched notifiers are useful for cmwq and possibly perf? -- error compiling committee.c: too many arguments to function