From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH 1/3] x86: allow kernel exception fixup for divide errors (#DE) Date: Tue, 24 Aug 2010 14:39:58 +0300 Message-ID: <4C73AF8E.60603@redhat.com> References: <1282648229-19575-1-git-send-email-avi@redhat.com> <1282648229-19575-2-git-send-email-avi@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Cc: x86@kernel.org, kvm@vger.kernel.org, Marcelo Tosatti , linux-kernel@vger.kernel.org To: Brian Gerst Return-path: Received: from mx1.redhat.com ([209.132.183.28]:61474 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751207Ab0HXLkC (ORCPT ); Tue, 24 Aug 2010 07:40:02 -0400 In-Reply-To: Sender: kvm-owner@vger.kernel.org List-ID: On 08/24/2010 02:22 PM, Brian Gerst wrote: > >> +dotraplinkage void do_divide_error(struct pt_regs *regs, long error_code) >> +{ >> + if (!user_mode_vm(regs)&& fixup_exception(regs)) >> + return; >> + do_divide_error_user(regs, error_code); >> +} >> + >> #ifdef CONFIG_X86_64 >> /* Runs on IST stack */ >> dotraplinkage void do_stack_segment(struct pt_regs *regs, long error_code) > Kernel mode exceptions should already be handled by do_trap(). This > is unnecessary. Correct. Please ignore this patch. I've verified that the other two work without it. -- error compiling committee.c: too many arguments to function