Re: [developer] Error scsi_vhci

["Udo Grabowski (IMK)" <udo.grabowski@kit.edu>]

[-- Attachment #1.1: Type: text/plain, Size: 2773 bytes --]

On 19-11-2024 19:13, Udo Grabowski (IMK) wrote:
> On 19-11-2024 19:07, Udo Grabowski (IMK) wrote:
>>
>>
>> On 19-11-2024 19:00, Udo Grabowski (IMK) wrote:
>>> On 19-11-2024 18:46, Udo Grabowski (IMK) wrote:
>>>> Hi,
>>>> for a while now, stmsboot drops the error message below, but after
>>>> reboot, it works fine. The message is actually caused by
>>>> /lib/mpxio/stmsboot_util -D mpt_sas -N, which spits out '|iport'
>>>> instead of 'iport', letting the search in /dev/dsk/* for it fail.
>>>> The consequence is that former automatic discovery for vhci-capable
>>>> disks fails and you have to revert to the failover list in
>>>> /kernel/drv/scsi_vhci.conf again. stmsboot_util is a compiled
>>>> executable.
>>>>
>>>> # stmsboot -e -D mpt_sas
>>>> WARNING: This operation will require a reboot.
>>>> Do you want to continue ? [y/n] (default: y) y
>>>> egrep: RE error in |iport: invalid regular expression  <===========
>>>> ...
>>>
>>>
>>> Looks like an oversight for an edge case:
>>>
>>>   -N
>>> 385       *    same as the -n option, except that we only print the
>>> 386       *    node-name (dev_info :: devi_node_name). Multiple 
>>> instances
>>> 387       *    through the libdevinfo snapshot are uniqified and 
>>> separated
>>> 388       *    by the "|" character for direct use by egrep(1).
>>>
>>> For only one item, there should be no "|" .
>>>
>>
>> This is the relevant section in  usr/src/cmd/stmsboot/stmsboot_util.c
>>
>> 1144              if (cap_N_option) {
>> 1145                  char *nodename = di_node_name(curnode);
>> 1146                  /* nodename is never going to be null */
>> 1147                  if (strstr(aliases, nodename) == NULL)
>> 1148                      /* haven't seen this nodename before */
>> 1149                      (void) snprintf(aliases,
>> 1150                          MAXPATHLEN + 1, "%s|%s",
>> 1151                          ((aliases != NULL) ? aliases : ""),
>> 1152                          nodename);
>> 1153              } else
>> 1154                  (void) printf("%s\n", path);
> 
> Setting the aliases part to " " instead of "" would fix it, but I
> don't know if that is correct for all situations.

No, there may exist blanks.
-- 
Dr.Udo Grabowski  Inst.of Meteorology & Climate Research IMK-ASF-SAT
https://www.imk-asf.kit.edu/english/sat.php
KIT - Karlsruhe Institute of Technology          https://www.kit.edu
Postfach 3640,76021 Karlsruhe,Germany T:(+49)721 608-26026 F:-926026


[-- Attachment #1.2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 6041 bytes --]

[-- Attachment #2: Type: text/plain, Size: 254 bytes --]


------------------------------------------
illumos: illumos-developer
Permalink: https://illumos.topicbox.com/groups/developer/Tf791f87e1cf696f7-Mc76de8992b98ba413fb086d6
Delivery options: https://illumos.topicbox.com/groups/developer/subscription