Completely agree, couldn't take a look but feel free to add my Acked-by: Christian König <christian.koenig at 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.StDenis at amd.com > <mailto:Tom.StDenis at amd.com>> wrote: > > Hi Alex, > > > I'll wait on pushing it until I hear back from one of them. > > > Cheers, > > Tom > > > > ------------------------------------------------------------------------ > *From:* Alex Deucher <alexdeucher at gmail.com > <mailto:alexdeucher at gmail.com>> > *Sent:* Wednesday, December 7, 2016 12:32 > *To:* StDenis, Tom > *Cc:* Edward O'Callaghan; amd-gfx at lists.freedesktop.org > <mailto:amd-gfx at 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.deucher at amd.com > <mailto:alexander.deucher at amd.com>> > > On Wed, Dec 7, 2016 at 12:05 PM, StDenis, Tom <Tom.StDenis at amd.com > <mailto:Tom.StDenis at amd.com>> wrote: > > Thanks. > > > Could use a RB/NAK from Christian or Alex when they get the > time :-) > > > Tom > > > > ------------------------------------------------------------------------ > *From:* amd-gfx <amd-gfx-bounces at lists.freedesktop.org > <mailto:amd-gfx-bounces at lists.freedesktop.org>> on behalf of > Edward O'Callaghan <funfunctor at folklore1984.net > <mailto:funfunctor at folklore1984.net>> > *Sent:* Tuesday, December 6, 2016 17:15 > *To:* Tom St Denis; amd-gfx at lists.freedesktop.org > <mailto:amd-gfx at lists.freedesktop.org> > *Subject:* Re: GPR read support via debugfs (v2) > This series is, > Acked-by: Edward O'Callaghan <funfunctor at folklore1984.net > <mailto:funfunctor at 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 at lists.freedesktop.org > <mailto:amd-gfx at 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 at lists.freedesktop.org > <mailto:amd-gfx at 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 at lists.freedesktop.org <mailto:amd-gfx at 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 at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx -------------- next part -------------- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20161207/e144cc16/attachment-0001.html>
