Re: 2.6.21-rc3-mm1 RSDL results

Con Kolivas
2007-03-11T12:52:45Z

On Sunday 11 March 2007 23:38, James Cloos wrote:
> |> See:
> |> http://webcvs.freedesktop.org/mesa/Mesa/src/mesa/drivers/dri/r200/r200_i
> |>octl.c?revision=1.37&view=markup
>
> OK.
>
> Mesa is in git, now, but that still applies. The gitweb url is:
>
> http://gitweb.freedesktop.org/?p=mesa/mesa.git
>
> and for the version of the above file in the master branch:
>
> http://gitweb.freedesktop.org/?p=mesa/mesa.git;a=blob;f=src/mesa/drivers/dr
>i/r200/r200_ioctl.c
>
> The recursive grep(1) on mesa shows:
>
> ,----[grep -r sched_yield mesa]
>
> | mesa/mesa/src/mesa/drivers/dri/r300/radeon_ioctl.c: sched_yield();
> | mesa/mesa/src/mesa/drivers/dri/i915tex/intel_batchpool.c:
> | sched_yield();
> | mesa/mesa/src/mesa/drivers/dri/i915tex/intel_batchbuffer.c:
> | sched_yield(); mesa/mesa/src/mesa/drivers/dri/common/vblank.h:#include
> | <sched.h> /* for sched_yield() */
> | mesa/mesa/src/mesa/drivers/dri/common/vblank.h:#include <sched.h> /*
> | for sched_yield() */ mesa/mesa/src/mesa/drivers/dri/common/vblank.h:
> | sched_yield(); \
> | mesa/mesa/src/mesa/drivers/dri/unichrome/via_ioctl.c: sched_yield();
> | mesa/mesa/src/mesa/drivers/dri/i915/intel_ioctl.c: sched_yield();
> | mesa/mesa/src/mesa/drivers/dri/r200/r200_ioctl.c: sched_yield();
>
> `----
>
> Thanks for the heads up. I must've grep(1)ed the xorg subdir rather
> than the parent dir, and so missed mesa.

I just wonder what the heck all these will do to testing when using any of
these drivers. Whether or not we do no yield, mild yield or full blown
expiration yield, somehow or other I can't get over the feeling that if the
code relies on yield() we can't really trust them to be meaningful cpu
scheduler tests. This means most 3d apps out there that aren't using binary
drivers, whether they be (fscking) glxgears, audio app visualisations or
what...

--
-ck