Re: [RFC] Bluetooth: AMP: Do not set name for AMP ctrl

[Andrei Emeltchenko <Andrei.Emeltchenko.news@gmail.com>]

Hi Johan,

On Wed, Jun 06, 2012 at 11:27:36PM +0800, Johan Hedberg wrote:
> Hi,
> 
> On Wed, Jun 06, 2012, Johan Hedberg wrote:
> > On Wed, Jun 06, 2012, Andrei Emeltchenko wrote:
> > > AMP controllers do not understand this command
> > > 
> > > ...
> > > < HCI Command: Write Local Name (0x03|0x0013) plen 248
> > >    name ''
> > > > HCI Event: Command Complete (0x0e) plen 4
> > >    Write Local Name (0x03|0x0013) ncmd 1
> > >    status 0x01
> > >    Error: Unknown HCI Command
> > > ...
> > > 
> > > Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@intel.com>
> > > ---
> > >  net/bluetooth/mgmt.c |    3 +++
> > >  1 file changed, 3 insertions(+)
> > > 
> > > diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
> > > index 958f764..9afcc84 100644
> > > --- a/net/bluetooth/mgmt.c
> > > +++ b/net/bluetooth/mgmt.c
> > > @@ -2135,6 +2135,9 @@ static int set_local_name(struct sock *sk, struct hci_dev *hdev, void *data,
> > >  
> > >  	BT_DBG("");
> > >  
> > > +	if (hdev->amp_type != HCI_BREDR)
> > > +		return -ENOTSUPP;
> > > +
> > >  	hci_dev_lock(hdev);
> > >  
> > >  	memcpy(hdev->short_name, cp->short_name, sizeof(hdev->short_name));
> > 
> > This will essentially make write() fail on the mgmt socket which isn't
> > necessarily the most intuitive behavior (one might think there's
> > something wrong with the socket). What would probably make more sense is
> > to send a proper cmd_status reply with MGMT_STATUS_NOT_SUPPORTED.
> 
> Actually, should we even be exposing AMP controllers to begin with
> through mgmt? Maybe we shouldn't even tell user space about them through
> mgmt?

This seems to be right. BTW: What is the best way to disable mgmt for
device?
 - Clear HCI_MGMT flag ?
 - Do not run mgmt_powered ?

BTW: Where that flag HCI_MGMT is set and how this check supposed to be
working?

test_and_set_bit(HCI_MGMT, &hdev->dev_flags))

Best regards 
Andrei Emeltchenko