On 20 March 2018 at 18:47, Eric Engestrom <eric.engestrom@xxxxxxxxxx> wrote:
> On Tuesday, 2018-03-20 17:36:51 +0000, Emil Velikov 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.
>> ---
>> intel/Android.mk | 3 +--
>> intel/intel_bufmgr.c | 12 ++++++++++++
>> 2 files changed, 13 insertions(+), 2 deletions(-)
>>
>> diff --git a/intel/Android.mk b/intel/Android.mk
>> index 3f9db785..dd881688 100644
>> --- a/intel/Android.mk
>> +++ b/intel/Android.mk
>> @@ -33,8 +33,7 @@ LOCAL_MODULE := libdrm_intel
>> LOCAL_SRC_FILES := $(LIBDRM_INTEL_FILES)
>>
>> LOCAL_SHARED_LIBRARIES := \
>> - libdrm \
>> - libpciaccess
>> + libdrm
>>
>> include $(LIBDRM_COMMON_MK)
>> include $(BUILD_SHARED_LIBRARY)
>> diff --git a/intel/intel_bufmgr.c b/intel/intel_bufmgr.c
>> index a2853400..42f5f62c 100644
>> --- a/intel/intel_bufmgr.c
>> +++ b/intel/intel_bufmgr.c
>> @@ -36,7 +36,9 @@
>> #include <errno.h>
>> #include <drm.h>
>> #include <i915_drm.h>
>> +#ifndef __ANDROID__
>> #include <pciaccess.h>
>> +#endif
>> #include "libdrm_macros.h"
>> #include "intel_bufmgr.h"
>> #include "intel_bufmgr_priv.h"
>> @@ -326,6 +328,7 @@ drm_intel_get_pipe_from_crtc_id(drm_intel_bufmgr *bufmgr, int crtc_id)
>> return -1;
>> }
>>
>> +#ifndef __ANDROID__
>> static size_t
>> drm_intel_probe_agp_aperture_size(int fd)
>> {
>> @@ -351,6 +354,15 @@ err:
>> pci_system_cleanup ();
>> return size;
>> }
>> +#else
>> +static size_t
>> +drm_intel_probe_agp_aperture_size(int fd)
>> +{
>> + /* Nothing seems to rely on this value on Android anyway... */
>> + fprintf(stderr, "%s: Mappable aperture size hardcoded to 64MiB\n");
>
> Guessing there's a __func__ missing at the end there ^
>
Indeed - fixed locally. Can resent if the Android related people are
keen on the overall idea.
-Emil
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
