RE: [PATCHv2 2/2] Bluetooth: Reorder HCI user channel socket release

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Oops, you're right.

Regards,
Loic
________________________________________
From: Marcel Holtmann [marcel@xxxxxxxxxxxx]
Sent: Tuesday, June 09, 2015 9:32 AM
To: Poulain, Loic
Cc: linux-bluetooth@xxxxxxxxxxxxxxx
Subject: Re: [PATCHv2 2/2] Bluetooth: Reorder HCI user channel socket release

Hi Loic,

> The hci close method needs to know if we are in user channel context.
> Only add the index to mgmt once close is performed.
>
> Signed-off-by: Loic Poulain <loic.poulain@xxxxxxxxx>
> ---
> v2: Rebase on d24d81444f8 (HCI_UP test)
> net/bluetooth/hci_core.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/net/bluetooth/hci_core.c b/net/bluetooth/hci_core.c
> index f6c9909..573711c 100644
> --- a/net/bluetooth/hci_core.c
> +++ b/net/bluetooth/hci_core.c
> @@ -1553,6 +1553,7 @@ static int hci_dev_do_close(struct hci_dev *hdev)
>       BT_DBG("%s %p", hdev->name, hdev);
>
>       if (!hci_dev_test_flag(hdev, HCI_UNREGISTER) &&
> +         !hci_dev_test_flag(hdev, HCI_USER_CHANNEL) &&
>           test_bit(HCI_UP, &hdev->flags)) {
>               /* Execute vendor specific shutdown routine */
>               if (hdev->shutdown)

are you sure that this is the correct commit message. It sounds like you copied the one from patch 1/2.

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




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux