Project

General

Profile

Bug #7218

pvscsi`pvscsi_bus_config uses wrong base while configuring targets

Added by Yuri Pankov about 4 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Yuri Pankov
Category:
driver - device drivers
Start date:
2016-07-24
Due date:
% Done:

100%

Estimated time:
Difficulty:
Bite-size
Tags:
Gerrit CR:

Description

pvscsi fails to configure targets 10-15 (a-f, that is) due to wrong base in ddi_strtol() call:

        case BUS_CONFIG_ONE:
                if ((p = strrchr((char *)arg, '@')) != NULL &&
                    ddi_strtol(p + 1, NULL, 10, &target) == 0)
                        ret = pvscsi_config_one(pdip, pvs, (int)target, childp);
                break;

The base argument should be 16.

History

#1

Updated by Electric Monk about 4 years ago

  • % Done changed from 50 to 100
  • Status changed from In Progress to Closed

git commit 6bd8a07093bddc0edfc07bfda4ca600e31c02c03

commit  6bd8a07093bddc0edfc07bfda4ca600e31c02c03
Author: Yuri Pankov <yuri.pankov@nexenta.com>
Date:   2016-08-03T16:11:49.000Z

    7218 pvscsi`pvscsi_bus_config uses wrong base while configuring targets
    7219 pvscsi panics while simultaneously deleting large number of disks
    Reviewed by: Hans Rosenfeld <hans.rosenfeld@nexenta.com>
    Reviewed by: Robert Mustacchi <rm@joyent.com>
    Approved by: Richard Lowe <richlowe@richlowe.net>

Also available in: Atom PDF