|
From: Nicolai H. <pre...@gm...> - 2009-09-21 13:07:11
Attachments:
quick-is-testone.patch
|
A lot of drivers expose a large number of visuals. So even though the --quick option reduces the time taken by texCombine significantly, "quick" is still kind of the wrong adjective to describe glean with the --quick option. The attached patch (which I've taken the liberty to apply to the copy of glean in piglit for now) changes the behaviour of --quick so that only the first selected visual gets selected, in the hope of making those test runs a bit less annoying for people. There's a potential pitfall here in that this might select a visual on which certain tests don't apply. On the limited systems I've tested here, the first visual is always double-buffered with Z buffer and stencil, so I hope this isn't a problem in practice. cu, Nicolai -- Lerne, wie die Welt wirklich ist, aber vergiss niemals, wie sie sein sollte. |
|
From: Brian P. <br...@vm...> - 2009-09-21 15:02:35
|
Nicolai Hähnle wrote: > A lot of drivers expose a large number of visuals. So even though the --quick > option reduces the time taken by texCombine significantly, "quick" is still > kind of the wrong adjective to describe glean with the --quick option. I guess I'm in the habbit of running glean on just one visual with the --visual option when I want a quick run. > The attached patch (which I've taken the liberty to apply to the copy of glean > in piglit for now) changes the behaviour of --quick so that only the first > selected visual gets selected, in the hope of making those test runs a bit > less annoying for people. > > There's a potential pitfall here in that this might select a visual on which > certain tests don't apply. On the limited systems I've tested here, the first > visual is always double-buffered with Z buffer and stencil, so I hope this > isn't a problem in practice. I think we really need to make sure that a visual with at least a Z buffer is chosen. Otherwise, quite a few tests won't run. I'll see if I can come up with a better patch. -Brian |
|
From: Brian P. <br...@vm...> - 2009-09-21 15:48:26
|
Brian Paul wrote: > Nicolai Hähnle wrote: >> A lot of drivers expose a large number of visuals. So even though the --quick >> option reduces the time taken by texCombine significantly, "quick" is still >> kind of the wrong adjective to describe glean with the --quick option. > > I guess I'm in the habbit of running glean on just one visual with the > --visual option when I want a quick run. > > >> The attached patch (which I've taken the liberty to apply to the copy of glean >> in piglit for now) changes the behaviour of --quick so that only the first >> selected visual gets selected, in the hope of making those test runs a bit >> less annoying for people. >> >> There's a potential pitfall here in that this might select a visual on which >> certain tests don't apply. On the limited systems I've tested here, the first >> visual is always double-buffered with Z buffer and stencil, so I hope this >> isn't a problem in practice. > > I think we really need to make sure that a visual with at least a Z > buffer is chosen. Otherwise, quite a few tests won't run. > > I'll see if I can come up with a better patch. OK, here's a patch series for review. When --quick but not --visuals is specified, testing is limited to the first rgb,z,stencil visual. -Brian |
|
From: Nicolai H. <pre...@gm...> - 2009-09-22 17:50:24
|
On Monday 21 September 2009 17:47:37 Brian Paul wrote: > OK, here's a patch series for review. > > When --quick but not --visuals is specified, testing is limited to the > first rgb,z,stencil visual. This is indeed a much better solution, thank you! cu, Nicolai > -Brian -- Lerne, wie die Welt wirklich ist, aber vergiss niemals, wie sie sein sollte. |
|
From: Allen A. <ak...@ar...> - 2009-09-21 17:45:04
|
On Mon, Sep 21, 2009 at 09:47:37AM -0600, Brian Paul wrote: >> I guess I'm in the habbit of running glean on just one visual with the >> --visual option when I want a quick run. Same here. > OK, here's a patch series for review. > > When --quick but not --visuals is specified, testing is limited to the > first rgb,z,stencil visual. Looks OK to me. The only change I might suggest is making sure the visual is conformant as well as rgb,z,stencil. Allen |
|
From: Brian P. <br...@vm...> - 2009-09-21 23:34:20
|
Allen Akin wrote: > On Mon, Sep 21, 2009 at 09:47:37AM -0600, Brian Paul wrote: >>> I guess I'm in the habbit of running glean on just one visual with the >>> --visual option when I want a quick run. > > Same here. > >> OK, here's a patch series for review. >> >> When --quick but not --visuals is specified, testing is limited to the >> first rgb,z,stencil visual. > > Looks OK to me. The only change I might suggest is making sure the > visual is conformant as well as rgb,z,stencil. Good idea. Done. -Brian |