Re: [PATCH 0/5] [v3] fix illegal use of __pa() in KVM code

2013-02-26 Thread Peter Hurley
On Mon, 2013-02-25 at 06:42 -0800, Dave Hansen wrote: > On 02/24/2013 01:28 PM, Peter Hurley wrote: > > Now that the alloc_remap() has been/is being removed, is most/all of > > this being reverted? > > I _believe_ alloc_remap() is the only case where we actually remapped > low memory. However, th

Re: [PATCH 0/5] [v3] fix illegal use of __pa() in KVM code

2013-02-25 Thread Dave Hansen
On 02/24/2013 01:28 PM, Peter Hurley wrote: > Now that the alloc_remap() has been/is being removed, is most/all of > this being reverted? I _believe_ alloc_remap() is the only case where we actually remapped low memory. However, there is still other code that does __pa() translations for percpu a

Re: [PATCH 0/5] [v3] fix illegal use of __pa() in KVM code

2013-02-25 Thread Gleb Natapov
On Sun, Feb 24, 2013 at 04:28:58PM -0500, Peter Hurley wrote: > > On Tue, 2013-01-22 at 13:24 -0800, Dave Hansen wrote: > > This series fixes a hard-to-debug early boot hang on 32-bit > > NUMA systems. It adds coverage to the debugging code, > > adds some helpers, and eventually fixes the origina

Re: [PATCH 0/5] [v3] fix illegal use of __pa() in KVM code

2013-02-24 Thread Peter Hurley
On Tue, 2013-01-22 at 13:24 -0800, Dave Hansen wrote: > This series fixes a hard-to-debug early boot hang on 32-bit > NUMA systems. It adds coverage to the debugging code, > adds some helpers, and eventually fixes the original bug I > was hitting. Hi Dave, Now that the alloc_remap() has been/is

Re: [PATCH 0/5] [v3] fix illegal use of __pa() in KVM code

2013-01-25 Thread Dave Hansen
On 01/22/2013 01:24 PM, Dave Hansen wrote: > This series fixes a hard-to-debug early boot hang on 32-bit > NUMA systems. It adds coverage to the debugging code, > adds some helpers, and eventually fixes the original bug I > was hitting. I got one more reviewed-by on this set, but otherwise no mor

[PATCH 0/5] [v3] fix illegal use of __pa() in KVM code

2013-01-22 Thread Dave Hansen
This series fixes a hard-to-debug early boot hang on 32-bit NUMA systems. It adds coverage to the debugging code, adds some helpers, and eventually fixes the original bug I was hitting. [v3] * Remove unnecessary initializations in slow_virt_to_phys(), and convert 'level' to use the new enum p