Completely agree, couldn't take a look but feel free to add my Acked-by: Christian König <christian.koe...@amd.com>.

Christian.

Am 07.12.2016 um 18:59 schrieb Marek Olšák:
Yeah, it can be useful for GPU hang debugging.

Marek

On Wed, Dec 7, 2016 at 6:49 PM, StDenis, Tom <tom.stde...@amd.com <mailto:tom.stde...@amd.com>> wrote:

    Hi Alex,


    I'll wait on pushing it until I hear back from one of them.


    Cheers,

    Tom



    ------------------------------------------------------------------------
    *From:* Alex Deucher <alexdeuc...@gmail.com
    <mailto:alexdeuc...@gmail.com>>
    *Sent:* Wednesday, December 7, 2016 12:32
    *To:* StDenis, Tom
    *Cc:* Edward O'Callaghan; amd-gfx@lists.freedesktop.org
    <mailto:amd-gfx@lists.freedesktop.org>

    *Subject:* Re: GPR read support via debugfs (v2)
    Patches look fine to me.  Assuming they are useful to Marek and
    Nicolai, the patches are:
    Reviewed-by: Alex Deucher <alexander.deuc...@amd.com
    <mailto:alexander.deuc...@amd.com>>

    On Wed, Dec 7, 2016 at 12:05 PM, StDenis, Tom <tom.stde...@amd.com
    <mailto:tom.stde...@amd.com>> wrote:

        Thanks.


        Could use a RB/NAK from Christian or Alex when they get the
        time :-)


        Tom



        ------------------------------------------------------------------------
        *From:* amd-gfx <amd-gfx-boun...@lists.freedesktop.org
        <mailto:amd-gfx-boun...@lists.freedesktop.org>> on behalf of
        Edward O'Callaghan <funfunc...@folklore1984.net
        <mailto:funfunc...@folklore1984.net>>
        *Sent:* Tuesday, December 6, 2016 17:15
        *To:* Tom St Denis; amd-gfx@lists.freedesktop.org
        <mailto:amd-gfx@lists.freedesktop.org>
        *Subject:* Re: GPR read support via debugfs (v2)
        This series is,
        Acked-by: Edward O'Callaghan <funfunc...@folklore1984.net
        <mailto:funfunc...@folklore1984.net>>

        On 12/07/2016 01:50 AM, Tom St Denis wrote:
        > These patches introduce debugfs access to read GPRs (SGPRs
        for now).
        >
        > (v2): Fixes white space, addressing is in dwords not bytes,
        and removes
        >       the buggy sanity checking.
        >
        > _______________________________________________
        > amd-gfx mailing list
        > amd-gfx@lists.freedesktop.org
        <mailto:amd-gfx@lists.freedesktop.org>
        > https://lists.freedesktop.org/mailman/listinfo/amd-gfx
        <https://lists.freedesktop.org/mailman/listinfo/amd-gfx>
        amd-gfx Info Page - lists.freedesktop.org
        <https://lists.freedesktop.org/mailman/listinfo/amd-gfx>
        lists.freedesktop.org <http://lists.freedesktop.org>
        To see the collection of prior postings to the list, visit the
        amd-gfx Archives. Using amd-gfx: To post a message to all the
        list members, send email ...



        >


        _______________________________________________
        amd-gfx mailing list
        amd-gfx@lists.freedesktop.org
        <mailto:amd-gfx@lists.freedesktop.org>
        https://lists.freedesktop.org/mailman/listinfo/amd-gfx
        <https://lists.freedesktop.org/mailman/listinfo/amd-gfx>



    _______________________________________________
    amd-gfx mailing list
    amd-gfx@lists.freedesktop.org <mailto:amd-gfx@lists.freedesktop.org>
    https://lists.freedesktop.org/mailman/listinfo/amd-gfx
    <https://lists.freedesktop.org/mailman/listinfo/amd-gfx>




_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx


_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to