Commits on Source (79)
-
Alex Deucher authored
Acked-by:
Christian König <christian.koenig@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Some devices can lack OF data or it may not be available in the uevent file. Fallback to the MODALIAS data in those cases. We strip any leading "MODALIAS=.*:" thus the resulting information is compatible with existing code in Mesa. v2: foo -> tmp_name Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Tested-by: Lucas Stach <l.stach@pengutronix.de> (v1)
-
The functions are virtually identical, fold them up. v2: foo -> tmp_name (Eric) Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Eric Engestrom <eric.engestrom@intel.com>
-
Rodrigo Vivi authored
Straight copy from the kernel file. Add more PCI Device IDs for Coffee Lake, Ice Lake, and Amber Lake. It also include a reorg on Whiskey Lake IDs. Align with kernel commits: 5e0f5a58b167 ("drm/i915/cfl: Adding another PCI Device ID.") 03ca3cf8e9aa ("drm/i915/icl: Adding few more device IDs for Ice Lake") c0c46ca461f1 ("drm/i915/aml: Add new Amber Lake PCI ID") c1c8f6fa731b ("drm/i915: Redefine some Whiskey Lake SKUs") Cc: José Roberto de Souza <jose.souza@intel.com> Signed-off-by:
Rodrigo Vivi <rodrigo.vivi@intel.com> Acked-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com>
-
We can't use drmSetMaster to query whether or not a drm fd is master because it requires CAP_SYS_ADMIN, even if the fd *is* a master fd. Pick DRM_IOCTL_MODE_ATTACHMODE as a long-deprecated ioctl that is DRM_MASTER but not DRM_ROOT_ONLY as the probe by which we can detect whether or not the fd is master. This is useful for code that might get master by open()ing the drm device while no other master exists, but can't call drmSetMaster itself because it's not running as root or is in a container, where container-root isn't real-root. v2: Use the AUTH_MAGIC request rather than MODE_ATTACHMODE, as it's more clearly related to master status. v3: [Emil] Don't expose internals, check for -EACCES. Signed-off-by:
Christopher James Halse Rogers <christopher.halse.rogers@canonical.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> (v2) Signed-off-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Eric Engestrom authored
Xserver has struct members named `bool`, which means the last commit breaks its build with errors like this: error: two or more data types in declaration specifiers Bool bool; ^ Fix this by making it return a 0/1 integer, with the same semantic as the boolean it was before. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=109587 Fixes: 17dfe3ac "xf86drm: Add drmIsMaster()" Cc: Christopher James Halse Rogers <christopher.halse.rogers@canonical.com> Signed-off-by:
Eric Engestrom <eric.engestrom@intel.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Eric Engestrom authored
This reverts 6d237985 "xf86atomic: #undef internal define" and b541d21a "freedreno: remove always-defined #ifdef". I didn't realise at the time that freedreno/freedreno_ringbuffer.h gets installed, and then used by Mesa for instance. These two commits were fine in libdrm, but broke Mesa which needs to use struct fd_ringbuffer but doesn't need to access ::refcnt. The hack that I removed serves to keep the struct at the correct size while only exposing the ::refcnt member within libdrm. Fixes: 6d237985 "xf86atomic: #undef internal define" Fixes: b541d21a "freedreno: remove always-defined #ifdef" Signed-off-by:
Eric Engestrom <eric.engestrom@intel.com>
-
Eric Engestrom authored
base/archlinux has been replaced with archlinux/base, which is maintained directly by the archlinux community. Signed-off-by:
Eric Engestrom <eric.engestrom@intel.com>
-
Emily Deng authored
For multiple GPUs which has the same BDF, but has different domain ID, the drmOpenByBusid will return the wrong fd when startx. The reproduce sequence as below: 1. Call drmOpenByBusid to open Card0, then will return the right fd0, and the fd0 is master privilege; 2. Call drmOpenByBusid to open Card1. In function drmOpenByBusid, it will open Card0 first, this time, the fd1 for opening Card0 is not master privilege, and will call drmSetInterfaceVersion to identify the domain ID feature, as the fd1 is not master privilege, then drmSetInterfaceVersion will fail, and then won't compare domain ID, then return the wrong fd for Card1. Solution: First loop search the best match fd about drm 1.4. Reviewed-by:
Alex Deucher <alexander.deucher@amd.com> Signed-off-by:
Emily Deng <Emily.Deng@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Seung-Woo Kim authored
After the commit 0926f0af ("meson,configure: include config.h automatically"), there is build error for autotools because config.h is not included. Fix the error by adding "-include config.h" to CPPFLAGS instead of CFLAGS from configure.ac. Reference: https://bugs.freedesktop.org/show_bug.cgi?id=106561 Signed-off-by:
Seung-Woo Kim <sw0312.kim@samsung.com> Reviewed-by:
Eric Engestrom <eric.engestrom@intel.com> Fixes: 0926f0af "meson,configure: include config.h automatically"
-
Tapani Pälli authored
Add new iris driver, remove deprecated ilo driver. Signed-off-by:
Tapani Pälli <tapani.palli@intel.com> Reviewed-by:
Eric Engestrom <eric.engestrom@intel.com>
-
Andreas Baierl authored
This fixes a bug, which was introduced with commit ee798b98 "xf85drm: de-duplicate drmParse{Platform.Host1x}{Bus,Device}Info". where accessing *compatible[i] with i>0 results in a segfault. Signed-off-by:
Andreas Baierl <ichgeh@imkreisrum.de> Fixes: ee798b98 "xf85drm: de-duplicate drmParse{Platform.Host1x}{Bus,Device}Info" Reviewed-by:
Eric Engestrom <eric.engestrom@intel.com> [Eric: add the same fix to the free() below] Signed-off-by:
Eric Engestrom <eric.engestrom@intel.com>
-
Alex Deucher authored
Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Cui, Flora authored
deadlock test for sdma will cause gpu recoverty. disable the test for now until GPU reset recovery could survive at least 1000 times test. v2: add modprobe parameter Reviewed-and-tested-by:
Evan Quan <evan.quan@amd.com> Acked-by:
Christian König <christian.koenig@amd.com> Signed-off-by:
Flora Cui <flora.cui@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Cui, Flora authored
add memset dispatch test for gfx9 v2: disable dispatch test for other ASICs Signed-off-by:
Flora Cui <flora.cui@amd.com> Tested-by:
Rui Teng <rui.teng@amd.com> Reviewed-by:
Hawking Zhang <Hawking.Zhang@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Cui, Flora authored
add memcpy dispatch test for gfx9 Signed-off-by:
Flora Cui <flora.cui@amd.com> Tested-by:
Rui Teng <rui.teng@amd.com> Reviewed-by:
Hawking Zhang <Hawking.Zhang@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Cui, Flora authored
add memset draw test for gfx9 Signed-off-by:
Flora Cui <flora.cui@amd.com> Tested-by:
Rui Teng <rui.teng@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Cui, Flora authored
add memcpy draw test for gfx9 Signed-off-by:
Flora Cui <flora.cui@amd.com> Tested-by:
Rui Teng <rui.teng@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Cui, Flora authored
1. clear cmd buffer 2. make amdgpu_memcpy_dispatch_test static 3. tab/space fix Reviewed-by:
Alex Deucher <alexander.deucher@amd.com> Signed-off-by:
Flora Cui <flora.cui@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Anusha authored
Add CML and EHL PCI IDs, and one more for ICL. This is in sync with kernel header as of b024ab9b2d3a ("drm/i915/bios: iterate over child devices to initialize ddi_port_info") Signed-off-by:
Anusha Srivatsa <anusha.srivatsa@intel.com> Reviewed-by:
Lucas De Marchi <lucas.demarchi@intel.com>
-
Gurchetan Singh authored
Generated using make headers_install. This brings in the in/out fence support for explicit synchronization. v2: don't use experimental kernel branch Signed-off-by:
Gurchetan Singh <gurchetansingh@chromium.org> Reviewed-by:
Robert Foss <robert.foss@collabora.com> Signed-off-by:
Rob Clark <robclark@freedesktop.org>
-
Alex Deucher authored
From drm-next commit b4e4538a0ab5079ae5dc401970e11f0ff2ba13a7 Adds support for: - RAS queries - context priority updates - CS chunks support for scheduled dependencies - IB flag for GDS max wave id Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
xinhui pan authored
Signed-off-by:
xinhui pan <xinhui.pan@amd.com> Reviewed-by:
Feifei Xu <Feifei.Xu@amd.com> Reviewed-by:
Hawking Zhang <Hawking.Zhang@amd.com> Acked-by:
Alex Deucher <alexander.deucher@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
xinhui pan authored
support per device test mask. Skip inject test on non-server card. Signed-off-by:
xinhui pan <xinhui.pan@amd.com> Reviewed-by:
Feifei Xu <Feifei.Xu@amd.com> Reviewed-by:
Hawking Zhang <Hawking.Zhang@amd.com> Acked-by:
Alex Deucher <alexander.deucher@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Ayan Halder authored
Generated using make headers_install from the drm-next tree - git://anongit.freedesktop.org/drm/drm branch - drm-next commit - 14d2bd53a47a7e1cb3e03d00a6b952734cf90f3f The changes were as follows :- core: (drm.h, drm_fourcc.h, drm_mode.h) - Added 'struct drm_syncobj_transfer', 'struct drm_syncobj_timeline_wait' and 'struct drm_syncobj_timeline_array' - Added various DRM_IOCTL_SYNCOBJ_ ioctls - Added some new RGB and YUV formats - Added 'DRM_FORMAT_MOD_VENDOR_ALLWINNER' - Added 'SAMSUNG' and Arm's 'AFBC' and 'ALLWINNER' format modifiers - Added 'struct drm_mode_rect' i915: - Added struct 'struct i915_user_extension' and various 'struct drm_i915_gem_context_' - Added different modes of per-process Graphics Translation Table Changes from v1:- - Removed the changes to 'msm_drm.h' as it breaks the build for 'freedreno' platform. Signed-off-by:
Ayan Kumar halder <ayan.halder@arm.com> Acked-by:
Eric Engestrom <eric.engestrom@intel.com>
-
Pan, Xinhui authored
struct drmPciBusInfo has been aligned to 6 bytes. So memcmp will access the last byte which is not initialized. Reviewed-by:
Alex Deucher <alexander.deucher@amd.com> Signed-off-by:
xinhui pan <xinhui.pan@amd.com> Signed-off-by:
Alex Deucher <alexander.deucher@amd.com>
-
Signed-off-by:
Eric Engestrom <eric.engestrom@intel.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Saw a couple of typos fixes in the patch DragonFlyBSD carries [1], so I ran codespell (a spell checker for code) on the whole repo. [1] https://github.com/DragonFlyBSD/DPorts/blob/master/graphics/libdrm/files/patch-xf86drm.c Signed-off-by:
Eric Engestrom <eric.engestrom@intel.com> Reviewed-by:
Daniel Vetter <daniel.vetter@ffwll.ch> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Signed-off-by:
Fritz Koenig <frkoenig@google.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Emil Velikov authored
This reverts commit 56c21f87. There were issues pointed out during review that were not addressed. Would love to have this re-land, once those are addressed.
-
This makes the test utilities work with the Armada driver without the necessity of using the -M argument. Signed-off-by:
Lubomir Rintel <lkundrak@v3.sk> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Bas Nieuwenhuizen authored
This way we can override the priority of a single context using a master fd. Since we cannot usefully create an amdgpu device of a master fd without the fd deduplication kicking in this takes a plain fd. This can be used by e.g. radv to get high priority contexts using a master fd from the primary node or a lease. Reviewed-by:
Samuel Pitoiset <samuel.pitoiset@gmail.com> Reviewed-by:
Andres Rodriguez <andresx7@gmail.com> Reviewed-by:
Chunming Zhou <david1.zhou@amd.com>
-
Signed-off-by:
Emil Velikov <emil.velikov@collabora.com>
-
Add a check to error out on Android version K(4.4) or lower. This is due to dependency added in a previous commit on mmap64, which was introduced with Android L. Cc: Sean Paul <seanpaul@chromium.org> Cc: Alistair Strachan <astrachan@google.com> Cc: Marissa Wall <marissaw@google.com> Suggested-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com> Signed-off-by:
John Stultz <john.stultz@linaro.org>
-
__mmap2 isn't supported on all platforms, mmap64 is the right way to do this in android. Also folds in a fix from Stéphane Marchesin <marcheu@chromium.org> Cc: Sean Paul <seanpaul@chromium.org> Cc: Alistair Strachan <astrachan@google.com> Cc: Marissa Wall <marissaw@google.com> Acked-by:
Alex Deucher <alexander.deucher@amd.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com> Signed-off-by:
Sean Paul <seanpaul@chromium.org> [jstultz: Folded in Stéphane's fix] Signed-off-by:
John Stultz <john.stultz@linaro.org>
-
Clang complains when initializing unions using "= {0}" so instead use memset. Cc: Sean Paul <seanpaul@chromium.org> Cc: Alistair Strachan <astrachan@google.com> Cc: Marissa Wall <marissaw@google.com> Reviewed-by:
Alex Deucher <alexander.deucher@amd.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com> Reviewed-by:
Christian König <christian.koenig@amd.com> Signed-off-by:
John Stultz <john.stultz@linaro.org>
-
Avoid additional drm device open and close. Cc: Sean Paul <seanpaul@chromium.org> Cc: Alistair Strachan <astrachan@google.com> Cc: Marissa Wall <marissaw@google.com> Reviewed-by:
Alex Deucher <alexander.deucher@amd.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com> Signed-off-by:
John Stultz <john.stultz@linaro.org>
-
When calling drmModeAtomicAddProperty allocation of memory happens as needed in increments of 16 elements. This can be very slow if there are multiple properties to be updated in an Atomic Commit call. Increase this to as many as can fit in a memory PAGE to avoid having to reallocate memory too often. Also this patch has a small one line perf tweak in drmModeAtomicDuplicate() to only memcpy items to the cursor position in order avoid copying the entire item array if its mostly empty. Cc: Sean Paul <seanpaul@chromium.org> Cc: Alistair Strachan <astrachan@google.com> Cc: Marissa Wall <marissaw@google.com> Reviewed-by:
Alex Deucher <alexander.deucher@amd.com> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com> [jstultz: Expanded commit message] Signed-off-by:
John Stultz <john.stultz@linaro.org>
-
Allows mmap on dmabuf fd with MAP_SHARED and PROT_WRITE. This fixes boot failures with Android (likely w/ closed source user-space drivers) that were caused due to mmap() returning error. Cc: Sean Paul <seanpaul@chromium.org> Cc: Alistair Strachan <astrachan@google.com> Cc: Marissa Wall <marissaw@google.com> Acked-by:
Emil Velikov <emil.velikov@collabora.com> Signed-off-by:
Hemant Hariyani <hemanthariyani@ti.com> [picked and updated commitmsg from http://git.ti.com/cgit/cgit.cgi/android/external-libdrm.git/ ] Signed-off-by:
Praneeth Bajjuri <praneeth@ti.com> Signed-off-by:
Alistair Strachan <astrachan@google.com> [jstultz: Tweaked commit message] Signed-off-by:
John Stultz <john.stultz@linaro.org>
-
The pointer p aquired with drmModeGetPlane() is not free in error path. Fix possible memory leak by calling drmModeFreePlane() in the error path. Reviewed-by:
Emil Velikov <emil.velikov@collabora.com> Signed-off-by:
Seung-Woo Kim <sw0312.kim@samsung.com>
-
Seung-Woo Kim authored
In drmModeGetPropertyPtr(), from upper error path, it calls free but with just next error path, it does not call. Fix the possible memory leak. Signed-off-by:
Seung-Woo Kim <sw0312.kim@samsung.com> Reviewed-by:
Eric Engestrom <eric.engestrom@intel.com>
-
Chunming Zhou authored
Signed-off-by:
Chunming Zhou <david1.zhou@amd.com> Acked-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com>
-
Chunming Zhou authored
v2: drop export/import Signed-off-by:
Chunming Zhou <david1.zhou@amd.com> Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com>
-
Chunming Zhou authored
v2: symbos are stored in lexical order. v3: drop export/import and extra query indirection Signed-off-by:
Chunming Zhou <david1.zhou@amd.com> Acked-by:
Christian König <christian.koenig@amd.com> Acked-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com>
-
Chunming Zhou authored
v2: use one transfer ioctl Signed-off-by:
Chunming Zhou <david1.zhou@amd.com> Reviewed-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com>
-
Chunming Zhou authored
v2: adapt to new one transfer ioctl Signed-off-by:
Chunming Zhou <david1.zhou@amd.com> Acked-by:
Christian König <christian.koenig@amd.com> Acked-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com>
-
Chunming Zhou authored
Signed-off-by:
Chunming Zhou <david1.zhou@amd.com> Acked-by:
Christian König <christian.koenig@amd.com> Acked-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com>
-
Chunming Zhou authored
v2: drop DRM_SYNCOBJ_CREATE_TYPE_TIMELINE, fix timeout calculation, fix some warnings v3: add export/import and cpu signal testing cases Signed-off-by:
Chunming Zhou <david1.zhou@amd.com> Acked-by:
Christian König <christian.koenig@amd.com> Acked-by:
Lionel Landwerlin <lionel.g.landwerlin@intel.com>
-
Michel Dänzer authored
Fixes make check. Trivial.
-
Chunming Zhou authored
a) delta: only DRM_CAP_SYNCOBJ_TIMELINE b) Generated using make headers_install. c) Generated from origin/drm-misc-next commit 982c0500fd1a8012c31d3c9dd8de285129904656" Signed-off-by:
Chunming Zhou <david1.zhou@amd.com> Suggested-by:
Michel Dänzer <michel@daenzer.net> Reviewed-by:
Michel Dänzer <michel.daenzer@amd.com>
-
Chunming Zhou authored
Feature is controlled by DRM_CAP_SYNCOBJ_TIMELINE drm capability. Signed-off-by:
Chunming Zhou <david1.zhou@amd.com> Reviewed-by:
Christian König <christian.koenig@amd.com>
-
Huang Rui authored
Reviewed-by:
Tim Writer <Tim.Writer@amd.com> Signed-off-by:
Huang Rui <ray.huang@amd.com> Reviewed-by:
Hawking Zhang <Hawking.Zhang@amd.com> Signed-off-by:
Marek Olšák <marek.olsak@amd.com>
-
Hawking Zhang authored
pa_sc_tile_steering_override is a new member introduced for gfx10 Signed-off-by:
Hawking Zhang <Hawking.Zhang@amd.com> Reviewed-by:
Marek Olšák <marek.olsak@amd.com> Signed-off-by:
Marek Olšák <marek.olsak@amd.com>
-
Tao Zhou authored
AMDGPU_VRAM_TYPE_GDDR6 is a new vram type for navi10 Reviewed-by:
Tim Writer <Tim.Writer@amd.com> Signed-off-by:
Tao Zhou <tao.zhou1@amd.com> Reviewed-by:
Hawking Zhang <Hawking.Zhang@amd.com> Signed-off-by:
Marek Olšák <marek.olsak@amd.com>
-
Leo Liu authored
With different register offsets from VCN1.0 Signed-off-by:
Leo Liu <leo.liu@amd.com> Reviewed-by:
James Zhu <James.Zhu@amd.com> Signed-off-by:
Marek Olšák <marek.olsak@amd.com>
-
Ilia Mirkin authored
This also adds a helper to generate a color LUT, which has to be used in conjunction with the C8 indexed format. Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Acked-by:
Emil Velikov <emil.velikov@collabora.com>
-
Ilia Mirkin authored
We need to shift the values up, otherwise we'd end up with a negative shift. This works for up-to 16-bit components, which is fine for now. Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Ilia Mirkin authored
The idea is to have a horizontal pattern split into two with the top and bottom halves having different precision. This allows one to see whether 10bpc support is working properly or not, as there are many pieces to the puzzle beyond the basic format support (gamma ramps, bpc encodings, etc). This is really only useful on 10bpc formats, but we also add support for 8bpc formats to ease testing. In the future, this could be applied to 16bpc formats as well. Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Ilia Mirkin authored
This change adds support for all current patterns. Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Ilia Mirkin authored
Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Ilia Mirkin authored
Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Ilia Mirkin authored
As new features are added and others are declared to be legacy, it's nice to be able to implement fallbacks. As such, create a property-setting variant that does not generate errors which can very well be entirely expected. Will be used for gamma control in a future change. Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Ilia Mirkin authored
This includes logic to configure the LUT accordingly. Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Ilia Mirkin authored
Instead of hacking the binary every time, we can now specify directly. Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Ilia Mirkin authored
Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
Emil Velikov <emil.velikov@collabora.com>
-
Ilia Mirkin authored
Signed-off-by:
Ilia Mirkin <imirkin@alum.mit.edu> Reviewed-by:
Caio Marcelo de Oliveira Filho <caio.oliveira@intel.com>
-
Lucas Stach authored
There is no implementation and also no users, so there is no point in keeping it in the API. Signed-off-by:
Lucas Stach <l.stach@pengutronix.de> Reviewed-by:
Philipp Zabel <p.zabel@pengutronix.de>
-
Michel Dänzer authored
Avoids compiler warning: ../../amdgpu/amdgpu_cs.c: In function 'amdgpu_cs_ctx_override_priority': ../../amdgpu/amdgpu_cs.c:155:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement] union drm_amdgpu_sched args; ^~~~~ Reviewed-by:
Christian König <christian.koenig@amd.com>
-
Michel Dänzer authored
To reflect current reality. Reviewed-by:
Christian König <christian.koenig@amd.com>
-
Marek Vasut authored
The following situation can happen in a multithreaded OpenGL application. A BO is submitted from etna_cmd_stream #1 with flags set for read. A BO is submitted from etna_cmd_stream #2 with flags set for write. This triggers a flush on stream #1 and clears the BO's current_stream pointer. If at this point, stream #2 attempts to queue BO again, which does happen, the BO will be added to the submit list twice. The Linux kernel driver correctly detects this and warns about it with "BO at index %u already on submit list" kernel message. However, when cleaning the BO cache in etna_bo_cache_free(), the BO which was submitted twice will also be free()d twice, this triggering a glibc double free detector. The fix is easy, even if the BO does not have current_stream set, iterate over current streams' list of BOs before adding the BO to it and verify that the BO is not yet there. Signed-off-by:
Marek Vasut <marex@denx.de> Cc: Christian Gmeiner <christian.gmeiner@gmail.com> Cc: Lucas Stach <l.stach@pengutronix.de> Reviewed-by:
Christian Gmeiner <christian.gmeiner@gmail.com> Reviewed-by:
Wladimir J. van der Laan <laanwj@gmail.com>
-
Michel Dänzer authored
And propagate drmIoctl's return value. This allows replacing all remaining open-coded DRM_IOCTL_GEM_CLOSE ioctl calls with amdgpu_close_kms_handle calls. Reviewed-by:
Emil Velikov <emil.l.velikov@gmail.com>
-
Michel Dänzer authored
Simplifies its callers. dev->bo_table_mutex is now always held when amdgpu_bo_create is called (this was already the case in amdgpu_bo_import). Reviewed-by:
Emil Velikov <emil.l.velikov@gmail.com>
-
Michel Dänzer authored
Seems to better reflect what they're for. Reviewed-by:
Emil Velikov <emil.l.velikov@gmail.com>
-
Marek Olšák authored
-
Timo Aaltonen authored
-
Timo Aaltonen authored
-
Timo Aaltonen authored
-
Timo Aaltonen authored
-
Timo Aaltonen authored
debian/patches/02_kfreebsd.diff
deleted
100644 → 0