* [PATCH v7 0/5] bits: Fixed-type GENMASK_U*() and BIT_U*()
@ 2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
0 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol via B4 Relay @ 2025-03-22 9:23 UTC (permalink / raw)
To: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton
Cc: linux-kernel, intel-gfx, dri-devel, Andi Shyti, David Laight,
Andy Shevchenko, Jani Nikula, Vincent Mailhol
Introduce some fixed width variant of the GENMASK() and the BIT()
macros in bits.h. Note that the main goal is not to get the correct
type, but rather to enforce more checks at compile time. For example:
GENMASK_U16(16, 0)
will raise a build bug.
This series is a continuation of:
https://lore.kernel.org/intel-xe/20240208074521.577076-1-lucas.demarchi@intel.com
from Lucas De Marchi. Above series is one year old. I really think
that this was a good idea and I do not want this series to die. So I
am volunteering to revive it.
Meanwhile, many changes occurred in bits.h. The most significant
change is that __GENMASK() was moved to the uapi headers. For this
reason, a new GENMASK_TYPE() is introduced instead and the uapi
__GENMASK() is left untouched.
Finally, I do not think it makes sense to expose the fixed width
variants to the asm. The fixed width integers type are a C concept. So
the GENMASK_U*() are only visible to the non-asm code. For asm, the
long and long long variants seems sufficient.
This series does not modify the actual GENMASK(), GENMASK_ULL() and
GENMASK_U128(). A consolidation of the existing genmasks will be
proposed later on in a separate series.
As requested, here are the bloat-o-meter stats:
$ ./scripts/bloat-o-meter vmlinux_before.o vmlinux_after.o
add/remove: 0/0 grow/shrink: 0/0 up/down: 0/0 (0)
Function old new delta
Total: Before=22723481, After=22723481, chg +0.00%
(done with GCC 12.4.1 on an x86_64 defconfig)
--
2.43.0
---
Changes from v6:
- Split the series in two: this series leave any existing GENMASK*()
unmodified. The consolidation will be done in a separate series.
- Collect some Reviewed-by tag.
- Address miscellaneous nitpick on the code comments and the line
wrapping (details in each patch).
- Link to v6: https://lore.kernel.org/r/20250308-fixed-type-genmasks-v6-0-f59315e73c29@wanadoo.fr
Changes from v5:
- Update the cover letter message. I was still refering to
GENMASK_t() instead of GENMASK_TYPE().
- Add a comment in the cover letter to explain that a common
GENMASK_TYPE() for C and asm wouldn't allow to generate the u128
variant.
- Restore the comment saying that BUILD_BUG_ON() is not available in
asm code.
- Add a FIXME message to highlight the absence of the asm GENMASK*()
unit tests.
- Use git's histogram diff algorithm
- Link to v5: https://lore.kernel.org/r/20250306-fixed-type-genmasks-v5-0-b443e9dcba63@wanadoo.fr
Changes from v4:
- Rebase on https://github.com/norov/linux/tree/bitmap-for-next
- Rename GENMASK_t() to GENMASK_TYPE()
- First patch of v4 (the typo fix 'init128' -> 'int128') is removed
because it was resent separately in:
https://lore.kernel.org/all/20250305-fix_init128_typo-v1-1-cbe5b8e54e7d@wanadoo.fr
- Replace the (t)~ULL(0) by type_max(t). This way, GENMASK_TYPE()
can now be used to generate GENMASK_U128().
- Get rid of the unsigned int cast for the U8 and U16 variants.
- Add the BIT_TYPE() helper macro.
- Link to v4: https://lore.kernel.org/r/20250305-fixed-type-genmasks-v4-0-1873dcdf6723@wanadoo.fr
Changes from v3:
- Rebase on v6.14-rc5
- Fix a typo in GENMASK_U128() comment.
- Split the asm and non-asm definition of
- Replace ~0ULL by ~ULL(0)
- Since v3, __GENMASK() was moved to the uapi and people started
using directly. Introduce GENMASK_t() instead.
- Link to v3: https://lore.kernel.org/intel-xe/20240208074521.577076-1-lucas.demarchi@intel.com
Changes from v2:
- Document both in commit message and code about the strict type
checking and give examples how it´d break with invalid params.
- Link to v2: https://lore.kernel.org/intel-xe/20240124050205.3646390-1-lucas.demarchi@intel.com
Link to v1: https://lore.kernel.org/intel-xe/20230509051403.2748545-1-lucas.demarchi@intel.com
---
Lucas De Marchi (3):
bits: introduce fixed-type BIT_U*()
drm/i915: Convert REG_GENMASK*() to fixed-width GENMASK_U*()
test_bits: add tests for GENMASK_U*()
Vincent Mailhol (2):
bits: introduce fixed-type GENMASK_U*()
test_bits: add tests for BIT_U*()
drivers/gpu/drm/i915/i915_reg_defs.h | 108 ++++-------------------------------
include/linux/bitops.h | 1 -
include/linux/bits.h | 57 +++++++++++++++++-
lib/test_bits.c | 30 ++++++++++
4 files changed, 96 insertions(+), 100 deletions(-)
---
base-commit: e3f42c436d7e0cb432935fe3ae275dd8d9b60f71
change-id: 20250228-fixed-type-genmasks-8d1a555f34e8
Best regards,
--
Vincent Mailhol <mailhol.vincent@wanadoo.fr>
^ permalink raw reply [flat|nested] 26+ messages in thread* [PATCH v7 1/5] bits: introduce fixed-type GENMASK_U*()
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
@ 2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
-1 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol @ 2025-03-22 9:23 UTC (permalink / raw)
To: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton
Cc: linux-kernel, intel-gfx, dri-devel, Andi Shyti, David Laight,
Andy Shevchenko, Jani Nikula, Vincent Mailhol
Add GENMASK_TYPE() which generalizes __GENMASK() to support different
types, and implement fixed-types versions of GENMASK() based on it.
The fixed-type version allows more strict checks to the min/max values
accepted, which is useful for defining registers like implemented by
i915 and xe drivers with their REG_GENMASK*() macros.
The strict checks rely on shift-count-overflow compiler check to fail
the build if a number outside of the range allowed is passed.
Example:
#define FOO_MASK GENMASK_U32(33, 4)
will generate a warning like:
include/linux/bits.h:51:27: error: right shift count >= width of type [-Werror=shift-count-overflow]
51 | type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))
| ^~
While GENMASK_TYPE() is crafted to cover all variants, including the
already existing GENMASK(), GENMASK_ULL() and GENMASK_U128(), for the
moment, only use it for the newly introduced GENMASK_U*(). The
consolidation will be done in a separate change.
Co-developed-by: Yury Norov <yury.norov@gmail.com>
Signed-off-by: Yury Norov <yury.norov@gmail.com>
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
---
Changelog:
v6 -> v7:
- Fix grammar in comment: 'GENMASK_U*() depends' -> 'GENMASK_U*()
depend'.
- Fix typo in comment: 'Nethertheless' -> 'Nevertheless'
- Do an artificial early line wrap in comment so that the next
patch only has a one line diff change.
- Re-wrap the comments to the 80th column.
- The patch changed a lot since Yury first version: put myself as
main author and Yury as Co-developer.
- Add a new paragraph to the patch description to explain that
consolidation will be done later.
v5 -> v6:
- No changes.
v4 -> v5:
- Rename GENMASK_t() to GENMASK_TYPE().
- Fix typo in patch description.
- Use tab indentations instead of single space to separate the
macro name from its body.
- s/__GENMASK_U*()/GENMASK_U*()/g in the comment.
- Add a tag to credit myself as Co-developer. Keep Yury as the
main author.
- Modify GENMASK_TYPE() to match the changes made to __GENMASK()
in: https://github.com/norov/linux/commit/1e7933a575ed
- Replace (t)~_ULL(0) with type_max(t). This is OK because
GENMASK_TYPE() is not available in asm.
- linux/const.h and asm/bitsperlong.h are not used anymore. Remove
them.
- Apply GENMASK_TYPE() to GENMASK_U128().
- Remove the unsigned int cast for the U8 and U16 variants. Cast
to the target type instead. Do that cast directly in
GENMASK_TYPE().
v3 -> v4:
- The v3 is one year old. Meanwhile people started using
__GENMASK() directly. So instead of generalizing __GENMASK() to
support different types, add a new GENMASK_t().
- replace ~0ULL by ~_ULL(0). Otherwise, GENMASK_t() would fail in
asm code.
- Make GENMASK_U8() and GENMASK_U16() return an unsigned int. In
v3, due to the integer promotion rules, these were returning a
signed integer. By casting these to unsigned int, at least the
signedness is kept.
---
include/linux/bitops.h | 1 -
include/linux/bits.h | 39 +++++++++++++++++++++++++++++++++++++--
2 files changed, 37 insertions(+), 3 deletions(-)
diff --git a/include/linux/bitops.h b/include/linux/bitops.h
index c1cb53cf2f0f8662ed3e324578f74330e63f935d..9be2d50da09a417966b3d11c84092bb2f4cd0bef 100644
--- a/include/linux/bitops.h
+++ b/include/linux/bitops.h
@@ -8,7 +8,6 @@
#include <uapi/linux/kernel.h>
-#define BITS_PER_TYPE(type) (sizeof(type) * BITS_PER_BYTE)
#define BITS_TO_LONGS(nr) __KERNEL_DIV_ROUND_UP(nr, BITS_PER_TYPE(long))
#define BITS_TO_U64(nr) __KERNEL_DIV_ROUND_UP(nr, BITS_PER_TYPE(u64))
#define BITS_TO_U32(nr) __KERNEL_DIV_ROUND_UP(nr, BITS_PER_TYPE(u32))
diff --git a/include/linux/bits.h b/include/linux/bits.h
index 14fd0ca9a6cd17339dd2f69e449558312a8a001b..beb3ee2f1bc74a9346dd72eb06c722a9bc536051 100644
--- a/include/linux/bits.h
+++ b/include/linux/bits.h
@@ -12,6 +12,7 @@
#define BIT_ULL_MASK(nr) (ULL(1) << ((nr) % BITS_PER_LONG_LONG))
#define BIT_ULL_WORD(nr) ((nr) / BITS_PER_LONG_LONG)
#define BITS_PER_BYTE 8
+#define BITS_PER_TYPE(type) (sizeof(type) * BITS_PER_BYTE)
/*
* Create a contiguous bitmask starting at bit position @l and ending at
@@ -19,16 +20,50 @@
* GENMASK_ULL(39, 21) gives us the 64bit vector 0x000000ffffe00000.
*/
#if !defined(__ASSEMBLY__)
+
+/*
+ * Missing asm support
+ *
+ * GENMASK_U*() depend on BITS_PER_TYPE() which relies on sizeof(),
+ * something not available in asm. Nevertheless, fixed width integers is a C
+ * concept. Assembly code can rely on the long and long long versions instead.
+ */
+
#include <linux/build_bug.h>
#include <linux/compiler.h>
+#include <linux/overflow.h>
+
#define GENMASK_INPUT_CHECK(h, l) BUILD_BUG_ON_ZERO(const_true((l) > (h)))
-#else
+
+/*
+ * Generate a mask for the specified type @t. Additional checks are made to
+ * guarantee the value returned fits in that type, relying on
+ * shift-count-overflow compiler check to detect incompatible arguments.
+ * For example, all these create build errors or warnings:
+ *
+ * - GENMASK(15, 20): wrong argument order
+ * - GENMASK(72, 15): doesn't fit unsigned long
+ * - GENMASK_U32(33, 15): doesn't fit in a u32
+ */
+#define GENMASK_TYPE(t, h, l) \
+ ((t)(GENMASK_INPUT_CHECK(h, l) + \
+ (type_max(t) << (l) & \
+ type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))
+
+#define GENMASK_U8(h, l) GENMASK_TYPE(u8, h, l)
+#define GENMASK_U16(h, l) GENMASK_TYPE(u16, h, l)
+#define GENMASK_U32(h, l) GENMASK_TYPE(u32, h, l)
+#define GENMASK_U64(h, l) GENMASK_TYPE(u64, h, l)
+
+#else /* defined(__ASSEMBLY__) */
+
/*
* BUILD_BUG_ON_ZERO is not available in h files included from asm files,
* disable the input check if that is the case.
*/
#define GENMASK_INPUT_CHECK(h, l) 0
-#endif
+
+#endif /* !defined(__ASSEMBLY__) */
#define GENMASK(h, l) \
(GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
--
2.48.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v7 1/5] bits: introduce fixed-type GENMASK_U*()
@ 2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
0 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol via B4 Relay @ 2025-03-22 9:23 UTC (permalink / raw)
To: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton
Cc: linux-kernel, intel-gfx, dri-devel, Andi Shyti, David Laight,
Andy Shevchenko, Jani Nikula, Vincent Mailhol
From: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
Add GENMASK_TYPE() which generalizes __GENMASK() to support different
types, and implement fixed-types versions of GENMASK() based on it.
The fixed-type version allows more strict checks to the min/max values
accepted, which is useful for defining registers like implemented by
i915 and xe drivers with their REG_GENMASK*() macros.
The strict checks rely on shift-count-overflow compiler check to fail
the build if a number outside of the range allowed is passed.
Example:
#define FOO_MASK GENMASK_U32(33, 4)
will generate a warning like:
include/linux/bits.h:51:27: error: right shift count >= width of type [-Werror=shift-count-overflow]
51 | type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))
| ^~
While GENMASK_TYPE() is crafted to cover all variants, including the
already existing GENMASK(), GENMASK_ULL() and GENMASK_U128(), for the
moment, only use it for the newly introduced GENMASK_U*(). The
consolidation will be done in a separate change.
Co-developed-by: Yury Norov <yury.norov@gmail.com>
Signed-off-by: Yury Norov <yury.norov@gmail.com>
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
---
Changelog:
v6 -> v7:
- Fix grammar in comment: 'GENMASK_U*() depends' -> 'GENMASK_U*()
depend'.
- Fix typo in comment: 'Nethertheless' -> 'Nevertheless'
- Do an artificial early line wrap in comment so that the next
patch only has a one line diff change.
- Re-wrap the comments to the 80th column.
- The patch changed a lot since Yury first version: put myself as
main author and Yury as Co-developer.
- Add a new paragraph to the patch description to explain that
consolidation will be done later.
v5 -> v6:
- No changes.
v4 -> v5:
- Rename GENMASK_t() to GENMASK_TYPE().
- Fix typo in patch description.
- Use tab indentations instead of single space to separate the
macro name from its body.
- s/__GENMASK_U*()/GENMASK_U*()/g in the comment.
- Add a tag to credit myself as Co-developer. Keep Yury as the
main author.
- Modify GENMASK_TYPE() to match the changes made to __GENMASK()
in: https://github.com/norov/linux/commit/1e7933a575ed
- Replace (t)~_ULL(0) with type_max(t). This is OK because
GENMASK_TYPE() is not available in asm.
- linux/const.h and asm/bitsperlong.h are not used anymore. Remove
them.
- Apply GENMASK_TYPE() to GENMASK_U128().
- Remove the unsigned int cast for the U8 and U16 variants. Cast
to the target type instead. Do that cast directly in
GENMASK_TYPE().
v3 -> v4:
- The v3 is one year old. Meanwhile people started using
__GENMASK() directly. So instead of generalizing __GENMASK() to
support different types, add a new GENMASK_t().
- replace ~0ULL by ~_ULL(0). Otherwise, GENMASK_t() would fail in
asm code.
- Make GENMASK_U8() and GENMASK_U16() return an unsigned int. In
v3, due to the integer promotion rules, these were returning a
signed integer. By casting these to unsigned int, at least the
signedness is kept.
---
include/linux/bitops.h | 1 -
include/linux/bits.h | 39 +++++++++++++++++++++++++++++++++++++--
2 files changed, 37 insertions(+), 3 deletions(-)
diff --git a/include/linux/bitops.h b/include/linux/bitops.h
index c1cb53cf2f0f8662ed3e324578f74330e63f935d..9be2d50da09a417966b3d11c84092bb2f4cd0bef 100644
--- a/include/linux/bitops.h
+++ b/include/linux/bitops.h
@@ -8,7 +8,6 @@
#include <uapi/linux/kernel.h>
-#define BITS_PER_TYPE(type) (sizeof(type) * BITS_PER_BYTE)
#define BITS_TO_LONGS(nr) __KERNEL_DIV_ROUND_UP(nr, BITS_PER_TYPE(long))
#define BITS_TO_U64(nr) __KERNEL_DIV_ROUND_UP(nr, BITS_PER_TYPE(u64))
#define BITS_TO_U32(nr) __KERNEL_DIV_ROUND_UP(nr, BITS_PER_TYPE(u32))
diff --git a/include/linux/bits.h b/include/linux/bits.h
index 14fd0ca9a6cd17339dd2f69e449558312a8a001b..beb3ee2f1bc74a9346dd72eb06c722a9bc536051 100644
--- a/include/linux/bits.h
+++ b/include/linux/bits.h
@@ -12,6 +12,7 @@
#define BIT_ULL_MASK(nr) (ULL(1) << ((nr) % BITS_PER_LONG_LONG))
#define BIT_ULL_WORD(nr) ((nr) / BITS_PER_LONG_LONG)
#define BITS_PER_BYTE 8
+#define BITS_PER_TYPE(type) (sizeof(type) * BITS_PER_BYTE)
/*
* Create a contiguous bitmask starting at bit position @l and ending at
@@ -19,16 +20,50 @@
* GENMASK_ULL(39, 21) gives us the 64bit vector 0x000000ffffe00000.
*/
#if !defined(__ASSEMBLY__)
+
+/*
+ * Missing asm support
+ *
+ * GENMASK_U*() depend on BITS_PER_TYPE() which relies on sizeof(),
+ * something not available in asm. Nevertheless, fixed width integers is a C
+ * concept. Assembly code can rely on the long and long long versions instead.
+ */
+
#include <linux/build_bug.h>
#include <linux/compiler.h>
+#include <linux/overflow.h>
+
#define GENMASK_INPUT_CHECK(h, l) BUILD_BUG_ON_ZERO(const_true((l) > (h)))
-#else
+
+/*
+ * Generate a mask for the specified type @t. Additional checks are made to
+ * guarantee the value returned fits in that type, relying on
+ * shift-count-overflow compiler check to detect incompatible arguments.
+ * For example, all these create build errors or warnings:
+ *
+ * - GENMASK(15, 20): wrong argument order
+ * - GENMASK(72, 15): doesn't fit unsigned long
+ * - GENMASK_U32(33, 15): doesn't fit in a u32
+ */
+#define GENMASK_TYPE(t, h, l) \
+ ((t)(GENMASK_INPUT_CHECK(h, l) + \
+ (type_max(t) << (l) & \
+ type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))
+
+#define GENMASK_U8(h, l) GENMASK_TYPE(u8, h, l)
+#define GENMASK_U16(h, l) GENMASK_TYPE(u16, h, l)
+#define GENMASK_U32(h, l) GENMASK_TYPE(u32, h, l)
+#define GENMASK_U64(h, l) GENMASK_TYPE(u64, h, l)
+
+#else /* defined(__ASSEMBLY__) */
+
/*
* BUILD_BUG_ON_ZERO is not available in h files included from asm files,
* disable the input check if that is the case.
*/
#define GENMASK_INPUT_CHECK(h, l) 0
-#endif
+
+#endif /* !defined(__ASSEMBLY__) */
#define GENMASK(h, l) \
(GENMASK_INPUT_CHECK(h, l) + __GENMASK(h, l))
--
2.48.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH v7 1/5] bits: introduce fixed-type GENMASK_U*()
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
(?)
@ 2025-03-24 7:22 ` Andy Shevchenko
2025-03-24 7:44 ` Vincent Mailhol
-1 siblings, 1 reply; 26+ messages in thread
From: Andy Shevchenko @ 2025-03-24 7:22 UTC (permalink / raw)
To: mailhol.vincent
Cc: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton, linux-kernel, intel-gfx, dri-devel,
Andi Shyti, David Laight, Jani Nikula
On Sat, Mar 22, 2025 at 06:23:12PM +0900, Vincent Mailhol via B4 Relay wrote:
>
> Add GENMASK_TYPE() which generalizes __GENMASK() to support different
> types, and implement fixed-types versions of GENMASK() based on it.
> The fixed-type version allows more strict checks to the min/max values
> accepted, which is useful for defining registers like implemented by
> i915 and xe drivers with their REG_GENMASK*() macros.
>
> The strict checks rely on shift-count-overflow compiler check to fail
> the build if a number outside of the range allowed is passed.
> Example:
>
> #define FOO_MASK GENMASK_U32(33, 4)
>
> will generate a warning like:
>
> include/linux/bits.h:51:27: error: right shift count >= width of type [-Werror=shift-count-overflow]
> 51 | type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))
> | ^~
>
> While GENMASK_TYPE() is crafted to cover all variants, including the
> already existing GENMASK(), GENMASK_ULL() and GENMASK_U128(), for the
> moment, only use it for the newly introduced GENMASK_U*(). The
> consolidation will be done in a separate change.
...
> #if !defined(__ASSEMBLY__)
> +
> -#else
> +#else /* defined(__ASSEMBLY__) */
> -#endif
> +
> +#endif /* !defined(__ASSEMBLY__) */
Up to you, but if new version is needed or maintainer require, I would move the
above changes either to a separate patch (prerequisite) or dropped them at all.
These are not big but unneeded churn,
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v7 1/5] bits: introduce fixed-type GENMASK_U*()
2025-03-24 7:22 ` Andy Shevchenko
@ 2025-03-24 7:44 ` Vincent Mailhol
0 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol @ 2025-03-24 7:44 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton, linux-kernel, intel-gfx, dri-devel,
Andi Shyti, David Laight, Jani Nikula
On 24/03/2025 at 16:22, Andy Shevchenko wrote:
> On Sat, Mar 22, 2025 at 06:23:12PM +0900, Vincent Mailhol via B4 Relay wrote:
>>
>> Add GENMASK_TYPE() which generalizes __GENMASK() to support different
>> types, and implement fixed-types versions of GENMASK() based on it.
>> The fixed-type version allows more strict checks to the min/max values
>> accepted, which is useful for defining registers like implemented by
>> i915 and xe drivers with their REG_GENMASK*() macros.
>>
>> The strict checks rely on shift-count-overflow compiler check to fail
>> the build if a number outside of the range allowed is passed.
>> Example:
>>
>> #define FOO_MASK GENMASK_U32(33, 4)
>>
>> will generate a warning like:
>>
>> include/linux/bits.h:51:27: error: right shift count >= width of type [-Werror=shift-count-overflow]
>> 51 | type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))
>> | ^~
>>
>> While GENMASK_TYPE() is crafted to cover all variants, including the
>> already existing GENMASK(), GENMASK_ULL() and GENMASK_U128(), for the
>> moment, only use it for the newly introduced GENMASK_U*(). The
>> consolidation will be done in a separate change.
>
> ...
>
>> #if !defined(__ASSEMBLY__)
>> +
>
>> -#else
>
>> +#else /* defined(__ASSEMBLY__) */
>
>> -#endif
>> +
>> +#endif /* !defined(__ASSEMBLY__) */
>
> Up to you, but if new version is needed or maintainer require, I would move the
> above changes either to a separate patch (prerequisite) or dropped them at all.
> These are not big but unneeded churn,
I do not want to drop this. After all the changes, there is a lot of
scrolling between the #if, #else and #endif, and the comments helps to
keep track of which context we are in.
As for putting this into another patch, OK but only if there is a need
for new version for other reasons.
Yours sincerely,
Vincent Mailhol
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v7 2/5] bits: introduce fixed-type BIT_U*()
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
@ 2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
-1 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol @ 2025-03-22 9:23 UTC (permalink / raw)
To: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton
Cc: linux-kernel, intel-gfx, dri-devel, Andi Shyti, David Laight,
Andy Shevchenko, Jani Nikula, Vincent Mailhol
From: Lucas De Marchi <lucas.demarchi@intel.com>
Implement fixed-type BIT_U*() to help drivers add stricter checks,
like it was done for GENMASK_U*().
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Co-developed-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
---
Changelog:
v6 -> v7:
v5 -> v6:
- No changes.
v4 -> v5:
- Rename GENMASK_t() to GENMASK_TYPE().
- Use tab indentations instead of single space to separate the
macro name from its body.
- Add a global comment at the beginning of the file to explain why
GENMASK_U*() and BIT_U*() are not available in asm.
- Add a new BIT_TYPE() helper function, similar to GENMASK_TYPE().
- Remove the unsigned int cast for the U8 and U16 variants. Move
the cast to BIT_TYPE().
- Rename the argument from BIT_U*(b) to BIT_U*(nr) for consistency
with vdso/bits.h.
v3 -> v4:
- Use const_true() to simplify BIT_INPUT_CHECK().
- Make BIT_U8() and BIT_U16() return an unsigned int instead of a
u8 and u16. Because of the integer promotion rules in C, an u8
or an u16 would become a signed integer as soon as these are
used in any expression. By casting these to unsigned ints, at
least the signedness is kept.
- Put the cast next to the BIT() macro.
- In BIT_U64(): use BIT_ULL() instead of BIT().
---
include/linux/bits.h | 20 +++++++++++++++++++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff --git a/include/linux/bits.h b/include/linux/bits.h
index beb3ee2f1bc74a9346dd72eb06c722a9bc536051..6a942ea9ab380d3bd0e521916caa1d59db8031c0 100644
--- a/include/linux/bits.h
+++ b/include/linux/bits.h
@@ -24,7 +24,7 @@
/*
* Missing asm support
*
- * GENMASK_U*() depend on BITS_PER_TYPE() which relies on sizeof(),
+ * GENMASK_U*() and BIT_U*() depend on BITS_PER_TYPE() which relies on sizeof(),
* something not available in asm. Nevertheless, fixed width integers is a C
* concept. Assembly code can rely on the long and long long versions instead.
*/
@@ -55,6 +55,24 @@
#define GENMASK_U32(h, l) GENMASK_TYPE(u32, h, l)
#define GENMASK_U64(h, l) GENMASK_TYPE(u64, h, l)
+/*
+ * Fixed-type variants of BIT(), with additional checks like GENMASK_TYPE(). The
+ * following examples generate compiler warnings due to shift-count-overflow:
+ *
+ * - BIT_U8(8)
+ * - BIT_U32(-1)
+ * - BIT_U32(40)
+ */
+#define BIT_INPUT_CHECK(type, nr) \
+ BUILD_BUG_ON_ZERO(const_true((nr) >= BITS_PER_TYPE(type)))
+
+#define BIT_TYPE(type, nr) ((type)(BIT_INPUT_CHECK(type, nr) + BIT_ULL(nr)))
+
+#define BIT_U8(nr) BIT_TYPE(u8, nr)
+#define BIT_U16(nr) BIT_TYPE(u16, nr)
+#define BIT_U32(nr) BIT_TYPE(u32, nr)
+#define BIT_U64(nr) BIT_TYPE(u64, nr)
+
#else /* defined(__ASSEMBLY__) */
/*
--
2.48.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v7 2/5] bits: introduce fixed-type BIT_U*()
@ 2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
0 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol via B4 Relay @ 2025-03-22 9:23 UTC (permalink / raw)
To: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton
Cc: linux-kernel, intel-gfx, dri-devel, Andi Shyti, David Laight,
Andy Shevchenko, Jani Nikula, Vincent Mailhol
From: Lucas De Marchi <lucas.demarchi@intel.com>
Implement fixed-type BIT_U*() to help drivers add stricter checks,
like it was done for GENMASK_U*().
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Co-developed-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
---
Changelog:
v6 -> v7:
v5 -> v6:
- No changes.
v4 -> v5:
- Rename GENMASK_t() to GENMASK_TYPE().
- Use tab indentations instead of single space to separate the
macro name from its body.
- Add a global comment at the beginning of the file to explain why
GENMASK_U*() and BIT_U*() are not available in asm.
- Add a new BIT_TYPE() helper function, similar to GENMASK_TYPE().
- Remove the unsigned int cast for the U8 and U16 variants. Move
the cast to BIT_TYPE().
- Rename the argument from BIT_U*(b) to BIT_U*(nr) for consistency
with vdso/bits.h.
v3 -> v4:
- Use const_true() to simplify BIT_INPUT_CHECK().
- Make BIT_U8() and BIT_U16() return an unsigned int instead of a
u8 and u16. Because of the integer promotion rules in C, an u8
or an u16 would become a signed integer as soon as these are
used in any expression. By casting these to unsigned ints, at
least the signedness is kept.
- Put the cast next to the BIT() macro.
- In BIT_U64(): use BIT_ULL() instead of BIT().
---
include/linux/bits.h | 20 +++++++++++++++++++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff --git a/include/linux/bits.h b/include/linux/bits.h
index beb3ee2f1bc74a9346dd72eb06c722a9bc536051..6a942ea9ab380d3bd0e521916caa1d59db8031c0 100644
--- a/include/linux/bits.h
+++ b/include/linux/bits.h
@@ -24,7 +24,7 @@
/*
* Missing asm support
*
- * GENMASK_U*() depend on BITS_PER_TYPE() which relies on sizeof(),
+ * GENMASK_U*() and BIT_U*() depend on BITS_PER_TYPE() which relies on sizeof(),
* something not available in asm. Nevertheless, fixed width integers is a C
* concept. Assembly code can rely on the long and long long versions instead.
*/
@@ -55,6 +55,24 @@
#define GENMASK_U32(h, l) GENMASK_TYPE(u32, h, l)
#define GENMASK_U64(h, l) GENMASK_TYPE(u64, h, l)
+/*
+ * Fixed-type variants of BIT(), with additional checks like GENMASK_TYPE(). The
+ * following examples generate compiler warnings due to shift-count-overflow:
+ *
+ * - BIT_U8(8)
+ * - BIT_U32(-1)
+ * - BIT_U32(40)
+ */
+#define BIT_INPUT_CHECK(type, nr) \
+ BUILD_BUG_ON_ZERO(const_true((nr) >= BITS_PER_TYPE(type)))
+
+#define BIT_TYPE(type, nr) ((type)(BIT_INPUT_CHECK(type, nr) + BIT_ULL(nr)))
+
+#define BIT_U8(nr) BIT_TYPE(u8, nr)
+#define BIT_U16(nr) BIT_TYPE(u16, nr)
+#define BIT_U32(nr) BIT_TYPE(u32, nr)
+#define BIT_U64(nr) BIT_TYPE(u64, nr)
+
#else /* defined(__ASSEMBLY__) */
/*
--
2.48.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* Re: [PATCH v7 2/5] bits: introduce fixed-type BIT_U*()
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
(?)
@ 2025-03-24 13:41 ` Andy Shevchenko
2025-03-24 14:16 ` Vincent Mailhol
-1 siblings, 1 reply; 26+ messages in thread
From: Andy Shevchenko @ 2025-03-24 13:41 UTC (permalink / raw)
To: mailhol.vincent
Cc: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton, linux-kernel, intel-gfx, dri-devel,
Andi Shyti, David Laight, Jani Nikula
On Sat, Mar 22, 2025 at 06:23:13PM +0900, Vincent Mailhol via B4 Relay wrote:
> From: Lucas De Marchi <lucas.demarchi@intel.com>
>
> Implement fixed-type BIT_U*() to help drivers add stricter checks,
> like it was done for GENMASK_U*().
...
> +/*
> + * Fixed-type variants of BIT(), with additional checks like GENMASK_TYPE(). The
> + * following examples generate compiler warnings due to shift-count-overflow:
"...due to -Wshift-count-overflow:" ?
Same idea — if you need a new version, since it's just a nit-pick.
> + *
> + * - BIT_U8(8)
> + * - BIT_U32(-1)
> + * - BIT_U32(40)
> + */
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v7 2/5] bits: introduce fixed-type BIT_U*()
2025-03-24 13:41 ` Andy Shevchenko
@ 2025-03-24 14:16 ` Vincent Mailhol
2025-03-24 14:32 ` Andy Shevchenko
0 siblings, 1 reply; 26+ messages in thread
From: Vincent Mailhol @ 2025-03-24 14:16 UTC (permalink / raw)
To: Andy Shevchenko
Cc: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton, linux-kernel, intel-gfx, dri-devel,
Andi Shyti, David Laight, Jani Nikula
On 24/03/2025 at 22:41, Andy Shevchenko wrote:
> On Sat, Mar 22, 2025 at 06:23:13PM +0900, Vincent Mailhol via B4 Relay wrote:
>> From: Lucas De Marchi <lucas.demarchi@intel.com>
>>
>> Implement fixed-type BIT_U*() to help drivers add stricter checks,
>> like it was done for GENMASK_U*().
>
> ...
>
>> +/*
>> + * Fixed-type variants of BIT(), with additional checks like GENMASK_TYPE(). The
>> + * following examples generate compiler warnings due to shift-count-overflow:
>
> "...due to -Wshift-count-overflow:" ?
>
> Same idea — if you need a new version, since it's just a nit-pick.
If you want. I staged this change locally, so if there is a v8, it will
be addressed. I applied the same to the previous patch which also
mentioned shift-count-overflow without the -W prefix.
But honestly, I am not convinced of the added value. This is from Lucas
original patch one year ago, and no one was bothered by this. IMHO, when
writing:
(...) generate compiler warnings due to shift-count-overflow:
I do not see where the ambiguity is. The sentence clearly say that this
is a compiler warning, so with or without the -W prefix, the sentence is
equally understandable.
Yours sincerely,
Vincent Mailhol
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v7 2/5] bits: introduce fixed-type BIT_U*()
2025-03-24 14:16 ` Vincent Mailhol
@ 2025-03-24 14:32 ` Andy Shevchenko
0 siblings, 0 replies; 26+ messages in thread
From: Andy Shevchenko @ 2025-03-24 14:32 UTC (permalink / raw)
To: Vincent Mailhol
Cc: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton, linux-kernel, intel-gfx, dri-devel,
Andi Shyti, David Laight, Jani Nikula
On Mon, Mar 24, 2025 at 11:16:30PM +0900, Vincent Mailhol wrote:
> On 24/03/2025 at 22:41, Andy Shevchenko wrote:
> > On Sat, Mar 22, 2025 at 06:23:13PM +0900, Vincent Mailhol via B4 Relay wrote:
...
> >> +/*
> >> + * Fixed-type variants of BIT(), with additional checks like GENMASK_TYPE(). The
> >> + * following examples generate compiler warnings due to shift-count-overflow:
> >
> > "...due to -Wshift-count-overflow:" ?
> >
> > Same idea — if you need a new version, since it's just a nit-pick.
>
> If you want. I staged this change locally, so if there is a v8, it will
> be addressed. I applied the same to the previous patch which also
> mentioned shift-count-overflow without the -W prefix.
>
> But honestly, I am not convinced of the added value. This is from Lucas
> original patch one year ago, and no one was bothered by this. IMHO, when
> writing:
>
> (...) generate compiler warnings due to shift-count-overflow:
>
> I do not see where the ambiguity is. The sentence clearly say that this
> is a compiler warning, so with or without the -W prefix, the sentence is
> equally understandable.
As I marked, it's a nit-pick, but from my point of view the added value
is immediate: The reader can be sure that we are talking about a compiler
warning and not something else (C standard? some special term?). So it adds
more context and makes it clearer.
--
With Best Regards,
Andy Shevchenko
^ permalink raw reply [flat|nested] 26+ messages in thread
* [PATCH v7 3/5] drm/i915: Convert REG_GENMASK*() to fixed-width GENMASK_U*()
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
@ 2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
-1 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol @ 2025-03-22 9:23 UTC (permalink / raw)
To: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton
Cc: linux-kernel, intel-gfx, dri-devel, Andi Shyti, David Laight,
Andy Shevchenko, Jani Nikula, Vincent Mailhol
From: Lucas De Marchi <lucas.demarchi@intel.com>
Now that include/linux/bits.h implements fixed-width GENMASK_U*(), use
them to implement the i915/xe specific macros. Converting each driver
to use the generic macros are left for later, when/if other
driver-specific macros are also generalized.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
---
Changelog:
v6 -> v7:
- Replace BIT_* and GENMASK_* by BIT_U*() and GENMASK_U*() in the
description.
- Add the information in the description that BIT_U*() and
GENMASK_U*() are fixed width.
v5 -> v6:
- No changes.
v4 -> v5:
- Add brackets to macro names in patch description,
e.g. 'REG_GENMASK*' -> 'REG_GENMASK*()'
v3 -> v4:
- Remove the prefixes in macro parameters,
e.g. 'REG_GENMASK(__high, __low)' -> 'REG_GENMASK(high, low)'
---
drivers/gpu/drm/i915/i915_reg_defs.h | 108 ++++-------------------------------
1 file changed, 11 insertions(+), 97 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_reg_defs.h b/drivers/gpu/drm/i915/i915_reg_defs.h
index e251bcc0c89f5710125bc70f07851b2cb978c89c..e0bc9cf5fa9ed98f7d36f6cb667999aa6522c384 100644
--- a/drivers/gpu/drm/i915/i915_reg_defs.h
+++ b/drivers/gpu/drm/i915/i915_reg_defs.h
@@ -9,76 +9,19 @@
#include <linux/bitfield.h>
#include <linux/bits.h>
-/**
- * REG_BIT() - Prepare a u32 bit value
- * @__n: 0-based bit number
- *
- * Local wrapper for BIT() to force u32, with compile time checks.
- *
- * @return: Value with bit @__n set.
+/*
+ * Wrappers over the generic fixed width BIT_U*() and GENMASK_U*()
+ * implementations, for compatibility reasons with previous implementation.
*/
-#define REG_BIT(__n) \
- ((u32)(BIT(__n) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__n) && \
- ((__n) < 0 || (__n) > 31))))
+#define REG_GENMASK(high, low) GENMASK_U32(high, low)
+#define REG_GENMASK64(high, low) GENMASK_U64(high, low)
+#define REG_GENMASK16(high, low) GENMASK_U16(high, low)
+#define REG_GENMASK8(high, low) GENMASK_U8(high, low)
-/**
- * REG_BIT8() - Prepare a u8 bit value
- * @__n: 0-based bit number
- *
- * Local wrapper for BIT() to force u8, with compile time checks.
- *
- * @return: Value with bit @__n set.
- */
-#define REG_BIT8(__n) \
- ((u8)(BIT(__n) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__n) && \
- ((__n) < 0 || (__n) > 7))))
-
-/**
- * REG_GENMASK() - Prepare a continuous u32 bitmask
- * @__high: 0-based high bit
- * @__low: 0-based low bit
- *
- * Local wrapper for GENMASK() to force u32, with compile time checks.
- *
- * @return: Continuous bitmask from @__high to @__low, inclusive.
- */
-#define REG_GENMASK(__high, __low) \
- ((u32)(GENMASK(__high, __low) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__high) && \
- __is_constexpr(__low) && \
- ((__low) < 0 || (__high) > 31 || (__low) > (__high)))))
-
-/**
- * REG_GENMASK64() - Prepare a continuous u64 bitmask
- * @__high: 0-based high bit
- * @__low: 0-based low bit
- *
- * Local wrapper for GENMASK_ULL() to force u64, with compile time checks.
- *
- * @return: Continuous bitmask from @__high to @__low, inclusive.
- */
-#define REG_GENMASK64(__high, __low) \
- ((u64)(GENMASK_ULL(__high, __low) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__high) && \
- __is_constexpr(__low) && \
- ((__low) < 0 || (__high) > 63 || (__low) > (__high)))))
-
-/**
- * REG_GENMASK8() - Prepare a continuous u8 bitmask
- * @__high: 0-based high bit
- * @__low: 0-based low bit
- *
- * Local wrapper for GENMASK() to force u8, with compile time checks.
- *
- * @return: Continuous bitmask from @__high to @__low, inclusive.
- */
-#define REG_GENMASK8(__high, __low) \
- ((u8)(GENMASK(__high, __low) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__high) && \
- __is_constexpr(__low) && \
- ((__low) < 0 || (__high) > 7 || (__low) > (__high)))))
+#define REG_BIT(n) BIT_U32(n)
+#define REG_BIT64(n) BIT_U64(n)
+#define REG_BIT16(n) BIT_U16(n)
+#define REG_BIT8(n) BIT_U8(n)
/*
* Local integer constant expression version of is_power_of_2().
@@ -143,35 +86,6 @@
*/
#define REG_FIELD_GET64(__mask, __val) ((u64)FIELD_GET(__mask, __val))
-/**
- * REG_BIT16() - Prepare a u16 bit value
- * @__n: 0-based bit number
- *
- * Local wrapper for BIT() to force u16, with compile time
- * checks.
- *
- * @return: Value with bit @__n set.
- */
-#define REG_BIT16(__n) \
- ((u16)(BIT(__n) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__n) && \
- ((__n) < 0 || (__n) > 15))))
-
-/**
- * REG_GENMASK16() - Prepare a continuous u8 bitmask
- * @__high: 0-based high bit
- * @__low: 0-based low bit
- *
- * Local wrapper for GENMASK() to force u16, with compile time
- * checks.
- *
- * @return: Continuous bitmask from @__high to @__low, inclusive.
- */
-#define REG_GENMASK16(__high, __low) \
- ((u16)(GENMASK(__high, __low) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__high) && \
- __is_constexpr(__low) && \
- ((__low) < 0 || (__high) > 15 || (__low) > (__high)))))
/**
* REG_FIELD_PREP16() - Prepare a u16 bitfield value
--
2.48.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v7 3/5] drm/i915: Convert REG_GENMASK*() to fixed-width GENMASK_U*()
@ 2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
0 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol via B4 Relay @ 2025-03-22 9:23 UTC (permalink / raw)
To: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton
Cc: linux-kernel, intel-gfx, dri-devel, Andi Shyti, David Laight,
Andy Shevchenko, Jani Nikula, Vincent Mailhol
From: Lucas De Marchi <lucas.demarchi@intel.com>
Now that include/linux/bits.h implements fixed-width GENMASK_U*(), use
them to implement the i915/xe specific macros. Converting each driver
to use the generic macros are left for later, when/if other
driver-specific macros are also generalized.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Acked-by: Jani Nikula <jani.nikula@intel.com>
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
---
Changelog:
v6 -> v7:
- Replace BIT_* and GENMASK_* by BIT_U*() and GENMASK_U*() in the
description.
- Add the information in the description that BIT_U*() and
GENMASK_U*() are fixed width.
v5 -> v6:
- No changes.
v4 -> v5:
- Add brackets to macro names in patch description,
e.g. 'REG_GENMASK*' -> 'REG_GENMASK*()'
v3 -> v4:
- Remove the prefixes in macro parameters,
e.g. 'REG_GENMASK(__high, __low)' -> 'REG_GENMASK(high, low)'
---
drivers/gpu/drm/i915/i915_reg_defs.h | 108 ++++-------------------------------
1 file changed, 11 insertions(+), 97 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_reg_defs.h b/drivers/gpu/drm/i915/i915_reg_defs.h
index e251bcc0c89f5710125bc70f07851b2cb978c89c..e0bc9cf5fa9ed98f7d36f6cb667999aa6522c384 100644
--- a/drivers/gpu/drm/i915/i915_reg_defs.h
+++ b/drivers/gpu/drm/i915/i915_reg_defs.h
@@ -9,76 +9,19 @@
#include <linux/bitfield.h>
#include <linux/bits.h>
-/**
- * REG_BIT() - Prepare a u32 bit value
- * @__n: 0-based bit number
- *
- * Local wrapper for BIT() to force u32, with compile time checks.
- *
- * @return: Value with bit @__n set.
+/*
+ * Wrappers over the generic fixed width BIT_U*() and GENMASK_U*()
+ * implementations, for compatibility reasons with previous implementation.
*/
-#define REG_BIT(__n) \
- ((u32)(BIT(__n) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__n) && \
- ((__n) < 0 || (__n) > 31))))
+#define REG_GENMASK(high, low) GENMASK_U32(high, low)
+#define REG_GENMASK64(high, low) GENMASK_U64(high, low)
+#define REG_GENMASK16(high, low) GENMASK_U16(high, low)
+#define REG_GENMASK8(high, low) GENMASK_U8(high, low)
-/**
- * REG_BIT8() - Prepare a u8 bit value
- * @__n: 0-based bit number
- *
- * Local wrapper for BIT() to force u8, with compile time checks.
- *
- * @return: Value with bit @__n set.
- */
-#define REG_BIT8(__n) \
- ((u8)(BIT(__n) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__n) && \
- ((__n) < 0 || (__n) > 7))))
-
-/**
- * REG_GENMASK() - Prepare a continuous u32 bitmask
- * @__high: 0-based high bit
- * @__low: 0-based low bit
- *
- * Local wrapper for GENMASK() to force u32, with compile time checks.
- *
- * @return: Continuous bitmask from @__high to @__low, inclusive.
- */
-#define REG_GENMASK(__high, __low) \
- ((u32)(GENMASK(__high, __low) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__high) && \
- __is_constexpr(__low) && \
- ((__low) < 0 || (__high) > 31 || (__low) > (__high)))))
-
-/**
- * REG_GENMASK64() - Prepare a continuous u64 bitmask
- * @__high: 0-based high bit
- * @__low: 0-based low bit
- *
- * Local wrapper for GENMASK_ULL() to force u64, with compile time checks.
- *
- * @return: Continuous bitmask from @__high to @__low, inclusive.
- */
-#define REG_GENMASK64(__high, __low) \
- ((u64)(GENMASK_ULL(__high, __low) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__high) && \
- __is_constexpr(__low) && \
- ((__low) < 0 || (__high) > 63 || (__low) > (__high)))))
-
-/**
- * REG_GENMASK8() - Prepare a continuous u8 bitmask
- * @__high: 0-based high bit
- * @__low: 0-based low bit
- *
- * Local wrapper for GENMASK() to force u8, with compile time checks.
- *
- * @return: Continuous bitmask from @__high to @__low, inclusive.
- */
-#define REG_GENMASK8(__high, __low) \
- ((u8)(GENMASK(__high, __low) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__high) && \
- __is_constexpr(__low) && \
- ((__low) < 0 || (__high) > 7 || (__low) > (__high)))))
+#define REG_BIT(n) BIT_U32(n)
+#define REG_BIT64(n) BIT_U64(n)
+#define REG_BIT16(n) BIT_U16(n)
+#define REG_BIT8(n) BIT_U8(n)
/*
* Local integer constant expression version of is_power_of_2().
@@ -143,35 +86,6 @@
*/
#define REG_FIELD_GET64(__mask, __val) ((u64)FIELD_GET(__mask, __val))
-/**
- * REG_BIT16() - Prepare a u16 bit value
- * @__n: 0-based bit number
- *
- * Local wrapper for BIT() to force u16, with compile time
- * checks.
- *
- * @return: Value with bit @__n set.
- */
-#define REG_BIT16(__n) \
- ((u16)(BIT(__n) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__n) && \
- ((__n) < 0 || (__n) > 15))))
-
-/**
- * REG_GENMASK16() - Prepare a continuous u8 bitmask
- * @__high: 0-based high bit
- * @__low: 0-based low bit
- *
- * Local wrapper for GENMASK() to force u16, with compile time
- * checks.
- *
- * @return: Continuous bitmask from @__high to @__low, inclusive.
- */
-#define REG_GENMASK16(__high, __low) \
- ((u16)(GENMASK(__high, __low) + \
- BUILD_BUG_ON_ZERO(__is_constexpr(__high) && \
- __is_constexpr(__low) && \
- ((__low) < 0 || (__high) > 15 || (__low) > (__high)))))
/**
* REG_FIELD_PREP16() - Prepare a u16 bitfield value
--
2.48.1
^ permalink raw reply related [flat|nested] 26+ messages in thread
* [PATCH v7 4/5] test_bits: add tests for GENMASK_U*()
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
@ 2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
-1 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol @ 2025-03-22 9:23 UTC (permalink / raw)
To: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton
Cc: linux-kernel, intel-gfx, dri-devel, Andi Shyti, David Laight,
Andy Shevchenko, Vincent Mailhol
From: Lucas De Marchi <lucas.demarchi@intel.com>
Add some additional tests in lib/test_bits.c to cover the
expected/non-expected values of the fixed-type GENMASK_U*() macros.
Also check that the result value matches the expected type. Since
those are known at build time, use static_assert() instead of normal
kunit tests.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
---
Changelog:
v6 -> v7:
v5 -> v6:
- No changes.
v4 -> v5:
- Revert v4 change. GENMASK_U8()/GENMASK_U16() are now back to
u8/u16.
v3 -> v4:
- Adjust the type of GENMASK_U8()/GENMASK_U16() from u8/u16 to
unsigned int.
- Reorder the tests to match the order in which the macros are
declared in bits.h.
---
lib/test_bits.c | 20 ++++++++++++++++++++
1 file changed, 20 insertions(+)
diff --git a/lib/test_bits.c b/lib/test_bits.c
index c7b38d91e1f16d42b7ca92e62fbd6c19b37e76a0..f443476f3265c463c1219b13c1ef9663d238d58b 100644
--- a/lib/test_bits.c
+++ b/lib/test_bits.c
@@ -5,6 +5,16 @@
#include <kunit/test.h>
#include <linux/bits.h>
+#include <linux/types.h>
+
+#define assert_type(t, x) _Generic(x, t: x, default: 0)
+
+static_assert(assert_type(unsigned long, GENMASK(31, 0)) == U32_MAX);
+static_assert(assert_type(unsigned long long, GENMASK_ULL(63, 0)) == U64_MAX);
+static_assert(assert_type(u8, GENMASK_U8(7, 0)) == U8_MAX);
+static_assert(assert_type(u16, GENMASK_U16(15, 0)) == U16_MAX);
+static_assert(assert_type(u32, GENMASK_U32(31, 0)) == U32_MAX);
+static_assert(assert_type(u64, GENMASK_U64(63, 0)) == U64_MAX);
static void genmask_test(struct kunit *test)
@@ -14,11 +24,21 @@ static void genmask_test(struct kunit *test)
KUNIT_EXPECT_EQ(test, 6ul, GENMASK(2, 1));
KUNIT_EXPECT_EQ(test, 0xFFFFFFFFul, GENMASK(31, 0));
+ KUNIT_EXPECT_EQ(test, 1u, GENMASK_U8(0, 0));
+ KUNIT_EXPECT_EQ(test, 3u, GENMASK_U16(1, 0));
+ KUNIT_EXPECT_EQ(test, 0x10000, GENMASK_U32(16, 16));
+
#ifdef TEST_GENMASK_FAILURES
/* these should fail compilation */
GENMASK(0, 1);
GENMASK(0, 10);
GENMASK(9, 10);
+
+ GENMASK_U32(0, 31);
+ GENMASK_U64(64, 0);
+ GENMASK_U32(32, 0);
+ GENMASK_U16(16, 0);
+ GENMASK_U8(8, 0);
#endif
--
2.48.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v7 4/5] test_bits: add tests for GENMASK_U*()
@ 2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
0 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol via B4 Relay @ 2025-03-22 9:23 UTC (permalink / raw)
To: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton
Cc: linux-kernel, intel-gfx, dri-devel, Andi Shyti, David Laight,
Andy Shevchenko, Vincent Mailhol
From: Lucas De Marchi <lucas.demarchi@intel.com>
Add some additional tests in lib/test_bits.c to cover the
expected/non-expected values of the fixed-type GENMASK_U*() macros.
Also check that the result value matches the expected type. Since
those are known at build time, use static_assert() instead of normal
kunit tests.
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
---
Changelog:
v6 -> v7:
v5 -> v6:
- No changes.
v4 -> v5:
- Revert v4 change. GENMASK_U8()/GENMASK_U16() are now back to
u8/u16.
v3 -> v4:
- Adjust the type of GENMASK_U8()/GENMASK_U16() from u8/u16 to
unsigned int.
- Reorder the tests to match the order in which the macros are
declared in bits.h.
---
lib/test_bits.c | 20 ++++++++++++++++++++
1 file changed, 20 insertions(+)
diff --git a/lib/test_bits.c b/lib/test_bits.c
index c7b38d91e1f16d42b7ca92e62fbd6c19b37e76a0..f443476f3265c463c1219b13c1ef9663d238d58b 100644
--- a/lib/test_bits.c
+++ b/lib/test_bits.c
@@ -5,6 +5,16 @@
#include <kunit/test.h>
#include <linux/bits.h>
+#include <linux/types.h>
+
+#define assert_type(t, x) _Generic(x, t: x, default: 0)
+
+static_assert(assert_type(unsigned long, GENMASK(31, 0)) == U32_MAX);
+static_assert(assert_type(unsigned long long, GENMASK_ULL(63, 0)) == U64_MAX);
+static_assert(assert_type(u8, GENMASK_U8(7, 0)) == U8_MAX);
+static_assert(assert_type(u16, GENMASK_U16(15, 0)) == U16_MAX);
+static_assert(assert_type(u32, GENMASK_U32(31, 0)) == U32_MAX);
+static_assert(assert_type(u64, GENMASK_U64(63, 0)) == U64_MAX);
static void genmask_test(struct kunit *test)
@@ -14,11 +24,21 @@ static void genmask_test(struct kunit *test)
KUNIT_EXPECT_EQ(test, 6ul, GENMASK(2, 1));
KUNIT_EXPECT_EQ(test, 0xFFFFFFFFul, GENMASK(31, 0));
+ KUNIT_EXPECT_EQ(test, 1u, GENMASK_U8(0, 0));
+ KUNIT_EXPECT_EQ(test, 3u, GENMASK_U16(1, 0));
+ KUNIT_EXPECT_EQ(test, 0x10000, GENMASK_U32(16, 16));
+
#ifdef TEST_GENMASK_FAILURES
/* these should fail compilation */
GENMASK(0, 1);
GENMASK(0, 10);
GENMASK(9, 10);
+
+ GENMASK_U32(0, 31);
+ GENMASK_U64(64, 0);
+ GENMASK_U32(32, 0);
+ GENMASK_U16(16, 0);
+ GENMASK_U8(8, 0);
#endif
--
2.48.1
^ permalink raw reply related [flat|nested] 26+ messages in thread
* [PATCH v7 5/5] test_bits: add tests for BIT_U*()
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
@ 2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
-1 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol @ 2025-03-22 9:23 UTC (permalink / raw)
To: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton
Cc: linux-kernel, intel-gfx, dri-devel, Andi Shyti, David Laight,
Andy Shevchenko, Vincent Mailhol
Add some additional tests in lib/test_bits.c to cover the expected
results of the fixed type BIT_U*() macros.
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
Changelog:
v6 -> v7:
- Add Lucas's Reviewed-by tag.
v5 -> v6:
- No changes.
v4 -> v5:
- BIT_U8()/BIT_U16() are now back to u8/u16.
v3 -> v4:
- New patch.
---
lib/test_bits.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/lib/test_bits.c b/lib/test_bits.c
index f443476f3265c463c1219b13c1ef9663d238d58b..47325b41515fde2c3ed434ed6f4094925c98886b 100644
--- a/lib/test_bits.c
+++ b/lib/test_bits.c
@@ -9,6 +9,16 @@
#define assert_type(t, x) _Generic(x, t: x, default: 0)
+static_assert(assert_type(u8, BIT_U8(0)) == 1u);
+static_assert(assert_type(u16, BIT_U16(0)) == 1u);
+static_assert(assert_type(u32, BIT_U32(0)) == 1u);
+static_assert(assert_type(u64, BIT_U64(0)) == 1ull);
+
+static_assert(assert_type(u8, BIT_U8(7)) == 0x80u);
+static_assert(assert_type(u16, BIT_U16(15)) == 0x8000u);
+static_assert(assert_type(u32, BIT_U32(31)) == 0x80000000u);
+static_assert(assert_type(u64, BIT_U64(63)) == 0x8000000000000000ull);
+
static_assert(assert_type(unsigned long, GENMASK(31, 0)) == U32_MAX);
static_assert(assert_type(unsigned long long, GENMASK_ULL(63, 0)) == U64_MAX);
static_assert(assert_type(u8, GENMASK_U8(7, 0)) == U8_MAX);
--
2.48.1
^ permalink raw reply related [flat|nested] 26+ messages in thread* [PATCH v7 5/5] test_bits: add tests for BIT_U*()
@ 2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
0 siblings, 0 replies; 26+ messages in thread
From: Vincent Mailhol via B4 Relay @ 2025-03-22 9:23 UTC (permalink / raw)
To: Yury Norov, Lucas De Marchi, Rasmus Villemoes, Jani Nikula,
Joonas Lahtinen, Rodrigo Vivi, Tvrtko Ursulin, David Airlie,
Simona Vetter, Andrew Morton
Cc: linux-kernel, intel-gfx, dri-devel, Andi Shyti, David Laight,
Andy Shevchenko, Vincent Mailhol
From: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
Add some additional tests in lib/test_bits.c to cover the expected
results of the fixed type BIT_U*() macros.
Signed-off-by: Vincent Mailhol <mailhol.vincent@wanadoo.fr>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
---
Changelog:
v6 -> v7:
- Add Lucas's Reviewed-by tag.
v5 -> v6:
- No changes.
v4 -> v5:
- BIT_U8()/BIT_U16() are now back to u8/u16.
v3 -> v4:
- New patch.
---
lib/test_bits.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/lib/test_bits.c b/lib/test_bits.c
index f443476f3265c463c1219b13c1ef9663d238d58b..47325b41515fde2c3ed434ed6f4094925c98886b 100644
--- a/lib/test_bits.c
+++ b/lib/test_bits.c
@@ -9,6 +9,16 @@
#define assert_type(t, x) _Generic(x, t: x, default: 0)
+static_assert(assert_type(u8, BIT_U8(0)) == 1u);
+static_assert(assert_type(u16, BIT_U16(0)) == 1u);
+static_assert(assert_type(u32, BIT_U32(0)) == 1u);
+static_assert(assert_type(u64, BIT_U64(0)) == 1ull);
+
+static_assert(assert_type(u8, BIT_U8(7)) == 0x80u);
+static_assert(assert_type(u16, BIT_U16(15)) == 0x8000u);
+static_assert(assert_type(u32, BIT_U32(31)) == 0x80000000u);
+static_assert(assert_type(u64, BIT_U64(63)) == 0x8000000000000000ull);
+
static_assert(assert_type(unsigned long, GENMASK(31, 0)) == U32_MAX);
static_assert(assert_type(unsigned long long, GENMASK_ULL(63, 0)) == U64_MAX);
static_assert(assert_type(u8, GENMASK_U8(7, 0)) == U8_MAX);
--
2.48.1
^ permalink raw reply related [flat|nested] 26+ messages in thread
* ✗ Fi.CI.CHECKPATCH: warning for bits: Fixed-type GENMASK_U*() and BIT_U*() (rev2)
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
` (5 preceding siblings ...)
(?)
@ 2025-03-22 10:07 ` Patchwork
-1 siblings, 0 replies; 26+ messages in thread
From: Patchwork @ 2025-03-22 10:07 UTC (permalink / raw)
To: Vincent Mailhol via B4 Relay; +Cc: intel-gfx
== Series Details ==
Series: bits: Fixed-type GENMASK_U*() and BIT_U*() (rev2)
URL : https://patchwork.freedesktop.org/series/146088/
State : warning
== Summary ==
Error: dim checkpatch failed
66702fd9fb3b bits: introduce fixed-type GENMASK_U*()
-:89: CHECK:MACRO_ARG_REUSE: Macro argument reuse 't' - possible side-effects?
#89: FILE: include/linux/bits.h:48:
+#define GENMASK_TYPE(t, h, l) \
+ ((t)(GENMASK_INPUT_CHECK(h, l) + \
+ (type_max(t) << (l) & \
+ type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))
-:89: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'h' - possible side-effects?
#89: FILE: include/linux/bits.h:48:
+#define GENMASK_TYPE(t, h, l) \
+ ((t)(GENMASK_INPUT_CHECK(h, l) + \
+ (type_max(t) << (l) & \
+ type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))
-:89: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'l' - possible side-effects?
#89: FILE: include/linux/bits.h:48:
+#define GENMASK_TYPE(t, h, l) \
+ ((t)(GENMASK_INPUT_CHECK(h, l) + \
+ (type_max(t) << (l) & \
+ type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))
total: 0 errors, 0 warnings, 3 checks, 66 lines checked
78a9e8e9feff bits: introduce fixed-type BIT_U*()
-:42: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'nr' - possible side-effects?
#42: FILE: include/linux/bits.h:69:
+#define BIT_TYPE(type, nr) ((type)(BIT_INPUT_CHECK(type, nr) + BIT_ULL(nr)))
total: 0 errors, 0 warnings, 1 checks, 32 lines checked
2c7dada269ed drm/i915: Convert REG_GENMASK*() to fixed-width GENMASK_U*()
300efb11156e test_bits: add tests for GENMASK_U*()
-:26: ERROR:SPACING: spaces required around that ':' (ctx:VxW)
#26: FILE: lib/test_bits.c:10:
+#define assert_type(t, x) _Generic(x, t: x, default: 0)
^
-:26: ERROR:SPACING: spaces required around that ':' (ctx:VxW)
#26: FILE: lib/test_bits.c:10:
+#define assert_type(t, x) _Generic(x, t: x, default: 0)
^
-:26: CHECK:MACRO_ARG_REUSE: Macro argument reuse 'x' - possible side-effects?
#26: FILE: lib/test_bits.c:10:
+#define assert_type(t, x) _Generic(x, t: x, default: 0)
total: 2 errors, 0 warnings, 1 checks, 37 lines checked
56e1dfba3900 test_bits: add tests for BIT_U*()
^ permalink raw reply [flat|nested] 26+ messages in thread* ✗ Fi.CI.SPARSE: warning for bits: Fixed-type GENMASK_U*() and BIT_U*() (rev2)
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
` (6 preceding siblings ...)
(?)
@ 2025-03-22 10:07 ` Patchwork
-1 siblings, 0 replies; 26+ messages in thread
From: Patchwork @ 2025-03-22 10:07 UTC (permalink / raw)
To: Vincent Mailhol via B4 Relay; +Cc: intel-gfx
== Series Details ==
Series: bits: Fixed-type GENMASK_U*() and BIT_U*() (rev2)
URL : https://patchwork.freedesktop.org/series/146088/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
^ permalink raw reply [flat|nested] 26+ messages in thread* ✓ i915.CI.BAT: success for bits: Fixed-type GENMASK_U*() and BIT_U*() (rev2)
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
` (7 preceding siblings ...)
(?)
@ 2025-03-22 16:19 ` Patchwork
-1 siblings, 0 replies; 26+ messages in thread
From: Patchwork @ 2025-03-22 16:19 UTC (permalink / raw)
To: Vincent Mailhol via B4 Relay; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 2826 bytes --]
== Series Details ==
Series: bits: Fixed-type GENMASK_U*() and BIT_U*() (rev2)
URL : https://patchwork.freedesktop.org/series/146088/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_16301 -> Patchwork_146088v2
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/index.html
Participating hosts (42 -> 40)
------------------------------
Missing (2): fi-snb-2520m fi-pnv-d510
Known issues
------------
Here are the changes found in Patchwork_146088v2 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_selftest@live@workarounds:
- bat-adlp-9: [PASS][1] -> [ABORT][2] ([i915#13696]) +1 other test abort
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/bat-adlp-9/igt@i915_selftest@live@workarounds.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/bat-adlp-9/igt@i915_selftest@live@workarounds.html
- bat-arls-6: [PASS][3] -> [DMESG-FAIL][4] ([i915#12061]) +1 other test dmesg-fail
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/bat-arls-6/igt@i915_selftest@live@workarounds.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/bat-arls-6/igt@i915_selftest@live@workarounds.html
#### Possible fixes ####
* igt@dmabuf@all-tests@dma_fence:
- bat-mtlp-9: [FAIL][5] -> [PASS][6] +1 other test pass
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/bat-mtlp-9/igt@dmabuf@all-tests@dma_fence.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/bat-mtlp-9/igt@dmabuf@all-tests@dma_fence.html
* igt@i915_pm_rpm@module-reload:
- fi-cfl-8700k: [DMESG-WARN][7] ([i915#13736]) -> [PASS][8]
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/fi-cfl-8700k/igt@i915_pm_rpm@module-reload.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/fi-cfl-8700k/igt@i915_pm_rpm@module-reload.html
[i915#12061]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12061
[i915#13696]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13696
[i915#13736]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13736
Build changes
-------------
* Linux: CI_DRM_16301 -> Patchwork_146088v2
CI-20190529: 20190529
CI_DRM_16301: a123616cf3afc205116835d8b083f48272583dcd @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_8278: b6673db372bd8987f65948d3a97f8dcd2ef42b01 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_146088v2: a123616cf3afc205116835d8b083f48272583dcd @ git://anongit.freedesktop.org/gfx-ci/linux
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/index.html
[-- Attachment #2: Type: text/html, Size: 3481 bytes --]
^ permalink raw reply [flat|nested] 26+ messages in thread* ✗ i915.CI.Full: failure for bits: Fixed-type GENMASK_U*() and BIT_U*() (rev2)
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
` (8 preceding siblings ...)
(?)
@ 2025-03-22 17:51 ` Patchwork
-1 siblings, 0 replies; 26+ messages in thread
From: Patchwork @ 2025-03-22 17:51 UTC (permalink / raw)
To: Vincent Mailhol via B4 Relay; +Cc: intel-gfx
[-- Attachment #1: Type: text/plain, Size: 102946 bytes --]
== Series Details ==
Series: bits: Fixed-type GENMASK_U*() and BIT_U*() (rev2)
URL : https://patchwork.freedesktop.org/series/146088/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_16301_full -> Patchwork_146088v2_full
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with Patchwork_146088v2_full absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_146088v2_full, please notify your bug team (I915-ci-infra@lists.freedesktop.org) to allow them
to document this new failure mode, which will reduce false positives in CI.
Participating hosts (9 -> 9)
------------------------------
No changes in participating hosts
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in Patchwork_146088v2_full:
### IGT changes ###
#### Possible regressions ####
* igt@kms_busy@extended-pageflip-hang-oldfb:
- shard-dg2: [PASS][1] -> [ABORT][2] +1 other test abort
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-11/igt@kms_busy@extended-pageflip-hang-oldfb.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-1/igt@kms_busy@extended-pageflip-hang-oldfb.html
* igt@kms_cursor_crc@cursor-onscreen-256x256:
- shard-snb: [PASS][3] -> [INCOMPLETE][4] +1 other test incomplete
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-snb2/igt@kms_cursor_crc@cursor-onscreen-256x256.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-snb4/igt@kms_cursor_crc@cursor-onscreen-256x256.html
New tests
---------
New tests have been introduced between CI_DRM_16301_full and Patchwork_146088v2_full:
### New IGT tests (36) ###
* igt@kms_busy@2x-single-buffer-flip-vs-dpms-off-vs-modeset-interruptible:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@bad-pad-fd-to-handle:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@bo-write-verify-threaded-none:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@bpc-switch-suspend:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@ccs-on-another-bo-y-tiled-gen12-rc-ccs-cc:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@crc:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@ctm-0-50:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@cursor-offscreen-32x10:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@cursor-random-32x10:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@cursor-rapid-movement-256x256:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@cursor-vs-flip-atomic-transitions:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@dontneed-before-pwrite:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@fbc-2p-primscrn-cur-indfb-draw-render:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@fbcpsr-1p-primscrn-pri-shrfb-draw-render:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@fbcpsr-2p-primscrn-spr-indfb-fullscreen:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@fbcpsr-2p-scndscrn-spr-indfb-draw-mmap-cpu:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@flip-32bpp-ytileccs-to-64bpp-ytile-downscaling:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@flip-64bpp-ytile-to-32bpp-ytilercccs-downscaling:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@lease-unleased-connector:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@legacy:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@linear-8bpp-rotate-180:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@linear-8bpp-rotate-180@pipe-d:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@load:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@media-rc6-accuracy:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@per-context-mode-unprivileged:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@plane-scaling:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@planes-unity-scaling-downscale-factor-0-75:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@psr-2p-primscrn-pri-indfb-draw-mmap-cpu:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@psr-suspend:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@query-garbage:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@reimport-vs-gem_close-race:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@reset-during-wait-for-submit:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@single-wait-signaled:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@system-suspend:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@x-tiled-8bpp-rotate-270:
- Statuses :
- Exec time: [None] s
* igt@kms_busy@y-tiled-to-vebox-linear:
- Statuses :
- Exec time: [None] s
Known issues
------------
Here are the changes found in Patchwork_146088v2_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@api_intel_bb@object-reloc-keep-cache:
- shard-rkl: NOTRUN -> [SKIP][5] ([i915#8411]) +1 other test skip
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-6/igt@api_intel_bb@object-reloc-keep-cache.html
- shard-dg2: NOTRUN -> [SKIP][6] ([i915#8411])
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@api_intel_bb@object-reloc-keep-cache.html
* igt@debugfs_test@basic-hwmon:
- shard-rkl: NOTRUN -> [SKIP][7] ([i915#9318])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-1/igt@debugfs_test@basic-hwmon.html
* igt@device_reset@cold-reset-bound:
- shard-tglu: NOTRUN -> [SKIP][8] ([i915#11078])
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-8/igt@device_reset@cold-reset-bound.html
- shard-rkl: NOTRUN -> [SKIP][9] ([i915#11078])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@device_reset@cold-reset-bound.html
* igt@device_reset@unbind-cold-reset-rebind:
- shard-mtlp: NOTRUN -> [SKIP][10] ([i915#11078])
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-6/igt@device_reset@unbind-cold-reset-rebind.html
* igt@drm_fdinfo@all-busy-idle-check-all:
- shard-mtlp: NOTRUN -> [SKIP][11] ([i915#8414])
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@drm_fdinfo@all-busy-idle-check-all.html
* igt@drm_fdinfo@virtual-busy-hang-all:
- shard-dg2-9: NOTRUN -> [SKIP][12] ([i915#8414])
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@drm_fdinfo@virtual-busy-hang-all.html
* igt@gem_ccs@block-multicopy-compressed:
- shard-rkl: NOTRUN -> [SKIP][13] ([i915#9323])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-1/igt@gem_ccs@block-multicopy-compressed.html
* igt@gem_ccs@suspend-resume@xmajor-compressed-compfmt0-smem-lmem0:
- shard-dg2: [PASS][14] -> [INCOMPLETE][15] ([i915#12392] / [i915#13356])
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-4/igt@gem_ccs@suspend-resume@xmajor-compressed-compfmt0-smem-lmem0.html
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-6/igt@gem_ccs@suspend-resume@xmajor-compressed-compfmt0-smem-lmem0.html
* igt@gem_close_race@multigpu-basic-process:
- shard-dg2-9: NOTRUN -> [SKIP][16] ([i915#7697])
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@gem_close_race@multigpu-basic-process.html
* igt@gem_compute@compute-square:
- shard-dg2: NOTRUN -> [FAIL][17] ([i915#13665])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@gem_compute@compute-square.html
* igt@gem_create@create-ext-set-pat:
- shard-dg2: NOTRUN -> [SKIP][18] ([i915#8562])
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@gem_create@create-ext-set-pat.html
- shard-rkl: NOTRUN -> [SKIP][19] ([i915#8562])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-6/igt@gem_create@create-ext-set-pat.html
* igt@gem_ctx_persistence@hang:
- shard-dg2: NOTRUN -> [SKIP][20] ([i915#8555])
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@gem_ctx_persistence@hang.html
* igt@gem_ctx_persistence@heartbeat-hang:
- shard-mtlp: NOTRUN -> [SKIP][21] ([i915#8555])
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-6/igt@gem_ctx_persistence@heartbeat-hang.html
* igt@gem_ctx_sseu@engines:
- shard-rkl: NOTRUN -> [SKIP][22] ([i915#280])
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@gem_ctx_sseu@engines.html
* igt@gem_ctx_sseu@invalid-args:
- shard-mtlp: NOTRUN -> [SKIP][23] ([i915#280])
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-6/igt@gem_ctx_sseu@invalid-args.html
* igt@gem_eio@hibernate:
- shard-tglu-1: NOTRUN -> [ABORT][24] ([i915#7975])
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@gem_eio@hibernate.html
- shard-dg2: NOTRUN -> [ABORT][25] ([i915#7975])
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-7/igt@gem_eio@hibernate.html
* igt@gem_eio@kms:
- shard-dg2: [PASS][26] -> [FAIL][27] ([i915#5784])
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-8/igt@gem_eio@kms.html
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-5/igt@gem_eio@kms.html
* igt@gem_exec_await@wide-all:
- shard-rkl: NOTRUN -> [DMESG-WARN][28] ([i915#12964]) +9 other tests dmesg-warn
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@gem_exec_await@wide-all.html
* igt@gem_exec_balancer@bonded-dual:
- shard-dg2: NOTRUN -> [SKIP][29] ([i915#4771])
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@gem_exec_balancer@bonded-dual.html
* igt@gem_exec_balancer@bonded-false-hang:
- shard-dg2-9: NOTRUN -> [SKIP][30] ([i915#4812]) +1 other test skip
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@gem_exec_balancer@bonded-false-hang.html
* igt@gem_exec_balancer@invalid-bonds:
- shard-mtlp: NOTRUN -> [SKIP][31] ([i915#4036])
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@gem_exec_balancer@invalid-bonds.html
* igt@gem_exec_balancer@parallel-balancer:
- shard-rkl: NOTRUN -> [SKIP][32] ([i915#4525]) +2 other tests skip
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-6/igt@gem_exec_balancer@parallel-balancer.html
* igt@gem_exec_balancer@parallel-keep-in-fence:
- shard-tglu: NOTRUN -> [SKIP][33] ([i915#4525])
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@gem_exec_balancer@parallel-keep-in-fence.html
* igt@gem_exec_balancer@parallel-out-fence:
- shard-tglu-1: NOTRUN -> [SKIP][34] ([i915#4525])
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@gem_exec_balancer@parallel-out-fence.html
* igt@gem_exec_big@single:
- shard-tglu: NOTRUN -> [ABORT][35] ([i915#11713])
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-8/igt@gem_exec_big@single.html
* igt@gem_exec_capture@capture-recoverable:
- shard-rkl: NOTRUN -> [SKIP][36] ([i915#6344])
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@gem_exec_capture@capture-recoverable.html
- shard-tglu-1: NOTRUN -> [SKIP][37] ([i915#6344])
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@gem_exec_capture@capture-recoverable.html
* igt@gem_exec_flush@basic-wb-prw-default:
- shard-dg2-9: NOTRUN -> [SKIP][38] ([i915#3539] / [i915#4852])
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@gem_exec_flush@basic-wb-prw-default.html
* igt@gem_exec_flush@basic-wb-rw-before-default:
- shard-dg2: NOTRUN -> [SKIP][39] ([i915#3539] / [i915#4852])
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@gem_exec_flush@basic-wb-rw-before-default.html
* igt@gem_exec_reloc@basic-active:
- shard-dg2: NOTRUN -> [SKIP][40] ([i915#3281]) +2 other tests skip
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@gem_exec_reloc@basic-active.html
* igt@gem_exec_reloc@basic-concurrent0:
- shard-mtlp: NOTRUN -> [SKIP][41] ([i915#3281]) +2 other tests skip
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@gem_exec_reloc@basic-concurrent0.html
* igt@gem_exec_reloc@basic-wc-read-noreloc:
- shard-rkl: NOTRUN -> [SKIP][42] ([i915#3281]) +15 other tests skip
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-1/igt@gem_exec_reloc@basic-wc-read-noreloc.html
* igt@gem_exec_reloc@basic-write-read-active:
- shard-dg2-9: NOTRUN -> [SKIP][43] ([i915#3281]) +5 other tests skip
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@gem_exec_reloc@basic-write-read-active.html
* igt@gem_exec_schedule@preempt-queue-chain:
- shard-dg2: NOTRUN -> [SKIP][44] ([i915#4537] / [i915#4812])
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@gem_exec_schedule@preempt-queue-chain.html
* igt@gem_exec_schedule@preempt-queue-contexts-chain:
- shard-mtlp: NOTRUN -> [SKIP][45] ([i915#4537] / [i915#4812])
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@gem_exec_schedule@preempt-queue-contexts-chain.html
* igt@gem_exec_suspend@basic-s0:
- shard-dg2: [PASS][46] -> [INCOMPLETE][47] ([i915#11441] / [i915#13304])
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-5/igt@gem_exec_suspend@basic-s0.html
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-1/igt@gem_exec_suspend@basic-s0.html
* igt@gem_exec_suspend@basic-s0@lmem0:
- shard-dg2: [PASS][48] -> [INCOMPLETE][49] ([i915#11441])
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-5/igt@gem_exec_suspend@basic-s0@lmem0.html
[49]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-1/igt@gem_exec_suspend@basic-s0@lmem0.html
* igt@gem_fence_thrash@bo-write-verify-none:
- shard-mtlp: NOTRUN -> [SKIP][50] ([i915#4860])
[50]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@gem_fence_thrash@bo-write-verify-none.html
* igt@gem_lmem_evict@dontneed-evict-race:
- shard-tglu-1: NOTRUN -> [SKIP][51] ([i915#4613] / [i915#7582])
[51]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@gem_lmem_evict@dontneed-evict-race.html
* igt@gem_lmem_swapping@massive:
- shard-tglu-1: NOTRUN -> [SKIP][52] ([i915#4613])
[52]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@gem_lmem_swapping@massive.html
- shard-mtlp: NOTRUN -> [SKIP][53] ([i915#4613])
[53]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@gem_lmem_swapping@massive.html
* igt@gem_lmem_swapping@parallel-random-verify-ccs:
- shard-tglu: NOTRUN -> [SKIP][54] ([i915#4613]) +1 other test skip
[54]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@gem_lmem_swapping@parallel-random-verify-ccs.html
* igt@gem_lmem_swapping@verify-random:
- shard-rkl: NOTRUN -> [SKIP][55] ([i915#4613]) +5 other tests skip
[55]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@gem_lmem_swapping@verify-random.html
* igt@gem_media_vme:
- shard-rkl: NOTRUN -> [SKIP][56] ([i915#284])
[56]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@gem_media_vme.html
* igt@gem_mmap_gtt@big-copy-odd:
- shard-dg2-9: NOTRUN -> [SKIP][57] ([i915#4077]) +4 other tests skip
[57]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@gem_mmap_gtt@big-copy-odd.html
* igt@gem_mmap_gtt@cpuset-medium-copy:
- shard-mtlp: NOTRUN -> [SKIP][58] ([i915#4077]) +1 other test skip
[58]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@gem_mmap_gtt@cpuset-medium-copy.html
* igt@gem_mmap_wc@close:
- shard-dg2: NOTRUN -> [SKIP][59] ([i915#4083]) +1 other test skip
[59]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@gem_mmap_wc@close.html
* igt@gem_mmap_wc@write-cpu-read-wc:
- shard-dg2-9: NOTRUN -> [SKIP][60] ([i915#4083]) +1 other test skip
[60]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@gem_mmap_wc@write-cpu-read-wc.html
* igt@gem_mmap_wc@write-gtt-read-wc:
- shard-mtlp: NOTRUN -> [SKIP][61] ([i915#4083])
[61]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@gem_mmap_wc@write-gtt-read-wc.html
* igt@gem_partial_pwrite_pread@writes-after-reads-uncached:
- shard-rkl: NOTRUN -> [SKIP][62] ([i915#3282]) +9 other tests skip
[62]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-6/igt@gem_partial_pwrite_pread@writes-after-reads-uncached.html
- shard-dg2: NOTRUN -> [SKIP][63] ([i915#3282])
[63]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@gem_partial_pwrite_pread@writes-after-reads-uncached.html
* igt@gem_pread@bench:
- shard-mtlp: NOTRUN -> [SKIP][64] ([i915#3282]) +1 other test skip
[64]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@gem_pread@bench.html
* igt@gem_pread@self:
- shard-dg2-9: NOTRUN -> [SKIP][65] ([i915#3282]) +1 other test skip
[65]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@gem_pread@self.html
* igt@gem_pxp@protected-raw-src-copy-not-readible:
- shard-rkl: NOTRUN -> [SKIP][66] ([i915#4270])
[66]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@gem_pxp@protected-raw-src-copy-not-readible.html
* igt@gem_pxp@reject-modify-context-protection-off-3:
- shard-rkl: [PASS][67] -> [TIMEOUT][68] ([i915#12917] / [i915#12964])
[67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-rkl-8/igt@gem_pxp@reject-modify-context-protection-off-3.html
[68]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@gem_pxp@reject-modify-context-protection-off-3.html
* igt@gem_pxp@verify-pxp-execution-after-suspend-resume:
- shard-rkl: NOTRUN -> [TIMEOUT][69] ([i915#12917] / [i915#12964]) +2 other tests timeout
[69]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@gem_pxp@verify-pxp-execution-after-suspend-resume.html
* igt@gem_render_copy@mixed-tiled-to-y-tiled-ccs:
- shard-mtlp: NOTRUN -> [SKIP][70] ([i915#8428]) +1 other test skip
[70]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-6/igt@gem_render_copy@mixed-tiled-to-y-tiled-ccs.html
* igt@gem_render_copy@yf-tiled-ccs-to-y-tiled:
- shard-dg2-9: NOTRUN -> [SKIP][71] ([i915#5190] / [i915#8428]) +3 other tests skip
[71]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@gem_render_copy@yf-tiled-ccs-to-y-tiled.html
* igt@gem_render_copy@yf-tiled-mc-ccs-to-vebox-y-tiled:
- shard-dg2: NOTRUN -> [SKIP][72] ([i915#5190] / [i915#8428]) +2 other tests skip
[72]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@gem_render_copy@yf-tiled-mc-ccs-to-vebox-y-tiled.html
* igt@gem_set_tiling_vs_pwrite:
- shard-dg2-9: NOTRUN -> [SKIP][73] ([i915#4079])
[73]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@gem_set_tiling_vs_pwrite.html
* igt@gem_softpin@evict-snoop-interruptible:
- shard-dg2-9: NOTRUN -> [SKIP][74] ([i915#4885])
[74]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@gem_softpin@evict-snoop-interruptible.html
* igt@gem_tiled_partial_pwrite_pread@writes:
- shard-dg2: NOTRUN -> [SKIP][75] ([i915#4077]) +4 other tests skip
[75]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@gem_tiled_partial_pwrite_pread@writes.html
* igt@gem_userptr_blits@access-control:
- shard-dg2: NOTRUN -> [SKIP][76] ([i915#3297])
[76]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-7/igt@gem_userptr_blits@access-control.html
- shard-tglu-1: NOTRUN -> [SKIP][77] ([i915#3297])
[77]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@gem_userptr_blits@access-control.html
* igt@gem_userptr_blits@coherency-sync:
- shard-tglu: NOTRUN -> [SKIP][78] ([i915#3297])
[78]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@gem_userptr_blits@coherency-sync.html
* igt@gem_userptr_blits@map-fixed-invalidate-overlap:
- shard-dg2: NOTRUN -> [SKIP][79] ([i915#3297] / [i915#4880])
[79]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-7/igt@gem_userptr_blits@map-fixed-invalidate-overlap.html
* igt@gem_userptr_blits@relocations:
- shard-rkl: NOTRUN -> [SKIP][80] ([i915#3281] / [i915#3297])
[80]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@gem_userptr_blits@relocations.html
* igt@gen9_exec_parse@bb-large:
- shard-tglu-1: NOTRUN -> [SKIP][81] ([i915#2527] / [i915#2856]) +1 other test skip
[81]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@gen9_exec_parse@bb-large.html
- shard-mtlp: NOTRUN -> [SKIP][82] ([i915#2856]) +1 other test skip
[82]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@gen9_exec_parse@bb-large.html
* igt@gen9_exec_parse@bb-oversize:
- shard-rkl: NOTRUN -> [SKIP][83] ([i915#2527]) +4 other tests skip
[83]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@gen9_exec_parse@bb-oversize.html
- shard-tglu: NOTRUN -> [SKIP][84] ([i915#2527] / [i915#2856]) +2 other tests skip
[84]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-8/igt@gen9_exec_parse@bb-oversize.html
* igt@gen9_exec_parse@cmd-crossing-page:
- shard-dg2-9: NOTRUN -> [SKIP][85] ([i915#2856]) +1 other test skip
[85]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@gen9_exec_parse@cmd-crossing-page.html
* igt@i915_pm_freq_api@freq-reset:
- shard-tglu-1: NOTRUN -> [SKIP][86] ([i915#8399]) +1 other test skip
[86]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@i915_pm_freq_api@freq-reset.html
- shard-rkl: NOTRUN -> [SKIP][87] ([i915#8399])
[87]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@i915_pm_freq_api@freq-reset.html
* igt@i915_pm_freq_api@freq-suspend@gt0:
- shard-dg2: [PASS][88] -> [INCOMPLETE][89] ([i915#12455] / [i915#13820]) +1 other test incomplete
[88]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-6/igt@i915_pm_freq_api@freq-suspend@gt0.html
[89]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-5/igt@i915_pm_freq_api@freq-suspend@gt0.html
* igt@i915_pm_freq_mult@media-freq@gt0:
- shard-rkl: NOTRUN -> [SKIP][90] ([i915#6590]) +1 other test skip
[90]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@i915_pm_freq_mult@media-freq@gt0.html
* igt@i915_pm_rc6_residency@rc6-idle:
- shard-tglu: NOTRUN -> [WARN][91] ([i915#13790] / [i915#2681]) +4 other tests warn
[91]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@i915_pm_rc6_residency@rc6-idle.html
* igt@i915_pm_rc6_residency@rc6-idle@gt0-bcs0:
- shard-dg1: [PASS][92] -> [FAIL][93] ([i915#3591]) +1 other test fail
[92]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg1-14/igt@i915_pm_rc6_residency@rc6-idle@gt0-bcs0.html
[93]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg1-17/igt@i915_pm_rc6_residency@rc6-idle@gt0-bcs0.html
* igt@i915_pm_rps@min-max-config-loaded:
- shard-dg2-9: NOTRUN -> [SKIP][94] ([i915#11681] / [i915#6621])
[94]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@i915_pm_rps@min-max-config-loaded.html
* igt@i915_pm_rps@thresholds-idle-park:
- shard-dg2-9: NOTRUN -> [SKIP][95] ([i915#11681])
[95]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@i915_pm_rps@thresholds-idle-park.html
* igt@i915_selftest@live@workarounds:
- shard-mtlp: [PASS][96] -> [DMESG-FAIL][97] ([i915#12061]) +1 other test dmesg-fail
[96]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-mtlp-2/igt@i915_selftest@live@workarounds.html
[97]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-2/igt@i915_selftest@live@workarounds.html
* igt@i915_selftest@mock@memory_region:
- shard-rkl: NOTRUN -> [DMESG-WARN][98] ([i915#9311]) +1 other test dmesg-warn
[98]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-1/igt@i915_selftest@mock@memory_region.html
* igt@i915_suspend@basic-s3-without-i915:
- shard-rkl: [PASS][99] -> [INCOMPLETE][100] ([i915#4817])
[99]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-rkl-5/igt@i915_suspend@basic-s3-without-i915.html
[100]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-3/igt@i915_suspend@basic-s3-without-i915.html
* igt@intel_hwmon@hwmon-write:
- shard-tglu: NOTRUN -> [SKIP][101] ([i915#7707])
[101]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@intel_hwmon@hwmon-write.html
* igt@kms_async_flips@async-flip-with-page-flip-events-atomic@pipe-c-hdmi-a-1-y-rc-ccs-cc:
- shard-tglu: NOTRUN -> [SKIP][102] ([i915#8709]) +3 other tests skip
[102]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_async_flips@async-flip-with-page-flip-events-atomic@pipe-c-hdmi-a-1-y-rc-ccs-cc.html
* igt@kms_async_flips@async-flip-with-page-flip-events@pipe-a-dp-3-4-mc-ccs:
- shard-dg2: NOTRUN -> [SKIP][103] ([i915#8709]) +15 other tests skip
[103]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-11/igt@kms_async_flips@async-flip-with-page-flip-events@pipe-a-dp-3-4-mc-ccs.html
* igt@kms_async_flips@async-flip-with-page-flip-events@pipe-b-hdmi-a-2-y-rc-ccs-cc:
- shard-rkl: NOTRUN -> [SKIP][104] ([i915#8709]) +1 other test skip
[104]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-1/igt@kms_async_flips@async-flip-with-page-flip-events@pipe-b-hdmi-a-2-y-rc-ccs-cc.html
* igt@kms_atomic@plane-primary-overlay-mutable-zpos:
- shard-rkl: NOTRUN -> [SKIP][105] ([i915#9531])
[105]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_atomic@plane-primary-overlay-mutable-zpos.html
- shard-tglu: NOTRUN -> [SKIP][106] ([i915#9531])
[106]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_atomic@plane-primary-overlay-mutable-zpos.html
* igt@kms_atomic_transition@plane-all-modeset-transition-fencing-internal-panels:
- shard-tglu: NOTRUN -> [SKIP][107] ([i915#1769] / [i915#3555]) +1 other test skip
[107]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_atomic_transition@plane-all-modeset-transition-fencing-internal-panels.html
* igt@kms_atomic_transition@plane-all-modeset-transition-fencing@pipe-a-hdmi-a-4:
- shard-dg1: [PASS][108] -> [FAIL][109] ([i915#5956]) +1 other test fail
[108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg1-14/igt@kms_atomic_transition@plane-all-modeset-transition-fencing@pipe-a-hdmi-a-4.html
[109]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg1-17/igt@kms_atomic_transition@plane-all-modeset-transition-fencing@pipe-a-hdmi-a-4.html
* igt@kms_atomic_transition@plane-all-modeset-transition-internal-panels:
- shard-rkl: NOTRUN -> [SKIP][110] ([i915#1769] / [i915#3555]) +1 other test skip
[110]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@kms_atomic_transition@plane-all-modeset-transition-internal-panels.html
* igt@kms_big_fb@4-tiled-32bpp-rotate-270:
- shard-tglu: NOTRUN -> [SKIP][111] ([i915#5286]) +2 other tests skip
[111]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_big_fb@4-tiled-32bpp-rotate-270.html
* igt@kms_big_fb@4-tiled-addfb:
- shard-rkl: NOTRUN -> [SKIP][112] ([i915#5286]) +7 other tests skip
[112]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-6/igt@kms_big_fb@4-tiled-addfb.html
* igt@kms_big_fb@4-tiled-max-hw-stride-32bpp-rotate-180-hflip:
- shard-tglu-1: NOTRUN -> [SKIP][113] ([i915#5286]) +2 other tests skip
[113]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_big_fb@4-tiled-max-hw-stride-32bpp-rotate-180-hflip.html
* igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-180-hflip:
- shard-mtlp: [PASS][114] -> [FAIL][115] ([i915#5138])
[114]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-mtlp-4/igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-180-hflip.html
[115]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-3/igt@kms_big_fb@4-tiled-max-hw-stride-64bpp-rotate-180-hflip.html
* igt@kms_big_fb@linear-16bpp-rotate-270:
- shard-mtlp: NOTRUN -> [SKIP][116] +8 other tests skip
[116]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-6/igt@kms_big_fb@linear-16bpp-rotate-270.html
* igt@kms_big_fb@y-tiled-64bpp-rotate-0:
- shard-dg2-9: NOTRUN -> [SKIP][117] ([i915#4538] / [i915#5190]) +4 other tests skip
[117]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_big_fb@y-tiled-64bpp-rotate-0.html
* igt@kms_big_fb@y-tiled-8bpp-rotate-270:
- shard-dg2: NOTRUN -> [SKIP][118] ([i915#4538] / [i915#5190]) +2 other tests skip
[118]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_big_fb@y-tiled-8bpp-rotate-270.html
- shard-rkl: NOTRUN -> [SKIP][119] ([i915#3638]) +4 other tests skip
[119]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-6/igt@kms_big_fb@y-tiled-8bpp-rotate-270.html
* igt@kms_busy@extended-pageflip-hang-oldfb@pipe-a:
- shard-dg2: [PASS][120] -> [DMESG-WARN][121] ([i915#13562])
[120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-11/igt@kms_busy@extended-pageflip-hang-oldfb@pipe-a.html
[121]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-1/igt@kms_busy@extended-pageflip-hang-oldfb@pipe-a.html
* igt@kms_ccs@bad-pixel-format-y-tiled-ccs@pipe-d-hdmi-a-1:
- shard-dg2: NOTRUN -> [SKIP][122] ([i915#10307] / [i915#10434] / [i915#6095]) +1 other test skip
[122]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_ccs@bad-pixel-format-y-tiled-ccs@pipe-d-hdmi-a-1.html
* igt@kms_ccs@bad-rotation-90-4-tiled-dg2-rc-ccs-cc@pipe-b-hdmi-a-1:
- shard-tglu-1: NOTRUN -> [SKIP][123] ([i915#6095]) +29 other tests skip
[123]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_ccs@bad-rotation-90-4-tiled-dg2-rc-ccs-cc@pipe-b-hdmi-a-1.html
* igt@kms_ccs@bad-rotation-90-4-tiled-lnl-ccs:
- shard-rkl: NOTRUN -> [SKIP][124] ([i915#12313]) +1 other test skip
[124]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-1/igt@kms_ccs@bad-rotation-90-4-tiled-lnl-ccs.html
* igt@kms_ccs@bad-rotation-90-4-tiled-mtl-mc-ccs@pipe-b-hdmi-a-2:
- shard-dg2-9: NOTRUN -> [SKIP][125] ([i915#10307] / [i915#6095]) +24 other tests skip
[125]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_ccs@bad-rotation-90-4-tiled-mtl-mc-ccs@pipe-b-hdmi-a-2.html
* igt@kms_ccs@bad-rotation-90-yf-tiled-ccs@pipe-a-hdmi-a-1:
- shard-tglu: NOTRUN -> [SKIP][126] ([i915#6095]) +39 other tests skip
[126]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_ccs@bad-rotation-90-yf-tiled-ccs@pipe-a-hdmi-a-1.html
* igt@kms_ccs@ccs-on-another-bo-yf-tiled-ccs@pipe-a-hdmi-a-3:
- shard-dg2: NOTRUN -> [SKIP][127] ([i915#10307] / [i915#6095]) +141 other tests skip
[127]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-7/igt@kms_ccs@ccs-on-another-bo-yf-tiled-ccs@pipe-a-hdmi-a-3.html
* igt@kms_ccs@crc-primary-rotation-180-4-tiled-lnl-ccs:
- shard-dg2: NOTRUN -> [SKIP][128] ([i915#12313])
[128]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_ccs@crc-primary-rotation-180-4-tiled-lnl-ccs.html
* igt@kms_ccs@crc-primary-suspend-4-tiled-dg2-mc-ccs@pipe-b-hdmi-a-2:
- shard-rkl: NOTRUN -> [SKIP][129] ([i915#6095]) +119 other tests skip
[129]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_ccs@crc-primary-suspend-4-tiled-dg2-mc-ccs@pipe-b-hdmi-a-2.html
* igt@kms_ccs@crc-primary-suspend-4-tiled-lnl-ccs:
- shard-tglu-1: NOTRUN -> [SKIP][130] ([i915#12805])
[130]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_ccs@crc-primary-suspend-4-tiled-lnl-ccs.html
* igt@kms_ccs@crc-primary-suspend-4-tiled-mtl-rc-ccs-cc@pipe-d-hdmi-a-3:
- shard-dg1: NOTRUN -> [SKIP][131] ([i915#6095]) +59 other tests skip
[131]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg1-13/igt@kms_ccs@crc-primary-suspend-4-tiled-mtl-rc-ccs-cc@pipe-d-hdmi-a-3.html
* igt@kms_ccs@crc-primary-suspend-y-tiled-ccs@pipe-b-dp-4:
- shard-dg2: NOTRUN -> [SKIP][132] ([i915#6095]) +11 other tests skip
[132]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-10/igt@kms_ccs@crc-primary-suspend-y-tiled-ccs@pipe-b-dp-4.html
* igt@kms_ccs@crc-primary-suspend-yf-tiled-ccs@pipe-a-hdmi-a-2:
- shard-dg2-9: NOTRUN -> [SKIP][133] ([i915#6095]) +9 other tests skip
[133]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_ccs@crc-primary-suspend-yf-tiled-ccs@pipe-a-hdmi-a-2.html
* igt@kms_ccs@random-ccs-data-4-tiled-bmg-ccs:
- shard-dg2-9: NOTRUN -> [SKIP][134] ([i915#12313])
[134]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_ccs@random-ccs-data-4-tiled-bmg-ccs.html
* igt@kms_ccs@random-ccs-data-4-tiled-dg2-rc-ccs-cc@pipe-b-edp-1:
- shard-mtlp: NOTRUN -> [SKIP][135] ([i915#6095]) +19 other tests skip
[135]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_ccs@random-ccs-data-4-tiled-dg2-rc-ccs-cc@pipe-b-edp-1.html
* igt@kms_cdclk@mode-transition:
- shard-rkl: NOTRUN -> [SKIP][136] ([i915#3742])
[136]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-1/igt@kms_cdclk@mode-transition.html
* igt@kms_cdclk@mode-transition@pipe-d-hdmi-a-3:
- shard-dg2: NOTRUN -> [SKIP][137] ([i915#13781]) +3 other tests skip
[137]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-2/igt@kms_cdclk@mode-transition@pipe-d-hdmi-a-3.html
* igt@kms_cdclk@plane-scaling@pipe-b-hdmi-a-3:
- shard-dg2: NOTRUN -> [SKIP][138] ([i915#13783]) +3 other tests skip
[138]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-1/igt@kms_cdclk@plane-scaling@pipe-b-hdmi-a-3.html
* igt@kms_chamelium_color@ctm-0-50:
- shard-dg2: NOTRUN -> [SKIP][139] +2 other tests skip
[139]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_chamelium_color@ctm-0-50.html
* igt@kms_chamelium_edid@hdmi-edid-change-during-suspend:
- shard-rkl: NOTRUN -> [SKIP][140] ([i915#11151] / [i915#7828]) +13 other tests skip
[140]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_chamelium_edid@hdmi-edid-change-during-suspend.html
* igt@kms_chamelium_edid@vga-edid-read:
- shard-mtlp: NOTRUN -> [SKIP][141] ([i915#11151] / [i915#7828]) +3 other tests skip
[141]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_chamelium_edid@vga-edid-read.html
* igt@kms_chamelium_frames@hdmi-aspect-ratio:
- shard-tglu: NOTRUN -> [SKIP][142] ([i915#11151] / [i915#7828]) +5 other tests skip
[142]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-8/igt@kms_chamelium_frames@hdmi-aspect-ratio.html
* igt@kms_chamelium_frames@hdmi-crc-nonplanar-formats:
- shard-dg2-9: NOTRUN -> [SKIP][143] ([i915#11151] / [i915#7828]) +3 other tests skip
[143]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_chamelium_frames@hdmi-crc-nonplanar-formats.html
* igt@kms_chamelium_hpd@dp-hpd-after-suspend:
- shard-tglu-1: NOTRUN -> [SKIP][144] ([i915#11151] / [i915#7828]) +4 other tests skip
[144]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_chamelium_hpd@dp-hpd-after-suspend.html
* igt@kms_chamelium_hpd@hdmi-hpd-storm-disable:
- shard-dg2: NOTRUN -> [SKIP][145] ([i915#11151] / [i915#7828])
[145]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_chamelium_hpd@hdmi-hpd-storm-disable.html
* igt@kms_color@deep-color:
- shard-dg2: [PASS][146] -> [SKIP][147] ([i915#3555])
[146]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-10/igt@kms_color@deep-color.html
[147]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-2/igt@kms_color@deep-color.html
- shard-rkl: NOTRUN -> [SKIP][148] ([i915#3555]) +5 other tests skip
[148]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-1/igt@kms_color@deep-color.html
* igt@kms_content_protection@atomic-dpms@pipe-a-dp-3:
- shard-dg2: NOTRUN -> [FAIL][149] ([i915#7173])
[149]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-11/igt@kms_content_protection@atomic-dpms@pipe-a-dp-3.html
* igt@kms_content_protection@dp-mst-lic-type-0:
- shard-tglu-1: NOTRUN -> [SKIP][150] ([i915#3116] / [i915#3299])
[150]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_content_protection@dp-mst-lic-type-0.html
* igt@kms_content_protection@dp-mst-type-1:
- shard-dg2-9: NOTRUN -> [SKIP][151] ([i915#3299])
[151]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_content_protection@dp-mst-type-1.html
* igt@kms_content_protection@srm:
- shard-rkl: NOTRUN -> [SKIP][152] ([i915#7118])
[152]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_content_protection@srm.html
- shard-tglu: NOTRUN -> [SKIP][153] ([i915#6944] / [i915#7116] / [i915#7118])
[153]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_content_protection@srm.html
* igt@kms_content_protection@type1:
- shard-rkl: NOTRUN -> [SKIP][154] ([i915#7118] / [i915#9424]) +2 other tests skip
[154]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-1/igt@kms_content_protection@type1.html
* igt@kms_content_protection@uevent:
- shard-mtlp: NOTRUN -> [SKIP][155] ([i915#6944] / [i915#9424])
[155]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_content_protection@uevent.html
- shard-tglu-1: NOTRUN -> [SKIP][156] ([i915#6944] / [i915#7116] / [i915#7118] / [i915#9424])
[156]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_content_protection@uevent.html
* igt@kms_cursor_crc@cursor-offscreen-256x85:
- shard-rkl: [PASS][157] -> [DMESG-WARN][158] ([i915#12964]) +8 other tests dmesg-warn
[157]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-rkl-3/igt@kms_cursor_crc@cursor-offscreen-256x85.html
[158]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-5/igt@kms_cursor_crc@cursor-offscreen-256x85.html
* igt@kms_cursor_crc@cursor-offscreen-512x512:
- shard-dg2: NOTRUN -> [SKIP][159] ([i915#13049])
[159]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_cursor_crc@cursor-offscreen-512x512.html
* igt@kms_cursor_crc@cursor-onscreen-512x170:
- shard-tglu-1: NOTRUN -> [SKIP][160] ([i915#13049])
[160]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_cursor_crc@cursor-onscreen-512x170.html
* igt@kms_cursor_crc@cursor-onscreen-max-size:
- shard-dg2: NOTRUN -> [SKIP][161] ([i915#3555]) +1 other test skip
[161]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_cursor_crc@cursor-onscreen-max-size.html
* igt@kms_cursor_crc@cursor-random-128x42:
- shard-mtlp: NOTRUN -> [SKIP][162] ([i915#8814])
[162]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_cursor_crc@cursor-random-128x42.html
* igt@kms_cursor_crc@cursor-random-128x42@pipe-a-hdmi-a-1:
- shard-tglu-1: NOTRUN -> [FAIL][163] ([i915#13566]) +1 other test fail
[163]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_cursor_crc@cursor-random-128x42@pipe-a-hdmi-a-1.html
* igt@kms_cursor_crc@cursor-random-512x512:
- shard-dg2-9: NOTRUN -> [SKIP][164] ([i915#13049])
[164]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_cursor_crc@cursor-random-512x512.html
* igt@kms_cursor_crc@cursor-rapid-movement-32x32:
- shard-mtlp: NOTRUN -> [SKIP][165] ([i915#3555] / [i915#8814])
[165]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-6/igt@kms_cursor_crc@cursor-rapid-movement-32x32.html
* igt@kms_cursor_crc@cursor-sliding-32x10:
- shard-dg2-9: NOTRUN -> [SKIP][166] ([i915#3555]) +1 other test skip
[166]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_cursor_crc@cursor-sliding-32x10.html
* igt@kms_cursor_crc@cursor-sliding-512x170:
- shard-tglu: NOTRUN -> [SKIP][167] ([i915#13049])
[167]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_cursor_crc@cursor-sliding-512x170.html
* igt@kms_cursor_crc@cursor-sliding-512x512:
- shard-rkl: NOTRUN -> [SKIP][168] ([i915#13049]) +4 other tests skip
[168]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@kms_cursor_crc@cursor-sliding-512x512.html
* igt@kms_cursor_legacy@2x-long-nonblocking-modeset-vs-cursor-atomic:
- shard-dg2: NOTRUN -> [SKIP][169] ([i915#13046] / [i915#5354]) +2 other tests skip
[169]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_cursor_legacy@2x-long-nonblocking-modeset-vs-cursor-atomic.html
* igt@kms_cursor_legacy@basic-flip-before-cursor-varying-size:
- shard-glk: [PASS][170] -> [FAIL][171] ([i915#2346])
[170]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-glk3/igt@kms_cursor_legacy@basic-flip-before-cursor-varying-size.html
[171]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-glk8/igt@kms_cursor_legacy@basic-flip-before-cursor-varying-size.html
* igt@kms_cursor_legacy@cursora-vs-flipb-atomic-transitions:
- shard-mtlp: NOTRUN -> [SKIP][172] ([i915#9809])
[172]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_cursor_legacy@cursora-vs-flipb-atomic-transitions.html
* igt@kms_cursor_legacy@cursorb-vs-flipb-varying-size:
- shard-dg2-9: NOTRUN -> [SKIP][173] ([i915#13046] / [i915#5354])
[173]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_cursor_legacy@cursorb-vs-flipb-varying-size.html
* igt@kms_cursor_legacy@modeset-atomic-cursor-hotspot:
- shard-mtlp: NOTRUN -> [SKIP][174] ([i915#9067])
[174]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_cursor_legacy@modeset-atomic-cursor-hotspot.html
- shard-rkl: NOTRUN -> [SKIP][175] ([i915#9067])
[175]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_cursor_legacy@modeset-atomic-cursor-hotspot.html
- shard-tglu-1: NOTRUN -> [SKIP][176] ([i915#9067])
[176]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_cursor_legacy@modeset-atomic-cursor-hotspot.html
* igt@kms_cursor_legacy@short-busy-flip-before-cursor-atomic-transitions-varying-size:
- shard-tglu-1: NOTRUN -> [SKIP][177] ([i915#4103])
[177]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_cursor_legacy@short-busy-flip-before-cursor-atomic-transitions-varying-size.html
* igt@kms_dirtyfb@psr-dirtyfb-ioctl:
- shard-rkl: NOTRUN -> [SKIP][178] ([i915#9723])
[178]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_dirtyfb@psr-dirtyfb-ioctl.html
- shard-tglu: NOTRUN -> [SKIP][179] ([i915#9723]) +1 other test skip
[179]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_dirtyfb@psr-dirtyfb-ioctl.html
* igt@kms_display_modes@extended-mode-basic:
- shard-rkl: NOTRUN -> [SKIP][180] ([i915#13691])
[180]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@kms_display_modes@extended-mode-basic.html
* igt@kms_dp_link_training@non-uhbr-mst:
- shard-rkl: NOTRUN -> [SKIP][181] ([i915#13749])
[181]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-1/igt@kms_dp_link_training@non-uhbr-mst.html
* igt@kms_dp_link_training@uhbr-mst:
- shard-mtlp: NOTRUN -> [SKIP][182] ([i915#13749])
[182]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-6/igt@kms_dp_link_training@uhbr-mst.html
* igt@kms_dp_linktrain_fallback@dp-fallback:
- shard-tglu-1: NOTRUN -> [SKIP][183] ([i915#13707])
[183]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_dp_linktrain_fallback@dp-fallback.html
* igt@kms_draw_crc@draw-method-mmap-gtt:
- shard-mtlp: NOTRUN -> [SKIP][184] ([i915#3555] / [i915#8812])
[184]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_draw_crc@draw-method-mmap-gtt.html
* igt@kms_dsc@dsc-fractional-bpp-with-bpc:
- shard-dg2-9: NOTRUN -> [SKIP][185] ([i915#3840])
[185]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_dsc@dsc-fractional-bpp-with-bpc.html
* igt@kms_dsc@dsc-with-formats:
- shard-tglu: NOTRUN -> [SKIP][186] ([i915#3555] / [i915#3840])
[186]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_dsc@dsc-with-formats.html
* igt@kms_dsc@dsc-with-output-formats:
- shard-dg2: NOTRUN -> [SKIP][187] ([i915#3555] / [i915#3840])
[187]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_dsc@dsc-with-output-formats.html
- shard-rkl: NOTRUN -> [SKIP][188] ([i915#3555] / [i915#3840])
[188]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-6/igt@kms_dsc@dsc-with-output-formats.html
* igt@kms_fbcon_fbt@psr-suspend:
- shard-tglu: NOTRUN -> [SKIP][189] ([i915#3469])
[189]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_fbcon_fbt@psr-suspend.html
* igt@kms_feature_discovery@display-2x:
- shard-tglu: NOTRUN -> [SKIP][190] ([i915#1839])
[190]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_feature_discovery@display-2x.html
* igt@kms_feature_discovery@dp-mst:
- shard-rkl: NOTRUN -> [SKIP][191] ([i915#9337])
[191]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_feature_discovery@dp-mst.html
- shard-tglu-1: NOTRUN -> [SKIP][192] ([i915#9337])
[192]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_feature_discovery@dp-mst.html
- shard-mtlp: NOTRUN -> [SKIP][193] ([i915#9337])
[193]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_feature_discovery@dp-mst.html
* igt@kms_fence_pin_leak:
- shard-mtlp: NOTRUN -> [SKIP][194] ([i915#4881])
[194]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-6/igt@kms_fence_pin_leak.html
* igt@kms_flip@2x-absolute-wf_vblank:
- shard-tglu-1: NOTRUN -> [SKIP][195] ([i915#3637]) +2 other tests skip
[195]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_flip@2x-absolute-wf_vblank.html
* igt@kms_flip@2x-absolute-wf_vblank-interruptible:
- shard-mtlp: NOTRUN -> [SKIP][196] ([i915#3637]) +1 other test skip
[196]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_flip@2x-absolute-wf_vblank-interruptible.html
* igt@kms_flip@2x-busy-flip:
- shard-dg2-9: NOTRUN -> [SKIP][197] ([i915#9934]) +2 other tests skip
[197]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_flip@2x-busy-flip.html
* igt@kms_flip@2x-flip-vs-dpms-off-vs-modeset-interruptible:
- shard-rkl: NOTRUN -> [SKIP][198] ([i915#9934]) +7 other tests skip
[198]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@kms_flip@2x-flip-vs-dpms-off-vs-modeset-interruptible.html
* igt@kms_flip@2x-flip-vs-expired-vblank-interruptible:
- shard-dg2: NOTRUN -> [SKIP][199] ([i915#9934]) +1 other test skip
[199]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-7/igt@kms_flip@2x-flip-vs-expired-vblank-interruptible.html
* igt@kms_flip@2x-flip-vs-modeset:
- shard-tglu: NOTRUN -> [SKIP][200] ([i915#3637]) +5 other tests skip
[200]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_flip@2x-flip-vs-modeset.html
* igt@kms_flip_scaled_crc@flip-32bpp-yftileccs-to-64bpp-yftile-downscaling@pipe-a-valid-mode:
- shard-tglu: NOTRUN -> [SKIP][201] ([i915#2587] / [i915#2672]) +1 other test skip
[201]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_flip_scaled_crc@flip-32bpp-yftileccs-to-64bpp-yftile-downscaling@pipe-a-valid-mode.html
* igt@kms_flip_scaled_crc@flip-32bpp-yftileccs-to-64bpp-yftile-upscaling:
- shard-dg2: NOTRUN -> [SKIP][202] ([i915#2672] / [i915#3555])
[202]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_flip_scaled_crc@flip-32bpp-yftileccs-to-64bpp-yftile-upscaling.html
* igt@kms_flip_scaled_crc@flip-32bpp-yftileccs-to-64bpp-yftile-upscaling@pipe-a-valid-mode:
- shard-dg2: NOTRUN -> [SKIP][203] ([i915#2672]) +2 other tests skip
[203]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_flip_scaled_crc@flip-32bpp-yftileccs-to-64bpp-yftile-upscaling@pipe-a-valid-mode.html
* igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-upscaling@pipe-a-valid-mode:
- shard-rkl: NOTRUN -> [SKIP][204] ([i915#2672]) +4 other tests skip
[204]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@kms_flip_scaled_crc@flip-32bpp-ytile-to-32bpp-ytileccs-upscaling@pipe-a-valid-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-16bpp-4tile-upscaling:
- shard-rkl: NOTRUN -> [SKIP][205] ([i915#2672] / [i915#3555]) +4 other tests skip
[205]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-16bpp-4tile-upscaling.html
- shard-tglu-1: NOTRUN -> [SKIP][206] ([i915#2672] / [i915#3555]) +1 other test skip
[206]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-16bpp-4tile-upscaling.html
* igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-16bpp-4tile-upscaling@pipe-a-valid-mode:
- shard-tglu-1: NOTRUN -> [SKIP][207] ([i915#2587] / [i915#2672]) +1 other test skip
[207]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_flip_scaled_crc@flip-64bpp-4tile-to-16bpp-4tile-upscaling@pipe-a-valid-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-downscaling:
- shard-dg2-9: NOTRUN -> [SKIP][208] ([i915#2672] / [i915#3555])
[208]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-downscaling.html
* igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-downscaling@pipe-a-valid-mode:
- shard-dg2-9: NOTRUN -> [SKIP][209] ([i915#2672])
[209]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-downscaling@pipe-a-valid-mode.html
* igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-upscaling:
- shard-tglu: NOTRUN -> [SKIP][210] ([i915#2672] / [i915#3555]) +1 other test skip
[210]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-8/igt@kms_flip_scaled_crc@flip-64bpp-yftile-to-16bpp-yftile-upscaling.html
* igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-16bpp-ytile-downscaling:
- shard-dg2: NOTRUN -> [SKIP][211] ([i915#2672] / [i915#3555] / [i915#5190]) +1 other test skip
[211]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_flip_scaled_crc@flip-64bpp-ytile-to-16bpp-ytile-downscaling.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-mmap-wc:
- shard-dg2: NOTRUN -> [SKIP][212] ([i915#8708]) +4 other tests skip
[212]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-cur-indfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-mmap-gtt:
- shard-mtlp: NOTRUN -> [SKIP][213] ([i915#8708]) +2 other tests skip
[213]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-mmap-gtt.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-mmap-wc:
- shard-rkl: NOTRUN -> [SKIP][214] ([i915#1825]) +56 other tests skip
[214]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-indfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-mmap-wc:
- shard-mtlp: NOTRUN -> [SKIP][215] ([i915#1825]) +10 other tests skip
[215]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-pri-shrfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-blt:
- shard-dg2: NOTRUN -> [SKIP][216] ([i915#5354]) +7 other tests skip
[216]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-spr-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbc-tiling-4:
- shard-rkl: NOTRUN -> [SKIP][217] ([i915#5439])
[217]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-1/igt@kms_frontbuffer_tracking@fbc-tiling-4.html
* igt@kms_frontbuffer_tracking@fbc-tiling-y:
- shard-dg2-9: NOTRUN -> [SKIP][218] ([i915#10055])
[218]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_frontbuffer_tracking@fbc-tiling-y.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-shrfb-draw-blt:
- shard-dg2-9: NOTRUN -> [SKIP][219] ([i915#3458]) +7 other tests skip
[219]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_frontbuffer_tracking@fbcpsr-1p-offscren-pri-shrfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-mmap-gtt:
- shard-dg2-9: NOTRUN -> [SKIP][220] ([i915#8708]) +8 other tests skip
[220]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-pri-shrfb-draw-mmap-gtt.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-pwrite:
- shard-dg2: NOTRUN -> [SKIP][221] ([i915#3458]) +4 other tests skip
[221]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-7/igt@kms_frontbuffer_tracking@fbcpsr-1p-primscrn-spr-indfb-draw-pwrite.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-indfb-fliptrack-mmap-gtt:
- shard-rkl: NOTRUN -> [SKIP][222] +28 other tests skip
[222]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_frontbuffer_tracking@fbcpsr-2p-indfb-fliptrack-mmap-gtt.html
* igt@kms_frontbuffer_tracking@fbcpsr-rgb101010-draw-mmap-wc:
- shard-rkl: NOTRUN -> [SKIP][223] ([i915#3023]) +31 other tests skip
[223]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@kms_frontbuffer_tracking@fbcpsr-rgb101010-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@fbcpsr-tiling-y:
- shard-dg2: NOTRUN -> [SKIP][224] ([i915#10055])
[224]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-7/igt@kms_frontbuffer_tracking@fbcpsr-tiling-y.html
* igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-mmap-wc:
- shard-tglu-1: NOTRUN -> [SKIP][225] +38 other tests skip
[225]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_frontbuffer_tracking@psr-1p-primscrn-cur-indfb-draw-mmap-wc.html
* igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-mmap-cpu:
- shard-dg2-9: NOTRUN -> [SKIP][226] ([i915#5354]) +14 other tests skip
[226]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_frontbuffer_tracking@psr-2p-primscrn-spr-indfb-draw-mmap-cpu.html
* igt@kms_frontbuffer_tracking@psr-2p-scndscrn-pri-shrfb-draw-render:
- shard-tglu: NOTRUN -> [SKIP][227] +49 other tests skip
[227]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-pri-shrfb-draw-render.html
* igt@kms_frontbuffer_tracking@psr-indfb-scaledprimary:
- shard-dg2: NOTRUN -> [SKIP][228] ([i915#10433] / [i915#3458])
[228]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_frontbuffer_tracking@psr-indfb-scaledprimary.html
* igt@kms_hdr@bpc-switch-suspend:
- shard-dg2: [PASS][229] -> [SKIP][230] ([i915#3555] / [i915#8228])
[229]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-11/igt@kms_hdr@bpc-switch-suspend.html
[230]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-1/igt@kms_hdr@bpc-switch-suspend.html
* igt@kms_hdr@invalid-hdr:
- shard-rkl: NOTRUN -> [SKIP][231] ([i915#3555] / [i915#8228])
[231]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@kms_hdr@invalid-hdr.html
* igt@kms_hdr@static-swap:
- shard-tglu: NOTRUN -> [SKIP][232] ([i915#3555] / [i915#8228]) +1 other test skip
[232]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_hdr@static-swap.html
* igt@kms_joiner@basic-force-big-joiner:
- shard-rkl: NOTRUN -> [SKIP][233] ([i915#12388])
[233]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@kms_joiner@basic-force-big-joiner.html
* igt@kms_joiner@invalid-modeset-big-joiner:
- shard-rkl: NOTRUN -> [SKIP][234] ([i915#10656])
[234]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_joiner@invalid-modeset-big-joiner.html
- shard-tglu: NOTRUN -> [SKIP][235] ([i915#10656])
[235]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_joiner@invalid-modeset-big-joiner.html
* igt@kms_joiner@invalid-modeset-ultra-joiner:
- shard-tglu: NOTRUN -> [SKIP][236] ([i915#12339])
[236]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-8/igt@kms_joiner@invalid-modeset-ultra-joiner.html
- shard-rkl: NOTRUN -> [SKIP][237] ([i915#12339])
[237]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@kms_joiner@invalid-modeset-ultra-joiner.html
* igt@kms_joiner@switch-modeset-ultra-joiner-big-joiner:
- shard-dg2: NOTRUN -> [SKIP][238] ([i915#13522])
[238]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_joiner@switch-modeset-ultra-joiner-big-joiner.html
- shard-rkl: NOTRUN -> [SKIP][239] ([i915#13522])
[239]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-6/igt@kms_joiner@switch-modeset-ultra-joiner-big-joiner.html
* igt@kms_multipipe_modeset@basic-max-pipe-crc-check:
- shard-rkl: NOTRUN -> [SKIP][240] ([i915#4816])
[240]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@kms_multipipe_modeset@basic-max-pipe-crc-check.html
* igt@kms_plane_scaling@plane-downscale-factor-0-25-with-rotation:
- shard-dg2: NOTRUN -> [SKIP][241] ([i915#12247] / [i915#9423])
[241]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_plane_scaling@plane-downscale-factor-0-25-with-rotation.html
- shard-rkl: NOTRUN -> [SKIP][242] ([i915#12247]) +9 other tests skip
[242]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-6/igt@kms_plane_scaling@plane-downscale-factor-0-25-with-rotation.html
* igt@kms_plane_scaling@plane-downscale-factor-0-25-with-rotation@pipe-d:
- shard-dg2: NOTRUN -> [SKIP][243] ([i915#12247]) +3 other tests skip
[243]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_plane_scaling@plane-downscale-factor-0-25-with-rotation@pipe-d.html
* igt@kms_plane_scaling@plane-scaler-unity-scaling-with-rotation@pipe-a:
- shard-tglu-1: NOTRUN -> [SKIP][244] ([i915#12247]) +4 other tests skip
[244]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_plane_scaling@plane-scaler-unity-scaling-with-rotation@pipe-a.html
* igt@kms_plane_scaling@planes-unity-scaling-downscale-factor-0-25:
- shard-dg2-9: NOTRUN -> [SKIP][245] ([i915#12247] / [i915#6953] / [i915#9423])
[245]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_plane_scaling@planes-unity-scaling-downscale-factor-0-25.html
* igt@kms_plane_scaling@planes-unity-scaling-downscale-factor-0-25@pipe-c:
- shard-dg2-9: NOTRUN -> [SKIP][246] ([i915#12247]) +3 other tests skip
[246]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_plane_scaling@planes-unity-scaling-downscale-factor-0-25@pipe-c.html
* igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-25:
- shard-mtlp: NOTRUN -> [SKIP][247] ([i915#12247] / [i915#3555])
[247]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-6/igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-25.html
* igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-25@pipe-b:
- shard-mtlp: NOTRUN -> [SKIP][248] ([i915#12247]) +3 other tests skip
[248]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-6/igt@kms_plane_scaling@planes-upscale-20x20-downscale-factor-0-25@pipe-b.html
* igt@kms_pm_backlight@basic-brightness:
- shard-rkl: NOTRUN -> [SKIP][249] ([i915#5354])
[249]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@kms_pm_backlight@basic-brightness.html
* igt@kms_pm_backlight@brightness-with-dpms:
- shard-rkl: NOTRUN -> [SKIP][250] ([i915#12343])
[250]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@kms_pm_backlight@brightness-with-dpms.html
* igt@kms_pm_backlight@fade-with-suspend:
- shard-tglu: NOTRUN -> [SKIP][251] ([i915#9812]) +1 other test skip
[251]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_pm_backlight@fade-with-suspend.html
* igt@kms_pm_rpm@basic-rte:
- shard-dg1: [PASS][252] -> [DMESG-WARN][253] ([i915#4423]) +1 other test dmesg-warn
[252]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg1-17/igt@kms_pm_rpm@basic-rte.html
[253]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg1-14/igt@kms_pm_rpm@basic-rte.html
* igt@kms_pm_rpm@dpms-mode-unset-non-lpsp:
- shard-dg2: NOTRUN -> [SKIP][254] ([i915#9519])
[254]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_pm_rpm@dpms-mode-unset-non-lpsp.html
* igt@kms_pm_rpm@dpms-non-lpsp:
- shard-tglu: NOTRUN -> [SKIP][255] ([i915#9519]) +1 other test skip
[255]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_pm_rpm@dpms-non-lpsp.html
* igt@kms_pm_rpm@modeset-lpsp-stress-no-wait:
- shard-dg2-9: NOTRUN -> [SKIP][256] ([i915#9519])
[256]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_pm_rpm@modeset-lpsp-stress-no-wait.html
- shard-rkl: [PASS][257] -> [SKIP][258] ([i915#9519])
[257]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-rkl-4/igt@kms_pm_rpm@modeset-lpsp-stress-no-wait.html
[258]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-3/igt@kms_pm_rpm@modeset-lpsp-stress-no-wait.html
* igt@kms_pm_rpm@modeset-non-lpsp-stress-no-wait:
- shard-dg2: [PASS][259] -> [SKIP][260] ([i915#9519])
[259]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-2/igt@kms_pm_rpm@modeset-non-lpsp-stress-no-wait.html
[260]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_pm_rpm@modeset-non-lpsp-stress-no-wait.html
* igt@kms_prime@basic-crc-vgem@second-to-first:
- shard-tglu-1: NOTRUN -> [ABORT][261] ([i915#13908]) +1 other test abort
[261]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_prime@basic-crc-vgem@second-to-first.html
* igt@kms_psr2_sf@fbc-psr2-cursor-plane-move-continuous-sf:
- shard-tglu: NOTRUN -> [SKIP][262] ([i915#11520]) +5 other tests skip
[262]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_psr2_sf@fbc-psr2-cursor-plane-move-continuous-sf.html
* igt@kms_psr2_sf@fbc-psr2-cursor-plane-update-sf:
- shard-dg2: NOTRUN -> [SKIP][263] ([i915#11520]) +2 other tests skip
[263]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-7/igt@kms_psr2_sf@fbc-psr2-cursor-plane-update-sf.html
* igt@kms_psr2_sf@fbc-psr2-overlay-primary-update-sf-dmg-area:
- shard-tglu-1: NOTRUN -> [SKIP][264] ([i915#11520]) +3 other tests skip
[264]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_psr2_sf@fbc-psr2-overlay-primary-update-sf-dmg-area.html
* igt@kms_psr2_sf@pr-cursor-plane-move-continuous-exceed-sf:
- shard-mtlp: NOTRUN -> [SKIP][265] ([i915#12316]) +1 other test skip
[265]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_psr2_sf@pr-cursor-plane-move-continuous-exceed-sf.html
* igt@kms_psr2_sf@pr-cursor-plane-move-continuous-sf:
- shard-dg2-9: NOTRUN -> [SKIP][266] ([i915#11520]) +3 other tests skip
[266]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_psr2_sf@pr-cursor-plane-move-continuous-sf.html
* igt@kms_psr2_sf@psr2-overlay-plane-update-sf-dmg-area:
- shard-rkl: NOTRUN -> [SKIP][267] ([i915#11520]) +13 other tests skip
[267]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_psr2_sf@psr2-overlay-plane-update-sf-dmg-area.html
* igt@kms_psr2_su@frontbuffer-xrgb8888:
- shard-tglu-1: NOTRUN -> [SKIP][268] ([i915#9683])
[268]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_psr2_su@frontbuffer-xrgb8888.html
* igt@kms_psr2_su@page_flip-p010:
- shard-rkl: NOTRUN -> [SKIP][269] ([i915#9683])
[269]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@kms_psr2_su@page_flip-p010.html
* igt@kms_psr@fbc-pr-primary-blt:
- shard-dg2: NOTRUN -> [SKIP][270] ([i915#1072] / [i915#9732]) +4 other tests skip
[270]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_psr@fbc-pr-primary-blt.html
* igt@kms_psr@fbc-pr-sprite-blt:
- shard-dg2-9: NOTRUN -> [SKIP][271] ([i915#1072] / [i915#9732]) +9 other tests skip
[271]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_psr@fbc-pr-sprite-blt.html
* igt@kms_psr@fbc-psr-cursor-mmap-cpu@edp-1:
- shard-mtlp: NOTRUN -> [SKIP][272] ([i915#9688]) +2 other tests skip
[272]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_psr@fbc-psr-cursor-mmap-cpu@edp-1.html
* igt@kms_psr@psr-sprite-plane-move:
- shard-rkl: NOTRUN -> [SKIP][273] ([i915#1072] / [i915#9732]) +29 other tests skip
[273]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_psr@psr-sprite-plane-move.html
* igt@kms_psr@psr2-cursor-plane-onoff:
- shard-tglu: NOTRUN -> [SKIP][274] ([i915#9732]) +12 other tests skip
[274]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_psr@psr2-cursor-plane-onoff.html
* igt@kms_psr@psr2-primary-mmap-cpu:
- shard-tglu-1: NOTRUN -> [SKIP][275] ([i915#9732]) +7 other tests skip
[275]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_psr@psr2-primary-mmap-cpu.html
* igt@kms_psr_stress_test@flip-primary-invalidate-overlay:
- shard-dg2-9: NOTRUN -> [SKIP][276] ([i915#9685])
[276]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_psr_stress_test@flip-primary-invalidate-overlay.html
* igt@kms_rotation_crc@primary-yf-tiled-reflect-x-0:
- shard-dg2-9: NOTRUN -> [SKIP][277] ([i915#5190])
[277]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_rotation_crc@primary-yf-tiled-reflect-x-0.html
* igt@kms_rotation_crc@primary-yf-tiled-reflect-x-180:
- shard-rkl: NOTRUN -> [SKIP][278] ([i915#5289]) +1 other test skip
[278]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_rotation_crc@primary-yf-tiled-reflect-x-180.html
- shard-tglu-1: NOTRUN -> [SKIP][279] ([i915#5289])
[279]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_rotation_crc@primary-yf-tiled-reflect-x-180.html
- shard-mtlp: NOTRUN -> [SKIP][280] ([i915#5289])
[280]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_rotation_crc@primary-yf-tiled-reflect-x-180.html
* igt@kms_rotation_crc@sprite-rotation-270:
- shard-dg2: NOTRUN -> [SKIP][281] ([i915#12755])
[281]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_rotation_crc@sprite-rotation-270.html
* igt@kms_scaling_modes@scaling-mode-full:
- shard-tglu: NOTRUN -> [SKIP][282] ([i915#3555]) +2 other tests skip
[282]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_scaling_modes@scaling-mode-full.html
* igt@kms_tiled_display@basic-test-pattern:
- shard-dg2: NOTRUN -> [SKIP][283] ([i915#8623])
[283]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_tiled_display@basic-test-pattern.html
- shard-rkl: NOTRUN -> [SKIP][284] ([i915#8623])
[284]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-6/igt@kms_tiled_display@basic-test-pattern.html
* igt@kms_vrr@flip-basic:
- shard-tglu-1: NOTRUN -> [SKIP][285] ([i915#3555])
[285]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_vrr@flip-basic.html
* igt@kms_vrr@flip-basic-fastset:
- shard-tglu: NOTRUN -> [SKIP][286] ([i915#9906])
[286]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-4/igt@kms_vrr@flip-basic-fastset.html
* igt@kms_vrr@lobf:
- shard-rkl: NOTRUN -> [SKIP][287] ([i915#11920])
[287]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_vrr@lobf.html
- shard-tglu-1: NOTRUN -> [SKIP][288] ([i915#11920])
[288]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@kms_vrr@lobf.html
- shard-mtlp: NOTRUN -> [SKIP][289] ([i915#11920])
[289]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@kms_vrr@lobf.html
* igt@kms_vrr@seamless-rr-switch-virtual:
- shard-dg2-9: NOTRUN -> [SKIP][290] ([i915#9906])
[290]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@kms_vrr@seamless-rr-switch-virtual.html
* igt@kms_vrr@seamless-rr-switch-vrr:
- shard-dg2: NOTRUN -> [SKIP][291] ([i915#9906])
[291]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_vrr@seamless-rr-switch-vrr.html
- shard-rkl: NOTRUN -> [SKIP][292] ([i915#9906]) +2 other tests skip
[292]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-6/igt@kms_vrr@seamless-rr-switch-vrr.html
* igt@kms_writeback@writeback-fb-id:
- shard-rkl: NOTRUN -> [SKIP][293] ([i915#2437])
[293]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@kms_writeback@writeback-fb-id.html
* igt@perf@gen8-unprivileged-single-ctx-counters:
- shard-rkl: NOTRUN -> [SKIP][294] ([i915#2436])
[294]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@perf@gen8-unprivileged-single-ctx-counters.html
* igt@perf@mi-rpc:
- shard-rkl: NOTRUN -> [SKIP][295] ([i915#2434])
[295]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-4/igt@perf@mi-rpc.html
* igt@perf_pmu@event-wait@rcs0:
- shard-dg2-9: NOTRUN -> [SKIP][296] +11 other tests skip
[296]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@perf_pmu@event-wait@rcs0.html
* igt@perf_pmu@module-unload:
- shard-mtlp: [PASS][297] -> [INCOMPLETE][298] ([i915#13520])
[297]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-mtlp-8/igt@perf_pmu@module-unload.html
[298]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-8/igt@perf_pmu@module-unload.html
* igt@prime_vgem@basic-fence-mmap:
- shard-dg2: NOTRUN -> [SKIP][299] ([i915#3708] / [i915#4077])
[299]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@prime_vgem@basic-fence-mmap.html
* igt@prime_vgem@basic-write:
- shard-dg2-9: NOTRUN -> [SKIP][300] ([i915#3291] / [i915#3708])
[300]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@prime_vgem@basic-write.html
* igt@prime_vgem@fence-flip-hang:
- shard-mtlp: NOTRUN -> [SKIP][301] ([i915#3708])
[301]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-6/igt@prime_vgem@fence-flip-hang.html
* igt@sriov_basic@enable-vfs-autoprobe-off:
- shard-rkl: NOTRUN -> [SKIP][302] ([i915#9917])
[302]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@sriov_basic@enable-vfs-autoprobe-off.html
* igt@sriov_basic@enable-vfs-autoprobe-off@numvfs-6:
- shard-mtlp: NOTRUN -> [FAIL][303] ([i915#12910]) +9 other tests fail
[303]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@sriov_basic@enable-vfs-autoprobe-off@numvfs-6.html
* igt@sriov_basic@enable-vfs-autoprobe-off@numvfs-all:
- shard-tglu-1: NOTRUN -> [FAIL][304] ([i915#12910]) +9 other tests fail
[304]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-1/igt@sriov_basic@enable-vfs-autoprobe-off@numvfs-all.html
* igt@sriov_basic@enable-vfs-autoprobe-on:
- shard-dg2-9: NOTRUN -> [SKIP][305] ([i915#9917])
[305]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-9/igt@sriov_basic@enable-vfs-autoprobe-on.html
#### Possible fixes ####
* igt@gem_eio@in-flight-10ms:
- shard-mtlp: [ABORT][306] ([i915#13193]) -> [PASS][307] +1 other test pass
[306]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-mtlp-7/igt@gem_eio@in-flight-10ms.html
[307]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-mtlp-5/igt@gem_eio@in-flight-10ms.html
* igt@gem_lmem_swapping@smem-oom@lmem0:
- shard-dg1: [TIMEOUT][308] ([i915#5493]) -> [PASS][309] +1 other test pass
[308]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg1-17/igt@gem_lmem_swapping@smem-oom@lmem0.html
[309]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg1-13/igt@gem_lmem_swapping@smem-oom@lmem0.html
* igt@gem_pxp@verify-pxp-stale-buf-execution:
- shard-rkl: [TIMEOUT][310] ([i915#12917] / [i915#12964]) -> [PASS][311]
[310]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-rkl-5/igt@gem_pxp@verify-pxp-stale-buf-execution.html
[311]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@gem_pxp@verify-pxp-stale-buf-execution.html
* igt@kms_cursor_crc@cursor-sliding-256x85:
- shard-tglu: [FAIL][312] ([i915#13566]) -> [PASS][313] +1 other test pass
[312]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-tglu-5/igt@kms_cursor_crc@cursor-sliding-256x85.html
[313]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-10/igt@kms_cursor_crc@cursor-sliding-256x85.html
* igt@kms_cursor_legacy@torture-bo@pipe-a:
- shard-rkl: [DMESG-WARN][314] ([i915#12964]) -> [PASS][315] +5 other tests pass
[314]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-rkl-5/igt@kms_cursor_legacy@torture-bo@pipe-a.html
[315]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-8/igt@kms_cursor_legacy@torture-bo@pipe-a.html
* igt@kms_dp_linktrain_fallback@dp-fallback:
- shard-dg2: [SKIP][316] ([i915#13707]) -> [PASS][317]
[316]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-3/igt@kms_dp_linktrain_fallback@dp-fallback.html
[317]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-10/igt@kms_dp_linktrain_fallback@dp-fallback.html
* igt@kms_flip@2x-blocking-wf_vblank@ab-vga1-hdmi-a1:
- shard-snb: [FAIL][318] ([i915#11832]) -> [PASS][319] +1 other test pass
[318]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-snb1/igt@kms_flip@2x-blocking-wf_vblank@ab-vga1-hdmi-a1.html
[319]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-snb6/igt@kms_flip@2x-blocking-wf_vblank@ab-vga1-hdmi-a1.html
* igt@kms_flip@flip-vs-expired-vblank@b-hdmi-a1:
- shard-glk: [FAIL][320] ([i915#13027]) -> [PASS][321] +1 other test pass
[320]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-glk1/igt@kms_flip@flip-vs-expired-vblank@b-hdmi-a1.html
[321]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-glk5/igt@kms_flip@flip-vs-expired-vblank@b-hdmi-a1.html
* igt@kms_hdr@invalid-metadata-sizes:
- shard-dg2: [SKIP][322] ([i915#3555] / [i915#8228]) -> [PASS][323] +2 other tests pass
[322]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-3/igt@kms_hdr@invalid-metadata-sizes.html
[323]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-10/igt@kms_hdr@invalid-metadata-sizes.html
* igt@kms_joiner@invalid-modeset-force-big-joiner:
- shard-dg2: [SKIP][324] ([i915#12388]) -> [PASS][325]
[324]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-1/igt@kms_joiner@invalid-modeset-force-big-joiner.html
[325]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-10/igt@kms_joiner@invalid-modeset-force-big-joiner.html
* igt@kms_plane_scaling@intel-max-src-size:
- shard-dg2: [SKIP][326] ([i915#6953] / [i915#9423]) -> [PASS][327]
[326]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-2/igt@kms_plane_scaling@intel-max-src-size.html
[327]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-11/igt@kms_plane_scaling@intel-max-src-size.html
* igt@kms_pm_dc@dc6-dpms:
- shard-tglu: [FAIL][328] ([i915#9295]) -> [PASS][329]
[328]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-tglu-5/igt@kms_pm_dc@dc6-dpms.html
[329]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-tglu-10/igt@kms_pm_dc@dc6-dpms.html
* igt@kms_pm_rpm@dpms-lpsp:
- shard-rkl: [SKIP][330] ([i915#9519]) -> [PASS][331]
[330]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-rkl-1/igt@kms_pm_rpm@dpms-lpsp.html
[331]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@kms_pm_rpm@dpms-lpsp.html
* igt@kms_vblank@query-idle-hang:
- shard-dg1: [DMESG-WARN][332] ([i915#4423]) -> [PASS][333]
[332]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg1-16/igt@kms_vblank@query-idle-hang.html
[333]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg1-16/igt@kms_vblank@query-idle-hang.html
* igt@kms_vrr@negative-basic:
- shard-dg2: [SKIP][334] ([i915#3555] / [i915#9906]) -> [PASS][335]
[334]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-1/igt@kms_vrr@negative-basic.html
[335]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-11/igt@kms_vrr@negative-basic.html
* igt@perf_pmu@most-busy-idle-check-all@rcs0:
- shard-rkl: [FAIL][336] ([i915#4349]) -> [PASS][337] +3 other tests pass
[336]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-rkl-1/igt@perf_pmu@most-busy-idle-check-all@rcs0.html
[337]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-rkl-7/igt@perf_pmu@most-busy-idle-check-all@rcs0.html
#### Warnings ####
* igt@gem_eio@in-flight-suspend:
- shard-glk: [INCOMPLETE][338] ([i915#13197] / [i915#13390]) -> [INCOMPLETE][339] ([i915#13390])
[338]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-glk5/igt@gem_eio@in-flight-suspend.html
[339]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-glk9/igt@gem_eio@in-flight-suspend.html
* igt@kms_big_fb@x-tiled-16bpp-rotate-270:
- shard-dg1: [SKIP][340] ([i915#3638] / [i915#4423]) -> [SKIP][341] ([i915#3638])
[340]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg1-16/igt@kms_big_fb@x-tiled-16bpp-rotate-270.html
[341]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg1-16/igt@kms_big_fb@x-tiled-16bpp-rotate-270.html
* igt@kms_ccs@crc-primary-basic-4-tiled-lnl-ccs:
- shard-dg1: [SKIP][342] ([i915#12313] / [i915#4423]) -> [SKIP][343] ([i915#12313])
[342]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg1-16/igt@kms_ccs@crc-primary-basic-4-tiled-lnl-ccs.html
[343]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg1-16/igt@kms_ccs@crc-primary-basic-4-tiled-lnl-ccs.html
* igt@kms_content_protection@atomic-dpms:
- shard-dg2: [SKIP][344] ([i915#7118] / [i915#9424]) -> [FAIL][345] ([i915#7173])
[344]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-7/igt@kms_content_protection@atomic-dpms.html
[345]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-11/igt@kms_content_protection@atomic-dpms.html
* igt@kms_content_protection@type1:
- shard-dg2: [SKIP][346] ([i915#7118] / [i915#7162] / [i915#9424]) -> [SKIP][347] ([i915#7118] / [i915#9424])
[346]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-10/igt@kms_content_protection@type1.html
[347]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-2/igt@kms_content_protection@type1.html
* igt@kms_content_protection@uevent:
- shard-dg2: [FAIL][348] ([i915#1339] / [i915#7173]) -> [SKIP][349] ([i915#7118] / [i915#9424])
[348]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-11/igt@kms_content_protection@uevent.html
[349]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-7/igt@kms_content_protection@uevent.html
* igt@kms_flip@2x-flip-vs-suspend-interruptible:
- shard-glk: [INCOMPLETE][350] ([i915#12314] / [i915#12745] / [i915#4839]) -> [INCOMPLETE][351] ([i915#12745] / [i915#4839])
[350]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-glk8/igt@kms_flip@2x-flip-vs-suspend-interruptible.html
[351]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-glk4/igt@kms_flip@2x-flip-vs-suspend-interruptible.html
* igt@kms_flip@2x-flip-vs-suspend-interruptible@ab-hdmi-a1-hdmi-a2:
- shard-glk: [INCOMPLETE][352] ([i915#12314] / [i915#4839]) -> [INCOMPLETE][353] ([i915#4839])
[352]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-glk8/igt@kms_flip@2x-flip-vs-suspend-interruptible@ab-hdmi-a1-hdmi-a2.html
[353]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-glk4/igt@kms_flip@2x-flip-vs-suspend-interruptible@ab-hdmi-a1-hdmi-a2.html
* igt@kms_flip@flip-vs-suspend-interruptible:
- shard-glk: [INCOMPLETE][354] ([i915#12745] / [i915#4839]) -> [INCOMPLETE][355] ([i915#12314] / [i915#12745] / [i915#4839])
[354]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-glk5/igt@kms_flip@flip-vs-suspend-interruptible.html
[355]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-glk8/igt@kms_flip@flip-vs-suspend-interruptible.html
* igt@kms_flip@flip-vs-suspend-interruptible@a-hdmi-a1:
- shard-glk: [INCOMPLETE][356] ([i915#12745]) -> [INCOMPLETE][357] ([i915#12314] / [i915#12745])
[356]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-glk5/igt@kms_flip@flip-vs-suspend-interruptible@a-hdmi-a1.html
[357]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-glk8/igt@kms_flip@flip-vs-suspend-interruptible@a-hdmi-a1.html
* igt@kms_frontbuffer_tracking@fbcpsr-shrfb-scaledprimary:
- shard-dg2: [SKIP][358] ([i915#10433] / [i915#3458]) -> [SKIP][359] ([i915#3458]) +3 other tests skip
[358]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-4/igt@kms_frontbuffer_tracking@fbcpsr-shrfb-scaledprimary.html
[359]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-3/igt@kms_frontbuffer_tracking@fbcpsr-shrfb-scaledprimary.html
* igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-blt:
- shard-dg2: [SKIP][360] ([i915#3458]) -> [SKIP][361] ([i915#10433] / [i915#3458])
[360]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg2-2/igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-blt.html
[361]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg2-4/igt@kms_frontbuffer_tracking@psr-1p-offscren-pri-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-mmap-cpu:
- shard-dg1: [SKIP][362] ([i915#4423]) -> [SKIP][363]
[362]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_16301/shard-dg1-15/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-mmap-cpu.html
[363]: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/shard-dg1-14/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-spr-indfb-draw-mmap-cpu.html
[i915#10055]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10055
[i915#10307]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10307
[i915#10433]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10433
[i915#10434]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10434
[i915#10656]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/10656
[i915#1072]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/1072
[i915#11078]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11078
[i915#11151]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11151
[i915#11441]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11441
[i915#11520]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11520
[i915#11681]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11681
[i915#11713]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11713
[i915#11832]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11832
[i915#11920]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/11920
[i915#12061]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12061
[i915#12247]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12247
[i915#12313]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12313
[i915#12314]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12314
[i915#12316]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12316
[i915#12339]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12339
[i915#12343]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12343
[i915#12388]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12388
[i915#12392]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12392
[i915#12455]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12455
[i915#12745]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12745
[i915#12755]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12755
[i915#12805]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12805
[i915#12910]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12910
[i915#12917]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12917
[i915#12964]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/12964
[i915#13027]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13027
[i915#13046]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13046
[i915#13049]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13049
[i915#13193]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13193
[i915#13197]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13197
[i915#13304]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13304
[i915#13356]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13356
[i915#1339]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/1339
[i915#13390]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13390
[i915#13520]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13520
[i915#13522]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13522
[i915#13562]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13562
[i915#13566]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13566
[i915#13665]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13665
[i915#13691]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13691
[i915#13707]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13707
[i915#13749]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13749
[i915#13781]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13781
[i915#13783]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13783
[i915#13790]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13790
[i915#13820]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13820
[i915#13908]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/13908
[i915#1769]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/1769
[i915#1825]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/1825
[i915#1839]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/1839
[i915#2346]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2346
[i915#2434]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2434
[i915#2436]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2436
[i915#2437]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2437
[i915#2527]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2527
[i915#2587]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2587
[i915#2672]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2672
[i915#2681]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2681
[i915#280]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/280
[i915#284]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/284
[i915#2856]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/2856
[i915#3023]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3023
[i915#3116]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3116
[i915#3281]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3281
[i915#3282]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3282
[i915#3291]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3291
[i915#3297]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3297
[i915#3299]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3299
[i915#3458]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3458
[i915#3469]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3469
[i915#3539]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3539
[i915#3555]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3555
[i915#3591]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3591
[i915#3637]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3637
[i915#3638]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3638
[i915#3708]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3708
[i915#3742]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3742
[i915#3840]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/3840
[i915#4036]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4036
[i915#4077]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4077
[i915#4079]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4079
[i915#4083]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4083
[i915#4103]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4103
[i915#4270]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4270
[i915#4349]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4349
[i915#4423]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4423
[i915#4525]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4525
[i915#4537]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4537
[i915#4538]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4538
[i915#4613]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4613
[i915#4771]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4771
[i915#4812]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4812
[i915#4816]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4816
[i915#4817]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4817
[i915#4839]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4839
[i915#4852]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4852
[i915#4860]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4860
[i915#4880]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4880
[i915#4881]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4881
[i915#4885]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4885
[i915#5138]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5138
[i915#5190]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5190
[i915#5286]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5286
[i915#5289]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5289
[i915#5354]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5354
[i915#5439]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5439
[i915#5493]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5493
[i915#5784]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5784
[i915#5956]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/5956
[i915#6095]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/6095
[i915#6344]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/6344
[i915#6590]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/6590
[i915#6621]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/6621
[i915#6944]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/6944
[i915#6953]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/6953
[i915#7116]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/7116
[i915#7118]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/7118
[i915#7162]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/7162
[i915#7173]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/7173
[i915#7582]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/7582
[i915#7697]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/7697
[i915#7707]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/7707
[i915#7828]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/7828
[i915#7975]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/7975
[i915#8228]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8228
[i915#8399]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8399
[i915#8411]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8411
[i915#8414]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8414
[i915#8428]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8428
[i915#8555]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8555
[i915#8562]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8562
[i915#8623]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8623
[i915#8708]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8708
[i915#8709]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8709
[i915#8812]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8812
[i915#8814]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/8814
[i915#9067]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9067
[i915#9295]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9295
[i915#9311]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9311
[i915#9318]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9318
[i915#9323]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9323
[i915#9337]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9337
[i915#9423]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9423
[i915#9424]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9424
[i915#9519]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9519
[i915#9531]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9531
[i915#9683]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9683
[i915#9685]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9685
[i915#9688]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9688
[i915#9723]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9723
[i915#9732]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9732
[i915#9809]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9809
[i915#9812]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9812
[i915#9906]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9906
[i915#9917]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9917
[i915#9934]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/9934
Build changes
-------------
* Linux: CI_DRM_16301 -> Patchwork_146088v2
CI-20190529: 20190529
CI_DRM_16301: a123616cf3afc205116835d8b083f48272583dcd @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_8278: b6673db372bd8987f65948d3a97f8dcd2ef42b01 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
Patchwork_146088v2: a123616cf3afc205116835d8b083f48272583dcd @ git://anongit.freedesktop.org/gfx-ci/linux
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_146088v2/index.html
[-- Attachment #2: Type: text/html, Size: 129859 bytes --]
^ permalink raw reply [flat|nested] 26+ messages in thread* Re: [PATCH v7 0/5] bits: Fixed-type GENMASK_U*() and BIT_U*()
2025-03-22 9:23 ` Vincent Mailhol via B4 Relay
` (9 preceding siblings ...)
(?)
@ 2025-03-24 14:28 ` Yury Norov
2025-03-24 16:23 ` Vincent Mailhol
-1 siblings, 1 reply; 26+ messages in thread
From: Yury Norov @ 2025-03-24 14:28 UTC (permalink / raw)
To: mailhol.vincent
Cc: Lucas De Marchi, Rasmus Villemoes, Jani Nikula, Joonas Lahtinen,
Rodrigo Vivi, Tvrtko Ursulin, David Airlie, Simona Vetter,
Andrew Morton, linux-kernel, intel-gfx, dri-devel, Andi Shyti,
David Laight, Andy Shevchenko, Jani Nikula
On Sat, Mar 22, 2025 at 06:23:11PM +0900, Vincent Mailhol via B4 Relay wrote:
> Introduce some fixed width variant of the GENMASK() and the BIT()
> macros in bits.h. Note that the main goal is not to get the correct
> type, but rather to enforce more checks at compile time. For example:
You say this, and then typecast both BIT and GENMASK. This may confuse
readers. Maybe add few words about promotion rules in C standard, or
just drop this note entirely? Doesn't require new submission, of
course.
> GENMASK_U16(16, 0)
>
> will raise a build bug.
>
> This series is a continuation of:
>
> https://lore.kernel.org/intel-xe/20240208074521.577076-1-lucas.demarchi@intel.com
>
> from Lucas De Marchi. Above series is one year old. I really think
> that this was a good idea and I do not want this series to die. So I
> am volunteering to revive it.
>
> Meanwhile, many changes occurred in bits.h. The most significant
> change is that __GENMASK() was moved to the uapi headers. For this
> reason, a new GENMASK_TYPE() is introduced instead and the uapi
> __GENMASK() is left untouched.
>
> Finally, I do not think it makes sense to expose the fixed width
> variants to the asm. The fixed width integers type are a C concept. So
> the GENMASK_U*() are only visible to the non-asm code. For asm, the
> long and long long variants seems sufficient.
>
> This series does not modify the actual GENMASK(), GENMASK_ULL() and
> GENMASK_U128(). A consolidation of the existing genmasks will be
> proposed later on in a separate series.
>
> As requested, here are the bloat-o-meter stats:
>
> $ ./scripts/bloat-o-meter vmlinux_before.o vmlinux_after.o
> add/remove: 0/0 grow/shrink: 0/0 up/down: 0/0 (0)
> Function old new delta
> Total: Before=22723481, After=22723481, chg +0.00%
>
> (done with GCC 12.4.1 on an x86_64 defconfig)
>
> --
> 2.43.0
>
> ---
> Changes from v6:
>
> - Split the series in two: this series leave any existing GENMASK*()
> unmodified. The consolidation will be done in a separate series.
>
> - Collect some Reviewed-by tag.
>
> - Address miscellaneous nitpick on the code comments and the line
> wrapping (details in each patch).
>
> - Link to v6: https://lore.kernel.org/r/20250308-fixed-type-genmasks-v6-0-f59315e73c29@wanadoo.fr
>
> Changes from v5:
>
> - Update the cover letter message. I was still refering to
> GENMASK_t() instead of GENMASK_TYPE().
>
> - Add a comment in the cover letter to explain that a common
> GENMASK_TYPE() for C and asm wouldn't allow to generate the u128
> variant.
>
> - Restore the comment saying that BUILD_BUG_ON() is not available in
> asm code.
>
> - Add a FIXME message to highlight the absence of the asm GENMASK*()
> unit tests.
>
> - Use git's histogram diff algorithm
>
> - Link to v5: https://lore.kernel.org/r/20250306-fixed-type-genmasks-v5-0-b443e9dcba63@wanadoo.fr
>
> Changes from v4:
>
> - Rebase on https://github.com/norov/linux/tree/bitmap-for-next
>
> - Rename GENMASK_t() to GENMASK_TYPE()
>
> - First patch of v4 (the typo fix 'init128' -> 'int128') is removed
> because it was resent separately in:
> https://lore.kernel.org/all/20250305-fix_init128_typo-v1-1-cbe5b8e54e7d@wanadoo.fr
>
> - Replace the (t)~ULL(0) by type_max(t). This way, GENMASK_TYPE()
> can now be used to generate GENMASK_U128().
>
> - Get rid of the unsigned int cast for the U8 and U16 variants.
>
> - Add the BIT_TYPE() helper macro.
>
> - Link to v4: https://lore.kernel.org/r/20250305-fixed-type-genmasks-v4-0-1873dcdf6723@wanadoo.fr
>
> Changes from v3:
>
> - Rebase on v6.14-rc5
>
> - Fix a typo in GENMASK_U128() comment.
>
> - Split the asm and non-asm definition of
>
> - Replace ~0ULL by ~ULL(0)
>
> - Since v3, __GENMASK() was moved to the uapi and people started
> using directly. Introduce GENMASK_t() instead.
>
> - Link to v3: https://lore.kernel.org/intel-xe/20240208074521.577076-1-lucas.demarchi@intel.com
>
> Changes from v2:
>
> - Document both in commit message and code about the strict type
> checking and give examples how it´d break with invalid params.
>
> - Link to v2: https://lore.kernel.org/intel-xe/20240124050205.3646390-1-lucas.demarchi@intel.com
>
> Link to v1: https://lore.kernel.org/intel-xe/20230509051403.2748545-1-lucas.demarchi@intel.com
>
> ---
> Lucas De Marchi (3):
> bits: introduce fixed-type BIT_U*()
> drm/i915: Convert REG_GENMASK*() to fixed-width GENMASK_U*()
> test_bits: add tests for GENMASK_U*()
>
> Vincent Mailhol (2):
> bits: introduce fixed-type GENMASK_U*()
> test_bits: add tests for BIT_U*()
>
> drivers/gpu/drm/i915/i915_reg_defs.h | 108 ++++-------------------------------
> include/linux/bitops.h | 1 -
> include/linux/bits.h | 57 +++++++++++++++++-
> lib/test_bits.c | 30 ++++++++++
> 4 files changed, 96 insertions(+), 100 deletions(-)
> ---
> base-commit: e3f42c436d7e0cb432935fe3ae275dd8d9b60f71
> change-id: 20250228-fixed-type-genmasks-8d1a555f34e8
>
> Best regards,
> --
> Vincent Mailhol <mailhol.vincent@wanadoo.fr>
>
^ permalink raw reply [flat|nested] 26+ messages in thread* Re: [PATCH v7 0/5] bits: Fixed-type GENMASK_U*() and BIT_U*()
2025-03-24 14:28 ` [PATCH v7 0/5] bits: Fixed-type GENMASK_U*() and BIT_U*() Yury Norov
@ 2025-03-24 16:23 ` Vincent Mailhol
2025-03-25 15:23 ` Yury Norov
0 siblings, 1 reply; 26+ messages in thread
From: Vincent Mailhol @ 2025-03-24 16:23 UTC (permalink / raw)
To: Yury Norov
Cc: Lucas De Marchi, Rasmus Villemoes, Jani Nikula, Joonas Lahtinen,
Rodrigo Vivi, Tvrtko Ursulin, David Airlie, Simona Vetter,
Andrew Morton, linux-kernel, intel-gfx, dri-devel, Andi Shyti,
David Laight, Andy Shevchenko, Jani Nikula
On 24/03/2025 at 23:28, Yury Norov wrote:
> On Sat, Mar 22, 2025 at 06:23:11PM +0900, Vincent Mailhol via B4 Relay wrote:
>> Introduce some fixed width variant of the GENMASK() and the BIT()
>> macros in bits.h. Note that the main goal is not to get the correct
>> type, but rather to enforce more checks at compile time. For example:
>
> You say this, and then typecast both BIT and GENMASK. This may confuse
> readers. Maybe add few words about promotion rules in C standard, or
> just drop this note entirely? Doesn't require new submission, of
> course.
I do not want to into this level of details in the cover letter, so I
will remove. Instead, I can add below paragraph to the "bits: introduce
fixed-type GENMASK_U*()" patch:
The result is casted to the corresponding fixed width type. For
example, GENMASK_U8() returns an u8. Note that because of the C
promotion rules, GENMASK_U8() and GENMASK_U16() will immediately be
promoted to int if used in an expression. Regardless, the main goal is
not to get the correct type, but rather to enforce more checks at
compile time.
I staged this change in the v8 together with the other nitpicks from
Andy. If you want that v8, let me know, it is ready. If you are happy
enough with the v7 (and if it doesn't receive more comments), then go
with it!
Yours sincerely,
Vincent Mailhol
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v7 0/5] bits: Fixed-type GENMASK_U*() and BIT_U*()
2025-03-24 16:23 ` Vincent Mailhol
@ 2025-03-25 15:23 ` Yury Norov
2025-03-25 16:13 ` Vincent Mailhol
0 siblings, 1 reply; 26+ messages in thread
From: Yury Norov @ 2025-03-25 15:23 UTC (permalink / raw)
To: Vincent Mailhol
Cc: Lucas De Marchi, Rasmus Villemoes, Jani Nikula, Joonas Lahtinen,
Rodrigo Vivi, Tvrtko Ursulin, David Airlie, Simona Vetter,
Andrew Morton, linux-kernel, intel-gfx, dri-devel, Andi Shyti,
David Laight, Andy Shevchenko, Jani Nikula
On Tue, Mar 25, 2025 at 01:23:22AM +0900, Vincent Mailhol wrote:
> On 24/03/2025 at 23:28, Yury Norov wrote:
> > On Sat, Mar 22, 2025 at 06:23:11PM +0900, Vincent Mailhol via B4 Relay wrote:
> >> Introduce some fixed width variant of the GENMASK() and the BIT()
> >> macros in bits.h. Note that the main goal is not to get the correct
> >> type, but rather to enforce more checks at compile time. For example:
> >
> > You say this, and then typecast both BIT and GENMASK. This may confuse
> > readers. Maybe add few words about promotion rules in C standard, or
> > just drop this note entirely? Doesn't require new submission, of
> > course.
>
> I do not want to into this level of details in the cover letter, so I
> will remove. Instead, I can add below paragraph to the "bits: introduce
> fixed-type GENMASK_U*()" patch:
>
> The result is casted to the corresponding fixed width type. For
> example, GENMASK_U8() returns an u8. Note that because of the C
> promotion rules, GENMASK_U8() and GENMASK_U16() will immediately be
> promoted to int if used in an expression. Regardless, the main goal is
> not to get the correct type, but rather to enforce more checks at
> compile time.
>
> I staged this change in the v8 together with the other nitpicks from
> Andy. If you want that v8, let me know, it is ready. If you are happy
> enough with the v7 (and if it doesn't receive more comments), then go
> with it!
This series doesn't apply on 6.15-rc1 because test_bits.c has moved to
lib/tests. Can you please rebase your v8 and submit? I see no other
issues to merge it in bitmap-for-next.
Thanks,
Yury
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v7 0/5] bits: Fixed-type GENMASK_U*() and BIT_U*()
2025-03-25 15:23 ` Yury Norov
@ 2025-03-25 16:13 ` Vincent Mailhol
2025-03-25 16:30 ` Yury Norov
0 siblings, 1 reply; 26+ messages in thread
From: Vincent Mailhol @ 2025-03-25 16:13 UTC (permalink / raw)
To: Yury Norov
Cc: Lucas De Marchi, Rasmus Villemoes, Jani Nikula, Joonas Lahtinen,
Rodrigo Vivi, Tvrtko Ursulin, David Airlie, Simona Vetter,
Andrew Morton, linux-kernel, intel-gfx, dri-devel, Andi Shyti,
David Laight, Andy Shevchenko, Jani Nikula
On 26/03/2025 at 00:23, Yury Norov wrote:
> On Tue, Mar 25, 2025 at 01:23:22AM +0900, Vincent Mailhol wrote:
(...)
> This series doesn't apply on 6.15-rc1 because test_bits.c has moved to
> lib/tests. Can you please rebase your v8 and submit? I see no other
> issues to merge it in bitmap-for-next.
git was smart enough to rebase everything automatically!
Here is the v8 (which includes the other few nitpicks from you and Andy):
https://lore.kernel.org/all/20250326-fixed-type-genmasks-v8-0-24afed16ca00@wanadoo.fr/
Do you also want me to rebase the other series which consolidates the
GENMASK(), GENMASK_ULL() and GENMASK_U128() now? Or should I wait a while?
Yours sincerely,
Vincent Mailhol
^ permalink raw reply [flat|nested] 26+ messages in thread
* Re: [PATCH v7 0/5] bits: Fixed-type GENMASK_U*() and BIT_U*()
2025-03-25 16:13 ` Vincent Mailhol
@ 2025-03-25 16:30 ` Yury Norov
0 siblings, 0 replies; 26+ messages in thread
From: Yury Norov @ 2025-03-25 16:30 UTC (permalink / raw)
To: Vincent Mailhol
Cc: Lucas De Marchi, Rasmus Villemoes, Jani Nikula, Joonas Lahtinen,
Rodrigo Vivi, Tvrtko Ursulin, David Airlie, Simona Vetter,
Andrew Morton, linux-kernel, intel-gfx, dri-devel, Andi Shyti,
David Laight, Andy Shevchenko, Jani Nikula
On Wed, Mar 26, 2025 at 01:13:28AM +0900, Vincent Mailhol wrote:
> On 26/03/2025 at 00:23, Yury Norov wrote:
> > On Tue, Mar 25, 2025 at 01:23:22AM +0900, Vincent Mailhol wrote:
>
> (...)
>
> > This series doesn't apply on 6.15-rc1 because test_bits.c has moved to
> > lib/tests. Can you please rebase your v8 and submit? I see no other
> > issues to merge it in bitmap-for-next.
>
> git was smart enough to rebase everything automatically!
>
> Here is the v8 (which includes the other few nitpicks from you and Andy):
>
> https://lore.kernel.org/all/20250326-fixed-type-genmasks-v8-0-24afed16ca00@wanadoo.fr/
>
> Do you also want me to rebase the other series which consolidates the
> GENMASK(), GENMASK_ULL() and GENMASK_U128() now? Or should I wait a while?
Let's wait for feedback, especially from ARM folks.
^ permalink raw reply [flat|nested] 26+ messages in thread