diff options
| author | Marcin Slusarz <marcin.slusarz@gmail.com> | 2012-06-09 20:56:37 +0200 |
|---|---|---|
| committer | Marcin Slusarz <marcin.slusarz@gmail.com> | 2012-06-09 23:23:32 +0200 |
| commit | 9e0026d35c56374b53a0b3837e24412bc9a4d8c9 (patch) | |
| tree | 361b197771e1669f0414a1047a7db00dcefb354c /nouveau/pushbuf.c | |
| parent | 25e4cb4659c62817aae2ca3b83f2d4f598d6474b (diff) | |
nouveau: silence some remaining valgrind warnings
Valgrind can't understand some of the fields passed to ioctls are overwritten
by kernel, so we need to initialize them. Almost all of our ioctl wrappers
already do it and the cost of remaining 3 is very small.
Signed-off-by: Marcin Slusarz <marcin.slusarz@gmail.com>
Diffstat (limited to 'nouveau/pushbuf.c')
| -rw-r--r-- | nouveau/pushbuf.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/nouveau/pushbuf.c b/nouveau/pushbuf.c index 7b9dbaad..0e37ce3d 100644 --- a/nouveau/pushbuf.c +++ b/nouveau/pushbuf.c @@ -336,6 +336,8 @@ pushbuf_submit(struct nouveau_pushbuf *push, struct nouveau_object *chan) req.push = (uint64_t)(unsigned long)krec->push; req.suffix0 = nvpb->suffix0; req.suffix1 = nvpb->suffix1; + req.vram_available = 0; /* for valgrind */ + req.gart_available = 0; if (dbg_on(0)) pushbuf_dump(krec, krec_id++, fifo->channel); @@ -534,7 +536,7 @@ nouveau_pushbuf_new(struct nouveau_client *client, struct nouveau_object *chan, struct nouveau_fifo *fifo = chan->data; struct nouveau_pushbuf_priv *nvpb; struct nouveau_pushbuf *push; - struct drm_nouveau_gem_pushbuf req; + struct drm_nouveau_gem_pushbuf req = {}; int ret; if (chan->oclass != NOUVEAU_FIFO_CHANNEL_CLASS) |