[PATCH libdrm] libdrm: Fix issue about differrent domainID but same BDF

Deng, Emily Emily.Deng at amd.com
Wed Apr 24 06:18:42 UTC 2019


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 at gmail.com>
>Sent: Thursday, April 18, 2019 2:26 AM
>To: Deucher, Alexander <Alexander.Deucher at amd.com>
>Cc: Alex Deucher <alexdeucher at gmail.com>; Deng, Emily
><Emily.Deng at amd.com>; Maling list - DRI developers <dri-
>devel at lists.freedesktop.org>; amd-gfx list <amd-gfx at 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.Deucher at amd.com> wrote:
>>
>> > -----Original Message-----
>> > From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> On Behalf Of
>> > Emil Velikov
>> > Sent: Monday, February 25, 2019 8:09 AM
>> > To: Alex Deucher <alexdeucher at gmail.com>
>> > Cc: Deng, Emily <Emily.Deng at amd.com>; Maling list - DRI developers
>> > <dri- devel at lists.freedesktop.org>; amd-gfx list
>> > <amd-gfx at 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


More information about the dri-devel mailing list