Skip to content

Commit

Permalink
IB/ipath: Make handling of one subport consistent
Browse files Browse the repository at this point in the history
Previously the driver and userspace code handled the case of 1 subport
somewhat inconsistently.  The new interpretation of this situation is
that if one subport is requested, the driver turns on the subport
mechanism and arranges for the port to be "shared" by one process.  In
normal use the userspace library does not use this configuration and
instead arranges for the port not to be shared at all.  This
particular idiom can be useful for testing purposes.

Signed-off-by: Mark Debbage <mark.debbage@qlogic.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
  • Loading branch information
Mark Debbage authored and Roland Dreier committed Jul 10, 2007
1 parent 0df6291 commit bacf401
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions drivers/infiniband/hw/ipath/ipath_file_ops.c
Original file line number Diff line number Diff line change
Expand Up @@ -1444,10 +1444,10 @@ static int init_subports(struct ipath_devdata *dd,
size_t size;

/*
* If the user is requesting zero or one port,
* If the user is requesting zero subports,
* skip the subport allocation.
*/
if (uinfo->spu_subport_cnt <= 1)
if (uinfo->spu_subport_cnt <= 0)
goto bail;

/* Self-consistency check for ipath_compatible_subports() */
Expand Down

0 comments on commit bacf401

Please sign in to comment.