* [PATCH i-g-t] tests/drv_missed_irq_hang: Fix gem_blt path
@ 2016-06-13 13:26 Mika Kuoppala
2016-06-15 11:56 ` Marius Vlad
0 siblings, 1 reply; 5+ messages in thread
From: Mika Kuoppala @ 2016-06-13 13:26 UTC (permalink / raw)
To: intel-gfx
Don't add SOURCE_DIR to the path for gem_blt as if this
script is invocated on some other directory, the path to
gem_blt will be concatenated two times.
References: https://bugs.freedesktop.org/show_bug.cgi?id=88437
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
---
tests/drv_missed_irq_hang | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/tests/drv_missed_irq_hang b/tests/drv_missed_irq_hang
index e76c7db674ac..96a653846005 100755
--- a/tests/drv_missed_irq_hang
+++ b/tests/drv_missed_irq_hang
@@ -6,12 +6,12 @@
SOURCE_DIR="$( dirname "${BASH_SOURCE[0]}" )"
. $SOURCE_DIR/drm_lib.sh
-oldpath=`pwd`
+oldpath=$PWD
cd $i915_dfs_path
function blt_wait {
- $oldpath/$SOURCE_DIR/../benchmarks/gem_blt -r 1 -b 64 -t 1 -S > /dev/null
+ ${oldpath}/../benchmarks/gem_blt -r 1 -b 64 -t 1 -S > /dev/null
}
function check_for_missed_irq {
--
2.7.4
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH i-g-t] tests/drv_missed_irq_hang: Fix gem_blt path
2016-06-13 13:26 [PATCH i-g-t] tests/drv_missed_irq_hang: Fix gem_blt path Mika Kuoppala
@ 2016-06-15 11:56 ` Marius Vlad
2016-06-15 12:28 ` Chris Wilson
0 siblings, 1 reply; 5+ messages in thread
From: Marius Vlad @ 2016-06-15 11:56 UTC (permalink / raw)
To: Mika Kuoppala; +Cc: intel-gfx
[-- Attachment #1.1: Type: text/plain, Size: 1505 bytes --]
On Mon, Jun 13, 2016 at 04:26:22PM +0300, Mika Kuoppala wrote:
> Don't add SOURCE_DIR to the path for gem_blt as if this
> script is invocated on some other directory, the path to
> gem_blt will be concatenated two times.
>
> References: https://bugs.freedesktop.org/show_bug.cgi?id=88437
> Cc: Chris Wilson <chris@chris-wilson.co.uk>
> Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
> ---
> tests/drv_missed_irq_hang | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/drv_missed_irq_hang b/tests/drv_missed_irq_hang
> index e76c7db674ac..96a653846005 100755
> --- a/tests/drv_missed_irq_hang
> +++ b/tests/drv_missed_irq_hang
> @@ -6,12 +6,12 @@
> SOURCE_DIR="$( dirname "${BASH_SOURCE[0]}" )"
> . $SOURCE_DIR/drm_lib.sh
>
> -oldpath=`pwd`
> +oldpath=$PWD
>
> cd $i915_dfs_path
>
> function blt_wait {
> - $oldpath/$SOURCE_DIR/../benchmarks/gem_blt -r 1 -b 64 -t 1 -S > /dev/null
> + ${oldpath}/../benchmarks/gem_blt -r 1 -b 64 -t 1 -S > /dev/null
But $SOURCE_DIR will always be '.'. If installed, benchmarks/ will be
underneath tests/ directory. Is that what you're trying to fix?
Posted a fix for this when running under piglit:
https://patchwork.freedesktop.org/patch/87378/
> }
>
> function check_for_missed_irq {
> --
> 2.7.4
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH i-g-t] tests/drv_missed_irq_hang: Fix gem_blt path
2016-06-15 11:56 ` Marius Vlad
@ 2016-06-15 12:28 ` Chris Wilson
2016-06-16 14:37 ` Mika Kuoppala
0 siblings, 1 reply; 5+ messages in thread
From: Chris Wilson @ 2016-06-15 12:28 UTC (permalink / raw)
To: Mika Kuoppala, intel-gfx
On Wed, Jun 15, 2016 at 02:56:39PM +0300, Marius Vlad wrote:
> On Mon, Jun 13, 2016 at 04:26:22PM +0300, Mika Kuoppala wrote:
> > Don't add SOURCE_DIR to the path for gem_blt as if this
> > script is invocated on some other directory, the path to
> > gem_blt will be concatenated two times.
> >
> > References: https://bugs.freedesktop.org/show_bug.cgi?id=88437
> > Cc: Chris Wilson <chris@chris-wilson.co.uk>
> > Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
> > ---
> > tests/drv_missed_irq_hang | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/tests/drv_missed_irq_hang b/tests/drv_missed_irq_hang
> > index e76c7db674ac..96a653846005 100755
> > --- a/tests/drv_missed_irq_hang
> > +++ b/tests/drv_missed_irq_hang
> > @@ -6,12 +6,12 @@
> > SOURCE_DIR="$( dirname "${BASH_SOURCE[0]}" )"
> > . $SOURCE_DIR/drm_lib.sh
> >
> > -oldpath=`pwd`
> > +oldpath=$PWD
> >
> > cd $i915_dfs_path
> >
> > function blt_wait {
> > - $oldpath/$SOURCE_DIR/../benchmarks/gem_blt -r 1 -b 64 -t 1 -S > /dev/null
> > + ${oldpath}/../benchmarks/gem_blt -r 1 -b 64 -t 1 -S > /dev/null
> But $SOURCE_DIR will always be '.'. If installed, benchmarks/ will be
> underneath tests/ directory. Is that what you're trying to fix?
>
> Posted a fix for this when running under piglit:
> https://patchwork.freedesktop.org/patch/87378/
Neither seem like a general solution for finding the testcase, for all of
the installed, not-installed, mixed-installed combinations.
Even more tricky is that we want tighter control over the batch to avoid
a fast GPU never enabling irqs and so never noticing the failed delivery
(either the operation is completed before the wait, or it will complete
within the spin).
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH i-g-t] tests/drv_missed_irq_hang: Fix gem_blt path
2016-06-15 12:28 ` Chris Wilson
@ 2016-06-16 14:37 ` Mika Kuoppala
2016-06-16 21:36 ` Daniel Vetter
0 siblings, 1 reply; 5+ messages in thread
From: Mika Kuoppala @ 2016-06-16 14:37 UTC (permalink / raw)
To: Chris Wilson, intel-gfx
Chris Wilson <chris@chris-wilson.co.uk> writes:
> On Wed, Jun 15, 2016 at 02:56:39PM +0300, Marius Vlad wrote:
>> On Mon, Jun 13, 2016 at 04:26:22PM +0300, Mika Kuoppala wrote:
>> > Don't add SOURCE_DIR to the path for gem_blt as if this
>> > script is invocated on some other directory, the path to
>> > gem_blt will be concatenated two times.
>> >
>> > References: https://bugs.freedesktop.org/show_bug.cgi?id=88437
>> > Cc: Chris Wilson <chris@chris-wilson.co.uk>
>> > Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>
>> > ---
>> > tests/drv_missed_irq_hang | 4 ++--
>> > 1 file changed, 2 insertions(+), 2 deletions(-)
>> >
>> > diff --git a/tests/drv_missed_irq_hang b/tests/drv_missed_irq_hang
>> > index e76c7db674ac..96a653846005 100755
>> > --- a/tests/drv_missed_irq_hang
>> > +++ b/tests/drv_missed_irq_hang
>> > @@ -6,12 +6,12 @@
>> > SOURCE_DIR="$( dirname "${BASH_SOURCE[0]}" )"
>> > . $SOURCE_DIR/drm_lib.sh
>> >
>> > -oldpath=`pwd`
>> > +oldpath=$PWD
>> >
>> > cd $i915_dfs_path
>> >
>> > function blt_wait {
>> > - $oldpath/$SOURCE_DIR/../benchmarks/gem_blt -r 1 -b 64 -t 1 -S > /dev/null
>> > + ${oldpath}/../benchmarks/gem_blt -r 1 -b 64 -t 1 -S > /dev/null
>> But $SOURCE_DIR will always be '.'. If installed, benchmarks/ will be
>> underneath tests/ directory. Is that what you're trying to fix?
>>
My stab at https://bugs.freedesktop.org/show_bug.cgi?id=88437
>> Posted a fix for this when running under piglit:
>> https://patchwork.freedesktop.org/patch/87378/
>
If it fixes the bug above, go for it.
> Neither seem like a general solution for finding the testcase, for all of
> the installed, not-installed, mixed-installed combinations.
>
> Even more tricky is that we want tighter control over the batch to avoid
> a fast GPU never enabling irqs and so never noticing the failed delivery
> (either the operation is completed before the wait, or it will complete
> within the spin).
> -Chris
How about we give up and convert that test C to gain control?
-Mika
>
> --
> Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH i-g-t] tests/drv_missed_irq_hang: Fix gem_blt path
2016-06-16 14:37 ` Mika Kuoppala
@ 2016-06-16 21:36 ` Daniel Vetter
0 siblings, 0 replies; 5+ messages in thread
From: Daniel Vetter @ 2016-06-16 21:36 UTC (permalink / raw)
To: Mika Kuoppala; +Cc: intel-gfx
On Thu, Jun 16, 2016 at 05:37:54PM +0300, Mika Kuoppala wrote:
> Chris Wilson <chris@chris-wilson.co.uk> writes:
> > Neither seem like a general solution for finding the testcase, for all of
> > the installed, not-installed, mixed-installed combinations.
> >
> > Even more tricky is that we want tighter control over the batch to avoid
> > a fast GPU never enabling irqs and so never noticing the failed delivery
> > (either the operation is completed before the wait, or it will complete
> > within the spin).
> > -Chris
>
> How about we give up and convert that test C to gain control?
+1
Actually: +1k ;-)
Cheers, Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-06-16 21:36 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-06-13 13:26 [PATCH i-g-t] tests/drv_missed_irq_hang: Fix gem_blt path Mika Kuoppala
2016-06-15 11:56 ` Marius Vlad
2016-06-15 12:28 ` Chris Wilson
2016-06-16 14:37 ` Mika Kuoppala
2016-06-16 21:36 ` Daniel Vetter
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.