Hi Dean, > Add global mutex lock to prevent reentrancy in the hci_uart_tty_ioctl() > ioctl handling. This prevents concurrency of all function calls within > hci_uart_tty_ioctl() handling including hci_uart_set_proto(). > > Signed-off-by: Vignesh Raman <Vignesh_Raman@xxxxxxxxxx> > Signed-off-by: Dean Jenkins <Dean_Jenkins@xxxxxxxxxx> > --- > drivers/bluetooth/hci_ldisc.c | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/drivers/bluetooth/hci_ldisc.c b/drivers/bluetooth/hci_ldisc.c > index 9497c46..01590f6 100644 > --- a/drivers/bluetooth/hci_ldisc.c > +++ b/drivers/bluetooth/hci_ldisc.c > @@ -51,6 +51,8 @@ > > #define VERSION "2.3" > > +static DEFINE_MUTEX(ioctl_mutex); > + explain to me why we need a global lock. I think that look can be per struct hci_uart. Regards Marcel -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html