On 05/18/2016 05:10 PM, Matthew Auld wrote: > There's an updated version of this patch already on the ml [1], which > I Cc'd you in on. I take it that your @tuebingen.mpg.de is in fact an > old email address? > > [1] https://patchwork.freedesktop.org/patch/86354/ >
Your patch looks good to me. I'd only keep that one dropped comment line in drmP.h about the vblank counter and ts also needing to be protected by the vblank_timelock in addition to the seqlock, as this is still needed, especially to get _irqsave part of spin_lock_irqsave, as the write seqlocks in don't do the local irq disable. I'll give it a test later this week. Reviewed-by: Mario Kleiner <mario.kleiner.de at gmail.com> Indeed the old inactive @tuebingen.mpg.de is only a forward to the gmail address, probably with some botched mail filter rules, so they can go unnoticed quite a while. thanks, -mario