Re: ntb_transport: Pick an unused queue

[Jon Mason <jdmason@kudzu.us>]

On Tue, Feb 14, 2017 at 09:20:00AM -0500, Allen Hubbe wrote:
> From: Aaron Sierra
> > From: Thomas VanSelus <tvanselus@xes-inc.com>
> > 
> > Fix typo causing ntb_transport_create_queue to select the first
> > queue every time, instead of using the next free queue.
> > 
> > Signed-off-by: Thomas VanSelus <tvanselus@xes-inc.com>
> > Signed-off-by: Aaron Sierra <asierra@xes-inc.com>
> 
> Acked-by: Allen Hubbe <Allen.Hubbe@dell.com>

Good catch!  This has been in there from the beginning.

Add a
    Fixes: fce8a7bb5 ("PCI-Express Non-Transparent Bridge Support")
and applied to my ntb branch.

Thanks,
Jon

> 
> > ---
> >  drivers/ntb/ntb_transport.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/ntb/ntb_transport.c b/drivers/ntb/ntb_transport.c
> > index f81aa4b..95b5540 100644
> > --- a/drivers/ntb/ntb_transport.c
> > +++ b/drivers/ntb/ntb_transport.c
> > @@ -1802,7 +1802,7 @@ ntb_transport_create_queue(void *data, struct device *client_dev,
> > 
> >  	node = dev_to_node(&ndev->dev);
> > 
> > -	free_queue = ffs(nt->qp_bitmap);
> > +	free_queue = ffs(nt->qp_bitmap_free);
> >  	if (!free_queue)
> >  		goto err;
> > 
>