| Message ID | 1463490282-23277-1-git-send-email-christophe.milard@linaro.org |
|---|---|
| State | New |
| Headers | show |
On 05/18/16 14:14, Savolainen, Petri (Nokia - FI/Espoo) wrote: > Reviewed patches 1-10 > > Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com> Petri, Christophe, are you ok with merging 1-10 first, and next after that? Maxim. > >> -----Original Message----- >> From: lng-odp [mailto:lng-odp-bounces@lists.linaro.org] On Behalf Of >> Christophe Milard >> Sent: Tuesday, May 17, 2016 4:04 PM >> To: lng-odp@lists.linaro.org; brian.brooks@linaro.org; >> mike.holmes@linaro.org >> Subject: [lng-odp] [PATCHv7 00/35] running things in process mode >> >> Pull can be performed from the URL at the end of this cover letter >> All patches sent to the list following Maxim's request. >> >> Since v6 >> -rebased >> -regrouping history, squashing patch 1 and 12; 4, 5 and 12 (Petri) >> -mixed mode test removed (but still claimed to be supported in ODP). >> (Petri) >> > _______________________________________________ > lng-odp mailing list > lng-odp@lists.linaro.org > https://lists.linaro.org/mailman/listinfo/lng-odp
This is, of course very strange to me, as the usage of the "old" functions will keep spreading, and we'll be stopping halv way done. I guess Brian can apply his review-by, as patch 11-35 are the same as v6 But if the alternative is nothing done... Christophe. On 18 May 2016 at 17:02, Maxim Uvarov <maxim.uvarov@linaro.org> wrote: > On 05/18/16 14:14, Savolainen, Petri (Nokia - FI/Espoo) wrote: > >> Reviewed patches 1-10 >> >> Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com> >> > Petri, Christophe, are you ok with merging 1-10 first, and next after that? > > Maxim. > >> >> -----Original Message----- >>> From: lng-odp [mailto:lng-odp-bounces@lists.linaro.org] On Behalf Of >>> Christophe Milard >>> Sent: Tuesday, May 17, 2016 4:04 PM >>> To: lng-odp@lists.linaro.org; brian.brooks@linaro.org; >>> mike.holmes@linaro.org >>> Subject: [lng-odp] [PATCHv7 00/35] running things in process mode >>> >>> Pull can be performed from the URL at the end of this cover letter >>> All patches sent to the list following Maxim's request. >>> >>> Since v6 >>> -rebased >>> -regrouping history, squashing patch 1 and 12; 4, 5 and 12 (Petri) >>> -mixed mode test removed (but still claimed to be supported in ODP). >>> (Petri) >>> >>> _______________________________________________ >> lng-odp mailing list >> lng-odp@lists.linaro.org >> https://lists.linaro.org/mailman/listinfo/lng-odp >> > > _______________________________________________ > lng-odp mailing list > lng-odp@lists.linaro.org > https://lists.linaro.org/mailman/listinfo/lng-odp >
From the call earlier I understood that we agreed to the whole series, we just need the Reviewed-by, they should all go in if reviewed unless there is a ncak that needs resloving. On 18 May 2016 at 11:18, Christophe Milard <christophe.milard@linaro.org> wrote: > This is, of course very strange to me, as the usage of the "old" functions > will keep spreading, and we'll be stopping halv way done. > I guess Brian can apply his review-by, as patch 11-35 are the same as v6 > > But if the alternative is nothing done... > > Christophe. > > On 18 May 2016 at 17:02, Maxim Uvarov <maxim.uvarov@linaro.org> wrote: > >> On 05/18/16 14:14, Savolainen, Petri (Nokia - FI/Espoo) wrote: >> >>> Reviewed patches 1-10 >>> >>> Reviewed-by: Petri Savolainen <petri.savolainen@nokia.com> >>> >> Petri, Christophe, are you ok with merging 1-10 first, and next after >> that? >> >> Maxim. >> >>> >>> -----Original Message----- >>>> From: lng-odp [mailto:lng-odp-bounces@lists.linaro.org] On Behalf Of >>>> Christophe Milard >>>> Sent: Tuesday, May 17, 2016 4:04 PM >>>> To: lng-odp@lists.linaro.org; brian.brooks@linaro.org; >>>> mike.holmes@linaro.org >>>> Subject: [lng-odp] [PATCHv7 00/35] running things in process mode >>>> >>>> Pull can be performed from the URL at the end of this cover letter >>>> All patches sent to the list following Maxim's request. >>>> >>>> Since v6 >>>> -rebased >>>> -regrouping history, squashing patch 1 and 12; 4, 5 and 12 (Petri) >>>> -mixed mode test removed (but still claimed to be supported in ODP). >>>> (Petri) >>>> >>>> _______________________________________________ >>> lng-odp mailing list >>> lng-odp@lists.linaro.org >>> https://lists.linaro.org/mailman/listinfo/lng-odp >>> >> >> _______________________________________________ >> lng-odp mailing list >> lng-odp@lists.linaro.org >> https://lists.linaro.org/mailman/listinfo/lng-odp >> > >
On 05/17 15:04:07, Christophe Milard wrote: > Pull can be performed from the URL at the end of this cover letter > All patches sent to the list following Maxim's request. > > Since v6 > -rebased > -regrouping history, squashing patch 1 and 12; 4, 5 and 12 (Petri) > -mixed mode test removed (but still claimed to be supported in ODP). (Petri) > > Since v5 > -rebased > -patch removing old unused helper code dropped. (Petri) > -Doc update to discourage usage of the old functions. (Petri, Mike) > > Since v4: > -updates following Brian's comments > > Since v3: > -fixed rebase error (Christophe) > -rebased > > Since v2: > -serious rebase following clash with dba05a28 (api: init: add instance handle) > -squashing the validation changes. > > Since v1: > -variable declaration gathered in function 's head (Petri) > -linux prefix removed from helper's types and function names (Mike, Christophe) This patch series allows existing tests and examples to run as pthread-per-core or process-per-core. The default pthread-per-core is preserved, and process-per-core is enabled via passing "--odph_proc" via the command line when invoking a test or example binary. The auxiliary helper code for launching pthreads and processes still exists, and a new "ODP thread" helper object was created as an abstraction of the two task types. This review is contingent upon the scope of ODP to include multi-process applications. Reviewed-by: Brian Brooks <brian.brooks@linaro.org>