From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Jan Beulich" Subject: Re: [Xen-devel] [RFC v4 2/2] x86/xen: allow privcmd hypercalls to be preempted Date: Mon, 26 Jan 2015 10:46:03 +0000 Message-ID: <54C628FB02000078000595C0@mail.emea.novell.com> References: <1421972951-3940-1-git-send-email-mcgrof@do-not-panic.com> <1421972951-3940-3-git-send-email-mcgrof@do-not-panic.com> <54C23442.9000008@citrix.com> <20150123185812.GX17887@wotan.suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 8BIT Cc: "Andy Lutomirski" , "David Vrabel" , "Luis R. Rodriguez" , , "Masami Hiramatsu" , , "Thomas Gleixner" , , , , , "Ingo Molnar" , "Borislav Petkov" , , , "H. Peter Anvin" To: "Luis R. Rodriguez" Return-path: In-Reply-To: <20150123185812.GX17887@wotan.suse.de> Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-Id: kvm.vger.kernel.org >>> On 23.01.15 at 19:58, wrote: > On Fri, Jan 23, 2015 at 11:45:06AM +0000, David Vrabel wrote: >> On 23/01/15 00:29, Luis R. Rodriguez wrote: >> > @@ -1243,6 +1247,25 @@ void xen_evtchn_do_upcall(struct pt_regs *regs) >> > set_irq_regs(old_regs); >> > } >> > >> > +/* >> > + * CONFIG_PREEMPT=n kernels can end up triggering the softlock >> > + * TASK_UNINTERRUPTIBLE hanger check (default 120 seconds) >> > + * when certain multicalls are used [0] on large systems, in >> > + * that case we need a way to voluntarily preempt. This is >> > + * only an issue on CONFIG_PREEMPT=n kernels. >> >> Rewrite this comment as; >> >> * Some hypercalls issued by the toolstack can take many 10s of > > Its not just hypercalls though, this is all about the interactions > with multicalls no? multicalls are just a special case of hypercalls. Jan