[Mesa-dev] [PATCH 2/2] gallium: don't use enum bitfields in p_video_state.h

[Christian König]

Am Donnerstag, den 14.07.2011, 15:36 +0000 schrieb younes.m at gmail.com:
> I wasn't going to bring this up until I had all the hardware decoding
> bits sorted out, however since you mention it, "vertex buffers" make
> no sense in a video decoding interface. You can't assume that a driver
> can provide such a buffer for motion vectors. Also, you moved the
> buffering policy out into the state tracker, which means that every
> driver will have to buffer an entire frame's worth of data even if the
> hardware doesn't support such things. I've backed out of some of these
> things in my local tree for my own purposes, but they'll have to be
> fixed properly eventually.

Yeah, I also thought about this. My overall feeling was to get it into
VRAM first and then bring it into the form needed by the hardware with a
shader if the need arise.

I slice level buffering doesn't makes any sense to me. It was one of the
big mistakes of XvMC, and I don't think we should repeat that. Decoding
single slices makes only sense if your under real memory pressure and
none of the modern interfaces still supports that.

Anyway that interface between state tracker and driver is only used for
anything except bitstream acceleration, and from what I know about UVD
that doesn't really support anything else.

Regards,
Christian.