Re: [PATCH libdrm 1/2] intel: Do not use libpciaccess on Android

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 27 March 2018 at 00:27, John Stultz <john.stultz@xxxxxxxxxx> wrote:
> On Tue, Mar 20, 2018 at 10:36 AM, Emil Velikov <emil.l.velikov@xxxxxxxxx> wrote:
>> From: Tomasz Figa <tfiga@xxxxxxxxxx>
>>
>> This patch makes the code not rely anymore on libpciaccess when compiled
>> for Android to eliminate ioperm() and iopl() syscalls required by that
>> library. As a side effect, the mappable aperture size is hardcoded to 64
>> MiB on Android, however nothing seems to rely on this value anyway, as
>> checked be grepping relevant code in drm_gralloc and Mesa.
>>
>> Cc: John Stultz <john.stultz@xxxxxxxxxx>
>> Cc: Rob Herring <rob.herring@xxxxxxxxxx>
>> Cc: John Stultz <john.stultz@xxxxxxxxxx>
>> Cc: Tomasz Figa <tfiga@xxxxxxxxxx>
>> Signed-off-by: Tomasz Figa <tfiga@xxxxxxxxxx>
>> [Emil Velikov: rebase against master]
>> Signed-off-by: Emil Velikov <emil.velikov@xxxxxxxxxxxxx>
>> ---
>> Tomasz, I've taken the liberty of pulling the patch from the Android
>> tree. Hope you don't mind.
>> ---
>
> Sorry, being abroad for a conference last week slowed me down and I
> didn't get to validate this.
>
> Many thanks for sending this out, I do agree its cleaner then my filtering fix.
>
> I've got this patch (along with the __func__ fix EricE noticed) built
> and tested for the HiKey boards and it works ok for me.
>
> Feel free to add my:
> Acked-by: John Stultz <john.stultz@xxxxxxxxxx>
>
> Emil: Do you mind respinning with the fix so Rob or someone can apply?
>
Added the __func__ piece and pushed the series.

Thanks guys!
Emil
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux