On Mon, 25 Feb 2019 at 19:53, Deucher, Alexander <Alexander.Deucher@xxxxxxx> wrote: > > > -----Original Message----- > > From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Emil > > Velikov > > Sent: Monday, February 25, 2019 8:09 AM > > To: Alex Deucher <alexdeucher@xxxxxxxxx> > > Cc: Deng, Emily <Emily.Deng@xxxxxxx>; Maling list - DRI developers <dri- > > devel@xxxxxxxxxxxxxxxxxxxxx>; amd-gfx list <amd-gfx@xxxxxxxxxxxxxxxxxxxxx> > > Subject: Re: [PATCH libdrm] libdrm: Fix issue about differrent domainID but > > same BDF > > > > Hi all, > > > > This patch causes unnecessary round trip by openning the nodes. As > > mentioned previously this could be trivially fixed [1]. > > > > Even Emily acknowledged that [1], yet the sub-par fix was merged. Can we > > revert+fixup this properly? > > > > Sorry, I totally missed your reply. I'm having Internet issues at the moment so if you want to revert for now, I'll work with Emily to address your suggestions later in the week or next. Emily, can you take a look at addressing Emil's concerns with an updated patch? > Doesn't seem like there's any follow-up work on this, so I've reverted this for now. I'm a sad panda for doing it - hopefully v2 will come shortly. -Emil _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
