* [igt-dev] [PATCH i-g-t v3] lib/i915: Add mmap_offset support
@ 2019-02-26 18:23 Antonio Argenziano
2019-02-26 18:29 ` Chris Wilson
2019-02-26 19:59 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork
0 siblings, 2 replies; 7+ messages in thread
From: Antonio Argenziano @ 2019-02-26 18:23 UTC (permalink / raw)
To: igt-dev; +Cc: Janulgue Abdiel, Matthew Auld
From: "Kalamarz, Lukasz" <lukasz.kalamarz@intel.com>
With recently proposed changes, IGT need to start supporting new
way of mmaping object, which will be used from now by default.
This patch modify gem_mmap_wc and gem_mmap functions to be
in sync with those changes.
v2:
- Fix IOCTL number. (Daniele)
- Move wrappers to new file. (Chris)
v3:
- Use mmap IOCTL for lower level wrappers. (Chris)
Signed-off-by: Lukasz Kalamarz <lukasz.kalamarz@intel.com>
Cc: Janulgue Abdiel <abdiel.janulgue@intel.com>
Cc: Matthew Auld <matthew.auld@intel.com>
Cc: Michal Winiarski <michal.winiarski@intel.com>
Cc: Antonio Argenziano <antonio.argenziano@intel.com>
Cc: Daniele Spurio Ceraolo <daniele.ceraolospurio@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Antonio Argenziano <antonio.argenziano@intel.com>
---
lib/i915/gem_mman.c | 125 ++++++++++++++++++++++++++++++++++++--------
lib/i915/gem_mman.h | 33 ++++++++++++
2 files changed, 136 insertions(+), 22 deletions(-)
diff --git a/lib/i915/gem_mman.c b/lib/i915/gem_mman.c
index 3cf9a6bb..f89d389e 100644
--- a/lib/i915/gem_mman.c
+++ b/lib/i915/gem_mman.c
@@ -106,36 +106,51 @@ bool gem_mmap__has_wc(int fd)
static int has_wc = -1;
if (has_wc == -1) {
- struct drm_i915_getparam gp;
- int mmap_version = -1;
- int gtt_version = -1;
has_wc = 0;
- memset(&gp, 0, sizeof(gp));
- gp.param = I915_PARAM_MMAP_GTT_VERSION;
- gp.value = >t_version;
- ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
-
- memset(&gp, 0, sizeof(gp));
- gp.param = I915_PARAM_MMAP_VERSION;
- gp.value = &mmap_version;
- ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
-
- /* Do we have the new mmap_ioctl with DOMAIN_WC? */
- if (mmap_version >= 1 && gtt_version >= 2) {
- struct drm_i915_gem_mmap arg;
+ /* Do we have the new mmap_offset ioctl? */
+ if (has_mmap_offset(fd)) {
+ struct local_drm_i915_gem_mmap_offset arg;
/* Does this device support wc-mmaps ? */
memset(&arg, 0, sizeof(arg));
arg.handle = gem_create(fd, 4096);
arg.offset = 0;
- arg.size = 4096;
- arg.flags = I915_MMAP_WC;
- has_wc = igt_ioctl(fd, DRM_IOCTL_I915_GEM_MMAP, &arg) == 0;
+ arg.flags = LOCAL_I915_MMAP_OFFSET_WC;
+ has_wc = igt_ioctl(fd, LOCAL_DRM_IOCTL_I915_GEM_MMAP_OFFSET, &arg) == 0;
gem_close(fd, arg.handle);
+ } else {
+ struct drm_i915_getparam gp;
+ int mmap_version = -1;
+ int gtt_version = -1;
+
+ memset(&gp, 0, sizeof(gp));
+ gp.param = I915_PARAM_MMAP_GTT_VERSION;
+ gp.value = >t_version;
+ ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
+
+ memset(&gp, 0, sizeof(gp));
+ gp.param = I915_PARAM_MMAP_VERSION;
+ gp.value = &mmap_version;
+ ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
+
+ /* Do we have the mmap_ioctl with DOMAIN_WC? */
+ if (mmap_version >= 1 && gtt_version >= 2) {
+ struct drm_i915_gem_mmap arg;
+
+ /* Does this device support wc-mmaps ? */
+ memset(&arg, 0, sizeof(arg));
+ arg.handle = gem_create(fd, 4096);
+ arg.offset = 0;
+ arg.size = 4096;
+ arg.flags = I915_MMAP_WC;
+ has_wc = igt_ioctl(fd, DRM_IOCTL_I915_GEM_MMAP, &arg) == 0;
+ gem_close(fd, arg.handle);
+ }
}
- errno = 0;
+
+ errno = 0;
}
return has_wc > 0;
@@ -211,7 +226,12 @@ void *__gem_mmap__wc(int fd, uint32_t handle, uint64_t offset, uint64_t size, un
*/
void *gem_mmap__wc(int fd, uint32_t handle, uint64_t offset, uint64_t size, unsigned prot)
{
- void *ptr = __gem_mmap__wc(fd, handle, offset, size, prot);
+ void *ptr;
+
+ ptr = __gem_mmap_offset(fd, handle, offset, size, prot, LOCAL_I915_MMAP_OFFSET_WC);
+ if (!ptr)
+ ptr = __gem_mmap__wc(fd, handle, offset, size, prot);
+
igt_assert(ptr);
return ptr;
}
@@ -248,7 +268,68 @@ void *__gem_mmap__cpu(int fd, uint32_t handle, uint64_t offset, uint64_t size, u
*/
void *gem_mmap__cpu(int fd, uint32_t handle, uint64_t offset, uint64_t size, unsigned prot)
{
- void *ptr = __gem_mmap__cpu(fd, handle, offset, size, prot);
+ void *ptr;
+
+ ptr = __gem_mmap_offset(fd, handle, offset, size, prot, LOCAL_I915_MMAP_OFFSET_WB);
+ if (!ptr)
+ ptr = __gem_mmap(fd, handle, offset, size, prot, 0);
+
igt_assert(ptr);
return ptr;
}
+
+bool has_mmap_offset(int fd)
+{
+ static int has_mmap_offset = -1;
+
+ if (has_mmap_offset == -1) {
+ struct drm_i915_getparam gp;
+
+ has_mmap_offset = 0;
+
+ memset(&gp, 0, sizeof(gp));
+ gp.param = 0x55; /* I915_PARAM_MMAP_OFFSET_VERSION */
+ gp.value = &has_mmap_offset;
+ ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
+ }
+
+ return has_mmap_offset > 0;
+}
+
+/**
+ * __gem_mmap_offset:
+ * @fd: open i915 drm file descriptor
+ * @handle: gem buffer object handle
+ * @offset: offset in the gem buffer of the mmap arena
+ * @size: size of the mmap arena
+ * @prot: memory protection bits as used by mmap()
+ * @flags: flags used to determine caching
+ *
+ * Similar to __gem_mmap but use MMAP_OFFSET IOCTL.
+ *
+ * Returns: A pointer to the created memory mapping, NULL on failure.
+ */
+void
+*__gem_mmap_offset(int fd, uint32_t handle, uint64_t offset, uint64_t size, unsigned int prot, uint64_t flags)
+{
+ struct local_drm_i915_gem_mmap_offset arg;
+ void *ptr;
+
+ memset(&arg, 0, sizeof(arg));
+ arg.handle = handle;
+ arg.offset = offset;
+ arg.flags = flags;
+
+ if (igt_ioctl(fd, LOCAL_DRM_IOCTL_I915_GEM_MMAP_OFFSET, &arg))
+ return NULL;
+
+ ptr = mmap64(0, size, prot, MAP_SHARED, fd, arg.offset);
+
+ if (ptr == MAP_FAILED)
+ ptr = NULL;
+ else
+ errno = 0;
+
+ return ptr;
+}
+
diff --git a/lib/i915/gem_mman.h b/lib/i915/gem_mman.h
index f7242ed7..c2c10249 100644
--- a/lib/i915/gem_mman.h
+++ b/lib/i915/gem_mman.h
@@ -51,5 +51,38 @@ int gem_munmap(void *ptr, uint64_t size);
*/
#define gem_require_mmap_wc(fd) igt_require(gem_mmap__has_wc(fd))
+struct local_drm_i915_gem_mmap_offset {
+ /** Handle for the object being mapped. */
+ __u32 handle;
+ __u32 pad;
+ /**
+ * Fake offset to use for subsequent mmap call
+ *
+ * This is a fixed-size type for 32/64 compatibility.
+ */
+ __u64 offset;
+
+ /**
+ * Flags for extended behaviour.
+ *
+ * It is mandatory that either one of the _WC/_WB flags
+ * should be passed here.
+ */
+ __u64 flags;
+};
+
+#define LOCAL_DRM_I915_GEM_MMAP_OFFSET 0x24
+#define LOCAL_I915_MMAP_OFFSET_WC (1 << 0)
+#define LOCAL_I915_MMAP_OFFSET_WB (1 << 1)
+#define LOCAL_I915_MMAP_OFFSET_UC (1 << 2)
+
+#define LOCAL_DRM_IOCTL_I915_GEM_MMAP_OFFSET \
+ DRM_IOWR(DRM_COMMAND_BASE + LOCAL_DRM_I915_GEM_MMAP_OFFSET, struct local_drm_i915_gem_mmap_offset)
+
+bool has_mmap_offset(int fd);
+
+void *__gem_mmap_offset(int fd, uint32_t handle, uint64_t offset, uint64_t size, unsigned int prot, uint64_t flags);
+
+
#endif /* GEM_MMAN_H */
--
2.20.1
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v3] lib/i915: Add mmap_offset support
2019-02-26 18:23 [igt-dev] [PATCH i-g-t v3] lib/i915: Add mmap_offset support Antonio Argenziano
@ 2019-02-26 18:29 ` Chris Wilson
2019-02-26 18:44 ` Antonio Argenziano
2019-02-26 19:59 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork
1 sibling, 1 reply; 7+ messages in thread
From: Chris Wilson @ 2019-02-26 18:29 UTC (permalink / raw)
To: Antonio Argenziano, igt-dev; +Cc: Janulgue Abdiel, Matthew Auld
Quoting Antonio Argenziano (2019-02-26 18:23:53)
> +bool has_mmap_offset(int fd)
Prefix please.
> +{
> + static int has_mmap_offset = -1;
What's our plan to avoiding the static?
I don't think we need the statics for these, as their use should be
limited to fixtures. Once plan has been decided upon, enact it for all
in gem_mman.c :)
> +
> + if (has_mmap_offset == -1) {
> + struct drm_i915_getparam gp;
> +
> + has_mmap_offset = 0;
> +
> + memset(&gp, 0, sizeof(gp));
> + gp.param = 0x55; /* I915_PARAM_MMAP_OFFSET_VERSION */
> + gp.value = &has_mmap_offset;
> + ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
> + }
> +
> + return has_mmap_offset > 0;
> +}
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v3] lib/i915: Add mmap_offset support
2019-02-26 18:29 ` Chris Wilson
@ 2019-02-26 18:44 ` Antonio Argenziano
2019-02-26 18:47 ` Chris Wilson
0 siblings, 1 reply; 7+ messages in thread
From: Antonio Argenziano @ 2019-02-26 18:44 UTC (permalink / raw)
To: Chris Wilson, igt-dev; +Cc: Janulgue Abdiel, Matthew Auld
On 26/02/19 10:29, Chris Wilson wrote:
> Quoting Antonio Argenziano (2019-02-26 18:23:53)
>> +bool has_mmap_offset(int fd)
>
> Prefix please.
>
>> +{
>> + static int has_mmap_offset = -1;
>
> What's our plan to avoiding the static?
How about: make the variable not static and remove the if?
>
> I don't think we need the statics for these, as their use should be
> limited to fixtures. Once plan has been decided upon, enact it for all
> in gem_mman.c :)
OK, there isn't really much in gem_mman, I'll add a patch before this
one to remove what is there and not add this here.
Antonio
>
>> +
>> + if (has_mmap_offset == -1) {
>> + struct drm_i915_getparam gp;
>> +
>> + has_mmap_offset = 0;
>> +
>> + memset(&gp, 0, sizeof(gp));
>> + gp.param = 0x55; /* I915_PARAM_MMAP_OFFSET_VERSION */
>> + gp.value = &has_mmap_offset;
>> + ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp);
>> + }
>> +
>> + return has_mmap_offset > 0;
>> +}
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v3] lib/i915: Add mmap_offset support
2019-02-26 18:44 ` Antonio Argenziano
@ 2019-02-26 18:47 ` Chris Wilson
2019-02-26 19:29 ` Antonio Argenziano
0 siblings, 1 reply; 7+ messages in thread
From: Chris Wilson @ 2019-02-26 18:47 UTC (permalink / raw)
To: Antonio Argenziano, igt-dev; +Cc: Janulgue Abdiel, Matthew Auld
Quoting Antonio Argenziano (2019-02-26 18:44:17)
>
>
> On 26/02/19 10:29, Chris Wilson wrote:
> > Quoting Antonio Argenziano (2019-02-26 18:23:53)
> >> +bool has_mmap_offset(int fd)
> >
> > Prefix please.
> >
> >> +{
> >> + static int has_mmap_offset = -1;
> >
> > What's our plan to avoiding the static?
>
> How about: make the variable not static and remove the if?
>
> >
> > I don't think we need the statics for these, as their use should be
> > limited to fixtures. Once plan has been decided upon, enact it for all
> > in gem_mman.c :)
>
> OK, there isn't really much in gem_mman, I'll add a patch before this
> one to remove what is there and not add this here.
>
> Antonio
>
> >
> >> +
> >> + if (has_mmap_offset == -1) {
> >> + struct drm_i915_getparam gp;
> >> +
> >> + has_mmap_offset = 0;
> >> +
> >> + memset(&gp, 0, sizeof(gp));
> >> + gp.param = 0x55; /* I915_PARAM_MMAP_OFFSET_VERSION */
Waitasec... I thought we would be overloading the MMAP_GTT_VERSION since
we are overloading the MMAP_GTT?
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v3] lib/i915: Add mmap_offset support
2019-02-26 18:47 ` Chris Wilson
@ 2019-02-26 19:29 ` Antonio Argenziano
2019-02-26 19:36 ` Chris Wilson
0 siblings, 1 reply; 7+ messages in thread
From: Antonio Argenziano @ 2019-02-26 19:29 UTC (permalink / raw)
To: Chris Wilson, igt-dev; +Cc: Janulgue Abdiel, Matthew Auld
On 26/02/19 10:47, Chris Wilson wrote:
> Quoting Antonio Argenziano (2019-02-26 18:44:17)
>>
>>
>> On 26/02/19 10:29, Chris Wilson wrote:
>>> Quoting Antonio Argenziano (2019-02-26 18:23:53)
>>>> +bool has_mmap_offset(int fd)
>>>
>>> Prefix please.
>>>
>>>> +{
>>>> + static int has_mmap_offset = -1;
>>>
>>> What's our plan to avoiding the static?
>>
>> How about: make the variable not static and remove the if?
>>
>>>
>>> I don't think we need the statics for these, as their use should be
>>> limited to fixtures. Once plan has been decided upon, enact it for all
>>> in gem_mman.c :)
>>
>> OK, there isn't really much in gem_mman, I'll add a patch before this
>> one to remove what is there and not add this here.
>>
>> Antonio
>>
>>>
>>>> +
>>>> + if (has_mmap_offset == -1) {
>>>> + struct drm_i915_getparam gp;
>>>> +
>>>> + has_mmap_offset = 0;
>>>> +
>>>> + memset(&gp, 0, sizeof(gp));
>>>> + gp.param = 0x55; /* I915_PARAM_MMAP_OFFSET_VERSION */
>
> Waitasec... I thought we would be overloading the MMAP_GTT_VERSION since
> we are overloading the MMAP_GTT?
I think the idea was to have them separate, if we were to merge them
what would you have return MMAP_GTT_VERSION, a new value or like a flag?
Antonio
> -Chris
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v3] lib/i915: Add mmap_offset support
2019-02-26 19:29 ` Antonio Argenziano
@ 2019-02-26 19:36 ` Chris Wilson
0 siblings, 0 replies; 7+ messages in thread
From: Chris Wilson @ 2019-02-26 19:36 UTC (permalink / raw)
To: Antonio Argenziano, igt-dev; +Cc: Janulgue Abdiel, Matthew Auld
Quoting Antonio Argenziano (2019-02-26 19:29:50)
>
>
> On 26/02/19 10:47, Chris Wilson wrote:
> > Quoting Antonio Argenziano (2019-02-26 18:44:17)
> >>
> >>
> >> On 26/02/19 10:29, Chris Wilson wrote:
> >>> Quoting Antonio Argenziano (2019-02-26 18:23:53)
> >>>> +bool has_mmap_offset(int fd)
> >>>
> >>> Prefix please.
> >>>
> >>>> +{
> >>>> + static int has_mmap_offset = -1;
> >>>
> >>> What's our plan to avoiding the static?
> >>
> >> How about: make the variable not static and remove the if?
> >>
> >>>
> >>> I don't think we need the statics for these, as their use should be
> >>> limited to fixtures. Once plan has been decided upon, enact it for all
> >>> in gem_mman.c :)
> >>
> >> OK, there isn't really much in gem_mman, I'll add a patch before this
> >> one to remove what is there and not add this here.
> >>
> >> Antonio
> >>
> >>>
> >>>> +
> >>>> + if (has_mmap_offset == -1) {
> >>>> + struct drm_i915_getparam gp;
> >>>> +
> >>>> + has_mmap_offset = 0;
> >>>> +
> >>>> + memset(&gp, 0, sizeof(gp));
> >>>> + gp.param = 0x55; /* I915_PARAM_MMAP_OFFSET_VERSION */
> >
> > Waitasec... I thought we would be overloading the MMAP_GTT_VERSION since
> > we are overloading the MMAP_GTT?
>
> I think the idea was to have them separate, if we were to merge them
> what would you have return MMAP_GTT_VERSION, a new value or like a flag?
It doesn't make any difference, the param namespace is huge. It just
struck me as being a little disconnected.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 7+ messages in thread
* [igt-dev] ✗ Fi.CI.BAT: failure for lib/i915: Add mmap_offset support
2019-02-26 18:23 [igt-dev] [PATCH i-g-t v3] lib/i915: Add mmap_offset support Antonio Argenziano
2019-02-26 18:29 ` Chris Wilson
@ 2019-02-26 19:59 ` Patchwork
1 sibling, 0 replies; 7+ messages in thread
From: Patchwork @ 2019-02-26 19:59 UTC (permalink / raw)
To: igt-dev
== Series Details ==
Series: lib/i915: Add mmap_offset support
URL : https://patchwork.freedesktop.org/series/57263/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_5663 -> IGTPW_2528
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with IGTPW_2528 absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in IGTPW_2528, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
External URL: https://patchwork.freedesktop.org/api/1.0/series/57263/revisions/1/
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in IGTPW_2528:
### IGT changes ###
#### Possible regressions ####
* igt@gem_exec_fence@basic-await-default:
- fi-hsw-4770r: PASS -> FAIL +3
- fi-byt-n2820: PASS -> FAIL +2
- fi-kbl-7560u: PASS -> FAIL +2
- fi-ivb-3770: PASS -> FAIL +2
- fi-bsw-n3050: PASS -> FAIL +2
- fi-skl-6600u: PASS -> FAIL +2
- fi-apl-guc: PASS -> DMESG-FAIL
- fi-hsw-4770: PASS -> FAIL +3
- fi-byt-j1900: PASS -> FAIL +2
- fi-blb-e6850: PASS -> FAIL +1
- fi-kbl-8809g: PASS -> FAIL +2
- fi-kbl-7500u: PASS -> FAIL +2
- fi-kbl-x1275: PASS -> FAIL +2
* igt@gem_exec_fence@nb-await-default:
- fi-bsw-kefka: PASS -> FAIL +2
- fi-bxt-j4205: PASS -> FAIL +2
- fi-icl-u3: PASS -> FAIL +2
- fi-ilk-650: PASS -> FAIL +3
- fi-ivb-3520m: PASS -> FAIL +2
- fi-snb-2520m: PASS -> FAIL +3
- fi-whl-u: PASS -> FAIL +2
- fi-bdw-5557u: PASS -> FAIL +2
- fi-skl-gvtdvm: PASS -> FAIL +2
* igt@gem_mmap_gtt@basic-small-bo-tiledy:
- fi-kbl-7567u: PASS -> FAIL +2
- fi-bwr-2160: PASS -> FAIL +1
- fi-bdw-gvtdvm: PASS -> FAIL +2
- fi-kbl-guc: PASS -> FAIL
- fi-cfl-8109u: PASS -> FAIL +2
- fi-kbl-r: PASS -> FAIL +2
- fi-cfl-guc: PASS -> FAIL
- fi-elk-e7500: PASS -> FAIL +3
- fi-cfl-8700k: PASS -> FAIL +2
- fi-pnv-d510: PASS -> FAIL +2
- fi-skl-guc: PASS -> FAIL
* igt@kms_frontbuffer_tracking@basic:
- fi-bsw-n3050: PASS -> CRASH +1
- fi-bsw-kefka: PASS -> CRASH +1
- fi-ivb-3770: PASS -> CRASH
- fi-byt-j1900: PASS -> CRASH
- fi-hsw-4770: PASS -> CRASH
- fi-kbl-7560u: PASS -> CRASH
- fi-skl-6600u: PASS -> CRASH
- fi-bdw-5557u: PASS -> CRASH
- fi-kbl-r: PASS -> CRASH
- fi-cfl-8109u: PASS -> CRASH
- fi-hsw-4770r: PASS -> CRASH
- fi-ilk-650: PASS -> CRASH
- fi-byt-n2820: PASS -> CRASH
- fi-snb-2520m: PASS -> CRASH
- fi-whl-u: PASS -> CRASH
- fi-cfl-8700k: PASS -> CRASH
- fi-ivb-3520m: PASS -> CRASH
- fi-bdw-gvtdvm: PASS -> CRASH
- fi-kbl-x1275: PASS -> CRASH
- fi-skl-gvtdvm: PASS -> CRASH
Known issues
------------
Here are the changes found in IGTPW_2528 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_exec_fence@basic-await-default:
- fi-kbl-guc: PASS -> FAIL [fdo#109474]
- fi-cfl-guc: PASS -> FAIL [fdo#109474]
- fi-skl-guc: PASS -> FAIL [fdo#109474]
* igt@gem_exec_reloc@basic-cpu-noreloc:
- fi-cfl-guc: PASS -> SKIP [fdo#109271] +63
* igt@gem_exec_reloc@basic-gtt-noreloc:
- fi-skl-guc: PASS -> SKIP [fdo#109271] +63
* igt@gem_exec_suspend@basic-s3:
- fi-blb-e6850: PASS -> INCOMPLETE [fdo#107718]
* igt@gem_mmap_gtt@basic-wc:
- fi-pnv-d510: PASS -> FAIL [fdo#107307]
- fi-kbl-guc: PASS -> FAIL [fdo#107307]
- fi-kbl-x1275: PASS -> FAIL [fdo#107307]
- fi-byt-j1900: PASS -> FAIL [fdo#107307]
- fi-kbl-r: PASS -> FAIL [fdo#107307]
- fi-byt-n2820: PASS -> FAIL [fdo#107307]
- fi-kbl-8809g: PASS -> FAIL [fdo#107307]
- fi-skl-gvtdvm: PASS -> FAIL [fdo#107307]
- fi-kbl-7560u: PASS -> FAIL [fdo#107307]
- fi-hsw-4770: PASS -> FAIL [fdo#107307]
- fi-cfl-guc: PASS -> FAIL [fdo#107307]
- fi-whl-u: PASS -> FAIL [fdo#107307]
- fi-skl-guc: PASS -> FAIL [fdo#107307]
- fi-kbl-7567u: PASS -> FAIL [fdo#107307]
- fi-bsw-n3050: PASS -> FAIL [fdo#107307]
- fi-ivb-3770: PASS -> FAIL [fdo#107307]
- fi-cfl-8700k: PASS -> FAIL [fdo#107307]
- fi-skl-6600u: PASS -> FAIL [fdo#107307]
- fi-kbl-7500u: PASS -> FAIL [fdo#107307]
- fi-gdg-551: PASS -> FAIL [fdo#107307]
- fi-cfl-8109u: PASS -> FAIL [fdo#107307]
- fi-ivb-3520m: PASS -> FAIL [fdo#107307]
- fi-bxt-j4205: PASS -> INCOMPLETE [fdo#103927]
- fi-icl-u3: PASS -> FAIL [fdo#107307]
- fi-ilk-650: PASS -> FAIL [fdo#107307]
- fi-bdw-gvtdvm: PASS -> FAIL [fdo#107307]
- fi-hsw-4770r: PASS -> FAIL [fdo#107307]
- fi-elk-e7500: PASS -> FAIL [fdo#107307]
- fi-snb-2520m: PASS -> FAIL [fdo#107307]
- fi-bsw-kefka: PASS -> FAIL [fdo#107307]
- fi-bwr-2160: PASS -> FAIL [fdo#107307]
- fi-bdw-5557u: PASS -> FAIL [fdo#107307]
* igt@i915_pm_rpm@basic-pci-d3-state:
- fi-bsw-kefka: PASS -> SKIP [fdo#109271]
* igt@i915_pm_rpm@basic-rte:
- fi-bsw-kefka: PASS -> FAIL [fdo#108800]
* igt@i915_pm_rps@basic-api:
- fi-kbl-guc: PASS -> SKIP [fdo#109271] +71
* igt@i915_selftest@live_evict:
- fi-bsw-kefka: PASS -> DMESG-WARN [fdo#107709]
* igt@kms_busy@basic-flip-a:
- fi-gdg-551: PASS -> FAIL [fdo#103182] +1
* igt@kms_frontbuffer_tracking@basic:
- fi-icl-u3: PASS -> FAIL [fdo#103167]
* igt@runner@aborted:
- fi-bsw-kefka: NOTRUN -> FAIL [fdo#107709]
#### Possible fixes ####
* igt@kms_busy@basic-flip-b:
- fi-gdg-551: FAIL [fdo#103182] -> PASS
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103182]: https://bugs.freedesktop.org/show_bug.cgi?id=103182
[fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#107307]: https://bugs.freedesktop.org/show_bug.cgi?id=107307
[fdo#107709]: https://bugs.freedesktop.org/show_bug.cgi?id=107709
[fdo#107718]: https://bugs.freedesktop.org/show_bug.cgi?id=107718
[fdo#108800]: https://bugs.freedesktop.org/show_bug.cgi?id=108800
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109474]: https://bugs.freedesktop.org/show_bug.cgi?id=109474
Participating hosts (44 -> 35)
------------------------------
Missing (9): fi-ilk-m540 fi-hsw-4200u fi-hsw-peppy fi-byt-squawks fi-icl-u2 fi-bsw-cyan fi-icl-y fi-bdw-samus fi-skl-6700k2
Build changes
-------------
* IGT: IGT_4859 -> IGTPW_2528
CI_DRM_5663: b3edf5fc71aad143ed214a72901a2c75da5535d2 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_2528: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2528/
IGT_4859: 1d8f3320cbc06fa73ad1487453a63993f17b9d57 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_2528/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2019-02-26 19:59 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-26 18:23 [igt-dev] [PATCH i-g-t v3] lib/i915: Add mmap_offset support Antonio Argenziano
2019-02-26 18:29 ` Chris Wilson
2019-02-26 18:44 ` Antonio Argenziano
2019-02-26 18:47 ` Chris Wilson
2019-02-26 19:29 ` Antonio Argenziano
2019-02-26 19:36 ` Chris Wilson
2019-02-26 19:59 ` [igt-dev] ✗ Fi.CI.BAT: failure for " Patchwork
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.