Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 07 Apr 2011 13:59:35 +0300
From:      Alexander Motin <mav@FreeBSD.org>
To:        Alexander Best <arundel@freebsd.org>
Cc:        freebsd-scsi@freebsd.org, freebsd-current@freebsd.org
Subject:   Re: multiple issues with devstat_*(9)
Message-ID:  <4D9D9917.3030102@FreeBSD.org>
In-Reply-To: <20110404204316.GA11367@freebsd.org>
References:  <20110331223339.GA13682@freebsd.org> <201104010843.47367.jhb@freebsd.org> <20110404204316.GA11367@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Alexander Best wrote:
> On Fri Apr  1 11, John Baldwin wrote:
>> On Thursday, March 31, 2011 6:33:39 pm Alexander Best wrote:
>>> i think there are multiple issues with devstat. i found the following in
>>> devicestat.h:

...

>>> funny thing is i found the following in scsi_pass.c:
>>>
>>>         softc->device_stats = devstat_new_entry("pass",
>>>                           periph->unit_number, 0,
>>>                           DEVSTAT_NO_BLOCKSIZE
>>>                           | (no_tags ? DEVSTAT_NO_ORDERED_TAGS : 0),
>>>                           softc->pd_type |
>>>                           DEVSTAT_TYPE_IF_SCSI |
>>>                           DEVSTAT_TYPE_PASS,
>>>                           DEVSTAT_PRIORITY_PASS);
>>>
>>> ...so pass* *should* show up under iostat -t scsi.

As I can see, this is a bug (or feature) of the libdevstat /
devstat_selectdevs(). If you specify any -t, then pass devices will be
reported only if you request "pass" specifically.

>> Hmm, pass devices for adaX should not be SCSI though, they should be ide I
>> think.
> 
> i think the situation with ATA_CAM should be discussed further. still besides
> this issue there are many more with devstat(3).
> 
> i'll try to track all the "devstat_new_entry()" occurrences and see if some
> issues can be fixed. maybe only the proper DEVSTAT_* args were forgotten.

Assuming that SCSI and IDE in -t option means transport type, and
assuming that we count everything except ATA and SATA as SCSI, I've made
following patch, that should fix issues from the CAM side:
http://people.freebsd.org/~mav/cam.devstat.patch

Any objections? Or SCSI/IDE there expected to mean command set?

-- 
Alexander Motin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4D9D9917.3030102>