On 12/10/2015 12:08, Savolainen, Petri (Nokia - FI/Espoo) wrote:

I’d really need the bug fix applied on api-next ASAP. It’s more urgent than a guide update. Buggy code slows down actual work. Buggy document is just annoying but does not prevent working efficiently.

Anyway, this used to be our policy: bug fixes (to master) are merged to api-next soon after master. What is the rationale **not** to merge bug fixes ASAP?

-Petri


working on sync master and api-next right now.

Maxim.

*From:*EXT Mike Holmes [mailto:mike.hol...@linaro.org]
*Sent:* Wednesday, December 09, 2015 8:34 PM
*To:* Maxim Uvarov
*Cc:* Savolainen, Petri (Nokia - FI/Espoo); lng-odp@lists.linaro.org
*Subject:* Re: [lng-odp] [PATCH 00/10] Pktio checks

Can we merge [1] that was ready and reviewed last week with no objections

Maxim Petri - can you review that patch and if there are changes needed I can update.

[1] [PATCH] doc: process-guide: add release process

On 9 December 2015 at 04:35, Maxim Uvarov <maxim.uva...@linaro.org <mailto:maxim.uva...@linaro.org>> wrote:

    On 12/09/2015 11:54, Savolainen, Petri (Nokia - FI/Espoo) wrote:


        So, critical bug fixes from master side are not visible to
        api-next until a month or so?


    Exactly. api-next was broken with TM. Until issue if fixed and
    branch operates as usual, there is no pull of critical bug and etc.
    https://ci.linaro.org/view/odp-ci/job/odp-api-check

    Now some people still see build fails with gcc4.9, but I can not
    reproduce it and CI also passes. So only now I can pull changes
    from master.

    Maxim.

        Most bug fixes should apply directly to api-next as well,
        since those are implementation/test code/build system fixes
        (not api changing fixes). Some fixes may touch the same lines
        that api-next have changed due to an api change (and would
        need more thinking), but generally that should not be the case.

        For example, without these fixes it’s impossible to write
        implementation (on api-next side) that is compatible to the
        API spec AND passes validation tests, since validation tests
        do not follow the spec. So, I could not send patches for
        api-next that follow the spec for one month (since ‘make
        check’ fails),  OR I would need to include these same (or
        similar bug fix) patches in the front of my patch set.

        -Petri

        *From:*EXT Mike Holmes [mailto:mike.hol...@linaro.org
        <mailto:mike.hol...@linaro.org>]
        *Sent:* Tuesday, December 08, 2015 5:34 PM
        *To:* Maxim Uvarov
        *Cc:* Savolainen, Petri (Nokia - FI/Espoo);
        lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
        *Subject:* Re: [lng-odp] [PATCH 00/10] Pktio checks

        The process states that API-NEXT takes master on master
        release points only.

        
https://docs.google.com/drawings/d/1kWkr2qBV4xbqOCXMDVsoINhdxaNeHC-8sCm6SBVFI8o/edit

        On 8 December 2015 at 09:48, Maxim Uvarov
        <maxim.uva...@linaro.org <mailto:maxim.uva...@linaro.org>
        <mailto:maxim.uva...@linaro.org
        <mailto:maxim.uva...@linaro.org>>> wrote:

            On 12/08/2015 17:20, Savolainen, Petri (Nokia - FI/Espoo)
        wrote:

                Could you merge this into api-next ASAP. Just lost
        couple of
                hours in debugging these same malfunctioning tests.

                -Petri


            If you can merge that master and api-next I can. But for
        now after
            time things 2 branches out of sync in many files.

            Maxim.



                    -----Original Message-----
                    From: lng-odp
        [mailto:lng-odp-boun...@lists.linaro.org
        <mailto:lng-odp-boun...@lists.linaro.org>
                    <mailto:lng-odp-boun...@lists.linaro.org
        <mailto:lng-odp-boun...@lists.linaro.org>>] On Behalf Of EXT
                    Maxim Uvarov
                    Sent: Tuesday, December 08, 2015 4:14 PM

                    To: lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>
        <mailto:lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>>
                    Subject: Re: [lng-odp] [PATCH 00/10] Pktio checks

                    Merged,
                    Maxim.

                    On 11/10/2015 18:49, Nicolas Morey-Chaisemartin wrote:

                        This series add several tests for pktios and
        fixes the
                        issue they

                    raised.

                        The main features checked are:
                        * RONLY pktio cannot be sent to
                        * WONLY pktio cannot be read from
                        * stopped pktio cannot be stopped
                        * started pktio cannot be started
                        * started pktio cannot be configured

                        Nicolas Morey-Chaisemartin (10):
                            linux-generic: pktio: check interface mode is
                        compatible before
                              receiving or sending
                            validation: pktio: add customizable out
        mode for
                        pktios
                            validation: pktio: add tests for rrecv()
        on WONLY,
                        and send on RONLY
                              pktios
                            validation: pktio: stop interfaces before
        removing
                        the default inq
                            validation: pktio: remove unneeded stop as
                        interface is stopped after
                              open()
                            validation: classification: start pktio after
                        setting inq and stop it
                              before removing it
                            validation: classification: stronger checks to
                        avoid SEGV on pktio
                              failure
                            linux-generic: pktio: check for
        pktio_start when
                        started and
                              pktio_stop when stopped
                            linux-generic: pktio: configuration functions
                        check that interface is
                              stopped
                            validation: pktio: add test for start when
        started
                        and stop when
                              stopped()

         platform/linux-generic/odp_packet_io.c |                 25 ++++-
         .../classification/odp_classification_common.c   |   2 +
.../classification/odp_classification_test_pmr.c | 25 +++--
         test/validation/pktio/pktio.c           | 116

                    ++++++++++++++++++---

                           4 files changed, 140 insertions(+), 28
        deletions(-)

        _______________________________________________
                        lng-odp mailing list

        lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
        <mailto:lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>>
        https://lists.linaro.org/mailman/listinfo/lng-odp
        <https://lists.linaro.org/mailman/listinfo/lng-odp>

        _______________________________________________
                    lng-odp mailing list
        lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
        <mailto:lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>>
        https://lists.linaro.org/mailman/listinfo/lng-odp
        <https://lists.linaro.org/mailman/listinfo/lng-odp>


            _______________________________________________
            lng-odp mailing list
        lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
        <mailto:lng-odp@lists.linaro.org
        <mailto:lng-odp@lists.linaro.org>>
        https://lists.linaro.org/mailman/listinfo/lng-odp



--
        Mike Holmes

        Technical Manager - Linaro Networking Group

        Linaro.org <http://www.linaro.org/>***│ *Open source software
        for ARM SoCs



--

Mike Holmes

Technical Manager - Linaro Networking Group

Linaro.org <http://www.linaro.org/>***│ *Open source software for ARM SoCs


_______________________________________________
lng-odp mailing list
lng-odp@lists.linaro.org
https://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to