drm/crtc: add sanity checks to create_dumb()
authorDavid Herrmann <dh.herrmann@gmail.com>
Mon, 20 Jan 2014 19:09:55 +0000 (20:09 +0100)
committerDavid Herrmann <dh.herrmann@gmail.com>
Sun, 16 Mar 2014 11:11:01 +0000 (12:11 +0100)
commitb28cd41f9e9bb8085f7362c80833fc129628d3d6
treedb129e135dc3e67d6cfddfd8fc63ae93efa84c97
parent77472347972add74a3d89a0b9152b8eebc0ad2b0
drm/crtc: add sanity checks to create_dumb()

Lets make sure some basic expressions are always true:
  bpp != NULL
  width != NULL
  height != NULL
  stride = bpp * width < 2^32
  size = stride * height < 2^32
  PAGE_ALIGN(size) < 2^32

At least the udl driver doesn't check for multiplication-overflows, so
lets just make sure it will never happen. These checks allow drivers to do
any 32bit math without having to test for mult-overflows themselves.

The two divisions might hurt performance a bit, but dumb_create() is only
used for scanout-buffers, so that should be fine. We could use 64bit math
to avoid the divisions, but that may be slow on 32bit machines.. Or maybe
there should just be a "safe_mult32()" helper, which currently doesn't
exist (I think?).

Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
drivers/gpu/drm/drm_crtc.c