[PATCH] drm: Move two seq_printf's outside of locked mutex

2014-12-31 Thread Jonas Lundqvist
Hi Jeremiah, On 12/30/2014 11:52 PM, Jeremiah Mahler wrote: > You changed 'i' but you didn't explain in your log message why you did this. I can change the commit message to something more generic. "Move code outside of locked mutex" or similar. > Does this change really improve anything? It ma

[PATCH] drm: Move two seq_printf's outside of locked mutex

2014-12-31 Thread Jeremiah Mahler
Jonas, On Wed, Dec 31, 2014 at 09:55:19AM +0100, Jonas Lundqvist wrote: > Hi Jeremiah, > > On 12/30/2014 11:52 PM, Jeremiah Mahler wrote: > > You changed 'i' but you didn't explain in your log message why you did this. > > I can change the commit message to something more generic. "Move code > o

[PATCH] drm: Move two seq_printf's outside of locked mutex

2014-12-30 Thread Jonas Lundqvist
In drm_info.c: drm_bufs_info() and drm_vm_info() two seq_printf() was done unnecessarily while locking a mutex. This patch flips the order of the print and lock/unlock where applicable. Signed-off-by: Jonas Lundqvist --- drivers/gpu/drm/drm_info.c | 7 +++ 1 file changed, 3 insertions(+), 4

[PATCH] drm: Move two seq_printf's outside of locked mutex

2014-12-30 Thread Jeremiah Mahler
Jonas, On Tue, Dec 30, 2014 at 10:54:26PM +0100, Jonas Lundqvist wrote: > In drm_info.c: drm_bufs_info() and drm_vm_info() two seq_printf() was > done unnecessarily while locking a mutex. This patch flips the order of > the print and lock/unlock where applicable. > > Signed-off-by: Jonas Lundqvis