Commits on Source (56)
-
Simona Vetter authored
Looks much neater on the gitlab UI, e.g. on my personal libdrm fork: https://gitlab.freedesktop.org/danvet/drm Acked-by: Eric Engestrom <eric.engestrom@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
9d07fbf5 -
Andrey Grodzovsky authored
Illegal access will cause CP hang followed by job timeout and recovery kicking in. Also, disable the suite for all APU ASICs until GPU reset issues for them will be resolved and GPU reset recovery will be enabled by default. v2: Add KV to deasbled APUs list and add comments regarding necessary kernel amdgpu paramteres to run the tests. Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky@amd.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
b3dec018 -
Andrey Grodzovsky authored
Since only for those ASICs gpu reset is enabled by deafult. Also update disable message and fix identation . Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky@amd.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
0be85044 -
Eric Engestrom authored
Reported-by: Jan Vesely <jano.vesely@gmail.com> Signed-off-by: Eric Engestrom <eric.engestrom@intel.com> Reviewed-by: Jan Vesely <jan.vesely@rutgers.edu>
0f2f38bf -
Eric Engestrom authored
Signed-off-by: Eric Engestrom <eric.engestrom@intel.com> Reviewed-by: Dylan Baker <dylan@pnwbakers.com>
8c1fddc6 -
Eric Engestrom authored
Error message was invalid too, negative values aren't the number of devices, they're errno error codes. Signed-off-by: Eric Engestrom <eric.engestrom@intel.com> Reviewed-by: Daniel Stone <daniels@collabora.com>
e642f480 -
Emma Anholt authored
Having people count characters is error-prone, when we could just have a computer do it. Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>
9b28c5ae -
Emma Anholt authored
For ARM systems with tinydrm displays attached to SPI, the bus name is /spi but we have platform device info for the rest. Fixes eglInitialize() failures on hx8357d since the EGL_EXT_device_drm changes. Acked-by: Eric Engestrom <eric.engestrom@intel.com>
89700ab0 -
Christian König authored
Make sure the kernel doesn't crash if we map something at the minimum/maximum address. Signed-off-by: Christian König <christian.koenig@amd.com> Reviewed-by: Junwei Zhang <Jerry.Zhang@amd.com>
b28b8237 -
Andrey Grodzovsky authored
I retested GPU recovery with Bonaire ASIC and it works. Signed-off-by: Andrey Grodzovsky <andrey.grodzovsky@amd.com>
ba45adb2 -
Like on OpenBSD, the DragonFly BSD kernel only contains pci drm drivers. Signed-off-by: François Tigeot <ftigeot@wolfpond.org>
200e9e98 -
It is a cleaner and less fragile way to get PCI IDs than the one currently used by local DPorts patches. Signed-off-by: François Tigeot <ftigeot@wolfpond.org>
8f2e0925 -
Leo Liu authored
Just make them properly i.e. put 0 to the Nop reg Signed-off-by: Leo Liu <leo.liu@amd.com> Reviewed-by: Boyuan Zhang <boyuan.zhang@amd.com> Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
866ada1b -
Lucas De Marchi authored
LANG=C sort -u .gitignore | sponge .gitignore This way it's easier to keep track of the entries. Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com> Acked-by: Emil Velikov <emil.velikov@collabora.com>
d8e615f9 -
Lucas De Marchi authored
This is the directory used by meson/autotools (at least in the .gitlab-ci configuration) so ignore the whole dir. Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com> Acked-by: Emil Velikov <emil.velikov@collabora.com>
2c02f1e6 -
maurossi authored
Seemingly the 64-bit int is always aligned to 8 in LP64. But this is not hold in LP32. Consequently sizeof(gralloc_drm_handle_t) are different between LP64 (which is 18 ints) and LP32 (which is 16 ints). As a result, 32-bit apps will crash in 64-bit OS since the checking handle->base.numInts != GRALLOC_GBM_HANDLE_NUM_INTS is true. Fix it by always aligning 64-bit int to 8. Besides, to avoid additional padding, just exchange the order of data_owner and modifier. It aligns modifier to 8 natually. This makes gralloc_drm_handle_t fit in 16 ints perfectly. (v2) gralloc_drm_handle.h patch now applied in gralloc_handle.h and GRALLOC_HANDLE_VERSION updated to 4 Reported-by: Mauro Rossi <issor.oruam@gmail.com> Signed-off-by: Chih-Wei Huang <cwhuang@linux.org.tw> Reviewed-by: Robert Foss <robert.foss@collabora.com>
-
Eric Engestrom authored
While at it, let's include xf86atomic.h explicitly, instead of relying on some other file accidentally including it before including this file. Signed-off-by: Eric Engestrom <eric.engestrom@intel.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
b541d21a -
Eric Engestrom authored
Thanks to the #error just above, any file including this header can only see one state for this macro: defined, with the value `1`. Let's just #undef it once we're done using it in here so that other files don't misconstrue any meaning to it. Signed-off-by: Eric Engestrom <eric.engestrom@intel.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
6d237985 -
Eric Engestrom authored
Also, move the sentence about "who would use libdrm" into its own paragraph, as it is something people discovering libdrm will want to know. Signed-off-by: Eric Engestrom <eric.engestrom@intel.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
403f3c92 -
Alex Deucher authored
Reviewed-by: Michel Dänzer <michel.daenzer@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
cfab2fc3 -
Marek Olšák authored
it's in kernel 5.0 Reviewed-by: Christian König <christian.koenig@amd.com>
98cff551 -
Marek Olšák authored
Reviewed-by: Chunming Zhou <david1.zhou@amd.com> Signed-off-by: Marek Olšák <marek.olsak@amd.com>
f19afaa5 -
Marek Olšák authoredb7a7a903
-
Alex Deucher authored
Acked-by: Christian König <christian.koenig@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
6415bd3c -
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)
3df8a7f0 -
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>
ee798b98 -
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>
70a1ae89 -
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>
17dfe3ac -
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>
eba6609b -
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>
e09f3277 -
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>
1592d471 -
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>
56c21f87 -
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"
d54e5468 -
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>
0b032a64 -
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>
4735ca71 -
Alex Deucher authored
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
c8bc69a1 -
fcui 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>
8c6dbd79 -
fcui 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>
8db4e2db -
fcui 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>
c1ced0ba -
fcui 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>
00dd9b72 -
fcui 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>
852a9d20 -
fcui 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>
a85b31c9 -
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>
ae836dec -
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>
4835d74c -
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>
028cbfff -
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>
b21d23e3 -
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>
b4fbc6d7 -
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>
210bfdf8 -
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>
fcf80e2f -
Signed-off-by: Eric Engestrom <eric.engestrom@intel.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
ad8bec1e -
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>
360292c7 -
Signed-off-by: Fritz Koenig <frkoenig@google.com> Reviewed-by: Emil Velikov <emil.velikov@collabora.com>
f8f8b2b9 -
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.
439a4c03 -
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>
0b474eab -
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>
232dc330 -
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
File moved