[Intel-gfx] [PATCH v5 i-g-t 13/15] lib/i915: Add helper for non-destructive engine property updates

2022-01-14 Thread John . C . Harrison
From: John Harrison Various tests want to configure engine properties such as pre-emption timeout and heartbeat interval. Some don't bother to restore the original values again afterwards. So, add a helper to make it easier to do this. v2: Fix for platforms with no pre-emption capability. Signe

[Intel-gfx] [PATCH v5 i-g-t 10/15] tests/i915/i915_hangman: Run background task on all engines

2022-01-14 Thread John . C . Harrison
From: John Harrison As opposed to only on the non-target engines. This means that there is some other workload present for the scheduler to switch between and so detet the hang immediately. Signed-off-by: John Harrison Reviewed-by: Matthew Brost --- tests/i915/i915_hangman.c | 10 ++

[Intel-gfx] [CI] PR for new GuC v69.0.3

2022-01-14 Thread John . C . Harrison
The following changes since commit b0e898fbaf377c99a36aac6fdeb7250003648ca4: linux-firmware: Update firmware file for Intel Bluetooth 9462 (2021-11-23 12:31:45 -0500) are available in the Git repository at: git://anongit.freedesktop.org/drm/drm-firmware guc_v69.0.3 for you to fetch changes

[Intel-gfx] [PATCH v2 i-g-t 0/1] Add test for new hw info query

2022-01-19 Thread John . C . Harrison
From: John Harrison Various UMDs require hardware configuration information about the current platform. A new interface has been added to the KMD to return this information. So, add a test for the new interface. v2: Rebased to newer baseline. Signed-off-by: John Harrison Reviewed-by: Matthew B

[Intel-gfx] [PATCH v2 i-g-t 1/1] tests/i915/query: Query, parse and validate the hwconfig table

2022-01-19 Thread John . C . Harrison
From: Rodrigo Vivi Newer platforms have an embedded table giving details about that platform's hardware configuration. This table can be retrieved from the KMD via the existing query API. So add a test for it as both an example of how to fetch the table and to validate the contents as much as is

[Intel-gfx] [PATCH v3 1/2] drm/i915/guc: Add fetch of hwconfig table

2022-01-19 Thread John . C . Harrison
From: John Harrison Implement support for fetching the hardware description table from the GuC. The call is made twice - once without a destination buffer to query the size and then a second time to fill in the buffer. Note that the table is only available on ADL-P and later platforms. Cc: Mich

[Intel-gfx] [PATCH v3 0/2] Add support for querying hw info that UMDs need

2022-01-19 Thread John . C . Harrison
From: John Harrison Various UMDs require hardware configuration information about the current platform. A bunch of static information is available in a fixed table that can be retrieved from the GuC. v2: Rebased to newer baseline and added a kerneldoc comment. v3: Rebased to newer baseline and n

[Intel-gfx] [PATCH v3 2/2] drm/i915/uapi: Add query for hwconfig table

2022-01-19 Thread John . C . Harrison
From: Rodrigo Vivi GuC contains a consolidated table with a bunch of information about the current device. Previously, this information was spread and hardcoded to all the components including GuC, i915 and various UMDs. The goal here is to consolidate the data into GuC in a way that all interes

[Intel-gfx] [CI] PR for new GuC v70.0.5

2022-03-09 Thread John . C . Harrison
The following changes since commit f011ccb490f952ea35e9ce4d73ca9b7d0d2453c3: linux-firmware: add firmware for MT7986 (2022-03-04 08:43:26 -0500) are available in the Git repository at: git://anongit.freedesktop.org/drm/drm-firmware guc_v70.0.5_dg2 for you to fetch changes up to 850f9be3f587

[Intel-gfx] [PATCH v3 i-g-t 0/2] Update DRM UAPI and add test for new hw info query

2022-06-02 Thread John . C . Harrison
From: John Harrison Various UMDs require hardware configuration information about the current platform. A new interface has been added to the KMD to return this information. So, add a test for the new interface. Also, update to the latest DRM UAPI header file that contains the new query enums.

[Intel-gfx] [PATCH v3 i-g-t 1/2] include/drm-uapi: Update to latest i915_drm.h

2022-06-02 Thread John . C . Harrison
From: John Harrison Update to the latest master version of the DRM UAPI header file. NB: Had to remove '__user' keywords as they do not appear to be supported outside of kernel builds. Signed-off-by: John Harrison --- include/drm-uapi/i915_drm.h | 410 1 f

[Intel-gfx] [PATCH v3 i-g-t 2/2] tests/i915/query: Query, parse and validate the hwconfig table

2022-06-02 Thread John . C . Harrison
From: Rodrigo Vivi Newer platforms have an embedded table giving details about that platform's hardware configuration. This table can be retrieved from the KMD via the existing query API. So add a test for it as both an example of how to fetch the table and to validate the contents as much as is

[Intel-gfx] [PATCH v4 i-g-t 0/2] Update DRM UAPI and add test for new hw info query

2022-06-03 Thread John . C . Harrison
From: John Harrison Various UMDs require hardware configuration information about the current platform. A new interface has been added to the KMD to return this information. So, add a test for the new interface. Also, update to the latest DRM UAPI header file that contains the new query enums.

[Intel-gfx] [PATCH v4 i-g-t 2/2] tests/i915/query: Query, parse and validate the hwconfig table

2022-06-03 Thread John . C . Harrison
From: Rodrigo Vivi Newer platforms have an embedded table giving details about that platform's hardware configuration. This table can be retrieved from the KMD via the existing query API. So add a test for it as both an example of how to fetch the table and to validate the contents as much as is

[Intel-gfx] [PATCH v4 i-g-t 1/2] include/drm-uapi: Update to latest i915_drm.h

2022-06-03 Thread John . C . Harrison
From: John Harrison Update to the latest master version of the DRM UAPI header file from git://anongit.freedesktop.org/git/drm/drm: c4955d9cd2fc Merge tag 'drm-intel-next-fixes-2022-05-24' of git://anongit.freedesktop.org/drm/drm-intel into drm-next Signed-off-by: John Harrison --- include

[Intel-gfx] [PATCH v5 i-g-t 3/3] tests/i915/query: Query, parse and validate the hwconfig table

2022-06-03 Thread John . C . Harrison
From: Rodrigo Vivi Newer platforms have an embedded table giving details about that platform's hardware configuration. This table can be retrieved from the KMD via the existing query API. So add a test for it as both an example of how to fetch the table and to validate the contents as much as is

[Intel-gfx] [PATCH v5 i-g-t 1/3] include/drm-uapi: Update to latest i915_drm.h

2022-06-03 Thread John . C . Harrison
From: John Harrison Update to the latest master version of the DRM UAPI header file from git://anongit.freedesktop.org/git/drm/drm: c4955d9cd2fc Merge tag 'drm-intel-next-fixes-2022-05-24' of git://anongit.freedesktop.org/drm/drm-intel into drm-next Signed-off-by: John Harrison --- include

[Intel-gfx] [PATCH v5 i-g-t 0/3] Update DRM UAPI and add test for new hw info query

2022-06-03 Thread John . C . Harrison
From: John Harrison Various UMDs require hardware configuration information about the current platform. A new interface has been added to the KMD to return this information. So, add a test for the new interface. Also, update to the latest DRM UAPI header file that contains the new query enums.

[Intel-gfx] [PATCH v5 i-g-t 2/3] tests/i915/query: Add descriptions to existing tests

2022-06-03 Thread John . C . Harrison
From: John Harrison None of the query tests had a description. So make some up. Signed-off-by: John Harrison --- tests/i915/i915_query.c | 12 1 file changed, 12 insertions(+) diff --git a/tests/i915/i915_query.c b/tests/i915/i915_query.c index 246a979af72a..35a91d245ec1 100644 -

[Intel-gfx] [PATCH v6 i-g-t 1/3] include/drm-uapi: Update to latest i915_drm.h

2022-06-07 Thread John . C . Harrison
From: John Harrison Update to the latest master version of the DRM UAPI header file from git://anongit.freedesktop.org/git/drm/drm: c4955d9cd2fc Merge tag 'drm-intel-next-fixes-2022-05-24' of git://anongit.freedesktop.org/drm/drm-intel into drm-next Signed-off-by: John Harrison Reviewed-by:

[Intel-gfx] [PATCH v6 i-g-t 0/3] Update DRM UAPI and add test for new hw info query

2022-06-07 Thread John . C . Harrison
From: John Harrison Various UMDs require hardware configuration information about the current platform. A new interface has been added to the KMD to return this information. So, add a test for the new interface. Also, update to the latest DRM UAPI header file that contains the new query enums.

[Intel-gfx] [PATCH v6 i-g-t 2/3] tests/i915/query: Add descriptions to existing tests

2022-06-07 Thread John . C . Harrison
From: John Harrison None of the query tests had a description. So make some up. Signed-off-by: John Harrison Reviewed-by: Kamil Konieczny --- tests/i915/i915_query.c | 12 1 file changed, 12 insertions(+) diff --git a/tests/i915/i915_query.c b/tests/i915/i915_query.c index 246a9

[Intel-gfx] [PATCH v6 i-g-t 3/3] tests/i915/query: Query, parse and validate the hwconfig table

2022-06-07 Thread John . C . Harrison
From: Rodrigo Vivi Newer platforms have an embedded table giving details about that platform's hardware configuration. This table can be retrieved from the KMD via the existing query API. So add a test for it as both an example of how to fetch the table and to validate the contents as much as is

[Intel-gfx] [PATCH] drm/i915/guc: Use drm_err instead of pr_err

2022-06-07 Thread John . C . Harrison
From: John Harrison Don't use pr_err in places where we have access to a struct_drm. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 10 ++--- drivers/gpu/drm/i915/gt/uc/selftest_guc.c | 38 +-- .../drm/i915/gt/uc/selftest_guc_multi_lrc.c

[Intel-gfx] [PATCH 02/12] drm/i915/guc: Don't call ring_is_idle in GuC submission

2022-07-12 Thread John . C . Harrison
From: Matthew Brost The engine registers really shouldn't be touched during GuC submission as the GuC owns the registers. Don't call ring_is_idle and tie intel_engine_is_idle strictly to the engine pm. Because intel_engine_is_idle tied to the engine pm, retire requests before checking intel_engi

[Intel-gfx] [PATCH 00/12] Random assortment of (mostly) GuC related patches

2022-07-12 Thread John . C . Harrison
From: John Harrison Pushing a bunch of patches which had gotten forgotten about. Signed-off-by: John Harrison Alan Previn (1): drm/i915/guc: Add a helper for log buffer size Chris Wilson (1): drm/i915/guc: Use streaming loads to speed up dumping the guc log John Harrison (4): drm/i915

[Intel-gfx] [PATCH 08/12] drm/i915/guc: Add selftest for a hung GuC

2022-07-12 Thread John . C . Harrison
From: Rahul Kumar Singh Add a test to check that the hangcheck will recover from a submission hang in the GuC. Signed-off-by: Rahul Kumar Singh --- .../gpu/drm/i915/gt/uc/intel_guc_submission.c | 1 + .../drm/i915/gt/uc/selftest_guc_hangcheck.c | 159 ++ .../drm/i915/selfte

[Intel-gfx] [PATCH 05/12] drm/i915/guc: Record CTB info in error logs

2022-07-12 Thread John . C . Harrison
From: John Harrison When debugging GuC communication issues, it is useful to have the CTB info available. So add the state and buffer contents to the error capture log. Also, add a sub-structure for the GuC specific error capture info as it is now becoming numerous. Signed-off-by: John Harrison

[Intel-gfx] [PATCH 01/12] drm/i915: Remove bogus GEM_BUG_ON in unpark

2022-07-12 Thread John . C . Harrison
From: Matthew Brost Remove bogus GEM_BUG_ON which compared kernel context timeline seqno to seqno in memory on engine PM unpark. If a GT reset occurred these values might not match as a kernel context could be skipped. This bug was hidden by always switching to a kernel context on park (execlists

[Intel-gfx] [PATCH 12/12] drm/i915/guc: Add a helper for log buffer size

2022-07-12 Thread John . C . Harrison
From: Alan Previn Add a helper to get GuC log buffer size. Signed-off-by: Alan Previn --- drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 49 -- 1 file changed, 27 insertions(+), 22 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/uc/intel_guc_log.c b/drivers/gpu/drm/i915/g

[Intel-gfx] [PATCH 06/12] drm/i915/guc: Use streaming loads to speed up dumping the guc log

2022-07-12 Thread John . C . Harrison
From: Chris Wilson Use a temporary page and mempy_from_wc to reduce the time it takes to dump the guc log to debugfs. Signed-off-by: Chris Wilson --- drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 24 -- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/drivers/g

[Intel-gfx] [PATCH 03/12] drm/i915/guc: Fix issues with live_preempt_cancel

2022-07-12 Thread John . C . Harrison
From: Matthew Brost Having semaphores results in different behavior when a dependent request is cancelled. In the case of semaphores the request could be on the HW and complete successfully while without the request is held in the driver and the error from the dependent request is propagated. Fix

[Intel-gfx] [PATCH 10/12] drm/i915/guc: Support larger contexts on newer hardware

2022-07-12 Thread John . C . Harrison
From: Matthew Brost The GuC needs a copy of a golden context for implementing watchdog resets (aka media resets). This context is larger on newer platforms. So adjust the size being allocated/copied accordingly. Signed-off-by: Matthew Brost --- drivers/gpu/drm/i915/gt/uc/intel_guc_ads.c | 10 +

[Intel-gfx] [PATCH 11/12] drm/i915/guc: Don't abort on CTB_UNUSED status

2022-07-12 Thread John . C . Harrison
From: John Harrison When the KMD sends a CLIENT_RESET request to GuC (as part of the suspend sequence), GuC will mark the CTB buffer as 'UNUSED'. If the KMD then checked the CTB queue, it would see a non-zero status value and report the buffer as corrupted. Technically, no G2H messages should be

[Intel-gfx] [PATCH 09/12] drm/i915/selftest: Cope with not having an RCS engine

2022-07-12 Thread John . C . Harrison
From: John Harrison It is no longer guaranteed that there will always be an RCS engine. So, use the helper function for finding the first available engine that can be used for general purpose selftets. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/selftest_hangcheck.c | 12 +

[Intel-gfx] [PATCH 04/12] drm/i915/guc: Add GuC <-> kernel time stamp translation information

2022-07-12 Thread John . C . Harrison
From: John Harrison It is useful to be able to match GuC events to kernel events when looking at the GuC log. That requires being able to convert GuC timestamps to kernel time. So, when dumping error captures and/or GuC logs, include a stamp in both time zones plus the clock frequency. Signed-of

[Intel-gfx] [PATCH 07/12] drm/i915/guc: Route semaphores to GuC for Gen12+

2022-07-12 Thread John . C . Harrison
From: Michał Winiarski Since we're going to use semaphores in selftests (and eventually in regular GuC submission), let's route semaphores to GuC. Signed-off-by: Michał Winiarski --- drivers/gpu/drm/i915/gt/uc/intel_guc_reg.h| 4 drivers/gpu/drm/i915/gt/uc/intel_guc_submission.c

[Intel-gfx] [PATCH] drm/i915/guc: Don't use pr_err when not necessary

2022-07-12 Thread John . C . Harrison
From: John Harrison A bunch of code was copy/pasted using pr_err as the default way to report errors. However, drm_err is significantly more useful in identifying where the error came from. So update the code to use that instead. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/inte

[Intel-gfx] [PATCH] drm/i915/guc: Don't use pr_err when not necessary

2022-07-14 Thread John . C . Harrison
From: John Harrison A bunch of code was copy/pasted using pr_err as the default way to report errors. However, drm_err is significantly more useful in identifying where the error came from. So update the code to use that instead. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/inte

[Intel-gfx] [CI] PR for new GuC v70.4.1 for DG2

2022-07-27 Thread John . C . Harrison
The following changes since commit 150864a4d73e8c448eb1e2c68e65f07635fe1a66: amdgpu partially revert "amdgpu: update beige goby to release 22.20" (2022-07-25 14:16:04 -0400) are available in the Git repository at: git://anongit.freedesktop.org/drm/drm-firmware dg2_guc_v70.4.1 for you to fe

[Intel-gfx] [PATCH 1/7] drm/i915/guc: Add a helper for log buffer size

2022-07-27 Thread John . C . Harrison
From: Alan Previn Add a helper to get GuC log buffer size. Signed-off-by: Alan Previn Signed-off-by: John Harrison Reviewed-by: Matthew Brost --- drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 49 -- 1 file changed, 27 insertions(+), 22 deletions(-) diff --git a/drivers/gp

[Intel-gfx] [PATCH 5/7] drm/i915/guc: Use streaming loads to speed up dumping the guc log

2022-07-27 Thread John . C . Harrison
From: Chris Wilson Use a temporary page and mempy_from_wc to reduce the time it takes to dump the guc log to debugfs. Signed-off-by: Chris Wilson Signed-off-by: John Harrison Reviewed-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_guc_log.c | 24 -- 1 file changed

[Intel-gfx] [PATCH 4/7] drm/i915/guc: Record CTB info in error logs

2022-07-27 Thread John . C . Harrison
From: John Harrison When debugging GuC communication issues, it is useful to have the CTB info available. So add the state and buffer contents to the error capture log. Also, add a sub-structure for the GuC specific error capture info as it is now becoming numerous. Signed-off-by: John Harrison

[Intel-gfx] [PATCH 0/7] Fixes and improvements to GuC logging and error capture

2022-07-27 Thread John . C . Harrison
From: John Harrison Fix bugs and improve the usability/effectiveness of GuC logging and GuC related error captures. Signed-off-by: John Harrison Alan Previn (1): drm/i915/guc: Add a helper for log buffer size Chris Wilson (1): drm/i915/guc: Use streaming loads to speed up dumping the guc

[Intel-gfx] [PATCH 7/7] drm/i915/guc: Reduce spam from error capture

2022-07-27 Thread John . C . Harrison
From: John Harrison Some debug code got left in when the GuC based register save for error capture was added. Remove that. Signed-off-by: John Harrison --- .../gpu/drm/i915/gt/uc/intel_guc_capture.c| 67 --- 1 file changed, 28 insertions(+), 39 deletions(-) diff --git a/dr

[Intel-gfx] [PATCH 6/7] drm/i915/guc: Make GuC log sizes runtime configurable

2022-07-27 Thread John . C . Harrison
From: John Harrison The GuC log buffer sizes had to be configured statically at compile time. This can be quite troublesome when needing to get larger logs out of a released driver. So re-organise the code to allow a boot time module parameter override. Signed-off-by: John Harrison --- drivers

[Intel-gfx] [PATCH 3/7] drm/i915/guc: Add GuC <-> kernel time stamp translation information

2022-07-27 Thread John . C . Harrison
From: John Harrison It is useful to be able to match GuC events to kernel events when looking at the GuC log. That requires being able to convert GuC timestamps to kernel time. So, when dumping error captures and/or GuC logs, include a stamp in both time zones plus the clock frequency. Signed-of

[Intel-gfx] [PATCH 2/7] drm/i915/guc: Fix capture size warning and bump the size

2022-07-27 Thread John . C . Harrison
From: John Harrison There was a size check to warn if the GuC error state capture buffer allocation would be too small to fit a reasonable amount of capture data for the current platform. Unfortunately, the test was done too early in the boot sequence and was actually testing 'if(-ENODEV > size)'

[Intel-gfx] [PATCH 0/6] Random assortment of (mostly) GuC related patches

2022-07-27 Thread John . C . Harrison
From: John Harrison Pushing a bunch of patches which had gotten forgotten about. Signed-off-by: John Harrison John Harrison (2): drm/i915/selftest: Cope with not having an RCS engine drm/i915/guc: Don't abort on CTB_UNUSED status Matthew Brost (2): drm/i915/guc: Fix issues with live_pr

[Intel-gfx] [PATCH 6/6] drm/i915/guc: Don't abort on CTB_UNUSED status

2022-07-27 Thread John . C . Harrison
From: John Harrison When the KMD sends a CLIENT_RESET request to GuC (as part of the suspend sequence), GuC will mark the CTB buffer as 'UNUSED'. If the KMD then checked the CTB queue, it would see a non-zero status value and report the buffer as corrupted. Technically, no G2H messages should be

[Intel-gfx] [PATCH 3/6] drm/i915/guc: Add selftest for a hung GuC

2022-07-27 Thread John . C . Harrison
From: Rahul Kumar Singh Add a test to check that the hangcheck will recover from a submission hang in the GuC. Signed-off-by: Rahul Kumar Singh Signed-off-by: John Harrison --- .../gpu/drm/i915/gt/uc/intel_guc_submission.c | 1 + .../drm/i915/gt/uc/selftest_guc_hangcheck.c | 159 +

[Intel-gfx] [PATCH 4/6] drm/i915/selftest: Cope with not having an RCS engine

2022-07-27 Thread John . C . Harrison
From: John Harrison It is no longer guaranteed that there will always be an RCS engine. So, use the helper function for finding the first available engine that can be used for general purpose selftets. Signed-off-by: John Harrison Reviewed-by: Matthew Brost --- drivers/gpu/drm/i915/gt/selftes

[Intel-gfx] [PATCH 1/6] drm/i915/guc: Route semaphores to GuC for Gen12+

2022-07-27 Thread John . C . Harrison
From: Michał Winiarski In GuC submission mode, there is an option to use auto-switch out semaphores and have GuC auto-switch in a waiting context. This requires routing the semaphore interrupt to GuC. Signed-off-by: Michał Winiarski Signed-off-by: John Harrison Reviewed-by: Matthew Brost Revi

[Intel-gfx] [PATCH 5/6] drm/i915/guc: Support larger contexts on newer hardware

2022-07-27 Thread John . C . Harrison
From: Matthew Brost The GuC needs a copy of a golden context for implementing watchdog resets (aka media resets). This context is larger on newer platforms. So adjust the size being allocated/copied accordingly. Signed-off-by: Matthew Brost Signed-off-by: John Harrison --- drivers/gpu/drm/i91

[Intel-gfx] [PATCH 2/6] drm/i915/guc: Fix issues with live_preempt_cancel

2022-07-27 Thread John . C . Harrison
From: Matthew Brost Having semaphores results in different behavior when a dependent request is cancelled. In the case of semaphores the request could be on the HW and complete successfully while without the request is held in the driver and the error from the dependent request is propagated. Fix

[Intel-gfx] [PATCH 3/6] drm/i915/guc: Add selftest for a hung GuC

2022-07-28 Thread John . C . Harrison
From: Rahul Kumar Singh Add a test to check that the hangcheck will recover from a submission hang in the GuC. Signed-off-by: Rahul Kumar Singh Signed-off-by: John Harrison --- .../gpu/drm/i915/gt/uc/intel_guc_submission.c | 1 + .../drm/i915/gt/uc/selftest_guc_hangcheck.c | 159 +

[Intel-gfx] [PATCH 0/1] Move DG2 to GuC v70.4.1

2022-07-28 Thread John . C . Harrison
From: John Harrison The latest GuC release contains a bunch of fixes for DG2. Start using it. Note that some of these fixes require i915 side support which will follow in separate patches. Signed-off-by: John Harrison John Harrison (1): drm/i915/dg2: Update DG2 to GuC v70.4.1 drivers/gpu

[Intel-gfx] [PATCH 1/1] drm/i915/dg2: Update DG2 to GuC v70.4.1

2022-07-28 Thread John . C . Harrison
From: John Harrison New release of GuC with a bunch of fixes specific to DG2. Some of these require follow up i915 changes to enable. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i9

[Intel-gfx] PR for new GuC v70.4.1 for DG2

2022-08-01 Thread John . C . Harrison
The following changes since commit 150864a4d73e8c448eb1e2c68e65f07635fe1a66: amdgpu partially revert "amdgpu: update beige goby to release 22.20" (2022-07-25 14:16:04 -0400) are available in the Git repository at: git://anongit.freedesktop.org/drm/drm-firmware dg2_guc_v70.4.1 for you to fe

[Intel-gfx] [PATCH 1/2] drm/i915/uc: Support for version reduced and multiple firmware files

2022-08-16 Thread John . C . Harrison
From: John Harrison There was a misunderstanding in how firmware file compatibility should be managed within i915. This has been clarified as: i915 must support all existing firmware releases forever new minor firmware releases should replace prior versions only backwards compatibility brea

[Intel-gfx] [PATCH 2/2] drm/i915/uc: Enable version reduced firmware files for newest platforms

2022-08-16 Thread John . C . Harrison
From: John Harrison Going forwards, the intention is for GuC firmware files to be named for their major version only and HuC firmware files to have no version number in the name at all. This patch adds those entries for DG2 and ADL-P/S. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/

[Intel-gfx] [PATCH 0/2] Drop version numbers from firmware files

2022-08-16 Thread John . C . Harrison
From: John Harrison Upstream direction is to include the bare minimum of version numbers in firmware files and to replace them in the repo rather than accumulating them. For HuC, that means going completely versionless. For GuC, the major version needs to be kept as that indicates a break in back

[Intel-gfx] [CI] PR for new GuC v70.1.1

2022-04-07 Thread John . C . Harrison
The following changes since commit 681281e49fb6778831370e5d94e6e1d97f0752d6: amdgpu: update PSP 13.0.8 firmware (2022-03-18 07:35:54 -0400) are available in the Git repository at: git://anongit.freedesktop.org/drm/drm-firmware guc_v70.1.1_dg2 for you to fetch changes up to 0a2cdf9b1b74322bf

[Intel-gfx] [PATCH 0/3] Update to GuC v70

2022-04-08 Thread John . C . Harrison
From: John Harrison Update to the latest GuC firmware release. Note that this includes some significant backwards breaking API changes. One is about context registration - the descriptor pool is gone, all parameters are passed via the CTB instead. The second is about scheduling policy updates -

[Intel-gfx] [PATCH 1/3] drm/i915/guc: Update context registration to new GuC API

2022-04-08 Thread John . C . Harrison
From: John Harrison The latest GuC firmware drops the context descriptor pool in favour of passing all creation data in the create H2G. It also greatly simplifies the work queue and removes the process descriptor used for multi-LRC submission. So, remove all mention of LRC and process descriptors

[Intel-gfx] [PATCH 3/3] drm/i915/guc: Update to GuC version 70.1.1

2022-04-08 Thread John . C . Harrison
From: John Harrison Update to the latest GuC firmware release. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c b/drivers/gpu/drm/i9

[Intel-gfx] [PATCH 2/3] drm/i915/guc: Update scheduling policies to new GuC API

2022-04-08 Thread John . C . Harrison
From: John Harrison The latest GuC firmware drops the individual scheduling policy update H2G commands in favour of a single KLV based H2G. So, change the update wrappers accordingly. Unfortunately, the API changes also mean losing the ability to set any scheduling policy values during context r

[Intel-gfx] [PATCH 1/1] drm/i915/guc: Update to GuC version 70.1.1

2022-04-12 Thread John . C . Harrison
From: John Harrison The latest GuC firmware drops the context descriptor pool in favour of passing all creation data in the create H2G. It also greatly simplifies the work queue and removes the process descriptor used for multi-LRC submission. So, remove all mention of LRC and process descriptors

[Intel-gfx] [PATCH 0/1] Update to GuC v70

2022-04-12 Thread John . C . Harrison
From: John Harrison Update to the latest GuC firmware release. Note that this includes some significant backwards breaking API changes. One is about context registration - the descriptor pool is gone, all parameters are passed via the CTB instead. The second is about scheduling policy updates -

[Intel-gfx] PR for new GuC v70.1.1

2022-04-19 Thread John . C . Harrison
The following changes since commit 681281e49fb6778831370e5d94e6e1d97f0752d6: amdgpu: update PSP 13.0.8 firmware (2022-03-18 07:35:54 -0400) are available in the Git repository at: git://anongit.freedesktop.org/drm/drm-firmware guc_v70.1.1 for you to fetch changes up to ab0d8c137d4235dbb09ac

[Intel-gfx] [CI] PR for new GuC v70.1.2 for DG2

2022-04-26 Thread John . C . Harrison
The following changes since commit ac21ab5d1de0de34201c90d32eee436f873d1e5b: Mellanox: Add lc_ini_bundle for xx.2010.1006 (2022-04-25 07:36:16 -0400) are available in the Git repository at: git://anongit.freedesktop.org/drm/drm-firmware guc_v70.1.2_dg2 for you to fetch changes up to 89ae5eb

[Intel-gfx] [PATCH 0/2] Initial GuC firmware release for DG2

2022-04-27 Thread John . C . Harrison
From: John Harrison Add GuC firmware for DG2. Note that an older version of this patch exists in the CI topic branch. Hence this set includes a revert of that patch before applying the new version. When merging, the revert would simply be dropped and the corresponding patch in the topic branch w

[Intel-gfx] [PATCH 1/2] Revert "drm/i915/dg2: Define GuC firmware version for DG2"

2022-04-27 Thread John . C . Harrison
From: John Harrison This reverts commit 55c7f980e48e56861496526e02ed5bbfdac49ede. The CI topic branch within drm-top contains an old patch for supporting GuC on DG2. That needs to be dropped and an updated patch merged to drm-gt-next. Hence this patch reverts it so the new patch can be sent in i

[Intel-gfx] [PATCH 2/2] drm/i915/dg2: Define GuC firmware version for DG2

2022-04-27 Thread John . C . Harrison
From: John Harrison First release of GuC for DG2. Signed-off-by: John Harrison CC: Tomasz Mistat CC: Ramalingam C CC: Daniele Ceraolo Spurio --- drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/i915/gt/uc/intel_uc_fw.c b/drivers/g

[Intel-gfx] PR for new GuC v70.1.2 for DG2

2022-04-28 Thread John . C . Harrison
The following changes since commit ac21ab5d1de0de34201c90d32eee436f873d1e5b: Mellanox: Add lc_ini_bundle for xx.2010.1006 (2022-04-25 07:36:16 -0400) are available in the Git repository at: git://anongit.freedesktop.org/drm/drm-firmware guc_v70.1.2_dg2 for you to fetch changes up to 89ae5eb

[Intel-gfx] [PATCH v2 i-g-t 6/8] lib/igt_sysfs: Support large files

2021-11-03 Thread John . C . Harrison
From: John Harrison The syfs helper functions were all using basic 'int' data types for sizs, offsets, etc. when reading from sysfs. This works fine for little files, but not for large error capture logs (which can be gigabytes in sizes). Signed-off-by: John Harrison Reviewed-by: Matthew Brost

[Intel-gfx] [PATCH v2 i-g-t 0/8] Fixes for gem_exec_capture

2021-11-03 Thread John . C . Harrison
From: John Harrison Fix a bunch of issues with gem_exec_capture with the ultimate aim of making it pass on GuC enabled platforms. v2: Abstract the 'find first available engine' block into a helper (review feedback from Matthew B). Note that for unknown reasons, this helper does not work as a fun

[Intel-gfx] [PATCH v2 i-g-t 3/8] tests/i915/gem_exec_capture: Make the error decode a common helper

2021-11-03 Thread John . C . Harrison
From: John Harrison The decode of the error capture contents was happening in two different sub-tests with two very different pieces of code. One being much more extensive than the other (actually decodes and verifies the contents of the captured buffers rather than just the address). So, move th

[Intel-gfx] [PATCH v2 i-g-t 2/8] tests/i915/gem_exec_capture: Cope with larger page sizes

2021-11-03 Thread John . C . Harrison
From: John Harrison At some point, larger than 4KB page sizes were added to the i915 driver. This included adding an informational line to the buffer entries in error capture logs. However, the error capture test was not updated to skip this string, thus it would silently abort processing. Signe

[Intel-gfx] [PATCH v2 i-g-t 7/8] lib/igt_gt: Allow per engine reset testing

2021-11-03 Thread John . C . Harrison
From: John Harrison With GuC submission, engine resets are handled entirely within GuC rather than within i915. Traditionally, IGT has disallowed engine based resets becuase they don't send the uevent which IGT uses to check for unexpected resets. However, it is important to be able to test all r

[Intel-gfx] [PATCH v2 i-g-t 1/8] tests/i915/gem_exec_capture: Remove pointless assert

2021-11-03 Thread John . C . Harrison
From: John Harrison The 'many' test ended with an 'assert(count)', presumably meaning to ensure that some objects were actually captured. However, 'count' is the number of objects created not how many were captured. Plus, there is already a 'require(count > 1)' at the start and count is invarient

[Intel-gfx] [PATCH v2 i-g-t 5/8] tests/i915/gem_exec_capture: Check for memory allocation failure

2021-11-03 Thread John . C . Harrison
From: John Harrison The sysfs file read helper does not actually report any errors if a realloc fails. It just silently returns a 'valid' but truncated buffer. This then leads to the decode of the buffer failing in random ways. So, add a check for ENOMEM being generated during the read. Signed-o

[Intel-gfx] [PATCH v2 i-g-t 4/8] tests/i915/gem_exec_capture: Use contexts and engines properly

2021-11-03 Thread John . C . Harrison
From: John Harrison Some of the capture tests were using explicit contexts, some not. Some were poking the per engine pre-emption timeout, some not. This would lead to sporadic failures due to random timeouts, contexts being banned depending upon how many subtests were run and/or how many engines

[Intel-gfx] [PATCH v2 i-g-t 8/8] tests/i915/gem_exec_capture: Update to support GuC based resets

2021-11-03 Thread John . C . Harrison
From: John Harrison When GuC submission is enabled, GuC itself manages hang detection and recovery. Therefore, any test that relies on being able to trigger an engine reset in the driver will fail. Full GT resets can still be triggered by the driver. However, in that situation detecting the speci

[Intel-gfx] [PATCH 4/4] drm/i915/guc: Don't go bang in GuC log if no GuC

2021-12-02 Thread John . C . Harrison
From: John Harrison If the GuC has failed to load for any reason and then the user pokes the debugfs GuC log interface, a BUG and/or null pointer deref can occur. Don't let that happen. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_guc_log_debugfs.c | 4 ++-- 1 file changed

[Intel-gfx] [PATCH 2/4] drm/i915/guc: Request RP0 before loading firmware

2021-12-02 Thread John . C . Harrison
From: Vinay Belgaumkar By default, GT (and GuC) run at RPn. Requesting for RP0 before firmware load can speed up DMA and HuC auth as well. In addition to writing to 0xA008, we also need to enable swreq in 0xA024 so that Punit will pay heed to our request. Signed-off-by: Vinay Belgaumkar --- dr

[Intel-gfx] [PATCH 0/4] Assorted fixes/tweaks to GuC support

2021-12-02 Thread John . C . Harrison
From: John Harrison Fix a potential null pointer dereference, improve debug crash reports, improve code separation, improve GuC/HuC load performance. Signed-off-by: John Harrison John Harrison (3): drm/i915/uc: Allow platforms to have GuC but not HuC drm/i915/guc: Increase GuC log size f

[Intel-gfx] [PATCH 1/4] drm/i915/uc: Allow platforms to have GuC but not HuC

2021-12-02 Thread John . C . Harrison
From: John Harrison It is possible for platforms to require GuC but not HuC firmware. Also, the firmware versions for GuC and HuC advance independently. So split the macros up to allow the lists to be maintained separately. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_uc_f

[Intel-gfx] [PATCH 3/4] drm/i915/guc: Increase GuC log size for CONFIG_DEBUG_GEM

2021-12-02 Thread John . C . Harrison
From: John Harrison Lots of testing is done with the DEBUG_GEM config option enabled but not the DEBUG_GUC option. That means we only get teeny-tiny GuC logs which are not hugely useful. Enabling full DEBUG_GUC also spews lots of other detailed output that is not generally desired. However, bigge

[Intel-gfx] [CI] PR for new GuC v69.0.0

2021-12-02 Thread John . C . Harrison
The following changes since commit b0e898fbaf377c99a36aac6fdeb7250003648ca4: linux-firmware: Update firmware file for Intel Bluetooth 9462 (2021-11-23 12:31:45 -0500) are available in the Git repository at: git://anongit.freedesktop.org/drm/drm-firmware guc_v69 for you to fetch changes up

[Intel-gfx] [PATCH 0/5] Update to GuC version 69.0.0

2021-12-03 Thread John . C . Harrison
From: John Harrison Update to the latest GuC version. This includes a suite of interface changes and new features with corresponding i915 side changes. Also, fix/improve a bunch of other things while at it. Signed-off-by: John Harrison John Harrison (5): drm/i915/uc: Allow platforms to ha

[Intel-gfx] [PATCH 2/5] drm/i915/guc: Increase GuC log size for CONFIG_DEBUG_GEM

2021-12-03 Thread John . C . Harrison
From: John Harrison Lots of testing is done with the DEBUG_GEM config option enabled but not the DEBUG_GUC option. That means we only get teeny-tiny GuC logs which are not hugely useful. Enabling full DEBUG_GUC also spews lots of other detailed output that is not generally desired. However, bigge

[Intel-gfx] [PATCH 4/5] drm/i915/guc: Update to GuC version 69.0.0

2021-12-03 Thread John . C . Harrison
From: John Harrison Update to the latest GuC release. The latest GuC firmware introduces a number of interface changes: GuC may return NO_RESPONSE_RETRY message for requests sent over CTB. Add support for this reply and try resending the request again as a new CTB message. A KLV (key-length-va

[Intel-gfx] [PATCH 1/5] drm/i915/uc: Allow platforms to have GuC but not HuC

2021-12-03 Thread John . C . Harrison
From: John Harrison It is possible for platforms to require GuC but not HuC firmware. Also, the firmware versions for GuC and HuC advance independently. So split the macros up to allow the lists to be maintained separately. Signed-off-by: John Harrison --- drivers/gpu/drm/i915/gt/uc/intel_uc_f

[Intel-gfx] [PATCH 3/5] drm/i915/guc: Don't go bang in GuC log if no GuC

2021-12-03 Thread John . C . Harrison
From: John Harrison If the GuC has failed to load for any reason and then the user pokes the debugfs GuC log interface, a BUG and/or null pointer deref can occur. Don't let that happen. Signed-off-by: John Harrison Reviewed-by: Lucas De Marchi --- drivers/gpu/drm/i915/gt/uc/intel_guc_log_debu

[Intel-gfx] [PATCH 5/5] drm/i915/guc: Improve GuC loading status check/error reports

2021-12-03 Thread John . C . Harrison
From: John Harrison If the GuC fails to load, it is useful to know what firmware file / version was attempted. So move the version info report to before the load attempt rather than only after a successful load. If the GuC does fail to load, then make the error messages visible rather than being

[Intel-gfx] [PATCH] drm/i915/guc: Fix flag query helper function to not modify state

2022-02-17 Thread John . C . Harrison
From: John Harrison A flag query helper was actually writing to the flags word rather than just reading. Fix that. Also update the function's comment as it was out of date. NB: No need for a 'Fixes' tag. The test was only ever used inside a BUG_ON during context registration. Rather than asserti

[Intel-gfx] [PATCH 5/8] drm/i915/guc: Move lrc desc setup to where it is needed

2022-02-17 Thread John . C . Harrison
From: John Harrison The LRC descriptor was being initialised early on in the context registration sequence. It could then be determined that the actual registration needs to be delayed and the descriptor would be wiped out. This is inefficient, so move the setup to later in the process after the

[Intel-gfx] [PATCH 0/8] Prep work for next GuC release

2022-02-17 Thread John . C . Harrison
From: John Harrison The next GuC firmware release includes some significant backwards breaking API changes. One such is that there is no longer an LRC descriptor pool. A bunch of prep work for that change can be done in advance - the descriptor pool was being used for things it shouldn't really h

[Intel-gfx] [PATCH 4/8] drm/i915/guc: Split guc_lrc_desc_pin apart

2022-02-17 Thread John . C . Harrison
From: John Harrison The LRC descriptor pool is going away. Further, the function that was populating it was also doing a bunch of logic about the context registration sequence. So, split that code apart into separate state setup and try to register functions. Note that some of those 'try to regis

<    1   2   3   4   5   6   7   8   9   10   >