diff for duplicates of <1686102.s8m2vfSUhC@avalon> diff --git a/a/1.txt b/N1/1.txt index 5ff60dc..8dce75f 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -3,7 +3,7 @@ Hi Magnus, Thank you for the patch. On Tuesday 15 March 2016 13:21:55 Magnus Damm wrote: -> From: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org> +> From: Magnus Damm <damm+renesas@opensource.se> > > Introduce a bitmap for context handing and convert the > interrupt routine to go handle all registered contexts. @@ -13,7 +13,7 @@ On Tuesday 15 March 2016 13:21:55 Magnus Damm wrote: > Also remove the use of the ARM specific mapping variable > from ipmmu_irq() to allow compile on ARM64. > -> Signed-off-by: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org> +> Signed-off-by: Magnus Damm <damm+renesas@opensource.se> > --- > > Changes since V1: (Thanks to Laurent for feedback!) @@ -77,7 +77,7 @@ structure. > */ > - domain->context_id = 0; > + ret = find_first_zero_bit(domain->mmu->ctx, IPMMU_CTX_MAX); -> + if (ret == IPMMU_CTX_MAX) { +> + if (ret = IPMMU_CTX_MAX) { > + free_io_pgtable_ops(domain->iop); > + return -EBUSY; > + } @@ -144,7 +144,7 @@ test_bit() isn't atomic. Let's use explicit locking in every location where the contexts bitmap is accessed in a racy way. > + continue; -> + if (ipmmu_domain_irq(mmu->domains[i]) == IRQ_HANDLED) +> + if (ipmmu_domain_irq(mmu->domains[i]) = IRQ_HANDLED) > + status = IRQ_HANDLED; > + } > diff --git a/a/content_digest b/N1/content_digest index 2eb0534..4884e65 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,8 +1,8 @@ "ref\020160315042136.22103.26570.sendpatchset@little-apple\0" "ref\020160315042155.22103.74587.sendpatchset@little-apple\0" - "From\0Laurent Pinchart <laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>\0" + "From\0Laurent Pinchart <laurent.pinchart@ideasonboard.com>\0" "Subject\0Re: [PATCH v2 02/04] iommu/ipmmu-vmsa: Rework interrupt code and use bitmap for context\0" - "Date\0Thu, 17 Mar 2016 11:56:11 +0200\0" + "Date\0Thu, 17 Mar 2016 09:56:11 +0000\0" "To\0Magnus Damm <magnus.damm-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>\0" "Cc\0laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org" geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org @@ -17,7 +17,7 @@ "Thank you for the patch.\n" "\n" "On Tuesday 15 March 2016 13:21:55 Magnus Damm wrote:\n" - "> From: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org>\n" + "> From: Magnus Damm <damm+renesas@opensource.se>\n" "> \n" "> Introduce a bitmap for context handing and convert the\n" "> interrupt routine to go handle all registered contexts.\n" @@ -27,7 +27,7 @@ "> Also remove the use of the ARM specific mapping variable\n" "> from ipmmu_irq() to allow compile on ARM64.\n" "> \n" - "> Signed-off-by: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org>\n" + "> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>\n" "> ---\n" "> \n" "> Changes since V1: (Thanks to Laurent for feedback!)\n" @@ -91,7 +91,7 @@ "> \t */\n" "> -\tdomain->context_id = 0;\n" "> +\tret = find_first_zero_bit(domain->mmu->ctx, IPMMU_CTX_MAX);\n" - "> +\tif (ret == IPMMU_CTX_MAX) {\n" + "> +\tif (ret = IPMMU_CTX_MAX) {\n" "> +\t\tfree_io_pgtable_ops(domain->iop);\n" "> +\t\treturn -EBUSY;\n" "> +\t}\n" @@ -158,7 +158,7 @@ "the contexts bitmap is accessed in a racy way.\n" "\n" "> +\t\t\tcontinue;\n" - "> +\t\tif (ipmmu_domain_irq(mmu->domains[i]) == IRQ_HANDLED)\n" + "> +\t\tif (ipmmu_domain_irq(mmu->domains[i]) = IRQ_HANDLED)\n" "> +\t\t\tstatus = IRQ_HANDLED;\n" "> +\t}\n" "> \n" @@ -181,4 +181,4 @@ "\n" Laurent Pinchart -9a6ac949f1a9f89cc1b71c6fce2f632a6bb1085953a673a9d2f6c53849d6cc07 +357684534ccb75fd7f483abc07e083ef7c79118a0b2aad78635b707fadbeec09
diff --git a/a/1.txt b/N2/1.txt index 5ff60dc..171c759 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -3,7 +3,7 @@ Hi Magnus, Thank you for the patch. On Tuesday 15 March 2016 13:21:55 Magnus Damm wrote: -> From: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org> +> From: Magnus Damm <damm+renesas@opensource.se> > > Introduce a bitmap for context handing and convert the > interrupt routine to go handle all registered contexts. @@ -13,7 +13,7 @@ On Tuesday 15 March 2016 13:21:55 Magnus Damm wrote: > Also remove the use of the ARM specific mapping variable > from ipmmu_irq() to allow compile on ARM64. > -> Signed-off-by: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org> +> Signed-off-by: Magnus Damm <damm+renesas@opensource.se> > --- > > Changes since V1: (Thanks to Laurent for feedback!) diff --git a/a/content_digest b/N2/content_digest index 2eb0534..e7aebaf 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,15 +1,16 @@ "ref\020160315042136.22103.26570.sendpatchset@little-apple\0" "ref\020160315042155.22103.74587.sendpatchset@little-apple\0" - "From\0Laurent Pinchart <laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>\0" + "From\0Laurent Pinchart <laurent.pinchart@ideasonboard.com>\0" "Subject\0Re: [PATCH v2 02/04] iommu/ipmmu-vmsa: Rework interrupt code and use bitmap for context\0" "Date\0Thu, 17 Mar 2016 11:56:11 +0200\0" - "To\0Magnus Damm <magnus.damm-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>\0" - "Cc\0laurent.pinchart+renesas-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org" - geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org - linux-sh-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org - iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org - " horms+renesas-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org\0" + "To\0Magnus Damm <magnus.damm@gmail.com>\0" + "Cc\0iommu@lists.linux-foundation.org" + laurent.pinchart+renesas@ideasonboard.com + geert+renesas@glider.be + linux-sh@vger.kernel.org + joro@8bytes.org + linux-kernel@vger.kernel.org + " horms+renesas@verge.net.au\0" "\00:1\0" "b\0" "Hi Magnus,\n" @@ -17,7 +18,7 @@ "Thank you for the patch.\n" "\n" "On Tuesday 15 March 2016 13:21:55 Magnus Damm wrote:\n" - "> From: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org>\n" + "> From: Magnus Damm <damm+renesas@opensource.se>\n" "> \n" "> Introduce a bitmap for context handing and convert the\n" "> interrupt routine to go handle all registered contexts.\n" @@ -27,7 +28,7 @@ "> Also remove the use of the ARM specific mapping variable\n" "> from ipmmu_irq() to allow compile on ARM64.\n" "> \n" - "> Signed-off-by: Magnus Damm <damm+renesas-yzvPICuk2ACczHhG9Qg4qA@public.gmane.org>\n" + "> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>\n" "> ---\n" "> \n" "> Changes since V1: (Thanks to Laurent for feedback!)\n" @@ -181,4 +182,4 @@ "\n" Laurent Pinchart -9a6ac949f1a9f89cc1b71c6fce2f632a6bb1085953a673a9d2f6c53849d6cc07 +f73942398747b43ed25212d8d7f7477926620509f566c3fe82fd0a2f76e29749
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.