diff for duplicates of <1374697969.15592.67@snotra> diff --git a/a/1.txt b/N1/1.txt index b4e0270..4489f31 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,33 +1,33 @@ On 07/24/2013 04:39:59 AM, Alexander Graf wrote: -> +>=20 > On 24.07.2013, at 11:35, Gleb Natapov wrote: -> +>=20 > > On Wed, Jul 24, 2013 at 11:21:11AM +0200, Alexander Graf wrote: -> >>> Are not we going to use page_is_ram() from +> >>> Are not we going to use page_is_ram() from =20 > e500_shadow_mas2_attrib() as Scott commented? > >> > >> rWhy aren't we using page_is_ram() in kvm_is_mmio_pfn()? > >> > >> -> > Because it is much slower and, IIRC, actually used to build pfn map +> > Because it is much slower and, IIRC, actually used to build pfn map =20 > that allow > > us to check quickly for valid pfn. -> +>=20 > Then why should we use page_is_ram()? :) -> -> I really don't want the e500 code to diverge too much from what the +>=20 +> I really don't want the e500 code to diverge too much from what the =20 > rest of the kvm code is doing. -I don't understand "actually used to build pfn map...". What code is -this? I don't see any calls to page_is_ram() in the KVM code, or in +I don't understand "actually used to build pfn map...". What code is =20 +this? I don't see any calls to page_is_ram() in the KVM code, or in =20 generic mm code. Is this a statement about what x86 does? -On PPC page_is_ram() is only called (AFAICT) for determining what -attributes to set on mmaps. We want to be sure that KVM always makes -the same decision. While pfn_valid() seems like it should be +On PPC page_is_ram() is only called (AFAICT) for determining what =20 +attributes to set on mmaps. We want to be sure that KVM always makes =20 +the same decision. While pfn_valid() seems like it should be =20 equivalent, it's not obvious from the PPC code that it is. -If pfn_valid() is better, why is that not used for mmap? Why are there +If pfn_valid() is better, why is that not used for mmap? Why are there =20 two different names for the same thing? --Scott +-Scott= diff --git a/a/content_digest b/N1/content_digest index 827b4ad..dfdc0a6 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,50 +1,50 @@ "ref\003EEFDFE-4603-44FC-8449-2450607F2864@suse.de\0" "From\0Scott Wood <scottwood@freescale.com>\0" "Subject\0Re: [PATCH 2/2] kvm: powerpc: set cache coherency only for kernel managed pages\0" - "Date\0Wed, 24 Jul 2013 20:32:49 +0000\0" + "Date\0Wed, 24 Jul 2013 15:32:49 -0500\0" "To\0Alexander Graf <agraf@suse.de>\0" - "Cc\0Gleb Natapov <gleb@redhat.com>" - Bhushan Bharat-R65777 <R65777@freescale.com> - " \342\200\234tiejun.chen\342\200\235 <tiejun.chen@windriver.com>" - kvm-ppc@vger.kernel.org <kvm-ppc@vger.kernel.org> + "Cc\0Wood Scott-B07421 <B07421@freescale.com>" + Gleb Natapov <gleb@redhat.com> kvm@vger.kernel.org list <kvm@vger.kernel.org> - Wood Scott-B07421 <B07421@freescale.com> + kvm-ppc@vger.kernel.org <kvm-ppc@vger.kernel.org> + " \342\200\234tiejun.chen\342\200\235 <tiejun.chen@windriver.com>" + Bhushan Bharat-R65777 <R65777@freescale.com> Paolo Bonzini <pbonzini@redhat.com> " linuxppc-dev@lists.ozlabs.org\0" "\00:1\0" "b\0" "On 07/24/2013 04:39:59 AM, Alexander Graf wrote:\n" - "> \n" + ">=20\n" "> On 24.07.2013, at 11:35, Gleb Natapov wrote:\n" - "> \n" + ">=20\n" "> > On Wed, Jul 24, 2013 at 11:21:11AM +0200, Alexander Graf wrote:\n" - "> >>> Are not we going to use page_is_ram() from \n" + "> >>> Are not we going to use page_is_ram() from =20\n" "> e500_shadow_mas2_attrib() as Scott commented?\n" "> >>\n" "> >> rWhy aren't we using page_is_ram() in kvm_is_mmio_pfn()?\n" "> >>\n" "> >>\n" - "> > Because it is much slower and, IIRC, actually used to build pfn map \n" + "> > Because it is much slower and, IIRC, actually used to build pfn map =20\n" "> that allow\n" "> > us to check quickly for valid pfn.\n" - "> \n" + ">=20\n" "> Then why should we use page_is_ram()? :)\n" - "> \n" - "> I really don't want the e500 code to diverge too much from what the \n" + ">=20\n" + "> I really don't want the e500 code to diverge too much from what the =20\n" "> rest of the kvm code is doing.\n" "\n" - "I don't understand \"actually used to build pfn map...\". What code is \n" - "this? I don't see any calls to page_is_ram() in the KVM code, or in \n" + "I don't understand \"actually used to build pfn map...\". What code is =20\n" + "this? I don't see any calls to page_is_ram() in the KVM code, or in =20\n" "generic mm code. Is this a statement about what x86 does?\n" "\n" - "On PPC page_is_ram() is only called (AFAICT) for determining what \n" - "attributes to set on mmaps. We want to be sure that KVM always makes \n" - "the same decision. While pfn_valid() seems like it should be \n" + "On PPC page_is_ram() is only called (AFAICT) for determining what =20\n" + "attributes to set on mmaps. We want to be sure that KVM always makes =20\n" + "the same decision. While pfn_valid() seems like it should be =20\n" "equivalent, it's not obvious from the PPC code that it is.\n" "\n" - "If pfn_valid() is better, why is that not used for mmap? Why are there \n" + "If pfn_valid() is better, why is that not used for mmap? Why are there =20\n" "two different names for the same thing?\n" "\n" - -Scott + -Scott= -3a1f1366caf669071b7b92aee6dbbe89ca49d10a2a3d35e8a6d67c2bfbdf03a6 +3f8ae4cd00fe6ed5d46b620a0fff93efd87431fc605edf8092b54b88cc89cecd
diff --git a/a/content_digest b/N2/content_digest index 827b4ad..b8d2e2b 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,7 +1,7 @@ "ref\003EEFDFE-4603-44FC-8449-2450607F2864@suse.de\0" "From\0Scott Wood <scottwood@freescale.com>\0" "Subject\0Re: [PATCH 2/2] kvm: powerpc: set cache coherency only for kernel managed pages\0" - "Date\0Wed, 24 Jul 2013 20:32:49 +0000\0" + "Date\0Wed, 24 Jul 2013 15:32:49 -0500\0" "To\0Alexander Graf <agraf@suse.de>\0" "Cc\0Gleb Natapov <gleb@redhat.com>" Bhushan Bharat-R65777 <R65777@freescale.com> @@ -47,4 +47,4 @@ "\n" -Scott -3a1f1366caf669071b7b92aee6dbbe89ca49d10a2a3d35e8a6d67c2bfbdf03a6 +11719364cd3791020da6756e79453d878676917abe6e784169c9294a9532d667
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.