[PATCH] drm: idiot-proof vblank

On Wed, Aug 06, 2014 at 01:16:59PM -0400, Rob Clark wrote: > After spending slightly more time than I'd care to admit debugging the > various and presumably spectacular way things fail when you pass too low > a value to drm_vblank_init() (thanks console-lock for not letting me see > the carnage!),

[PATCH] drm: idiot-proof vblank

On Wed, Aug 06, 2014 at 03:06:40PM -0400, Rob Clark wrote: > On Wed, Aug 6, 2014 at 2:12 PM, Ville Syrj?l? > wrote: > > On Wed, Aug 06, 2014 at 01:16:59PM -0400, Rob Clark wrote: > >> After spending slightly more time than I'd care to admit debugging the > >> various and presumably spectacular way

[PATCH] drm: idiot-proof vblank

On Wed, Aug 06, 2014 at 01:16:59PM -0400, Rob Clark wrote: > After spending slightly more time than I'd care to admit debugging the > various and presumably spectacular way things fail when you pass too low > a value to drm_vblank_init() (thanks console-lock for not letting me see > the carnage!),

[PATCH] drm: idiot-proof vblank

On Wed, Aug 6, 2014 at 4:28 PM, Daniel Vetter wrote: > On Wed, Aug 06, 2014 at 03:06:40PM -0400, Rob Clark wrote: >> On Wed, Aug 6, 2014 at 2:12 PM, Ville Syrj?l? >> wrote: >> > On Wed, Aug 06, 2014 at 01:16:59PM -0400, Rob Clark wrote: >> >> After spending slightly more time than I'd care to adm

[PATCH] drm: idiot-proof vblank

On Wed, Aug 6, 2014 at 2:12 PM, Ville Syrj?l? wrote: > On Wed, Aug 06, 2014 at 01:16:59PM -0400, Rob Clark wrote: >> After spending slightly more time than I'd care to admit debugging the >> various and presumably spectacular way things fail when you pass too low >> a value to drm_vblank_init() (t

[PATCH] drm: idiot-proof vblank

After spending slightly more time than I'd care to admit debugging the various and presumably spectacular way things fail when you pass too low a value to drm_vblank_init() (thanks console-lock for not letting me see the carnage!), I decided it might be a good idea to add some sanity checking. Sig