linux-toradex.git/drivers/gpu/drm/sis, branch v3.4.73

drm sis: initialize object_idr

2012-06-22T18:36:53+00:00

commit 648ccc7d35e3416fdc739d2e520e85de3125361b upstream.

The filed object_idr of struct drm_sis_private was introduced with
commit http://git.kernel.org/?p=linux/kernel/git/torvalds/linux.git;a=commitdiff;h=6de8a748881f1cd9d795454da2b6db616d5ca3d7 .

The idr_init(&dev->object_name_idr) is called instead of
idr_init(&dev_priv->object_idr) by mistake, leaving object_idr
uninitialized. Correct this.

This patch was not tested because of lack of hardware.

Signed-off-by: Márton Németh 
Reviewed-by: Daniel Vetter 
Signed-off-by: Dave Airlie 
Signed-off-by: Greg Kroah-Hartman

drm: move pci bus master enable into driver.

2012-02-16T18:31:07+00:00

The current enabling of bus mastering in the drm midlayer allows a large
race condition under kexec. When a kexec'ed kernel re-enables bus mastering
for the GPU, previously setup dma blocks may cause writes to random pieces
of memory. On radeon the writeback mechanism can cause these sorts of issues.

This patch doesn't fix the problem, but it moves the bus master enable under
the individual drivers control so they can move enabling it until later in
their load cycle and close the race.

Fix for radeon kms driver will be in a follow-up patch.

Signed-off-by: Dave Airlie

gpu, drm, sis: Don't return uninitialized variable from sis_driver_load()

2012-01-23T11:00:47+00:00

In sis_driver_load(), the only use of 'ret' is as the return value
from the function, unfortunately it is never initialized, so the
function just returns garbage when it succeeds.
To fix that, remove the variable and just return 0 directly on success.

Signed-off-by: Jesper Juhl 
Reviewed-by: Daniel Vetter 
Signed-off-by: Dave Airlie

drm/sis|via: don't return stack garbage from free_mem ioctl

2012-01-09T12:11:39+00:00

Fallout from my "kill drm_sman" refactor. Unfortunately gcc seems to
have failed me and not warned about this.

Tested-by: Jakob Bornecrantz  (on via)
Signed-off-by: Daniel Vetter 
Signed-off-by: Dave Airlie

drm/sis: use drm_mm instead of drm_sman

2011-12-21T23:33:23+00:00

v2: Smash compile fix from Tormod Volden  for
CONFIG_FB_SIS on top of this.

Signed-off-by: Daniel Vetter

drm/sis: track user->memblock mapping with idr

2011-12-21T23:33:21+00:00

Signed-off-by: Daniel Vetter

drm/sman: rip out owner tracking

2011-12-21T23:33:20+00:00

In contrast to kms drivers, sis/via _always_ associated a buffer with
a drm fd. So by the time we reach lastclose, all open drm fds are gone
and with them their associated objects.

So when sis/via call drm_sman_cleanup in their lastclose funcs, that
will free 0 objects.

The owner tracking now serves no purpose at all, hence rip it ou. We
can't kill the corresponding fields in struct drm_memblock_item yet
because we hijack these in the new driver private owner tracking. But
now that drm_sman.c doesn't touch ->owner_list anymore, we need to
kill the list_move hack and properly add the item to the file_priv
list.

Also leave the list_del(&obj->owner_list) in drm_sman_free for the
moment, it will move to the drivers when sman disappears completely.

v2: Remove the redundant INIT_LIST_HEAD as noted by Chris Wilson

Signed-off-by: Daniel Vetter

drm/sis: track obj->drm_fd relations in the driver

2011-12-21T23:33:15+00:00

By attach a driver private struct to each open drm fd.

Because we steal the owner_list from drm_sman until things settle,
use list_move instead of list_add.

This requires to export a drm_sman function temporarily before
drm_sman will die for real completely.

Signed-off-by: Daniel Vetter

drm: Make the per-driver file_operations struct const

2011-11-11T11:14:47+00:00

From fdf1fdebaa00f81de18c227f32f8074c8b352d50 Mon Sep 17 00:00:00 2001
From: Arjan van de Ven 
Date: Sun, 30 Oct 2011 19:06:07 -0700
Subject: [PATCH] drm: Make the per-driver file_operations struct const

The DRM layer keeps a copy of struct file_operations inside its
big driver struct... which prevents it from being consistent and static.
For consistency (and the general security objective of having such things
static), it's desirable to get this fixed.

This patch splits out the file_operations field to its own struct,
which is then "static const", and just stick a pointer to this into
the driver struct, making it more consistent with how the rest of the
kernel does this.

Signed-off-by: Arjan van de Ven 
Signed-off-by: Dave Airlie

gpu: add module.h to drivers/gpu files as required.

2011-10-31T23:32:03+00:00

So that we don't get build failures once the implicit module.h
presence is removed.

Signed-off-by: Paul Gortmaker