diff for duplicates of <20180517150516.000067ca@huawei.com> diff --git a/a/1.txt b/N1/1.txt index 3760bfe..d39af98 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,5 +1,5 @@ On Fri, 11 May 2018 20:06:04 +0100 -Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> wrote: +Jean-Philippe Brucker <jean-philippe.brucker@arm.com> wrote: > Allocate IOMMU mm structures and binding them to devices. Four operations > are added to IOMMU drivers: @@ -44,7 +44,7 @@ Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> wrote > Handling process exit with regard to unbind() is tricky, so it is left for > another patch and we explicitly fail mm_alloc() for the moment. > -> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> +> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker@arm.com> A few minor bits and bobs inline. Looks good in general + nice diags! diff --git a/a/content_digest b/N1/content_digest index 047cbb5..27c0bd1 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,31 +1,46 @@ "ref\020180511190641.23008-1-jean-philippe.brucker@arm.com\0" "ref\020180511190641.23008-4-jean-philippe.brucker@arm.com\0" - "ref\020180511190641.23008-4-jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org\0" - "From\0Jonathan Cameron <Jonathan.Cameron-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>\0" + "From\0Jonathan Cameron <Jonathan.Cameron@huawei.com>\0" "Subject\0Re: [PATCH v2 03/40] iommu/sva: Manage process address spaces\0" "Date\0Thu, 17 May 2018 15:25:07 +0100\0" - "To\0Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org>\0" - "Cc\0kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" - linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - xuzaibo-hv44wF8Li93QT0dZR+AlfA@public.gmane.org - will.deacon-5wv7dgnIgG8@public.gmane.org - okaya-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org - linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org - ashok.raj-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org - bharatku-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org - linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - rfranz-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org - devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - rgummal-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org - linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org - dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org - ilias.apalodimas-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org - iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org - " christian.koenig-5C7GfCeVMHo@public.gmane.org\0" + "To\0Jean-Philippe Brucker <jean-philippe.brucker@arm.com>\0" + "Cc\0<linux-arm-kernel@lists.infradead.org>" + <linux-pci@vger.kernel.org> + <linux-acpi@vger.kernel.org> + <devicetree@vger.kernel.org> + <iommu@lists.linux-foundation.org> + <kvm@vger.kernel.org> + <linux-mm@kvack.org> + <joro@8bytes.org> + <will.deacon@arm.com> + <robin.murphy@arm.com> + <alex.williamson@redhat.com> + <tn@semihalf.com> + <liubo95@huawei.com> + <thunder.leizhen@huawei.com> + <xieyisheng1@huawei.com> + <xuzaibo@huawei.com> + <ilias.apalodimas@linaro.org> + <liudongdong3@huawei.com> + <shunyong.yang@hxt-semitech.com> + <nwatters@codeaurora.org> + <okaya@codeaurora.org> + <jcrouse@codeaurora.org> + <rfranz@cavium.com> + <dwmw2@infradead.org> + <jacob.jun.pan@linux.intel.com> + <yi.l.liu@intel.com> + <ashok.raj@intel.com> + <kevin.tian@intel.com> + <baolu.lu@linux.intel.com> + <robdclark@gmail.com> + <christian.koenig@amd.com> + <bharatku@xilinx.com> + " <rgummal@xilinx.com>\0" "\00:1\0" "b\0" "On Fri, 11 May 2018 20:06:04 +0100\n" - "Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> wrote:\n" + "Jean-Philippe Brucker <jean-philippe.brucker@arm.com> wrote:\n" "\n" "> Allocate IOMMU mm structures and binding them to devices. Four operations\n" "> are added to IOMMU drivers:\n" @@ -70,7 +85,7 @@ "> Handling process exit with regard to unbind() is tricky, so it is left for\n" "> another patch and we explicitly fail mm_alloc() for the moment.\n" "> \n" - "> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org>\n" + "> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker@arm.com>\n" "\n" "A few minor bits and bobs inline. Looks good in general + nice diags!\n" "\n" @@ -611,4 +626,4 @@ "> \t\t phys_addr_t paddr, size_t size, int prot);\n" "> \tsize_t (*unmap)(struct iommu_domain *domain, unsigned long iova," -add84b9dae79fb4798afdb50c1862a80b66675d2a0331e4d6abe4155f5bcdc6e +17c3b6fd4ada0d4cfff4d4806c74571e946be54be943227d7863f51d49e2e27f
diff --git a/a/1.txt b/N2/1.txt index 3760bfe..d39af98 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,5 +1,5 @@ On Fri, 11 May 2018 20:06:04 +0100 -Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> wrote: +Jean-Philippe Brucker <jean-philippe.brucker@arm.com> wrote: > Allocate IOMMU mm structures and binding them to devices. Four operations > are added to IOMMU drivers: @@ -44,7 +44,7 @@ Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> wrote > Handling process exit with regard to unbind() is tricky, so it is left for > another patch and we explicitly fail mm_alloc() for the moment. > -> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> +> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker@arm.com> A few minor bits and bobs inline. Looks good in general + nice diags! diff --git a/a/content_digest b/N2/content_digest index 047cbb5..163bd7f 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,31 +1,13 @@ "ref\020180511190641.23008-1-jean-philippe.brucker@arm.com\0" "ref\020180511190641.23008-4-jean-philippe.brucker@arm.com\0" - "ref\020180511190641.23008-4-jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org\0" - "From\0Jonathan Cameron <Jonathan.Cameron-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>\0" - "Subject\0Re: [PATCH v2 03/40] iommu/sva: Manage process address spaces\0" + "From\0Jonathan.Cameron@huawei.com (Jonathan Cameron)\0" + "Subject\0[PATCH v2 03/40] iommu/sva: Manage process address spaces\0" "Date\0Thu, 17 May 2018 15:25:07 +0100\0" - "To\0Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org>\0" - "Cc\0kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" - linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - xuzaibo-hv44wF8Li93QT0dZR+AlfA@public.gmane.org - will.deacon-5wv7dgnIgG8@public.gmane.org - okaya-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org - linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org - ashok.raj-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org - bharatku-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org - linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - rfranz-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org - devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - rgummal-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org - linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org - dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org - ilias.apalodimas-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org - iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org - " christian.koenig-5C7GfCeVMHo@public.gmane.org\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Fri, 11 May 2018 20:06:04 +0100\n" - "Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> wrote:\n" + "Jean-Philippe Brucker <jean-philippe.brucker@arm.com> wrote:\n" "\n" "> Allocate IOMMU mm structures and binding them to devices. Four operations\n" "> are added to IOMMU drivers:\n" @@ -70,7 +52,7 @@ "> Handling process exit with regard to unbind() is tricky, so it is left for\n" "> another patch and we explicitly fail mm_alloc() for the moment.\n" "> \n" - "> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org>\n" + "> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker@arm.com>\n" "\n" "A few minor bits and bobs inline. Looks good in general + nice diags!\n" "\n" @@ -611,4 +593,4 @@ "> \t\t phys_addr_t paddr, size_t size, int prot);\n" "> \tsize_t (*unmap)(struct iommu_domain *domain, unsigned long iova," -add84b9dae79fb4798afdb50c1862a80b66675d2a0331e4d6abe4155f5bcdc6e +a4c26c0b33ad43479fe049c6ab6025cbaf6d494a7470466d409ff0f9981810ff
diff --git a/a/1.txt b/N3/1.txt index 3760bfe..ab82918 100644 --- a/a/1.txt +++ b/N3/1.txt @@ -1,5 +1,5 @@ On Fri, 11 May 2018 20:06:04 +0100 -Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> wrote: +Jean-Philippe Brucker <jean-philippe.brucker@arm.com> wrote: > Allocate IOMMU mm structures and binding them to devices. Four operations > are added to IOMMU drivers: @@ -44,7 +44,7 @@ Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> wrote > Handling process exit with regard to unbind() is tricky, so it is left for > another patch and we explicitly fail mm_alloc() for the moment. > -> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> +> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker@arm.com> A few minor bits and bobs inline. Looks good in general + nice diags! @@ -242,7 +242,7 @@ free.. If so a comment to explain this would be great. > + > + /* TODO: keep track of mm. For the moment, abort. */ ->From later patches, I can now see why we didn't init the kref +From later patches, I can now see why we didn't init the kref here, but perhaps a comment would make that clear rather than people checking it is correctly used throughout? Actually just grab the comment from patch 5 and put it in this one and that will diff --git a/a/content_digest b/N3/content_digest index 047cbb5..ec5cf8d 100644 --- a/a/content_digest +++ b/N3/content_digest @@ -1,31 +1,46 @@ "ref\020180511190641.23008-1-jean-philippe.brucker@arm.com\0" "ref\020180511190641.23008-4-jean-philippe.brucker@arm.com\0" - "ref\020180511190641.23008-4-jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org\0" - "From\0Jonathan Cameron <Jonathan.Cameron-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>\0" + "From\0Jonathan Cameron <Jonathan.Cameron@huawei.com>\0" "Subject\0Re: [PATCH v2 03/40] iommu/sva: Manage process address spaces\0" "Date\0Thu, 17 May 2018 15:25:07 +0100\0" - "To\0Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org>\0" - "Cc\0kvm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" - linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - xuzaibo-hv44wF8Li93QT0dZR+AlfA@public.gmane.org - will.deacon-5wv7dgnIgG8@public.gmane.org - okaya-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org - linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org - ashok.raj-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org - bharatku-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org - linux-acpi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - rfranz-YGCgFSpz5w/QT0dZR+AlfA@public.gmane.org - devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - rgummal-gjFFaj9aHVfQT0dZR+AlfA@public.gmane.org - linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org - dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org - ilias.apalodimas-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org - iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org - " christian.koenig-5C7GfCeVMHo@public.gmane.org\0" + "To\0Jean-Philippe Brucker <jean-philippe.brucker@arm.com>\0" + "Cc\0linux-arm-kernel@lists.infradead.org" + linux-pci@vger.kernel.org + linux-acpi@vger.kernel.org + devicetree@vger.kernel.org + iommu@lists.linux-foundation.org + kvm@vger.kernel.org + linux-mm@kvack.org + joro@8bytes.org + will.deacon@arm.com + robin.murphy@arm.com + alex.williamson@redhat.com + tn@semihalf.com + liubo95@huawei.com + thunder.leizhen@huawei.com + xieyisheng1@huawei.com + xuzaibo@huawei.com + ilias.apalodimas@linaro.org + liudongdong3@huawei.com + shunyong.yang@hxt-semitech.com + nwatters@codeaurora.org + okaya@codeaurora.org + jcrouse@codeaurora.org + rfranz@cavium.com + dwmw2@infradead.org + jacob.jun.pan@linux.intel.com + yi.l.liu@intel.com + ashok.raj@intel.com + kevin.tian@intel.com + baolu.lu@linux.intel.com + robdclark@gmail.com + christian.koenig@amd.com + bharatku@xilinx.com + " rgummal@xilinx.com\0" "\00:1\0" "b\0" "On Fri, 11 May 2018 20:06:04 +0100\n" - "Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org> wrote:\n" + "Jean-Philippe Brucker <jean-philippe.brucker@arm.com> wrote:\n" "\n" "> Allocate IOMMU mm structures and binding them to devices. Four operations\n" "> are added to IOMMU drivers:\n" @@ -70,7 +85,7 @@ "> Handling process exit with regard to unbind() is tricky, so it is left for\n" "> another patch and we explicitly fail mm_alloc() for the moment.\n" "> \n" - "> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker-5wv7dgnIgG8@public.gmane.org>\n" + "> Signed-off-by: Jean-Philippe Brucker <jean-philippe.brucker@arm.com>\n" "\n" "A few minor bits and bobs inline. Looks good in general + nice diags!\n" "\n" @@ -268,7 +283,7 @@ "> +\n" "> +\t/* TODO: keep track of mm. For the moment, abort. */\n" "\n" - ">From later patches, I can now see why we didn't init the kref\n" + "From later patches, I can now see why we didn't init the kref\n" "here, but perhaps a comment would make that clear rather than\n" "people checking it is correctly used throughout? Actually just grab\n" "the comment from patch 5 and put it in this one and that will\n" @@ -611,4 +626,4 @@ "> \t\t phys_addr_t paddr, size_t size, int prot);\n" "> \tsize_t (*unmap)(struct iommu_domain *domain, unsigned long iova," -add84b9dae79fb4798afdb50c1862a80b66675d2a0331e4d6abe4155f5bcdc6e +2a2c501b663566eb6e1a87d64322500a983efbcad2c67f7b7c7ddf75507014a4
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.