[PATCH] drm: check return values in drm_version
From: Andres Salomon
Date: Tue Nov 03 2009 - 15:26:31 EST
On Tue, 3 Nov 2009 15:09:54 -0500
Andres Salomon <dilinger@xxxxxxxxxxxxxxx> wrote:
> On Mon, 2 Nov 2009 23:41:24 -0800
> Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:
> > On Mon, 19 Oct 2009 01:37:08 -0400 Andres Salomon
> > <dilinger@xxxxxxxxxxxxxxx> wrote:
> > > There are a few more macros in drmP.h that are unused;
> > > DRM_GET_PRIV_SAREA, DRM_ARRAY_SIZE, and DRM_WAITCOUNT can go away
> > > completely.
> > >
> > > Unfortunately, DRM_COPY is still used in one place, but we can at
> > > least move it to where it's used. It's an awful looking macro..
> > It would have been nice to fix the (valid) checkpatch warnings while
> > you were there.
> How about I do one better and replace the macro w/ a function?
> From 535091717408e4ec4974e8f309212f61aabd1880 Mon Sep 17 00:00:00 2001
> From: Andres Salomon <dilinger@xxxxxxxxxxxxxxx>
> Date: Tue, 3 Nov 2009 15:03:52 -0500
> Subject: [PATCH] drm: replace DRM_COPY macro w/ a function
> Don't inline it; the compiler can figure it out. Comments added that
> are based upon my interpretation of the code. Hopefully they're
> correct. :)
And here's one that actually checks the return values from
drm_copy_field. This is separate from the pure cleanup patch(es)
as it changes behavior that userspace
sees, so it can be bisected and reverted apart from the others if