* [PATCH] drm/i915: Remove superfluous GEN8_LR_CONTEXT_ALIGN
@ 2018-07-27 9:29 Chris Wilson
2018-07-27 9:31 ` Matthew Auld
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Chris Wilson @ 2018-07-27 9:29 UTC (permalink / raw)
To: intel-gfx
As GEN8_LR_CONTEXT_ALIGN is I915_GTT_MIN_ALIGNMENT is it functionally
equivalent to 0, and we will not be able to reduce the min-alignment for
the GTT, so passing 0 is and will remain equivalent.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
---
drivers/gpu/drm/i915/intel_lrc.c | 2 +-
drivers/gpu/drm/i915/intel_lrc.h | 2 --
2 files changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/i915/intel_lrc.c b/drivers/gpu/drm/i915/intel_lrc.c
index 986f84920290..66c7252526f3 100644
--- a/drivers/gpu/drm/i915/intel_lrc.c
+++ b/drivers/gpu/drm/i915/intel_lrc.c
@@ -1306,7 +1306,7 @@ static int __context_pin(struct i915_gem_context *ctx, struct i915_vma *vma)
if (ctx->ggtt_offset_bias)
flags |= PIN_OFFSET_BIAS | ctx->ggtt_offset_bias;
- return i915_vma_pin(vma, 0, GEN8_LR_CONTEXT_ALIGN, flags);
+ return i915_vma_pin(vma, 0, 0, flags);
}
static struct intel_context *
diff --git a/drivers/gpu/drm/i915/intel_lrc.h b/drivers/gpu/drm/i915/intel_lrc.h
index 4dfb78e3ec7e..f5a5502ecf70 100644
--- a/drivers/gpu/drm/i915/intel_lrc.h
+++ b/drivers/gpu/drm/i915/intel_lrc.h
@@ -27,8 +27,6 @@
#include "intel_ringbuffer.h"
#include "i915_gem_context.h"
-#define GEN8_LR_CONTEXT_ALIGN I915_GTT_MIN_ALIGNMENT
-
/* Execlists regs */
#define RING_ELSP(engine) _MMIO((engine)->mmio_base + 0x230)
#define RING_EXECLIST_STATUS_LO(engine) _MMIO((engine)->mmio_base + 0x234)
--
2.18.0
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] drm/i915: Remove superfluous GEN8_LR_CONTEXT_ALIGN
2018-07-27 9:29 [PATCH] drm/i915: Remove superfluous GEN8_LR_CONTEXT_ALIGN Chris Wilson
@ 2018-07-27 9:31 ` Matthew Auld
2018-07-27 10:26 ` ✓ Fi.CI.BAT: success for " Patchwork
2018-07-27 11:30 ` ✓ Fi.CI.IGT: " Patchwork
2 siblings, 0 replies; 4+ messages in thread
From: Matthew Auld @ 2018-07-27 9:31 UTC (permalink / raw)
To: Chris Wilson; +Cc: Intel Graphics Development
On 27 July 2018 at 10:29, Chris Wilson <chris@chris-wilson.co.uk> wrote:
> As GEN8_LR_CONTEXT_ALIGN is I915_GTT_MIN_ALIGNMENT is it functionally
> equivalent to 0, and we will not be able to reduce the min-alignment for
> the GTT, so passing 0 is and will remain equivalent.
>
> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 4+ messages in thread
* ✓ Fi.CI.BAT: success for drm/i915: Remove superfluous GEN8_LR_CONTEXT_ALIGN
2018-07-27 9:29 [PATCH] drm/i915: Remove superfluous GEN8_LR_CONTEXT_ALIGN Chris Wilson
2018-07-27 9:31 ` Matthew Auld
@ 2018-07-27 10:26 ` Patchwork
2018-07-27 11:30 ` ✓ Fi.CI.IGT: " Patchwork
2 siblings, 0 replies; 4+ messages in thread
From: Patchwork @ 2018-07-27 10:26 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
== Series Details ==
Series: drm/i915: Remove superfluous GEN8_LR_CONTEXT_ALIGN
URL : https://patchwork.freedesktop.org/series/47331/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4559 -> Patchwork_9790 =
== Summary - SUCCESS ==
No regressions found.
External URL: https://patchwork.freedesktop.org/api/1.0/series/47331/revisions/1/mbox/
== Known issues ==
Here are the changes found in Patchwork_9790 that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@drv_selftest@live_hangcheck:
fi-kbl-7560u: PASS -> DMESG-FAIL (fdo#106560, fdo#106947)
igt@gem_exec_suspend@basic-s3:
{fi-cfl-8109u}: PASS -> INCOMPLETE (fdo#107187)
==== Possible fixes ====
igt@drv_selftest@live_workarounds:
{fi-bsw-kefka}: DMESG-FAIL (fdo#107292) -> PASS
igt@kms_pipe_crc_basic@suspend-read-crc-pipe-c:
fi-skl-6700k2: FAIL (fdo#103191) -> PASS
==== Warnings ====
{igt@kms_psr@primary_page_flip}:
fi-cnl-psr: DMESG-FAIL (fdo#107372) -> DMESG-WARN (fdo#107372)
{name}: This element is suppressed. This means it is ignored when computing
the status of the difference (SUCCESS, WARNING, or FAILURE).
fdo#103191 https://bugs.freedesktop.org/show_bug.cgi?id=103191
fdo#106560 https://bugs.freedesktop.org/show_bug.cgi?id=106560
fdo#106947 https://bugs.freedesktop.org/show_bug.cgi?id=106947
fdo#107187 https://bugs.freedesktop.org/show_bug.cgi?id=107187
fdo#107292 https://bugs.freedesktop.org/show_bug.cgi?id=107292
fdo#107372 https://bugs.freedesktop.org/show_bug.cgi?id=107372
== Participating hosts (53 -> 47) ==
Missing (6): fi-ilk-m540 fi-hsw-4200u fi-byt-squawks fi-bsw-cyan fi-ctg-p8600 fi-byt-clapper
== Build changes ==
* Linux: CI_DRM_4559 -> Patchwork_9790
CI_DRM_4559: 749fbb7efb867a15963eb08a942ef6f9e0b3335b @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4576: bcb37a9b20eeec97f15fac2222408cc2e0b77631 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_9790: a44cb39453afc4acfa1e44fa78e54d701dfadbf9 @ git://anongit.freedesktop.org/gfx-ci/linux
== Linux commits ==
a44cb39453af drm/i915: Remove superfluous GEN8_LR_CONTEXT_ALIGN
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/Patchwork_9790/issues.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 4+ messages in thread* ✓ Fi.CI.IGT: success for drm/i915: Remove superfluous GEN8_LR_CONTEXT_ALIGN
2018-07-27 9:29 [PATCH] drm/i915: Remove superfluous GEN8_LR_CONTEXT_ALIGN Chris Wilson
2018-07-27 9:31 ` Matthew Auld
2018-07-27 10:26 ` ✓ Fi.CI.BAT: success for " Patchwork
@ 2018-07-27 11:30 ` Patchwork
2 siblings, 0 replies; 4+ messages in thread
From: Patchwork @ 2018-07-27 11:30 UTC (permalink / raw)
To: Chris Wilson; +Cc: intel-gfx
== Series Details ==
Series: drm/i915: Remove superfluous GEN8_LR_CONTEXT_ALIGN
URL : https://patchwork.freedesktop.org/series/47331/
State : success
== Summary ==
= CI Bug Log - changes from CI_DRM_4559_full -> Patchwork_9790_full =
== Summary - WARNING ==
Minor unknown changes coming with Patchwork_9790_full need to be verified
manually.
If you think the reported changes have nothing to do with the changes
introduced in Patchwork_9790_full, please notify your bug team to allow them
to document this new failure mode, which will reduce false positives in CI.
== Possible new issues ==
Here are the unknown changes that may have been introduced in Patchwork_9790_full:
=== IGT changes ===
==== Warnings ====
igt@gem_mocs_settings@mocs-rc6-ctx-dirty-render:
shard-kbl: PASS -> SKIP
== Known issues ==
Here are the changes found in Patchwork_9790_full that come from known issues:
=== IGT changes ===
==== Issues hit ====
igt@kms_vblank@pipe-b-ts-continuation-dpms-suspend:
shard-apl: PASS -> FAIL (fdo#103375)
==== Possible fixes ====
igt@kms_flip@plain-flip-ts-check:
shard-glk: FAIL (fdo#100368) -> PASS
igt@kms_flip@plain-flip-ts-check-interruptible:
shard-glk: FAIL (fdo#103928) -> PASS
fdo#100368 https://bugs.freedesktop.org/show_bug.cgi?id=100368
fdo#103375 https://bugs.freedesktop.org/show_bug.cgi?id=103375
fdo#103928 https://bugs.freedesktop.org/show_bug.cgi?id=103928
== Participating hosts (5 -> 5) ==
No changes in participating hosts
== Build changes ==
* Linux: CI_DRM_4559 -> Patchwork_9790
CI_DRM_4559: 749fbb7efb867a15963eb08a942ef6f9e0b3335b @ git://anongit.freedesktop.org/gfx-ci/linux
IGT_4576: bcb37a9b20eeec97f15fac2222408cc2e0b77631 @ git://anongit.freedesktop.org/xorg/app/intel-gpu-tools
Patchwork_9790: a44cb39453afc4acfa1e44fa78e54d701dfadbf9 @ 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_9790/shards.html
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-07-27 11:30 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-27 9:29 [PATCH] drm/i915: Remove superfluous GEN8_LR_CONTEXT_ALIGN Chris Wilson
2018-07-27 9:31 ` Matthew Auld
2018-07-27 10:26 ` ✓ Fi.CI.BAT: success for " Patchwork
2018-07-27 11:30 ` ✓ 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.