Re: [PATCH] Moving spinlock to struct usb_hcd

2008-01-27 Thread Romit Dasgupta
Hi, Should I go ahead and submit the patch with the usual "signed-off" thingie? Or is it totally useless patch that is going to be ignored? Thanks, -Romit On Jan 26, 2008 9:06 PM, Romit Dasgupta <[EMAIL PROTECTED]> wrote: > > > > > > Looking at

Re: [PATCH] Moving spinlock to struct usb_hcd

2008-01-26 Thread Romit Dasgupta
> > > Looking at how this lock is used, contention doesn't look likely > to be an issue. It's never held for long ... yes in the general case but in usb_hcd_flush_endpoint routine it seems to be held for longer than other routines. I agree that usb_hcd_flush_endpoint is an infrequently called rout

[PATCH] Moving spinlock to struct usb_hcd

2008-01-25 Thread Romit Dasgupta
Hi, This is an attempt to move the hcd_urb_list_lock to struct usb_hcd. The lock is taken on functions that try to add/delete/use urb against a given hcd. I have not seen any association of an urb with multiple hcds. Hence I thought this can be moved within usb_hcd. This should help reduce conte