Re: Re: [PATCH] Bluetooth: Fix missing hci_dev_lock/unlock

Re: Re: [PATCH] Bluetooth: Fix missing hci_dev_lock/unlock

[JAGANATH KANAKKASSERY]

SGkgTWFyY2VsLA0KDQo+PiBtZ210X3BlbmRpbmdfcmVtb3ZlKCkgc2hvdWxkIGJlIGNhbGxlZCB3
aXRoIGhjaV9kZXZfbG9jayBwcm90ZWN0aW9uDQo+PiBhbmQgdGhpcyBwYXRjaCBmaXhlcyB0aGUg
c2FtZSBpbiB0aGUgbWlzc2luZyBwbGFjZXMuDQo+ID4NCj4+V2l0aG91dCB0aGlzIHBhdGNoIHRo
ZXJlIGlzIGEgY2hhbmNlIG9mIGludmFsaWQgbWVtb3J5IGFjY2VzcyB3aGlsZQ0KPj4gYWNjZXNz
aW5nIHRoZSBtZ210X3BlbmRpbmcgbGlzdCBsaWtlIGJlbG93DQo+PiANCj4+IGJsdWV0b290aGQ6
ICAzOTJdIFswXSBCYWNrdHJhY2U6DQo+PiBibHVldG9vdGhkOiAgMzkyXSBbMF0gWzxjMDRlYzc3
MD5dIChwZW5kaW5nX2Vpcl9vcl9jbGFzcysweDAvMHg2OCkgZnJvbSBbPGMwNGYxODMwPl0gKGFk
ZF91dWlkKzB4MzQvMHgxYzQpDQo+PiBibHVldG9vdGhkOiAgMzkyXSBbMF0gWzxjMDRmMTdmYz5d
IChhZGRfdXVpZCsweDAvMHgxYzQpIGZyb20gWzxjMDRmM2NjND5dIChtZ210X2NvbnRyb2wrMHgy
MDQvMHgyNzQpDQo+PiBibHVldG9vdGhkOiAgMzkyXSBbMF0gWzxjMDRmM2FjMD5dIChtZ210X2Nv
bnRyb2wrMHgwLzB4Mjc0KSBmcm9tIFs8YzA0ZjYwOWM+XSAoaGNpX3NvY2tfc2VuZG1zZysweDgw
LzB4MzA4KQ0KPj4gYmx1ZXRvb3RoZDogIDM5Ml0gWzBdIFs8YzA0ZjYwMWM+XSAoaGNpX3NvY2tf
c2VuZG1zZysweDAvMHgzMDgpIGZyb20gWzxjMDNkNGQ2OD5dIChzb2NrX2Fpb193cml0ZSsweDE0
NC8weDE3NCkNCj4+IGJsdWV0b290aGQ6ICAzOTJdIFswXSAgcjg6MDAwMDAwMDAgcjcgN2MxYmU5
MCByNiA3YzFiZTE4IHI1OjAwMDAwMDE3IHI0IGE5MGVhODANCj4+IGJsdWV0b290aGQ6ICAzOTJd
IFswXSBbPGMwM2Q0YzI0Pl0gKHNvY2tfYWlvX3dyaXRlKzB4MC8weDE3NCkgZnJvbSBbPGMwMGUy
ZDRjPl0gKGRvX3N5bmNfd3JpdGUrMHhiMC8weGUwKQ0KPj4gYmx1ZXRvb3RoZDogIDM5Ml0gWzBd
IFs8YzAwZTJjOWM+XSAoZG9fc3luY193cml0ZSsweDAvMHhlMCkgZnJvbSBbPGMwMGUzNzFjPl0g
KHZmc193cml0ZSsweDEzNC8weDEzYykNCj4+IGJsdWV0b290aGQ6ICAzOTJdIFswXSAgcjg6MDAw
MDAwMDAgcjcgN2MxYmY3MCByNjpiZWVjYTVjOCByNTowMDAwMDAxNyByNCA3YzA1OTAwDQo+PiBi
bHVldG9vdGhkOiAgMzkyXSBbMF0gWzxjMDBlMzVlOD5dICh2ZnNfd3JpdGUrMHgwLzB4MTNjKSBm
cm9tIFs8YzAwZTM5MTA+XSAoc3lzX3dyaXRlKzB4NDQvMHg3MCkNCj4+IGJsdWV0b290aGQ6ICAz
OTJdIFswXSAgcjg6MDAwMDAwMDAgcjc6MDAwMDAwMDQgcjY6MDAwMDAwMTcgcjU6YmVlY2E1Yzgg
cjQgN2MwNTkwMA0KPj4gYmx1ZXRvb3RoZDogIDM5Ml0gWzBdIFs8YzAwZTM4Y2M+XSAoc3lzX3dy
aXRlKzB4MC8weDcwKSBmcm9tIFs8YzAwMGUzYzA+XSAocmV0X2Zhc3Rfc3lzY2FsbCsweDAvMHgz
MCkNCj4+IGJsdWV0b290aGQ6ICAzOTJdIFswXSAgcjkgN2MxYTAwMCByODpjMDAwZTU2OCByNjo0
MDBiNWYxMCByNTo0MDM4OTZkOCByNDpiZWVjYTYwNA0KPj4gYmx1ZXRvb3RoZDogIDM5Ml0gWzBd
IENvZGU6IGUyOGNjMDBjIGUxNTIwMDBjIDBhMDAwMDBmIGUzYTAwMDAxIChlMWQyMTBiOCkNCj4+
IGJsdWV0b290aGQ6ICAzOTJdIFswXSAtLS1bIGVuZCB0cmFjZSA2N2I2YWM2NzQzNTg2NGM0IF0t
LS0NCj4+IGJsdWV0b290aGQ6ICAzOTJdIFswXSBLZXJuZWwgcGFuaWMgLSBub3Qgc3luY2luZzog
RmF0YWwgZXhjZXB0aW9uDQo+PiANCj4+U2lnbmVkLW9mZi1ieTogSmFnYW5hdGggS2FuYWtrYXNz
ZXJ5IDxqYWdhbmF0aC5rQHNhbXN1bmcuY29tPg0KPj4gLS0tDQo+PiBuZXQvYmx1ZXRvb3RoL2hj
aV9jb3JlLmMgIHwgICAgMiArKw0KPj4gbmV0L2JsdWV0b290aC9oY2lfZXZlbnQuYyB8ICAgMjIg
KysrKysrKysrKysrKysrKysrKystLQ0KPj4gbmV0L2JsdWV0b290aC9tZ210LmMgICAgICB8ICAg
MTggKysrKysrKysrKysrLS0tLS0tDQo+PiAzIGZpbGVzIGNoYW5nZWQsIDM0IGluc2VydGlvbnMo
KyksIDggZGVsZXRpb25zKC0pDQoNCj5JIHdvdWxkIHNwbGl0IHRoaXMgaW50byB0d28gcGF0Y2hl
cy4gT25lIGZvciB0aGUgbWdtdC5jIGFuZCBoY2lfY29yZS5jIGNoYW5nZXMgYW5kIG9uZSBmb3Ig
aGNpX2V2ZW50LmMgY2hhbmdlcy4NCg0KT2ssIEkgd2lsbCBzcGxpdCB0aGUgcGF0Y2ggaW50byB0
d28gYW5kIHJhaXNlIGFnYWluDQoNCj4+DQo+PiBkaWZmIC0tZ2l0IGEvbmV0L2JsdWV0b290aC9o
Y2lfY29yZS5jIGIvbmV0L2JsdWV0b290aC9oY2lfY29yZS5jDQo+PmluZGV4IDk2ZTczMjEuLmVj
ZDdjMDEgMTAwNjQ0DQo+PiAtLS0gYS9uZXQvYmx1ZXRvb3RoL2hjaV9jb3JlLmMNCj4+ICsrKyBi
L25ldC9ibHVldG9vdGgvaGNpX2NvcmUuYw0KPj4gQEAgLTMwODMsNyArMzA4Myw5IEBAIHN0YXRp
YyB2b2lkIGhjaV9wb3dlcl9vbihzdHJ1Y3Qgd29ya19zdHJ1Y3QgKndvcmspDQo+PiANCj4+IAll
cnIgPSBoY2lfZGV2X2RvX29wZW4oaGRldik7DQo+PiAJaWYgKGVyciA8IDApIHsNCj4+ICsJCWhj
aV9kZXZfbG9jayhoZGV2KTsNCj4+IAkJbWdtdF9zZXRfcG93ZXJlZF9mYWlsZWQoaGRldiwgZXJy
KTsNCj4+ICsJCWhjaV9kZXZfdW5sb2NrKGhkZXYpOw0KPj4gCQlyZXR1cm47DQo+PiAJfQ0KDQo+
SSB3b25kZXIgaXMgc29tZSBvZiB0aGUgbWdtdF8gZnVuY3Rpb24gc2hvdWxkIGp1c3QgdGFrZSB0
aGUgaGNpX2RldiBsb2NrLiBBcmUgdGhlcmUgY2FzZXMgd2hlcmUgd2UgZG9uJ3Qgd2FudCB0aGVt
IHRvIHRha2UgdGhlIGxvb2s/DQoNClRoZXJlIGFyZSBtYW55IG1nbXRfZnVuY3Rpb25zIGNhbGxl
ZCBmcm9tIGhjaV9ldmVudC5jIHdoaWNoIGRvbid0IHJlcXVpcmUgbG9jay4NCllvdSBwcmVmZXIg
bW92aW5nIHRoZSBsb2NrIGluc2lkZSBtZ210X3NldF9wb3dlcmVkX2ZhaWxlZCgpPw0KPj4gDQo+
PiBkaWZmIC0tZ2l0IGEvbmV0L2JsdWV0b290aC9oY2lfZXZlbnQuYyBiL25ldC9ibHVldG9vdGgv
aGNpX2V2ZW50LmMNCj4+IGluZGV4IDMyMmFiYmIuLmJhYTZjOTYgMTAwNjQ0DQo+PiAtLS0gYS9u
ZXQvYmx1ZXRvb3RoL2hjaV9ldmVudC5jDQo+PiArKysgYi9uZXQvYmx1ZXRvb3RoL2hjaV9ldmVu
dC5jDQo+PiBAQCAtMjU3LDYgKzI1Nyw4IEBAIHN0YXRpYyB2b2lkIGhjaV9jY193cml0ZV9hdXRo
X2VuYWJsZShzdHJ1Y3QgaGNpX2RldiAqaGRldiwgc3RydWN0IHNrX2J1ZmYgKnNrYikNCj4+IAlp
ZiAoIXNlbnQpDQo+PiAJCXJldHVybjsNCj4+IA0KPj4gKwloY2lfZGV2X2xvY2soaGRldik7DQo+
PiArDQo+PiAJaWYgKCFzdGF0dXMpIHsNCj4+IAkJX191OCBwYXJhbSA9ICooKF9fdTggKikgc2Vu
dCk7DQo+Pg0KPj4gQEAgLTI2OCw2ICsyNzAsOCBAQCBzdGF0aWMgdm9pZCBoY2lfY2Nfd3JpdGVf
YXV0aF9lbmFibGUoc3RydWN0IGhjaV9kZXYgKmhkZXYsIHN0cnVjdCBza19idWZmICpza2IpDQo+
PiANCj4+IAlpZiAodGVzdF9iaXQoSENJX01HTVQsICZoZGV2LT5kZXZfZmxhZ3MpKQ0KPj4gCQlt
Z210X2F1dGhfZW5hYmxlX2NvbXBsZXRlKGhkZXYsIHN0YXR1cyk7DQo+PiArDQo+PiArCWhjaV9k
ZXZfdW5sb2NrKGhkZXYpOw0KPj4gfQ0KPj4gDQo+PiBzdGF0aWMgdm9pZCBoY2lfY2Nfd3JpdGVf
ZW5jcnlwdF9tb2RlKHN0cnVjdCBoY2lfZGV2ICpoZGV2LCBzdHJ1Y3Qgc2tfYnVmZiAqc2tiKQ0K
Pj4gQEAgLTQ0Myw2ICs0NDcsOCBAQCBzdGF0aWMgdm9pZCBoY2lfY2Nfd3JpdGVfc3NwX21vZGUo
c3RydWN0IGhjaV9kZXYgKmhkZXYsIHN0cnVjdCBza19idWZmICpza2IpDQo+PiAJaWYgKCFzZW50
KQ0KPj4gCQlyZXR1cm47DQo+PiANCj4+ICsJaGNpX2Rldl9sb2NrKGhkZXYpOw0KPj4gKw0KPj4g
CWlmICghc3RhdHVzKSB7DQo+PiAJCWlmIChzZW50LT5tb2RlKQ0KPj4gCQkJaGRldi0+ZmVhdHVy
ZXNbMV1bMF0gfD0gTE1QX0hPU1RfU1NQOw0KPj4gQEAgLTQ1OCw2ICs0NjQsOCBAQCBzdGF0aWMg
dm9pZCBoY2lfY2Nfd3JpdGVfc3NwX21vZGUoc3RydWN0IGhjaV9kZXYgKmhkZXYsIHN0cnVjdCBz
a19idWZmICpza2IpDQo+PiAJCWVsc2UNCj4+IAkJCWNsZWFyX2JpdChIQ0lfU1NQX0VOQUJMRUQs
ICZoZGV2LT5kZXZfZmxhZ3MpOw0KPj4gCX0NCj4+ICsNCj4+ICsJaGNpX2Rldl91bmxvY2soaGRl
dik7DQo+PiB9DQo+PiANCj4+IHN0YXRpYyB2b2lkIGhjaV9jY193cml0ZV9zY19zdXBwb3J0KHN0
cnVjdCBoY2lfZGV2ICpoZGV2LCBzdHJ1Y3Qgc2tfYnVmZiAqc2tiKQ0KPj4gQEAgLTQ3MSw2ICs0
NzksOCBAQCBzdGF0aWMgdm9pZCBoY2lfY2Nfd3JpdGVfc2Nfc3VwcG9ydChzdHJ1Y3QgaGNpX2Rl
diAqaGRldiwgc3RydWN0IHNrX2J1ZmYgKnNrYikNCj4+IAlpZiAoIXNlbnQpDQo+PiAJCXJldHVy
bjsNCj4+IA0KPj4rCWhjaV9kZXZfbG9jayhoZGV2KTsNCj4+ICsNCj4+IAlpZiAoIXN0YXR1cykg
ew0KPj4gCQlpZiAoc2VudC0+c3VwcG9ydCkNCj4+IAkJCWhkZXYtPmZlYXR1cmVzWzFdWzBdIHw9
IExNUF9IT1NUX1NDOw0KPj4gQEAgLTQ4Niw2ICs0OTYsOCBAQCBzdGF0aWMgdm9pZCBoY2lfY2Nf
d3JpdGVfc2Nfc3VwcG9ydChzdHJ1Y3QgaGNpX2RldiAqaGRldiwgc3RydWN0IHNrX2J1ZmYgKnNr
YikNCj4+IAkJZWxzZQ0KPj4gCQkJY2xlYXJfYml0KEhDSV9TQ19FTkFCTEVELCAmaGRldi0+ZGV2
X2ZsYWdzKTsNCj4+IAl9DQo+PiArDQo+PiArCWhjaV9kZXZfdW5sb2NrKGhkZXYpOw0KPj4gfQ0K
Pj4gDQo+PiBzdGF0aWMgdm9pZCBoY2lfY2NfcmVhZF9sb2NhbF92ZXJzaW9uKHN0cnVjdCBoY2lf
ZGV2ICpoZGV2LCBzdHJ1Y3Qgc2tfYnVmZiAqc2tiKQ0KPj4gQEAgLTExNzIsOSArMTE4NCwxMSBA
QCBzdGF0aWMgdm9pZCBoY2lfY2NfbGVfc2V0X3NjYW5fZW5hYmxlKHN0cnVjdCBoY2lfZGV2ICpo
ZGV2LA0KPj4gCQkgKiByZS1lbmFibGUgaXQgYWdhaW4gaWYgbmVjZXNzYXJ5Lg0KPj4gCQkgKi8N
Cj4+IAkJaWYgKHRlc3RfYW5kX2NsZWFyX2JpdChIQ0lfTEVfU0NBTl9JTlRFUlJVUFRFRCwNCj4+
LQkJCQkgICAgICAgJmhkZXYtPmRldl9mbGFncykpDQo+PiArCQkJCSAgICAgICAmaGRldi0+ZGV2
X2ZsYWdzKSkgew0KPj4gKwkJCWhjaV9kZXZfbG9jayhoZGV2KTsNCj4+IAkJCWhjaV9kaXNjb3Zl
cnlfc2V0X3N0YXRlKGhkZXYsIERJU0NPVkVSWV9TVE9QUEVEKTsNCj4+IC0JCWVsc2UgaWYgKCF0
ZXN0X2JpdChIQ0lfTEVfQURWLCAmaGRldi0+ZGV2X2ZsYWdzKSAmJg0KPj4gKwkJCWhjaV9kZXZf
dW5sb2NrKGhkZXYpOw0KPj4gKwkJfSBlbHNlIGlmICghdGVzdF9iaXQoSENJX0xFX0FEViwgJmhk
ZXYtPmRldl9mbGFncykgJiYNCj4+IAkJCSBoZGV2LT5kaXNjb3Zlcnkuc3RhdGUgPT0gRElTQ09W
RVJZX0ZJTkRJTkcpDQoNCj5UaGlzIGlzIG5vdyBhIGNvZGluZyBzdHlsZSB2aW9sYXRpb24uIFlv
dSBuZWVkIHRvIG1vdmUgdGhpcyBsaW5lIGFzIHdlbGwuDQoNCk9rIEkgd2lsbCBmaXggaXQgaW4g
dGhlIG5leHQgcGF0Y2ggc2V0DQoNCj4+IAkJCW1nbXRfcmVlbmFibGVfYWR2ZXJ0aXNpbmcoaGRl
dik7DQo+PiANCj4+IEBAIC0xMjc4LDYgKzEyOTIsOCBAQCBzdGF0aWMgdm9pZCBoY2lfY2Nfd3Jp
dGVfbGVfaG9zdF9zdXBwb3J0ZWQoc3RydWN0IGhjaV9kZXYgKmhkZXYsDQo+PiAJaWYgKCFzZW50
KQ0KPj4gCQlyZXR1cm47DQo+PiANCj4+ICsJaGNpX2Rldl9sb2NrKGhkZXYpOw0KPj4gKw0KPj4g
CWlmIChzZW50LT5sZSkgew0KPj4gCQloZGV2LT5mZWF0dXJlc1sxXVswXSB8PSBMTVBfSE9TVF9M
RTsNCj4+IAkJc2V0X2JpdChIQ0lfTEVfRU5BQkxFRCwgJmhkZXYtPmRldl9mbGFncyk7DQo+PiBA
QCAtMTI5MSw2ICsxMzA3LDggQEAgc3RhdGljIHZvaWQgaGNpX2NjX3dyaXRlX2xlX2hvc3Rfc3Vw
cG9ydGVkKHN0cnVjdCBoY2lfZGV2ICpoZGV2LA0KPj4gCQloZGV2LT5mZWF0dXJlc1sxXVswXSB8
PSBMTVBfSE9TVF9MRV9CUkVEUjsNCj4+IAllbHNlDQo+PiAJCWhkZXYtPmZlYXR1cmVzWzFdWzBd
ICY9IH5MTVBfSE9TVF9MRV9CUkVEUjsNCj4+ICsNCj4+ICsJaGNpX2Rldl91bmxvY2soaGRldik7
DQo+PiB9DQo+PiANCj4+IHN0YXRpYyB2b2lkIGhjaV9jY19zZXRfYWR2X3BhcmFtKHN0cnVjdCBo
Y2lfZGV2ICpoZGV2LCBzdHJ1Y3Qgc2tfYnVmZiAqc2tiKQ0KPj4gZGlmZiAtLWdpdCBhL25ldC9i
bHVldG9vdGgvbWdtdC5jIGIvbmV0L2JsdWV0b290aC9tZ210LmMNCj4+IGluZGV4IDQ0YjIwZGUu
LjE2YWMwMzcgMTAwNjQ0DQo+PiAtLS0gYS9uZXQvYmx1ZXRvb3RoL21nbXQuYw0KPj4gKysrIGIv
bmV0L2JsdWV0b290aC9tZ210LmMNCj4+IEBAIC0yMTk5LDEyICsyMTk5LDE0IEBAIHN0YXRpYyB2
b2lkIGxlX2VuYWJsZV9jb21wbGV0ZShzdHJ1Y3QgaGNpX2RldiAqaGRldiwgdTggc3RhdHVzKQ0K
Pj4gew0KPj4gCXN0cnVjdCBjbWRfbG9va3VwIG1hdGNoID0geyBOVUxMLCBoZGV2IH07DQo+PiAN
Cj4+ICsJaGNpX2Rldl9sb2NrKGhkZXYpOw0KPj4gKw0KPj4gCWlmIChzdGF0dXMpIHsNCj4+IAkJ
dTggbWdtdF9lcnIgPSBtZ210X3N0YXR1cyhzdGF0dXMpOw0KPj4gDQo+PiAJCW1nbXRfcGVuZGlu
Z19mb3JlYWNoKE1HTVRfT1BfU0VUX0xFLCBoZGV2LCBjbWRfc3RhdHVzX3JzcCwNCj4+IAkJCQkg
ICAgICZtZ210X2Vycik7DQo+PiAtCQlyZXR1cm47DQo+PiArCQlnb3RvIHVubG9jazsNCj4+IAl9
DQo+PiANCj4+IAltZ210X3BlbmRpbmdfZm9yZWFjaChNR01UX09QX1NFVF9MRSwgaGRldiwgc2V0
dGluZ3NfcnNwLCAmbWF0Y2gpOw0KPj4gQEAgLTIyMjIsMTcgKzIyMjQsMTYgQEAgc3RhdGljIHZv
aWQgbGVfZW5hYmxlX2NvbXBsZXRlKHN0cnVjdCBoY2lfZGV2ICpoZGV2LCB1OCBzdGF0dXMpDQo+
PiAJaWYgKHRlc3RfYml0KEhDSV9MRV9FTkFCTEVELCAmaGRldi0+ZGV2X2ZsYWdzKSkgew0KPj4g
CQlzdHJ1Y3QgaGNpX3JlcXVlc3QgcmVxOw0KPj4NCj4+LQkJaGNpX2Rldl9sb2NrKGhkZXYpOw0K
Pj4gLQ0KPj4gCQloY2lfcmVxX2luaXQoJnJlcSwgaGRldik7DQo+PiAJCXVwZGF0ZV9hZHZfZGF0
YSgmcmVxKTsNCj4+IAkJdXBkYXRlX3NjYW5fcnNwX2RhdGEoJnJlcSk7DQo+PiAJCWhjaV9yZXFf
cnVuKCZyZXEsIE5VTEwpOw0KPj4gDQo+PiAJCWhjaV91cGRhdGVfYmFja2dyb3VuZF9zY2FuKGhk
ZXYpOw0KPj4gLQ0KPj4gLQkJaGNpX2Rldl91bmxvY2soaGRldik7DQo+PiAJfQ0KPj4gKw0KPj4g
K3VubG9jazoNCj4+ICsJaGNpX2Rldl91bmxvY2soaGRldik7DQo+PiB9DQo+PiANCj4+IHN0YXRp
YyBpbnQgc2V0X2xlKHN0cnVjdCBzb2NrICpzaywgc3RydWN0IGhjaV9kZXYgKmhkZXYsIHZvaWQg
KmRhdGEsIHUxNiBsZW4pDQo+PiBAQCAtNDI3OSwxMiArNDI4MCwxNCBAQCBzdGF0aWMgdm9pZCBz
ZXRfYWR2ZXJ0aXNpbmdfY29tcGxldGUoc3RydWN0IGhjaV9kZXYgKmhkZXYsIHU4IHN0YXR1cykN
Cj4+IHsNCj4+IAlzdHJ1Y3QgY21kX2xvb2t1cCBtYXRjaCA9IHsgTlVMTCwgaGRldiB9Ow0KPj4g
DQo+PiArCWhjaV9kZXZfbG9jayhoZGV2KTsNCj4+ICsNCj4+IAlpZiAoc3RhdHVzKSB7DQo+PiAJ
CXU4IG1nbXRfZXJyID0gbWdtdF9zdGF0dXMoc3RhdHVzKTsNCj4+IA0KPj4gCQltZ210X3BlbmRp
bmdfZm9yZWFjaChNR01UX09QX1NFVF9BRFZFUlRJU0lORywgaGRldiwNCj4+IAkJCQkgICAgIGNt
ZF9zdGF0dXNfcnNwLCAmbWdtdF9lcnIpOw0KPj4gLQkJcmV0dXJuOw0KPj4gKwkJZ290byB1bmxv
Y2s7DQo+PiAJfQ0KPj4gDQo+PiAJaWYgKHRlc3RfYml0KEhDSV9MRV9BRFYsICZoZGV2LT5kZXZf
ZmxhZ3MpKQ0KPj4gQEAgLTQyOTksNiArNDMwMiw5IEBAIHN0YXRpYyB2b2lkIHNldF9hZHZlcnRp
c2luZ19jb21wbGV0ZShzdHJ1Y3QgaGNpX2RldiAqaGRldiwgdTggc3RhdHVzKQ0KPj4gDQo+PiAJ
aWYgKG1hdGNoLnNrKQ0KPj4gCQlzb2NrX3B1dChtYXRjaC5zayk7DQo+PiArDQo+PiArdW5sb2Nr
Og0KPj4gKwloY2lfZGV2X3VubG9jayhoZGV2KTsNCj4+IH0NCj4+IA0KDQpUaGFua3MsDQpKYWdh
bmF0aA==

Re: [PATCH] Bluetooth: Fix missing hci_dev_lock/unlock

[Johan Hedberg]

Hi Jaganath & Marcel,

On Wed, Dec 10, 2014, JAGANATH KANAKKASSERY wrote:
> >> 	err = hci_dev_do_open(hdev);
> >> 	if (err < 0) {
> >> +		hci_dev_lock(hdev);
> >> 		mgmt_set_powered_failed(hdev, err);
> >> +		hci_dev_unlock(hdev);
> >> 		return;
> >> 	}
> 
> >I wonder is some of the mgmt_ function should just take the hci_dev
> >lock. Are there cases where we don't want them to take the look?
> 
> There are many mgmt_functions called from hci_event.c which don't
> require lock.  You prefer moving the lock inside
> mgmt_set_powered_failed()?

This should be kept consistent imo. Either all mgmt_* functions called
from hci_core/event.c should be responsible for taking the lock
themselves or none should be. I think right now the general rule is that
hci_core/event.c take the lock first, and this is mainly because a lot
of code that needed calling into mgmt.c was already holding the lock.
I.e. moving the taking into mgmt.c would for many places have required
awkward looking hdev_unlock(); mgmt_foo(); hdev_lock(); constructions.

Johan

Re: [PATCH] Bluetooth: Fix missing hci_dev_lock/unlock

[Marcel Holtmann]

Hi Johan,

>>>> 	err = hci_dev_do_open(hdev);
>>>> 	if (err < 0) {
>>>> +		hci_dev_lock(hdev);
>>>> 		mgmt_set_powered_failed(hdev, err);
>>>> +		hci_dev_unlock(hdev);
>>>> 		return;
>>>> 	}
>> 
>>> I wonder is some of the mgmt_ function should just take the hci_dev
>>> lock. Are there cases where we don't want them to take the look?
>> 
>> There are many mgmt_functions called from hci_event.c which don't
>> require lock.  You prefer moving the lock inside
>> mgmt_set_powered_failed()?
> 
> This should be kept consistent imo. Either all mgmt_* functions called
> from hci_core/event.c should be responsible for taking the lock
> themselves or none should be. I think right now the general rule is that
> hci_core/event.c take the lock first, and this is mainly because a lot
> of code that needed calling into mgmt.c was already holding the lock.
> I.e. moving the taking into mgmt.c would for many places have required
> awkward looking hdev_unlock(); mgmt_foo(); hdev_lock(); constructions.

we could also rename some to __mgmt_foo which means they are not taking the lock and others to mgmt_bar which take the lock by themselves. And then see if they actually have variations where we would need both.

Regards

Marcel

Re: Re: [PATCH] Bluetooth: Fix missing hci_dev_lock/unlock

[JAGANATH KANAKKASSERY]

SGkgTWFyY2VsIGFuZCBKb2hhbiwNCg0KPj4+Pj4gCWVyciA9IGhjaV9kZXZfZG9fb3BlbihoZGV2
KTsNCj4+Pj4+IAlpZiAoZXJyIDwgMCkgew0KPj4+Pj4gKwkJaGNpX2Rldl9sb2NrKGhkZXYpOw0K
Pj4+Pj4gCQltZ210X3NldF9wb3dlcmVkX2ZhaWxlZChoZGV2LCBlcnIpOw0KPj4+Pj4gKwkJaGNp
X2Rldl91bmxvY2soaGRldik7DQo+Pj4+PiAJCXJldHVybjsNCj4+Pj4+IAl9DQo+Pj4gDQo+Pj4+
IEkgd29uZGVyIGlzIHNvbWUgb2YgdGhlIG1nbXRfIGZ1bmN0aW9uIHNob3VsZCBqdXN0IHRha2Ug
dGhlIGhjaV9kZXYNCj4+Pj4gbG9jay4gQXJlIHRoZXJlIGNhc2VzIHdoZXJlIHdlIGRvbid0IHdh
bnQgdGhlbSB0byB0YWtlIHRoZSBsb29rPw0KPj4+IA0KPj4+IFRoZXJlIGFyZSBtYW55IG1nbXRf
ZnVuY3Rpb25zIGNhbGxlZCBmcm9tIGhjaV9ldmVudC5jIHdoaWNoIGRvbid0DQo+Pj4gcmVxdWly
ZSBsb2NrLiAgWW91IHByZWZlciBtb3ZpbmcgdGhlIGxvY2sgaW5zaWRlDQo+Pj4gbWdtdF9zZXRf
cG93ZXJlZF9mYWlsZWQoKT8NCj4+IA0KPj4gVGhpcyBzaG91bGQgYmUga2VwdCBjb25zaXN0ZW50
IGltby4gRWl0aGVyIGFsbCBtZ210XyogZnVuY3Rpb25zIGNhbGxlZA0KPj4gZnJvbSBoY2lfY29y
ZS9ldmVudC5jIHNob3VsZCBiZSByZXNwb25zaWJsZSBmb3IgdGFraW5nIHRoZSBsb2NrDQo+PiB0
aGVtc2VsdmVzIG9yIG5vbmUgc2hvdWxkIGJlLiBJIHRoaW5rIHJpZ2h0IG5vdyB0aGUgZ2VuZXJh
bCBydWxlIGlzIHRoYXQNCj4+IGhjaV9jb3JlL2V2ZW50LmMgdGFrZSB0aGUgbG9jayBmaXJzdCwg
YW5kIHRoaXMgaXMgbWFpbmx5IGJlY2F1c2UgYSBsb3QNCj4+IG9mIGNvZGUgdGhhdCBuZWVkZWQg
Y2FsbGluZyBpbnRvIG1nbXQuYyB3YXMgYWxyZWFkeSBob2xkaW5nIHRoZSBsb2NrLg0KPj4gSS5l
LiBtb3ZpbmcgdGhlIHRha2luZyBpbnRvIG1nbXQuYyB3b3VsZCBmb3IgbWFueSBwbGFjZXMgaGF2
ZSByZXF1aXJlZA0KPj4gYXdrd2FyZCBsb29raW5nIGhkZXZfdW5sb2NrKCk7IG1nbXRfZm9vKCk7
IGhkZXZfbG9jaygpOyBjb25zdHJ1Y3Rpb25zLg0KPg0KPndlIGNvdWxkIGFsc28gcmVuYW1lIHNv
bWUgdG8gX19tZ210X2ZvbyB3aGljaCBtZWFucyB0aGV5IGFyZSBub3QgdGFraW5nDQo+dGhlIGxv
Y2sgYW5kIG90aGVycyB0byBtZ210X2JhciB3aGljaCB0YWtlIHRoZSBsb2NrIGJ5IHRoZW1zZWx2
ZXMuIEFuZCB0aGVuDQo+c2VlIGlmIHRoZXkgYWN0dWFsbHkgaGF2ZSB2YXJpYXRpb25zIHdoZXJl
IHdlIHdvdWxkIG5lZWQgYm90aC4NCg0KSSBoYXZlIHJhaXNlZCB0aGUgc3BsaXQgcGF0Y2ggc2V0
cyBhbmQgZml4ZWQgY29kaW5nIHZpb2xhdGlvbiBhcyB3ZWxsLiBSZW5hbWluZyBvZg0KbWdtdC4g
ZnVuY3Rpb25zIGNhbiBiZSBkb25lIG9uIHRvcCBvZiB0aGlzIGlmIHJlcXVpcmVkLg0KDQoNClRo
YW5rcywNCkphZ2FuYXRo