diff for duplicates of <1493133369.25985.10.camel@synopsys.com> diff --git a/a/1.txt b/N1/1.txt index c505d91..d886e9f 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,10 +1,10 @@ -On Mon, 2017-04-24@19:56 +0300, Andy Shevchenko wrote: -> On Mon, 2017-04-24@15:55 +0000, Eugeniy Paltsev wrote: +On Mon, 2017-04-24 at 19:56 +0300, Andy Shevchenko wrote: +> On Mon, 2017-04-24 at 15:55 +0000, Eugeniy Paltsev wrote: > > Hi, -> > On Fri, 2017-04-21@18:13 +0300, Andy Shevchenko wrote: -> > > On Fri, 2017-04-21@14:29 +0000, Eugeniy Paltsev wrote: -> > > > On Tue, 2017-04-18@15:31 +0300, Andy Shevchenko wrote: -> > > > > On Fri, 2017-04-07@17:04 +0300, Eugeniy Paltsev wrote: +> > On Fri, 2017-04-21 at 18:13 +0300, Andy Shevchenko wrote: +> > > On Fri, 2017-04-21 at 14:29 +0000, Eugeniy Paltsev wrote: +> > > > On Tue, 2017-04-18 at 15:31 +0300, Andy Shevchenko wrote: +> > > > > On Fri, 2017-04-07 at 17:04 +0300, Eugeniy Paltsev wrote: > > > > > > This patch adds support for the DW AXI DMAC controller. > > > > > > +static inline void > > > > > > +axi_dma_iowrite32(struct axi_dma_chip *chip, u32 reg, u32 @@ -102,7 +102,7 @@ There is no performance profit: we anyway need to re-splitt sg_list If we want to use same descriptors several times we just can use DMA_CTRL_REUSE option - the descriptors will be created one time and -re-splitting will be ?ompleted only one time. +re-splitting will be Ñompleted only one time. But there are cons of this implementation: we need to implement re-splitting mechanism in each place we use dma @@ -177,12 +177,12 @@ Ok, I'll drop prioritization untill I implement cyclic transfers. > See below. > > > I checked hsu/hsu.c dma driver implementation: -> >???vdma descriptor is deleted from desc_issued list when transfer -> >???starts. When descriptor marked as error descriptor -> >???vchan_cookie_complete isn't called for this descriptor. And this -> >???descriptor isn't placed in any list. So error descriptors *never* -> >???will be freed. -> >???I don't actually like this approach. +> >   vdma descriptor is deleted from desc_issued list when transfer +> >   starts. When descriptor marked as error descriptor +> >   vchan_cookie_complete isn't called for this descriptor. And this +> >   descriptor isn't placed in any list. So error descriptors *never* +> >   will be freed. +> >   I don't actually like this approach. > > Descriptor is active until terminate_all() is called or new > descriptor @@ -323,4 +323,4 @@ Ok, I'll recheck others implementation. Ok. -- -?Eugeniy Paltsev + Eugeniy PaltsevN‹§²æìr¸›yúèšØb²X¬¶Ç§vØ^–)Þº{.nÇ+‰·zøœzÚÞz)í…æèw*\x1fjg¬±¨\x1e¶‰šŽŠÝ¢j.ïÛ°\½½MŽúgjÌæa×\x02››–' ™©Þ¢¸\f¢·¦j:+v‰¨ŠwèjØm¶Ÿÿ¾\a«‘êçzZ+ƒùšŽŠÝ¢j"ú!¶i diff --git a/a/content_digest b/N1/content_digest index 4d94c86..6aba2c3 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -5,19 +5,29 @@ "ref\01492787583.24567.120.camel@linux.intel.com\0" "ref\01493049305.25985.4.camel@synopsys.com\0" "ref\01493052970.24567.168.camel@linux.intel.com\0" - "From\0Eugeniy.Paltsev@synopsys.com (Eugeniy Paltsev)\0" - "Subject\0[PATCH v2 2/2] dmaengine: Add DW AXI DMAC driver\0" + "ref\01493052970.24567.168.camel-VuQAYsv1563Yd54FQh9/CA@public.gmane.org\0" + "From\0Eugeniy Paltsev <Eugeniy.Paltsev-HKixBCOQz3hWk0Htik3J/w@public.gmane.org>\0" + "Subject\0Re: [PATCH v2 2/2] dmaengine: Add DW AXI DMAC driver\0" "Date\0Tue, 25 Apr 2017 15:16:10 +0000\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org <andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>\0" + "Cc\0vinod.koul-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org <vinod.koul-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>" + linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> + robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> + Alexey.Brodkin-HKixBCOQz3hWk0Htik3J/w@public.gmane.org <Alexey.Brodkin-HKixBCOQz3hWk0Htik3J/w@public.gmane.org> + devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org> + Eugeniy.Paltsev-HKixBCOQz3hWk0Htik3J/w@public.gmane.org <Eugeniy.Paltsev-HKixBCOQz3hWk0Htik3J/w@public.gmane.org> + linux-snps-arc-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org <linux-snps-arc-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org> + dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org <dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org> + " dmaengine-u79uwXL29TY76Z2rM5mHXA@public.gmane.org <dmaengine-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>\0" "\00:1\0" "b\0" - "On Mon, 2017-04-24@19:56 +0300, Andy Shevchenko wrote:\n" - "> On Mon, 2017-04-24@15:55 +0000, Eugeniy Paltsev wrote:\n" + "On Mon, 2017-04-24 at 19:56 +0300, Andy Shevchenko wrote:\n" + "> On Mon, 2017-04-24 at 15:55 +0000, Eugeniy Paltsev wrote:\n" "> > Hi,\n" - "> > On Fri, 2017-04-21@18:13 +0300, Andy Shevchenko wrote:\n" - "> > > On Fri, 2017-04-21@14:29 +0000, Eugeniy Paltsev wrote:\n" - "> > > > On Tue, 2017-04-18@15:31 +0300, Andy Shevchenko wrote:\n" - "> > > > > On Fri, 2017-04-07@17:04 +0300, Eugeniy Paltsev wrote:\n" + "> > On Fri, 2017-04-21 at 18:13 +0300, Andy Shevchenko wrote:\n" + "> > > On Fri, 2017-04-21 at 14:29 +0000, Eugeniy Paltsev wrote:\n" + "> > > > On Tue, 2017-04-18 at 15:31 +0300, Andy Shevchenko wrote:\n" + "> > > > > On Fri, 2017-04-07 at 17:04 +0300, Eugeniy Paltsev wrote:\n" "> > > > > > This patch adds support for the DW AXI DMAC controller.\n" "> > > > > > +static inline void\n" "> > > > > > +axi_dma_iowrite32(struct axi_dma_chip *chip, u32 reg, u32\n" @@ -115,7 +125,7 @@ "\n" "If we want to use same descriptors several times we just can use\n" "DMA_CTRL_REUSE option - the descriptors will be created one time and\n" - "re-splitting will be ?ompleted only one time.\n" + "re-splitting will be \303\221\302\201ompleted only one time.\n" "\n" "But there are cons of this implementation:\n" "we need to implement re-splitting mechanism in each place we use dma\n" @@ -190,12 +200,12 @@ "> See below.\n" ">\n" "> > I checked hsu/hsu.c dma driver implementation:\n" - "> >???vdma descriptor is deleted from desc_issued list when transfer\n" - "> >???starts. When descriptor marked as error descriptor\n" - "> >???vchan_cookie_complete isn't called for this descriptor. And this\n" - "> >???descriptor isn't placed in any list. So error descriptors *never*\n" - "> >???will be freed.\n" - "> >???I don't actually like this approach.\n" + "> >\303\202\302\240\303\202\302\240\303\202\302\240vdma descriptor is deleted from desc_issued list when transfer\n" + "> >\303\202\302\240\303\202\302\240\303\202\302\240starts. When descriptor marked as error descriptor\n" + "> >\303\202\302\240\303\202\302\240\303\202\302\240vchan_cookie_complete isn't called for this descriptor. And this\n" + "> >\303\202\302\240\303\202\302\240\303\202\302\240descriptor isn't placed in any list. So error descriptors *never*\n" + "> >\303\202\302\240\303\202\302\240\303\202\302\240will be freed.\n" + "> >\303\202\302\240\303\202\302\240\303\202\302\240I don't actually like this approach.\n" ">\n" "> Descriptor is active until terminate_all() is called or new\n" "> descriptor\n" @@ -336,6 +346,6 @@ "Ok.\n" "\n" "--\n" - ?Eugeniy Paltsev + "\303\202\302\240Eugeniy PaltsevN\302\213\302\247\302\262\303\246\303\254r\302\270\302\233y\303\272\303\250\302\232\303\230b\302\262X\302\254\302\266\303\207\302\247v\303\230^\302\226)\303\236\302\272{.n\303\207+\302\211\302\267\302\235z\303\270\302\234z\303\232\303\236z)\303\255\302\205\303\246\303\250w*\037jg\302\254\302\261\302\250\036\302\266\302\211\302\232\302\216\302\212\303\235\302\242j.\303\257\303\233\302\260\\\302\275\302\275M\302\216\303\272gj\303\214\303\246a\303\227\002\302\233\302\233\302\226' \302\231\302\251\303\236\302\242\302\270\f\302\242\302\267\302\246j:+v\302\211\302\250\302\212w\303\250j\303\230m\302\266\302\237\303\277\302\276\a\302\253\302\221\303\252\303\247zZ+\302\203\303\271\302\232\302\216\302\212\303\235\302\242j\"\302\235\303\272!\302\266i" -df92fd212910b7e77f12b463dcfbbedccc0955755a075ca844cd4e52227ab02f +df77e2d8828f54772eda2d06c3a8413ad0effb30113d634e54ed9e30f55d2a9e
diff --git a/a/1.txt b/N2/1.txt index c505d91..e690ae9 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,10 +1,10 @@ -On Mon, 2017-04-24@19:56 +0300, Andy Shevchenko wrote: -> On Mon, 2017-04-24@15:55 +0000, Eugeniy Paltsev wrote: +On Mon, 2017-04-24 at 19:56 +0300, Andy Shevchenko wrote: +> On Mon, 2017-04-24 at 15:55 +0000, Eugeniy Paltsev wrote: > > Hi, -> > On Fri, 2017-04-21@18:13 +0300, Andy Shevchenko wrote: -> > > On Fri, 2017-04-21@14:29 +0000, Eugeniy Paltsev wrote: -> > > > On Tue, 2017-04-18@15:31 +0300, Andy Shevchenko wrote: -> > > > > On Fri, 2017-04-07@17:04 +0300, Eugeniy Paltsev wrote: +> > On Fri, 2017-04-21 at 18:13 +0300, Andy Shevchenko wrote: +> > > On Fri, 2017-04-21 at 14:29 +0000, Eugeniy Paltsev wrote: +> > > > On Tue, 2017-04-18 at 15:31 +0300, Andy Shevchenko wrote: +> > > > > On Fri, 2017-04-07 at 17:04 +0300, Eugeniy Paltsev wrote: > > > > > > This patch adds support for the DW AXI DMAC controller. > > > > > > +static inline void > > > > > > +axi_dma_iowrite32(struct axi_dma_chip *chip, u32 reg, u32 @@ -102,7 +102,7 @@ There is no performance profit: we anyway need to re-splitt sg_list If we want to use same descriptors several times we just can use DMA_CTRL_REUSE option - the descriptors will be created one time and -re-splitting will be ?ompleted only one time. +re-splitting will be сompleted only one time. But there are cons of this implementation: we need to implement re-splitting mechanism in each place we use dma @@ -177,12 +177,12 @@ Ok, I'll drop prioritization untill I implement cyclic transfers. > See below. > > > I checked hsu/hsu.c dma driver implementation: -> >???vdma descriptor is deleted from desc_issued list when transfer -> >???starts. When descriptor marked as error descriptor -> >???vchan_cookie_complete isn't called for this descriptor. And this -> >???descriptor isn't placed in any list. So error descriptors *never* -> >???will be freed. -> >???I don't actually like this approach. +> > vdma descriptor is deleted from desc_issued list when transfer +> > starts. When descriptor marked as error descriptor +> > vchan_cookie_complete isn't called for this descriptor. And this +> > descriptor isn't placed in any list. So error descriptors *never* +> > will be freed. +> > I don't actually like this approach. > > Descriptor is active until terminate_all() is called or new > descriptor @@ -323,4 +323,4 @@ Ok, I'll recheck others implementation. Ok. -- -?Eugeniy Paltsev + Eugeniy Paltsev diff --git a/a/content_digest b/N2/content_digest index 4d94c86..de429e0 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -5,19 +5,28 @@ "ref\01492787583.24567.120.camel@linux.intel.com\0" "ref\01493049305.25985.4.camel@synopsys.com\0" "ref\01493052970.24567.168.camel@linux.intel.com\0" - "From\0Eugeniy.Paltsev@synopsys.com (Eugeniy Paltsev)\0" - "Subject\0[PATCH v2 2/2] dmaengine: Add DW AXI DMAC driver\0" + "From\0Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com>\0" + "Subject\0Re: [PATCH v2 2/2] dmaengine: Add DW AXI DMAC driver\0" "Date\0Tue, 25 Apr 2017 15:16:10 +0000\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0andriy.shevchenko@linux.intel.com <andriy.shevchenko@linux.intel.com>\0" + "Cc\0vinod.koul@intel.com <vinod.koul@intel.com>" + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + robh+dt@kernel.org <robh+dt@kernel.org> + Alexey.Brodkin@synopsys.com <Alexey.Brodkin@synopsys.com> + devicetree@vger.kernel.org <devicetree@vger.kernel.org> + Eugeniy.Paltsev@synopsys.com <Eugeniy.Paltsev@synopsys.com> + linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org> + dan.j.williams@intel.com <dan.j.williams@intel.com> + " dmaengine@vger.kernel.org <dmaengine@vger.kernel.org>\0" "\00:1\0" "b\0" - "On Mon, 2017-04-24@19:56 +0300, Andy Shevchenko wrote:\n" - "> On Mon, 2017-04-24@15:55 +0000, Eugeniy Paltsev wrote:\n" + "On Mon, 2017-04-24 at 19:56 +0300, Andy Shevchenko wrote:\n" + "> On Mon, 2017-04-24 at 15:55 +0000, Eugeniy Paltsev wrote:\n" "> > Hi,\n" - "> > On Fri, 2017-04-21@18:13 +0300, Andy Shevchenko wrote:\n" - "> > > On Fri, 2017-04-21@14:29 +0000, Eugeniy Paltsev wrote:\n" - "> > > > On Tue, 2017-04-18@15:31 +0300, Andy Shevchenko wrote:\n" - "> > > > > On Fri, 2017-04-07@17:04 +0300, Eugeniy Paltsev wrote:\n" + "> > On Fri, 2017-04-21 at 18:13 +0300, Andy Shevchenko wrote:\n" + "> > > On Fri, 2017-04-21 at 14:29 +0000, Eugeniy Paltsev wrote:\n" + "> > > > On Tue, 2017-04-18 at 15:31 +0300, Andy Shevchenko wrote:\n" + "> > > > > On Fri, 2017-04-07 at 17:04 +0300, Eugeniy Paltsev wrote:\n" "> > > > > > This patch adds support for the DW AXI DMAC controller.\n" "> > > > > > +static inline void\n" "> > > > > > +axi_dma_iowrite32(struct axi_dma_chip *chip, u32 reg, u32\n" @@ -115,7 +124,7 @@ "\n" "If we want to use same descriptors several times we just can use\n" "DMA_CTRL_REUSE option - the descriptors will be created one time and\n" - "re-splitting will be ?ompleted only one time.\n" + "re-splitting will be \321\201ompleted only one time.\n" "\n" "But there are cons of this implementation:\n" "we need to implement re-splitting mechanism in each place we use dma\n" @@ -190,12 +199,12 @@ "> See below.\n" ">\n" "> > I checked hsu/hsu.c dma driver implementation:\n" - "> >???vdma descriptor is deleted from desc_issued list when transfer\n" - "> >???starts. When descriptor marked as error descriptor\n" - "> >???vchan_cookie_complete isn't called for this descriptor. And this\n" - "> >???descriptor isn't placed in any list. So error descriptors *never*\n" - "> >???will be freed.\n" - "> >???I don't actually like this approach.\n" + "> >\302\240\302\240\302\240vdma descriptor is deleted from desc_issued list when transfer\n" + "> >\302\240\302\240\302\240starts. When descriptor marked as error descriptor\n" + "> >\302\240\302\240\302\240vchan_cookie_complete isn't called for this descriptor. And this\n" + "> >\302\240\302\240\302\240descriptor isn't placed in any list. So error descriptors *never*\n" + "> >\302\240\302\240\302\240will be freed.\n" + "> >\302\240\302\240\302\240I don't actually like this approach.\n" ">\n" "> Descriptor is active until terminate_all() is called or new\n" "> descriptor\n" @@ -336,6 +345,6 @@ "Ok.\n" "\n" "--\n" - ?Eugeniy Paltsev + "\302\240Eugeniy Paltsev" -df92fd212910b7e77f12b463dcfbbedccc0955755a075ca844cd4e52227ab02f +e6076062bd2a6f10b7575aaec919e978ce186d199a55fe2293e200c221a848fe
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.