Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754293AbbFJWMC (ORCPT ); Wed, 10 Jun 2015 18:12:02 -0400 Received: from mail-wg0-f66.google.com ([74.125.82.66]:34722 "EHLO mail-wg0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751479AbbFJWL5 (ORCPT ); Wed, 10 Jun 2015 18:11:57 -0400 MIME-Version: 1.0 In-Reply-To: <1433270522-1973-1-git-send-email-mcgrof@do-not-panic.com> References: <1433270522-1973-1-git-send-email-mcgrof@do-not-panic.com> Date: Wed, 10 Jun 2015 15:11:56 -0700 Message-ID: Subject: Re: [PATCH] ia64: remove paravirt code From: Tony Luck To: "Luis R. Rodriguez" Cc: "Yu, Fenghua" , "linux-ia64@vger.kernel.org" , Linux Kernel Mailing List , pbonzini@redhat.com, Boris Ostrovsky , Chris Wright , Jeremy Fitzhardinge , akataria@vmware.com, Rusty Russell , "Luis R. Rodriguez" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2050 Lines: 52 On Tue, Jun 2, 2015 at 11:42 AM, Luis R. Rodriguez wrote: > From: "Luis R. Rodriguez" > > All the ia64 pvops code is now dead code since both > xen and kvm support have been ripped out [0] [1]. Just > that no one had troubled to rip this stuff out. The only > useful remaining pieces were the old pvops docs but that > was recently also generalized and moved out from ia64 [2]. > > This has been run time tested on an ia64 Madison system. > > [0] 003f7de625890 "KVM: ia64: remove" since v3.19-rc1 > [1] d52eefb47d4eb "ia64/xen: Remove Xen support for ia64" since v3.14-rc1 > [2] "virtual: Documentation: simplify and generalize paravirt_ops.txt" > > Signed-off-by: Luis R. Rodriguez > --- > I've build and run time tested this against linux-next tag next-20150529. > The build will fail if you've enabled the LKDTM (Linux Kernel Dump Test > Tool Module) enabled on SLE11 ia64 builds, a fix for which I've just > posted [3]. > > [3] https://marc.info/?l=linux-kernel&m=143327018731361&w=2 So this builds cleanly on all my test configurations, and boots on the one machine that I tried it on - good job. But I'm wondering how much of this was auto-generated (and so perhaps free from typos) and how much was done by hand? It isn't a full revert of the patch series that added paravirt ... you simplified macros like: #define MOV_TO_KR(kr, reg, clob0, clob1) \ - mov IA64_KR(kr) = reg \ - CLOBBER(clob0) \ - CLOBBER(clob1) + mov IA64_KR(kr) = reg but didn't remove them completely. So what can I do to validate this 4000+ line patch? Can I compare disassembled ".o" files pre/post patch somehow to be sure this doesn't have some small typo errors somewhere? -Tony -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/