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.velikov@xxxxxxxxx>
>Sent: Thursday, April 18, 2019 2:26 AM
>To: Deucher, Alexander <Alexander.Deucher@xxxxxxx>
>Cc: Alex Deucher <alexdeucher@xxxxxxxxx>; 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
>
>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
