Re: [PATCH v2 0/8] Improve performance of VM translation on x86_64

2012-11-01 Thread Alexander Duyck
On 10/11/2012 03:58 PM, H. Peter Anvin wrote: > On 10/12/2012 06:40 AM, Andi Kleen wrote: >> Patch series looks good to me. Thanks for doing this properly. >> Reviewed-by: Andi Kleen >> > Agreed. > > Acked-by: H. Peter Anvin > > I will pick this up after the merge window closes unless Ingo beats

Re: [PATCH v2 0/8] Improve performance of VM translation on x86_64

2012-10-12 Thread Alexander Duyck
On 10/12/2012 08:15 AM, Andi Kleen wrote: >> Could you also add a blurb in the Documentation/< appropriate file >> for device driver writes> mentioning the usage of __pa_symbol is >> preferred? > Device driver writer's shouldn't use any of this anyways, they should always > use the PCI DMA APIs an

Re: [PATCH v2 0/8] Improve performance of VM translation on x86_64

2012-10-12 Thread Andi Kleen
> Could you also add a blurb in the Documentation/< appropriate file > for device driver writes> mentioning the usage of __pa_symbol is > preferred? Device driver writer's shouldn't use any of this anyways, they should always use the PCI DMA APIs and never DMA to the stack or to static variables.

Re: [PATCH v2 0/8] Improve performance of VM translation on x86_64

2012-10-12 Thread Konrad Rzeszutek Wilk
On Thu, Oct 11, 2012 at 4:49 PM, Alexander Duyck wrote: > This patch series is meant to address several issues I encountered with VM > translations on x86_64. In my testing I found that swiotlb was incurring up > to a 5% processing overhead due to calls to __phys_addr. To address that I > have u

Re: [PATCH v2 0/8] Improve performance of VM translation on x86_64

2012-10-11 Thread H. Peter Anvin
On 10/12/2012 06:40 AM, Andi Kleen wrote: > > Patch series looks good to me. Thanks for doing this properly. > Reviewed-by: Andi Kleen > Agreed. Acked-by: H. Peter Anvin I will pick this up after the merge window closes unless Ingo beats me to it. (I'm currently traveling.) -hpa

Re: [PATCH v2 0/8] Improve performance of VM translation on x86_64

2012-10-11 Thread Andi Kleen
Patch series looks good to me. Thanks for doing this properly. Reviewed-by: Andi Kleen -Andi -- a...@linux.intel.com -- Speaking for myself only. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info

[PATCH v2 0/8] Improve performance of VM translation on x86_64

2012-10-11 Thread Alexander Duyck
This patch series is meant to address several issues I encountered with VM translations on x86_64. In my testing I found that swiotlb was incurring up to a 5% processing overhead due to calls to __phys_addr. To address that I have updated swiotlb to use physical addresses instead of virtual addre