[EMAIL PROTECTED] wrote: > Hi, > > Selon "Zhang, Xiantao" <[EMAIL PROTECTED]>: > >>> From 5f82ea88c095cf89cbae920944c05e578f35365f Mon Sep 17 00:00:00 >>> 2001 >> From: Xiantao Zhang <[EMAIL PROTECTED]> >> Date: Wed, 12 Mar 2008 14:48:09 +0800 >> Subject: [PATCH] kvm/ia64: Add mmio decoder for kvm/ia64. [...] >> + post_update = (inst.M5.i << 7) + inst.M5.imm7; >> + if (inst.M5.s) >> + temp -= post_update; >> + else >> + temp += post_update; > > The sign extension is not done correctly here. (This has been fixed > in Xen code).
Include the fix in the latest merge candidate patchset. Thanks! :) git://git.kernel.org/pub/scm/linux/kernel/git/xiantao/kvm-ia64.git kvm-ia64-mc8 > >> + post_update = (inst.M3.i << 7) + inst.M3.imm7; >> + if (inst.M3.s) >> + temp -= post_update; >> + else >> + temp += post_update; > > Ditto. > >> + post_update = (inst.M10.i << 7) + inst.M10.imm7; + if >> (inst.M10.s) + temp -= post_update; >> + else >> + temp += post_update; > > Ditto. > >> + post_update = (inst.M10.i << 7) + inst.M10.imm7; + if >> (inst.M10.s) + temp -= post_update; >> + else >> + temp += post_update; > > Ditto. > >> + post_update = (inst.M15.i << 7) + inst.M15.imm7; + if >> (inst.M15.s) + temp -= post_update; >> + else >> + temp += post_update; > > Ditto. > > Tristan. ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace _______________________________________________ kvm-devel mailing list kvm-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kvm-devel
