drm: don't cast a pointer to pointer of list_head

The casting is safe only when the list_head member is the first member of
the structure.

While reading some code I stumbled across the use of 'err' in
drivers/char/drm/mga_dma.c::mga_do_cleanup_dma() and I think there's a small
problem.

The variable is only used inside #if __OS_HAS_AGP which is fine, but all
that
ever happens is an assignment to the variable - it is never actually used
for
anything.  The variable is nicely initialized to zero which is also what the
return statement at the end of function returns (always at the moment).

It looks to me like that function should be returning 'err' instead of
always
just returning 0.  Here's a patch to do that.

Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

Document drm_ttm_set_user.

Add a comment explaining the parameters for this function

Document drm_buffer_object_validate function.

Just add documentation for this function, no code changes.

Document fence_class mess in drm_bo_setstatus_ioctl

drmBOSetStatus does not bother to set the fence_class parameter.
Fortunately, drm_bo_setstatus_ioctl doesn't end up using it as it
calls drm_bo_handle_validate with use_old_fence_class = 1.

Document drm_bo_handle_validate. Match drm_bo_do_validate parameter order.

Document parameters and usage for drm_bo_handle_validate. Change parameter
order to match drm_bo_do_validate (fence_class has been moved to after
flags, hint and mask values). Existing users of this function have been
changed, but out-of-tree users must be modified separately.

Document drm_bo_do_validate. Remove spurious 'do_wait' parameter.

Add comments about the parameters to drm_bo_do_validate, along
with comments for the DRM_BO_HINT options. Remove the 'do_wait'
parameter as it is duplicated by DRM_BO_HINT_DONT_BLOCK.

Make ttm create/destroy APIs consistent. Pass page_flags in create.

Creating a ttm was done with drm_ttm_init while destruction was done with
drm_destroy_ttm. Renaming these to drm_ttm_create and drm_ttm_destroy makes
their use clearer. Passing page_flags to the create function will allow that
to know whether user or kernel pages are needed, with the goal of allowing
kernel ttms to be saved for later reuse.

Revert "nouveau: nv30: missing ramin init, does it brake other hw?"

This reverts commit 46235ea4595152d8dd5f016c18c6845a77db30b0.

catch an out of memory condition

Make relocation validate client computed values when debugging

i915: wait for buffer idle before writing relocations

When writing a relocation entry, make sure the target buffer is idle,
otherwise the GPU may see inconsistent data.

Bump driver minor for relocation optimzations

Allow relocation to be skipped when buffers don't move.

One of the costs of superioctl has been the need to perform relocations
inside the kernel. The cost of mapping the buffers to the CPU and writing
data is fairly high, especially if those buffers have been mapped and read
by the GPU.

If we assume that buffers don't move around very often, we can have the
client compute the relocations itself using the previous GPU address. When
that object doesn't move, the kernel can skip computing and writing the
updated data.

Here's a patch which adds a new field to struct drm_bo_info_req called
'presumed_offset', and a new DRM_BO_HINT_PRESUMED_OFFSET that is set when
this field has been filled in by the client.

There are two separate optimizations performed when the presumed_offset is
correct:

 1. i915_exec_reloc checks to see if all previous buffer offsets were guessed
    correctly. If so, there's no need for it to look at *any* of the
    relocations for a buffer. When this happens, it skips the whole
    relocation process, simply returning success.

 2. i915_apply_reloc checks to see if the target buffer offset was guessed
    correctly. If so, it skips mapping the relocatee, computing the
    relocation and writing the value. If no relocations are needed, the
    relocatee should never be mapped to the CPU, and so the kernel shouldn't
    need to wait for any fences to pass.

Merge branch 'master' into r500-support

drm: move agp include outside CONFIG_AGP as it isn't dependant on agp in kernel

mach64: comment bus master / ring buffer behavior and security

Fix pipe<->plane mapping vs. vblank handling (again)

If drmMinor >= 6, the intel DDX driver will enable vblank events on both
pipes.  If drmMinor >= 10 on pre-965 chipsets, the intel DDX driver will
swap the pipe<->plane mapping to allow for framebuffer compression on
laptop screens.  This means the secondary vblank counter (corresponding
to pipe B) will be incremented when vblank interrupts occur.

Now Mesa waits for vblank events on whichever plane has a greater
portion of the displayed window.  So it will happly ask to wait for the
primary counter even though that one won't increment.

So we can fix this in either the DDX driver, Mesa or the kernel (though
I thought we already had several times).

Since current (and previous) userspace assumes it's talking about a pipe
== plane situation and now uses planes when talking to the kernel, we
should probably just hide the mapping details there (indeed they already
are hidden there for vblank swaps), which this patch does.

So as far as userland is concerned, whether we call things planes or
pipes is irrelevant, as long as kernel developers understand that
userland hands them planes and they have to figure out which pipe that
corresponds to (which will typically be the same on 965+ hardware and
reversed on pre-965 mobile chips).

take down stuff after asking driver to unload

mach64: make buffer emission macros normal functions

mach64: use utf-8

Remove references to the sarea_priv perf_boxes field.

This field isn't touched or read by any other code in the stack so it's
time to retire these last few references.

patch from -mm kernel to use upper_32_bits

bsd: Replace other occurrences of msleep with mtx_sleep

bsd: Now make secondary vblank work

We needed to specifically check for driver support and test the correct
vbl_received value.  Also pulled over support for _DRM_VBLANK_NEXTONMISS
from the linux code.

bsd: Hook secondary vblank support.

bsd: Fix typo in i915_drv.c

mtx_sleep is preferred to msleep

Calling semantics are the same and both were introduced in 5.0

Clarify order of operations

DRM_DEBUG already prints the function name.

drm: Add _DRM_DRIVER map flag.

This flag indicates that the driver is responsible for the map.

bsd: Move counter initialization to load time.

nouveau: Properly identify NV40 and NV44 generation.

Beside the emitted warning, the added cast (u64 -> unsigned) strips out
part of address on 64 bit. Cast to unsigned long instead.

Signed-off-by: Jiri Slaby <jirislaby@gmail.com>

drm: enable udev node creation

drm: oops not a cleanup..

Fix up drm_ati_pcigart_info

drm: more cleanups

i965: oops force mi batchbuffer start

r500: add a bunch of all r5xx pci ids..

fix up a range that may be needed for r500 mesa

drm: major whitespace/coding style realignment with kernel

drm: cleanup drm_regman.c coding style

i915: add context handle to superioctl struct

This will be used later for lockless operation.

r500: add pci id for X1650

drm: don't reset to 0 irq_enabled when client open file descriptor

r500: suggestion from glisse to not add cliprect offset on r5xx

radeon: add initial r5xx support

Fix capitalization of __linux__ define.

drm: fix dead lock in drm_buffer_object_transfer

Bug #13233: Fix build on FreeBSD.

Bug #13233: Chase move of agp headers on FreeBSD.

radeon: refactor out the fb/agp location read/write.

Add a new get param to get the fb location into userspace. Mesa currently
hits MMIO to do this, but this isn't always possible.

nouveau: also mention the number of succcessfully copied bios bytes.

nouveau: rename

nouveau: flip buffer into gart.

nouveau: be verbose about PPC bios for now.

nouveau: revert the nv34 context size change, it was not the culprit after all.

nouveau: use get_property instead of of_get_property on pre-2.6.22 kernels.

mm fixups.

i915: remove excess debug output

nouveau: Copy the PPC bios to RAMIN on init, that lets us do proper output detection in user space.

intel: add flushing for i8xx chipsets.

Add a nut vs hammer style chipset flush for the i8xx chipsets - reenable TTM
code paths

nouveau: nv30: missing ramin init, does it brake other hw?

Add new shared header file drm_internal.h.

This header file is shared across linux and bsd, but is not installed
for user space to access.  It's the place to put prototypes and data
types that aren't platform or chipset specific, but still internal to
the drm.

nouveau: adjust the size of the NV34 context. That fixes mobile PPC cards.

Fix ttmtest.

nouveau: Also wait until CACHE1 gets emptied.

Revert "nouveau: stub superioctl"

This reverts commit 2370ded79b4176d76cda1ec5f495fd33c2d566ed.

Err.. didn't mean for that to slip in :)

Merge branch 'fifo-cleanup' into upstream-master

nouveau: Attempt to wait for channel idle before we destroy it.

nouveau: Use "new" NV40 USER control regs.

Probably entirely pointless, but a simple change in any case.

nouveau: store user control reg offsets in channel struct

nouveau: funcs to determine active channel on PFIFO.

nouveau: stub superioctl

Make sure PLLs are enabled before writing pipe configuration regs

Fix from the X driver.  Make sure the PLLs are enabled and not in VGA mode
before writing PIPE(A|B)CONF regs to avoid hangs or crashes.

suspend() and resume() need kernel 2.6.22 or later

Don't overwrite TTM page flags.

i915: oops disable TTM is backwards

Avoid buffers not ending up on a list in some cases.

Add missing drm_regman.c file.

i915: disable TTM on 8xx chips for now until flushing is solved

i915: compat code doesn't work in i8xx hw.

[PATCH] i915: fix missing G33 detect in IS_I9XX

G33 detect seems missing with Jesse's suspend/resume patch.

drm/agp: kernel style fixes

i915: cleanup most of the whitespace

Export a symbol.

Fix a user-buffer check.

drm: more kernel coding style cleanups

drm/ttm: apply linux kernel coding style to bo_lock/move/object/ttm.c

drm/ttm: kernel coding style for bo.c and objects.h

drm: fix the the typo

drm: fix memset size error

The size passing to memset is wrong.

Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>

drm: update kernel generator script using v4l script

drm: remove lots of spurious whitespace.

Kernel "cleanfile" script run.

drm: move some of the OS stuff into the OS header

nouveau: more nv20_graph_init.

This patch is originally from malc0_, but since it used some NV40_*
regs, I edited them into hex values with a comment.
This seems to correspond quite well with my own mmio-trace,
for the parts I cared to check.

nouveau: Use a sw method instead of notify interrupt to signal fence completion.

nouveau: disable m2mf buffer move for the moment

Fill fence sequence after emit ioctl.

nouveau: cleanups