* [PATCH i-g-t v4] lib/i915/gem_mman: Add a helper for obtaining MMAP_GTT interface version
@ 2019-05-31 9:29 ` Janusz Krzysztofik
0 siblings, 0 replies; 8+ messages in thread
From: Janusz Krzysztofik @ 2019-05-31 9:29 UTC (permalink / raw)
To: Arkadiusz Hiler, Petri Latvala; +Cc: Janusz Krzysztofik, intel-gfx, igt-dev
From: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
If a test calls a function which depends on availability of a specific
version of MMAP_GTT interface, an error may occur on unsupported hardware.
That may negatively affect results reported by a test framework even if
that test ignores the failure and succeedes.
This helper wraps up an IOCTL call which returns a version number of
MMAP_GTT interface. It may be used by tests which should adjust their
scope depending on availability of a specific version of MMAP_GTT
interface.
Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
Cc: Antonio Argenziano <antonio.argenziano@intel.com>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
---
Changelog:
v3 -> v4:
- return errno value on failure (Chris - thanks!),
- clear errno before return, as other helpers do,
- reword the helper documentation and commit message again (Chris -
thanks!).
v2 (internal) -> v3:
- make the code less obsucre, more explicit (Antonio - thanks!),
- reword the helper documentation and commit message.
v1 (internal) -> v2 (internal):
- minimize future potential conflicts with
https://patchwork.freedesktop.org/patch/294053/?series=58551&rev=1
(no progress with than one so not waiting for it any longer):
- convert the helper to a drop-in replacement of the one from the
above mentioned patch, returning mappable aperture version, not
only information on its availability,
- drop any other wrappers,
- document the helper,
- reword commit message.
lib/i915/gem_mman.c | 25 +++++++++++++++++++++++++
lib/i915/gem_mman.h | 1 +
2 files changed, 26 insertions(+)
diff --git a/lib/i915/gem_mman.c b/lib/i915/gem_mman.c
index 3cf9a6bb..2c3d6971 100644
--- a/lib/i915/gem_mman.c
+++ b/lib/i915/gem_mman.c
@@ -40,6 +40,31 @@
#define VG(x) do {} while (0)
#endif
+/**
+ * gem_mmap__gtt_version:
+ * @fd: open i915 drm file descriptor
+ *
+ * This functions wraps up an IOCTL to obtain MMAP_GTT interface version
+ *
+ * Returns: MMAP_GTT interface version, kernel error code on failure.
+ */
+int gem_mmap__gtt_version(int fd)
+{
+ int gtt_version, ret;
+ struct drm_i915_getparam gp = {
+ .param = I915_PARAM_MMAP_GTT_VERSION,
+ .value = >t_version,
+ };
+
+ if (ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp))
+ ret = errno;
+ else
+ ret = gtt_version;
+
+ errno = 0;
+ return ret;
+}
+
/**
* __gem_mmap__gtt:
* @fd: open i915 drm file descriptor
diff --git a/lib/i915/gem_mman.h b/lib/i915/gem_mman.h
index f7242ed7..ab12e566 100644
--- a/lib/i915/gem_mman.h
+++ b/lib/i915/gem_mman.h
@@ -25,6 +25,7 @@
#ifndef GEM_MMAN_H
#define GEM_MMAN_H
+int gem_mmap__gtt_version(int fd);
void *gem_mmap__gtt(int fd, uint32_t handle, uint64_t size, unsigned prot);
void *gem_mmap__cpu(int fd, uint32_t handle, uint64_t offset, uint64_t size, unsigned prot);
--
2.21.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [igt-dev] [PATCH i-g-t v4] lib/i915/gem_mman: Add a helper for obtaining MMAP_GTT interface version
@ 2019-05-31 9:29 ` Janusz Krzysztofik
0 siblings, 0 replies; 8+ messages in thread
From: Janusz Krzysztofik @ 2019-05-31 9:29 UTC (permalink / raw)
To: Arkadiusz Hiler, Petri Latvala; +Cc: Janusz Krzysztofik, intel-gfx, igt-dev
From: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
If a test calls a function which depends on availability of a specific
version of MMAP_GTT interface, an error may occur on unsupported hardware.
That may negatively affect results reported by a test framework even if
that test ignores the failure and succeedes.
This helper wraps up an IOCTL call which returns a version number of
MMAP_GTT interface. It may be used by tests which should adjust their
scope depending on availability of a specific version of MMAP_GTT
interface.
Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
Cc: Antonio Argenziano <antonio.argenziano@intel.com>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
---
Changelog:
v3 -> v4:
- return errno value on failure (Chris - thanks!),
- clear errno before return, as other helpers do,
- reword the helper documentation and commit message again (Chris -
thanks!).
v2 (internal) -> v3:
- make the code less obsucre, more explicit (Antonio - thanks!),
- reword the helper documentation and commit message.
v1 (internal) -> v2 (internal):
- minimize future potential conflicts with
https://patchwork.freedesktop.org/patch/294053/?series=58551&rev=1
(no progress with than one so not waiting for it any longer):
- convert the helper to a drop-in replacement of the one from the
above mentioned patch, returning mappable aperture version, not
only information on its availability,
- drop any other wrappers,
- document the helper,
- reword commit message.
lib/i915/gem_mman.c | 25 +++++++++++++++++++++++++
lib/i915/gem_mman.h | 1 +
2 files changed, 26 insertions(+)
diff --git a/lib/i915/gem_mman.c b/lib/i915/gem_mman.c
index 3cf9a6bb..2c3d6971 100644
--- a/lib/i915/gem_mman.c
+++ b/lib/i915/gem_mman.c
@@ -40,6 +40,31 @@
#define VG(x) do {} while (0)
#endif
+/**
+ * gem_mmap__gtt_version:
+ * @fd: open i915 drm file descriptor
+ *
+ * This functions wraps up an IOCTL to obtain MMAP_GTT interface version
+ *
+ * Returns: MMAP_GTT interface version, kernel error code on failure.
+ */
+int gem_mmap__gtt_version(int fd)
+{
+ int gtt_version, ret;
+ struct drm_i915_getparam gp = {
+ .param = I915_PARAM_MMAP_GTT_VERSION,
+ .value = >t_version,
+ };
+
+ if (ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp))
+ ret = errno;
+ else
+ ret = gtt_version;
+
+ errno = 0;
+ return ret;
+}
+
/**
* __gem_mmap__gtt:
* @fd: open i915 drm file descriptor
diff --git a/lib/i915/gem_mman.h b/lib/i915/gem_mman.h
index f7242ed7..ab12e566 100644
--- a/lib/i915/gem_mman.h
+++ b/lib/i915/gem_mman.h
@@ -25,6 +25,7 @@
#ifndef GEM_MMAN_H
#define GEM_MMAN_H
+int gem_mmap__gtt_version(int fd);
void *gem_mmap__gtt(int fd, uint32_t handle, uint64_t size, unsigned prot);
void *gem_mmap__cpu(int fd, uint32_t handle, uint64_t offset, uint64_t size, unsigned prot);
--
2.21.0
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH i-g-t v4] lib/i915/gem_mman: Add a helper for obtaining MMAP_GTT interface version
2019-05-31 9:29 ` [igt-dev] " Janusz Krzysztofik
@ 2019-05-31 9:35 ` Chris Wilson
-1 siblings, 0 replies; 8+ messages in thread
From: Chris Wilson @ 2019-05-31 9:35 UTC (permalink / raw)
To: Arkadiusz Hiler, Janusz Krzysztofik, Petri Latvala
Cc: igt-dev, intel-gfx, Janusz Krzysztofik
Quoting Janusz Krzysztofik (2019-05-31 10:29:16)
> From: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
>
> If a test calls a function which depends on availability of a specific
> version of MMAP_GTT interface, an error may occur on unsupported hardware.
> That may negatively affect results reported by a test framework even if
> that test ignores the failure and succeedes.
>
> This helper wraps up an IOCTL call which returns a version number of
> MMAP_GTT interface. It may be used by tests which should adjust their
> scope depending on availability of a specific version of MMAP_GTT
> interface.
>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
> Cc: Antonio Argenziano <antonio.argenziano@intel.com>
> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> ---
> Changelog:
> v3 -> v4:
> - return errno value on failure (Chris - thanks!),
> - clear errno before return, as other helpers do,
> - reword the helper documentation and commit message again (Chris -
> thanks!).
>
> v2 (internal) -> v3:
> - make the code less obsucre, more explicit (Antonio - thanks!),
> - reword the helper documentation and commit message.
>
> v1 (internal) -> v2 (internal):
> - minimize future potential conflicts with
> https://patchwork.freedesktop.org/patch/294053/?series=58551&rev=1
> (no progress with than one so not waiting for it any longer):
> - convert the helper to a drop-in replacement of the one from the
> above mentioned patch, returning mappable aperture version, not
> only information on its availability,
> - drop any other wrappers,
> - document the helper,
> - reword commit message.
>
> lib/i915/gem_mman.c | 25 +++++++++++++++++++++++++
> lib/i915/gem_mman.h | 1 +
> 2 files changed, 26 insertions(+)
>
> diff --git a/lib/i915/gem_mman.c b/lib/i915/gem_mman.c
> index 3cf9a6bb..2c3d6971 100644
> --- a/lib/i915/gem_mman.c
> +++ b/lib/i915/gem_mman.c
> @@ -40,6 +40,31 @@
> #define VG(x) do {} while (0)
> #endif
>
> +/**
> + * gem_mmap__gtt_version:
> + * @fd: open i915 drm file descriptor
> + *
> + * This functions wraps up an IOCTL to obtain MMAP_GTT interface version
> + *
> + * Returns: MMAP_GTT interface version, kernel error code on failure.
> + */
> +int gem_mmap__gtt_version(int fd)
> +{
> + int gtt_version, ret;
> + struct drm_i915_getparam gp = {
> + .param = I915_PARAM_MMAP_GTT_VERSION,
> + .value = >t_version,
> + };
> +
> + if (ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp))
> + ret = errno;
ret = -errno; :)
Petri also like it when we then say igt_assume(ret);
Or one could use
{
int result = -EIO;
struct ... gp = {
.param = I915_PARAM_MMAP_GTT_VERSION,
.value = &result,
};
if (ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp)) {
result = -errno;
igt_assume(result);
}
errno = 0;
return result;
}
Now just put it to use somewhere.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v4] lib/i915/gem_mman: Add a helper for obtaining MMAP_GTT interface version
@ 2019-05-31 9:35 ` Chris Wilson
0 siblings, 0 replies; 8+ messages in thread
From: Chris Wilson @ 2019-05-31 9:35 UTC (permalink / raw)
To: Arkadiusz Hiler, Janusz Krzysztofik, Petri Latvala
Cc: igt-dev, intel-gfx, Janusz Krzysztofik
Quoting Janusz Krzysztofik (2019-05-31 10:29:16)
> From: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
>
> If a test calls a function which depends on availability of a specific
> version of MMAP_GTT interface, an error may occur on unsupported hardware.
> That may negatively affect results reported by a test framework even if
> that test ignores the failure and succeedes.
>
> This helper wraps up an IOCTL call which returns a version number of
> MMAP_GTT interface. It may be used by tests which should adjust their
> scope depending on availability of a specific version of MMAP_GTT
> interface.
>
> Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
> Cc: Antonio Argenziano <antonio.argenziano@intel.com>
> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> ---
> Changelog:
> v3 -> v4:
> - return errno value on failure (Chris - thanks!),
> - clear errno before return, as other helpers do,
> - reword the helper documentation and commit message again (Chris -
> thanks!).
>
> v2 (internal) -> v3:
> - make the code less obsucre, more explicit (Antonio - thanks!),
> - reword the helper documentation and commit message.
>
> v1 (internal) -> v2 (internal):
> - minimize future potential conflicts with
> https://patchwork.freedesktop.org/patch/294053/?series=58551&rev=1
> (no progress with than one so not waiting for it any longer):
> - convert the helper to a drop-in replacement of the one from the
> above mentioned patch, returning mappable aperture version, not
> only information on its availability,
> - drop any other wrappers,
> - document the helper,
> - reword commit message.
>
> lib/i915/gem_mman.c | 25 +++++++++++++++++++++++++
> lib/i915/gem_mman.h | 1 +
> 2 files changed, 26 insertions(+)
>
> diff --git a/lib/i915/gem_mman.c b/lib/i915/gem_mman.c
> index 3cf9a6bb..2c3d6971 100644
> --- a/lib/i915/gem_mman.c
> +++ b/lib/i915/gem_mman.c
> @@ -40,6 +40,31 @@
> #define VG(x) do {} while (0)
> #endif
>
> +/**
> + * gem_mmap__gtt_version:
> + * @fd: open i915 drm file descriptor
> + *
> + * This functions wraps up an IOCTL to obtain MMAP_GTT interface version
> + *
> + * Returns: MMAP_GTT interface version, kernel error code on failure.
> + */
> +int gem_mmap__gtt_version(int fd)
> +{
> + int gtt_version, ret;
> + struct drm_i915_getparam gp = {
> + .param = I915_PARAM_MMAP_GTT_VERSION,
> + .value = >t_version,
> + };
> +
> + if (ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp))
> + ret = errno;
ret = -errno; :)
Petri also like it when we then say igt_assume(ret);
Or one could use
{
int result = -EIO;
struct ... gp = {
.param = I915_PARAM_MMAP_GTT_VERSION,
.value = &result,
};
if (ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp)) {
result = -errno;
igt_assume(result);
}
errno = 0;
return result;
}
Now just put it to use somewhere.
-Chris
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH i-g-t v4] lib/i915/gem_mman: Add a helper for obtaining MMAP_GTT interface version
2019-05-31 9:35 ` [igt-dev] " Chris Wilson
@ 2019-05-31 9:42 ` Janusz Krzysztofik
-1 siblings, 0 replies; 8+ messages in thread
From: Janusz Krzysztofik @ 2019-05-31 9:42 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx, igt-dev
On Friday, May 31, 2019 11:35:39 AM CEST Chris Wilson wrote:
> Quoting Janusz Krzysztofik (2019-05-31 10:29:16)
> > From: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
> >
> > If a test calls a function which depends on availability of a specific
> > version of MMAP_GTT interface, an error may occur on unsupported hardware.
> > That may negatively affect results reported by a test framework even if
> > that test ignores the failure and succeedes.
> >
> > This helper wraps up an IOCTL call which returns a version number of
> > MMAP_GTT interface. It may be used by tests which should adjust their
> > scope depending on availability of a specific version of MMAP_GTT
> > interface.
> >
> > Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
> > Cc: Antonio Argenziano <antonio.argenziano@intel.com>
> > Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> > ---
> > Changelog:
> > v3 -> v4:
> > - return errno value on failure (Chris - thanks!),
> > - clear errno before return, as other helpers do,
> > - reword the helper documentation and commit message again (Chris -
> > thanks!).
> >
> > v2 (internal) -> v3:
> > - make the code less obsucre, more explicit (Antonio - thanks!),
> > - reword the helper documentation and commit message.
> >
> > v1 (internal) -> v2 (internal):
> > - minimize future potential conflicts with
> > https://patchwork.freedesktop.org/patch/294053/?series=58551&rev=1
> > (no progress with than one so not waiting for it any longer):
> > - convert the helper to a drop-in replacement of the one from the
> > above mentioned patch, returning mappable aperture version, not
> > only information on its availability,
> > - drop any other wrappers,
> > - document the helper,
> > - reword commit message.
> >
> > lib/i915/gem_mman.c | 25 +++++++++++++++++++++++++
> > lib/i915/gem_mman.h | 1 +
> > 2 files changed, 26 insertions(+)
> >
> > diff --git a/lib/i915/gem_mman.c b/lib/i915/gem_mman.c
> > index 3cf9a6bb..2c3d6971 100644
> > --- a/lib/i915/gem_mman.c
> > +++ b/lib/i915/gem_mman.c
> > @@ -40,6 +40,31 @@
> > #define VG(x) do {} while (0)
> > #endif
> >
> > +/**
> > + * gem_mmap__gtt_version:
> > + * @fd: open i915 drm file descriptor
> > + *
> > + * This functions wraps up an IOCTL to obtain MMAP_GTT interface version
> > + *
> > + * Returns: MMAP_GTT interface version, kernel error code on failure.
> > + */
> > +int gem_mmap__gtt_version(int fd)
> > +{
> > + int gtt_version, ret;
> > + struct drm_i915_getparam gp = {
> > + .param = I915_PARAM_MMAP_GTT_VERSION,
> > + .value = >t_version,
> > + };
> > +
> > + if (ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp))
> > + ret = errno;
>
> ret = -errno; :)
Sorry.
> Petri also like it when we then say igt_assume(ret);
>
> Or one could use
>
> {
> int result = -EIO;
> struct ... gp = {
> .param = I915_PARAM_MMAP_GTT_VERSION,
> .value = &result,
> };
>
> if (ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp)) {
> result = -errno;
> igt_assume(result);
OK, I'll learn what igt_assume() is first then use it.
Thanks,
Janusz
> }
>
> errno = 0;
> return result;
> }
>
> Now just put it to use somewhere.
> -Chris
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [igt-dev] [PATCH i-g-t v4] lib/i915/gem_mman: Add a helper for obtaining MMAP_GTT interface version
@ 2019-05-31 9:42 ` Janusz Krzysztofik
0 siblings, 0 replies; 8+ messages in thread
From: Janusz Krzysztofik @ 2019-05-31 9:42 UTC (permalink / raw)
To: Chris Wilson; +Cc: Petri Latvala, intel-gfx, igt-dev
On Friday, May 31, 2019 11:35:39 AM CEST Chris Wilson wrote:
> Quoting Janusz Krzysztofik (2019-05-31 10:29:16)
> > From: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
> >
> > If a test calls a function which depends on availability of a specific
> > version of MMAP_GTT interface, an error may occur on unsupported hardware.
> > That may negatively affect results reported by a test framework even if
> > that test ignores the failure and succeedes.
> >
> > This helper wraps up an IOCTL call which returns a version number of
> > MMAP_GTT interface. It may be used by tests which should adjust their
> > scope depending on availability of a specific version of MMAP_GTT
> > interface.
> >
> > Signed-off-by: Janusz Krzysztofik <janusz.krzysztofik@intel.com>
> > Cc: Antonio Argenziano <antonio.argenziano@intel.com>
> > Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> > ---
> > Changelog:
> > v3 -> v4:
> > - return errno value on failure (Chris - thanks!),
> > - clear errno before return, as other helpers do,
> > - reword the helper documentation and commit message again (Chris -
> > thanks!).
> >
> > v2 (internal) -> v3:
> > - make the code less obsucre, more explicit (Antonio - thanks!),
> > - reword the helper documentation and commit message.
> >
> > v1 (internal) -> v2 (internal):
> > - minimize future potential conflicts with
> > https://patchwork.freedesktop.org/patch/294053/?series=58551&rev=1
> > (no progress with than one so not waiting for it any longer):
> > - convert the helper to a drop-in replacement of the one from the
> > above mentioned patch, returning mappable aperture version, not
> > only information on its availability,
> > - drop any other wrappers,
> > - document the helper,
> > - reword commit message.
> >
> > lib/i915/gem_mman.c | 25 +++++++++++++++++++++++++
> > lib/i915/gem_mman.h | 1 +
> > 2 files changed, 26 insertions(+)
> >
> > diff --git a/lib/i915/gem_mman.c b/lib/i915/gem_mman.c
> > index 3cf9a6bb..2c3d6971 100644
> > --- a/lib/i915/gem_mman.c
> > +++ b/lib/i915/gem_mman.c
> > @@ -40,6 +40,31 @@
> > #define VG(x) do {} while (0)
> > #endif
> >
> > +/**
> > + * gem_mmap__gtt_version:
> > + * @fd: open i915 drm file descriptor
> > + *
> > + * This functions wraps up an IOCTL to obtain MMAP_GTT interface version
> > + *
> > + * Returns: MMAP_GTT interface version, kernel error code on failure.
> > + */
> > +int gem_mmap__gtt_version(int fd)
> > +{
> > + int gtt_version, ret;
> > + struct drm_i915_getparam gp = {
> > + .param = I915_PARAM_MMAP_GTT_VERSION,
> > + .value = >t_version,
> > + };
> > +
> > + if (ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp))
> > + ret = errno;
>
> ret = -errno; :)
Sorry.
> Petri also like it when we then say igt_assume(ret);
>
> Or one could use
>
> {
> int result = -EIO;
> struct ... gp = {
> .param = I915_PARAM_MMAP_GTT_VERSION,
> .value = &result,
> };
>
> if (ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp)) {
> result = -errno;
> igt_assume(result);
OK, I'll learn what igt_assume() is first then use it.
Thanks,
Janusz
> }
>
> errno = 0;
> return result;
> }
>
> Now just put it to use somewhere.
> -Chris
>
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 8+ messages in thread
* [igt-dev] ✓ Fi.CI.BAT: success for lib/i915/gem_mman: Add a helper for obtaining MMAP_GTT interface version
2019-05-31 9:29 ` [igt-dev] " Janusz Krzysztofik
(?)
(?)
@ 2019-05-31 14:49 ` Patchwork
-1 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2019-05-31 14:49 UTC (permalink / raw)
To: Janusz Krzysztofik; +Cc: igt-dev
== Series Details ==
Series: lib/i915/gem_mman: Add a helper for obtaining MMAP_GTT interface version
URL : https://patchwork.freedesktop.org/series/61417/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6173 -> IGTPW_3086
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/61417/revisions/1/mbox/
Known issues
------------
Here are the changes found in IGTPW_3086 that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@gem_ctx_create@basic-files:
- fi-icl-y: [PASS][1] -> [INCOMPLETE][2] ([fdo#107713] / [fdo#109100])
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/fi-icl-y/igt@gem_ctx_create@basic-files.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/fi-icl-y/igt@gem_ctx_create@basic-files.html
* igt@i915_selftest@live_contexts:
- fi-skl-gvtdvm: [PASS][3] -> [DMESG-FAIL][4] ([fdo#110235])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/fi-skl-gvtdvm/igt@i915_selftest@live_contexts.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/fi-skl-gvtdvm/igt@i915_selftest@live_contexts.html
#### Possible fixes ####
* {igt@i915_selftest@live_blt}:
- fi-skl-iommu: [INCOMPLETE][5] -> [PASS][6]
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/fi-skl-iommu/igt@i915_selftest@live_blt.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/fi-skl-iommu/igt@i915_selftest@live_blt.html
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100
[fdo#110235]: https://bugs.freedesktop.org/show_bug.cgi?id=110235
Participating hosts (48 -> 44)
------------------------------
Additional (1): fi-icl-dsi
Missing (5): fi-byt-squawks fi-bsw-cyan fi-apl-guc fi-icl-u3 fi-bdw-samus
Build changes
-------------
* IGT: IGT_5026 -> IGTPW_3086
CI_DRM_6173: f4b0adf2423ad1c24b655ad956f9463b70f96b89 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_3086: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/
IGT_5026: 4108c74c3b15460de25ab989f4e2031594559dfc @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 8+ messages in thread
* [igt-dev] ✓ Fi.CI.IGT: success for lib/i915/gem_mman: Add a helper for obtaining MMAP_GTT interface version
2019-05-31 9:29 ` [igt-dev] " Janusz Krzysztofik
` (2 preceding siblings ...)
(?)
@ 2019-06-01 12:41 ` Patchwork
-1 siblings, 0 replies; 8+ messages in thread
From: Patchwork @ 2019-06-01 12:41 UTC (permalink / raw)
To: Janusz Krzysztofik; +Cc: igt-dev
== Series Details ==
Series: lib/i915/gem_mman: Add a helper for obtaining MMAP_GTT interface version
URL : https://patchwork.freedesktop.org/series/61417/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_6173_full -> IGTPW_3086_full
====================================================
Summary
-------
**SUCCESS**
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/61417/revisions/1/mbox/
Known issues
------------
Here are the changes found in IGTPW_3086_full that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@i915_suspend@debugfs-reader:
- shard-apl: [PASS][1] -> [DMESG-WARN][2] ([fdo#108566]) +5 similar issues
[1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-apl4/igt@i915_suspend@debugfs-reader.html
[2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-apl1/igt@i915_suspend@debugfs-reader.html
* igt@kms_flip@dpms-vs-vblank-race:
- shard-glk: [PASS][3] -> [FAIL][4] ([fdo#103060])
[3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-glk8/igt@kms_flip@dpms-vs-vblank-race.html
[4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-glk7/igt@kms_flip@dpms-vs-vblank-race.html
* igt@kms_flip@plain-flip-fb-recreate-interruptible:
- shard-glk: [PASS][5] -> [FAIL][6] ([fdo#100368])
[5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-glk1/igt@kms_flip@plain-flip-fb-recreate-interruptible.html
[6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-glk9/igt@kms_flip@plain-flip-fb-recreate-interruptible.html
* igt@kms_flip_tiling@flip-to-x-tiled:
- shard-iclb: [PASS][7] -> [FAIL][8] ([fdo#108134])
[7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-iclb5/igt@kms_flip_tiling@flip-to-x-tiled.html
[8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-iclb6/igt@kms_flip_tiling@flip-to-x-tiled.html
* igt@kms_frontbuffer_tracking@fbc-1p-rte:
- shard-iclb: [PASS][9] -> [FAIL][10] ([fdo#103167] / [fdo#110378])
[9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-iclb7/igt@kms_frontbuffer_tracking@fbc-1p-rte.html
[10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-iclb2/igt@kms_frontbuffer_tracking@fbc-1p-rte.html
* igt@kms_frontbuffer_tracking@fbc-badstride:
- shard-iclb: [PASS][11] -> [FAIL][12] ([fdo#103167]) +6 similar issues
[11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-iclb4/igt@kms_frontbuffer_tracking@fbc-badstride.html
[12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-iclb1/igt@kms_frontbuffer_tracking@fbc-badstride.html
* igt@kms_psr2_su@page_flip:
- shard-iclb: [PASS][13] -> [SKIP][14] ([fdo#109642])
[13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-iclb2/igt@kms_psr2_su@page_flip.html
[14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-iclb1/igt@kms_psr2_su@page_flip.html
* igt@kms_psr@psr2_primary_mmap_cpu:
- shard-iclb: [PASS][15] -> [SKIP][16] ([fdo#109441]) +1 similar issue
[15]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-iclb2/igt@kms_psr@psr2_primary_mmap_cpu.html
[16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-iclb1/igt@kms_psr@psr2_primary_mmap_cpu.html
* igt@kms_setmode@basic:
- shard-apl: [PASS][17] -> [FAIL][18] ([fdo#99912])
[17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-apl1/igt@kms_setmode@basic.html
[18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-apl1/igt@kms_setmode@basic.html
- shard-snb: [PASS][19] -> [FAIL][20] ([fdo#99912])
[19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-snb6/igt@kms_setmode@basic.html
[20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-snb2/igt@kms_setmode@basic.html
#### Possible fixes ####
* igt@gem_exec_params@sol-reset-invalid:
- shard-glk: [INCOMPLETE][21] ([fdo#103359] / [k.org#198133]) -> [PASS][22]
[21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-glk9/igt@gem_exec_params@sol-reset-invalid.html
[22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-glk7/igt@gem_exec_params@sol-reset-invalid.html
* igt@gem_fence_thrash@bo-write-verify-threaded-y:
- shard-iclb: [INCOMPLETE][23] ([fdo#107713] / [fdo#109100]) -> [PASS][24]
[23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-iclb7/igt@gem_fence_thrash@bo-write-verify-threaded-y.html
[24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-iclb3/igt@gem_fence_thrash@bo-write-verify-threaded-y.html
* igt@gem_tiled_swapping@non-threaded:
- shard-kbl: [FAIL][25] ([fdo#108686]) -> [PASS][26]
[25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-kbl1/igt@gem_tiled_swapping@non-threaded.html
[26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-kbl4/igt@gem_tiled_swapping@non-threaded.html
* igt@gem_workarounds@suspend-resume-context:
- shard-apl: [DMESG-WARN][27] ([fdo#108566]) -> [PASS][28] +3 similar issues
[27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-apl4/igt@gem_workarounds@suspend-resume-context.html
[28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-apl8/igt@gem_workarounds@suspend-resume-context.html
* igt@i915_pm_rc6_residency@rc6-accuracy:
- shard-kbl: [SKIP][29] ([fdo#109271]) -> [PASS][30]
[29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-kbl1/igt@i915_pm_rc6_residency@rc6-accuracy.html
[30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-kbl3/igt@i915_pm_rc6_residency@rc6-accuracy.html
* igt@kms_cursor_edge_walk@pipe-c-256x256-right-edge:
- shard-apl: [INCOMPLETE][31] ([fdo#103927]) -> [PASS][32]
[31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-apl5/igt@kms_cursor_edge_walk@pipe-c-256x256-right-edge.html
[32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-apl7/igt@kms_cursor_edge_walk@pipe-c-256x256-right-edge.html
* igt@kms_frontbuffer_tracking@fbc-stridechange:
- shard-iclb: [FAIL][33] ([fdo#103167]) -> [PASS][34] +2 similar issues
[33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-iclb8/igt@kms_frontbuffer_tracking@fbc-stridechange.html
[34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-iclb6/igt@kms_frontbuffer_tracking@fbc-stridechange.html
* igt@kms_plane_lowres@pipe-a-tiling-x:
- shard-iclb: [FAIL][35] ([fdo#103166]) -> [PASS][36]
[35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-iclb4/igt@kms_plane_lowres@pipe-a-tiling-x.html
[36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-iclb2/igt@kms_plane_lowres@pipe-a-tiling-x.html
* igt@kms_psr@psr2_no_drrs:
- shard-iclb: [SKIP][37] ([fdo#109441]) -> [PASS][38]
[37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-iclb8/igt@kms_psr@psr2_no_drrs.html
[38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-iclb2/igt@kms_psr@psr2_no_drrs.html
* igt@kms_sequence@get-forked:
- shard-snb: [SKIP][39] ([fdo#109271]) -> [PASS][40] +4 similar issues
[39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-snb6/igt@kms_sequence@get-forked.html
[40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-snb1/igt@kms_sequence@get-forked.html
* igt@kms_setmode@basic:
- shard-kbl: [FAIL][41] ([fdo#99912]) -> [PASS][42]
[41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-kbl2/igt@kms_setmode@basic.html
[42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-kbl3/igt@kms_setmode@basic.html
* igt@perf_pmu@rc6-runtime-pm-long:
- shard-glk: [FAIL][43] ([fdo#105010]) -> [PASS][44]
[43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-glk7/igt@perf_pmu@rc6-runtime-pm-long.html
[44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-glk9/igt@perf_pmu@rc6-runtime-pm-long.html
#### Warnings ####
* igt@gem_mmap_gtt@forked-big-copy-xy:
- shard-iclb: [TIMEOUT][45] ([fdo#109673]) -> [INCOMPLETE][46] ([fdo#107713] / [fdo#109100])
[45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-iclb2/igt@gem_mmap_gtt@forked-big-copy-xy.html
[46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-iclb4/igt@gem_mmap_gtt@forked-big-copy-xy.html
* igt@kms_vblank@pipe-a-ts-continuation-modeset-rpm:
- shard-snb: [SKIP][47] ([fdo#109271]) -> [SKIP][48] ([fdo#109271] / [fdo#109278])
[47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_6173/shard-snb7/igt@kms_vblank@pipe-a-ts-continuation-modeset-rpm.html
[48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/shard-snb6/igt@kms_vblank@pipe-a-ts-continuation-modeset-rpm.html
[fdo#100368]: https://bugs.freedesktop.org/show_bug.cgi?id=100368
[fdo#103060]: https://bugs.freedesktop.org/show_bug.cgi?id=103060
[fdo#103166]: https://bugs.freedesktop.org/show_bug.cgi?id=103166
[fdo#103167]: https://bugs.freedesktop.org/show_bug.cgi?id=103167
[fdo#103359]: https://bugs.freedesktop.org/show_bug.cgi?id=103359
[fdo#103927]: https://bugs.freedesktop.org/show_bug.cgi?id=103927
[fdo#105010]: https://bugs.freedesktop.org/show_bug.cgi?id=105010
[fdo#107713]: https://bugs.freedesktop.org/show_bug.cgi?id=107713
[fdo#108134]: https://bugs.freedesktop.org/show_bug.cgi?id=108134
[fdo#108566]: https://bugs.freedesktop.org/show_bug.cgi?id=108566
[fdo#108686]: https://bugs.freedesktop.org/show_bug.cgi?id=108686
[fdo#109100]: https://bugs.freedesktop.org/show_bug.cgi?id=109100
[fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
[fdo#109278]: https://bugs.freedesktop.org/show_bug.cgi?id=109278
[fdo#109441]: https://bugs.freedesktop.org/show_bug.cgi?id=109441
[fdo#109642]: https://bugs.freedesktop.org/show_bug.cgi?id=109642
[fdo#109673]: https://bugs.freedesktop.org/show_bug.cgi?id=109673
[fdo#110378]: https://bugs.freedesktop.org/show_bug.cgi?id=110378
[fdo#99912]: https://bugs.freedesktop.org/show_bug.cgi?id=99912
[k.org#198133]: https://bugzilla.kernel.org/show_bug.cgi?id=198133
Participating hosts (10 -> 6)
------------------------------
Missing (4): pig-skl-6260u shard-skl pig-hsw-4770r pig-glk-j5005
Build changes
-------------
* IGT: IGT_5026 -> IGTPW_3086
* Piglit: piglit_4509 -> None
CI_DRM_6173: f4b0adf2423ad1c24b655ad956f9463b70f96b89 @ git://anongit.freedesktop.org/gfx-ci/linux
IGTPW_3086: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/
IGT_5026: 4108c74c3b15460de25ab989f4e2031594559dfc @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
piglit_4509: fdc5a4ca11124ab8413c7988896eec4c97336694 @ git://anongit.freedesktop.org/piglit
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_3086/
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-06-01 12:41 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-31 9:29 [PATCH i-g-t v4] lib/i915/gem_mman: Add a helper for obtaining MMAP_GTT interface version Janusz Krzysztofik
2019-05-31 9:29 ` [igt-dev] " Janusz Krzysztofik
2019-05-31 9:35 ` Chris Wilson
2019-05-31 9:35 ` [igt-dev] " Chris Wilson
2019-05-31 9:42 ` Janusz Krzysztofik
2019-05-31 9:42 ` [igt-dev] " Janusz Krzysztofik
2019-05-31 14:49 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2019-06-01 12:41 ` [igt-dev] ✓ Fi.CI.IGT: " 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.