Hi Emil and Alex,
Sorry for miss your emails. I will update a new patch as Emil's suggestion.
Best wishes
Emily Deng
>-----Original Message-----
>From: Emil Velikov <emil.l.veli...@gmail.com>
>Sent: Thursday, April 18, 2019 2:26 AM
>To: Deucher, Alexander <alexander.deuc...@amd.com>
>Cc: Alex Deucher <alexdeuc...@gmail.com>; Deng, Emily
><emily.d...@amd.com>; Maling list - DRI developers <dri-
>de...@lists.freedesktop.org>; amd-gfx list <amd-...@lists.freedesktop.org>
>Subject: Re: [PATCH libdrm] libdrm: Fix issue about differrent domainID but
>same BDF
>
>On Mon, 25 Feb 2019 at 19:53, Deucher, Alexander
><alexander.deuc...@amd.com> wrote:
>>
>> > -----Original Message-----
>> > From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of
>> > Emil Velikov
>> > Sent: Monday, February 25, 2019 8:09 AM
>> > To: Alex Deucher <alexdeuc...@gmail.com>
>> > Cc: Deng, Emily <emily.d...@amd.com>; Maling list - DRI developers
>> > <dri- de...@lists.freedesktop.org>; amd-gfx list
>> > <amd-...@lists.freedesktop.org>
>> > 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
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
