diff for duplicates of <20190418163512.GA25347@lst.de> diff --git a/a/1.txt b/N1/1.txt index 4ed9eaa..8083f08 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -7,9 +7,13 @@ On Thu, Apr 18, 2019 at 04:06:56PM +0100, Robin Murphy wrote: > > Please let me know what you think. ->From a very high level POV this looks ok, but sometimes a bit to +From a very high level POV this looks ok, but sometimes a bit to convoluted to me. The major issue why I went with the version I posted is that I can cleanly ifdef out the remap code in just a few sections. In this version it is spread out a lot more, and the use of IS_ENABLED means that we'd need a lot more stubs for functionality that won't ever be called but needs to be compilable. +_______________________________________________ +iommu mailing list +iommu@lists.linux-foundation.org +https://lists.linuxfoundation.org/mailman/listinfo/iommu diff --git a/a/content_digest b/N1/content_digest index 5fefe88..097c18e 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -8,14 +8,13 @@ "Subject\0Re: [PATCH 12/21] dma-iommu: factor atomic pool allocations into helpers\0" "Date\0Thu, 18 Apr 2019 18:35:12 +0200\0" "To\0Robin Murphy <robin.murphy@arm.com>\0" - "Cc\0Christoph Hellwig <hch@lst.de>" - Joerg Roedel <joro@8bytes.org> + "Cc\0Tom Lendacky <thomas.lendacky@amd.com>" Catalin Marinas <catalin.marinas@arm.com> Will Deacon <will.deacon@arm.com> - Tom Lendacky <thomas.lendacky@amd.com> + linux-kernel@vger.kernel.org iommu@lists.linux-foundation.org - linux-arm-kernel@lists.infradead.org - " linux-kernel@vger.kernel.org\0" + Christoph Hellwig <hch@lst.de> + " linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Thu, Apr 18, 2019 at 04:06:56PM +0100, Robin Murphy wrote:\n" @@ -27,11 +26,15 @@ ">\n" "> Please let me know what you think.\n" "\n" - ">From a very high level POV this looks ok, but sometimes a bit to\n" + "From a very high level POV this looks ok, but sometimes a bit to\n" "convoluted to me. The major issue why I went with the version I posted\n" "is that I can cleanly ifdef out the remap code in just a few sections.\n" "In this version it is spread out a lot more, and the use of IS_ENABLED\n" "means that we'd need a lot more stubs for functionality that won't\n" - ever be called but needs to be compilable. + "ever be called but needs to be compilable.\n" + "_______________________________________________\n" + "iommu mailing list\n" + "iommu@lists.linux-foundation.org\n" + https://lists.linuxfoundation.org/mailman/listinfo/iommu -a2afb35a0442a9b371247570e9c3697745eafe288a2588cc4efd0aff7f3d7357 +aadfd9b56bfa4068c682f37445817400e23f2e0c58de16a0ee65eb8e9cef8494
diff --git a/a/1.txt b/N2/1.txt index 4ed9eaa..3f37ff8 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -7,9 +7,14 @@ On Thu, Apr 18, 2019 at 04:06:56PM +0100, Robin Murphy wrote: > > Please let me know what you think. ->From a very high level POV this looks ok, but sometimes a bit to +From a very high level POV this looks ok, but sometimes a bit to convoluted to me. The major issue why I went with the version I posted is that I can cleanly ifdef out the remap code in just a few sections. In this version it is spread out a lot more, and the use of IS_ENABLED means that we'd need a lot more stubs for functionality that won't ever be called but needs to be compilable. + +_______________________________________________ +linux-arm-kernel mailing list +linux-arm-kernel@lists.infradead.org +http://lists.infradead.org/mailman/listinfo/linux-arm-kernel diff --git a/a/content_digest b/N2/content_digest index 5fefe88..6bdd04d 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -8,14 +8,14 @@ "Subject\0Re: [PATCH 12/21] dma-iommu: factor atomic pool allocations into helpers\0" "Date\0Thu, 18 Apr 2019 18:35:12 +0200\0" "To\0Robin Murphy <robin.murphy@arm.com>\0" - "Cc\0Christoph Hellwig <hch@lst.de>" - Joerg Roedel <joro@8bytes.org> + "Cc\0Tom Lendacky <thomas.lendacky@amd.com>" Catalin Marinas <catalin.marinas@arm.com> + Joerg Roedel <joro@8bytes.org> Will Deacon <will.deacon@arm.com> - Tom Lendacky <thomas.lendacky@amd.com> + linux-kernel@vger.kernel.org iommu@lists.linux-foundation.org - linux-arm-kernel@lists.infradead.org - " linux-kernel@vger.kernel.org\0" + Christoph Hellwig <hch@lst.de> + " linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Thu, Apr 18, 2019 at 04:06:56PM +0100, Robin Murphy wrote:\n" @@ -27,11 +27,16 @@ ">\n" "> Please let me know what you think.\n" "\n" - ">From a very high level POV this looks ok, but sometimes a bit to\n" + "From a very high level POV this looks ok, but sometimes a bit to\n" "convoluted to me. The major issue why I went with the version I posted\n" "is that I can cleanly ifdef out the remap code in just a few sections.\n" "In this version it is spread out a lot more, and the use of IS_ENABLED\n" "means that we'd need a lot more stubs for functionality that won't\n" - ever be called but needs to be compilable. + "ever be called but needs to be compilable.\n" + "\n" + "_______________________________________________\n" + "linux-arm-kernel mailing list\n" + "linux-arm-kernel@lists.infradead.org\n" + http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -a2afb35a0442a9b371247570e9c3697745eafe288a2588cc4efd0aff7f3d7357 +96d67538acbab759c924e0e46ac542e4f0c268727552f8154a65695a01440217
diff --git a/a/1.txt b/N3/1.txt index 4ed9eaa..4925250 100644 --- a/a/1.txt +++ b/N3/1.txt @@ -7,7 +7,7 @@ On Thu, Apr 18, 2019 at 04:06:56PM +0100, Robin Murphy wrote: > > Please let me know what you think. ->From a very high level POV this looks ok, but sometimes a bit to +From a very high level POV this looks ok, but sometimes a bit to convoluted to me. The major issue why I went with the version I posted is that I can cleanly ifdef out the remap code in just a few sections. In this version it is spread out a lot more, and the use of IS_ENABLED diff --git a/a/content_digest b/N3/content_digest index 5fefe88..1b9f4c1 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -27,11 +27,11 @@ ">\n" "> Please let me know what you think.\n" "\n" - ">From a very high level POV this looks ok, but sometimes a bit to\n" + "From a very high level POV this looks ok, but sometimes a bit to\n" "convoluted to me. The major issue why I went with the version I posted\n" "is that I can cleanly ifdef out the remap code in just a few sections.\n" "In this version it is spread out a lot more, and the use of IS_ENABLED\n" "means that we'd need a lot more stubs for functionality that won't\n" ever be called but needs to be compilable. -a2afb35a0442a9b371247570e9c3697745eafe288a2588cc4efd0aff7f3d7357 +fc19cd5a40d2c8e27a736c20738ee4e4013932d66ac33791b51470480626d32e
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.