On Fri, Sep 22, 2023 at 01:05:17PM -0500, Brian Conway wrote:
> Greetings. I noticed the above behavior with the latest 7.4-beta when
> scripting some fw_update runs. Perhaps a corner case was missed in the
> recent work? Steps below, let me know if more information is required
> to reproduce. Thanks in advance.

Yup, I bet I missed several exit code issues.  This one is obviously
wrong, but I have to decide what to do in the:

    fw_update -d intel nonexist
    fw_update -d intel # not installed, success?
    fw_update    intel nonexist
    fw_update    intel # already installed, success?

I'll take a look at it.  Not entirely sure if I'll have time before a
week from tomorrow, but we shall see.

> root:current-amd64.124:0:~# fw_update -d intel; echo ${?}
> fw_update: delete intel
> 0
> root:current-amd64.124:0:~# fw_update; echo ${?}          
> fw_update: add intel; update none; keep inteldrm,vmm
> 0
> root:current-amd64.124:0:~# fw_update -d intel; echo ${?} 
> fw_update: delete intel
> 0
> root:current-amd64.124:0:~# fw_update intel; echo ${?} 
> fw_update: add intel; update none
> 1

That is a bit surprising though, I kind of assumed if it got to printing
the end of the status it would always exit 0.


> 
> Brian Conway
> Owner
> RCE Software, LLC
> 
> OpenBSD 7.4-beta (GENERIC.MP) #1373: Thu Sep 21 09:26:32 MDT 2023
>     dera...@amd64.openbsd.org:/usr/src/sys/arch/amd64/compile/GENERIC.MP
> real mem = 4160446464 (3967MB)
> avail mem = 4014624768 (3828MB)
> random: good seed from bootblocks
> mpath0 at root
> scsibus0 at mpath0: 256 targets
> mainbus0 at root
> bios0 at mainbus0: SMBIOS rev. 3.2 @ 0x9b810000 (97 entries)
> bios0: vendor American Megatrends Inc. version "5.17" date 03/25/2021
> bios0: Default string Default string
> efi0 at bios0: UEFI 2.7
> efi0: American Megatrends rev 0x50011
> acpi0 at bios0: ACPI 6.2
> acpi0: sleep states S0 S5
> acpi0: tables DSDT FACP MCFG SSDT FIDT SSDT HPET SSDT SSDT SSDT NHLT LPIT 
> SSDT SSDT DBGP DBG2 SSDT DMAR WSMT APIC FPDT
> acpi0: wakeup devices PS2K(S0) PS2M(S0) RP01(S0) PXSX(S0) RP02(S0) PXSX(S0) 
> RP03(S0) PXSX(S0) RP04(S0) PXSX(S0) RP05(S0) PXSX(S0) RP06(S0) PXSX(S0) 
> RP07(S0) PXSX(S0) [...]
> acpitimer0 at acpi0: 3579545 Hz, 24 bits
> acpimcfg0 at acpi0
> acpimcfg0: addr 0xe0000000, bus 0-255
> acpihpet0 at acpi0: 23999999 Hz
> acpimadt0 at acpi0 addr 0xfee00000: PC-AT compat
> cpu0 at mainbus0: apid 0 (boot processor)
> cpu0: Intel(R) Celeron(R) CPU 5305U @ 2.30GHz, 2194.89 MHz, 06-8e-0c, patch 
> 000000f8
> cpu0: 
> FPU,VME,DE,PSE,TSC,MSR,PAE,MCE,CX8,APIC,SEP,MTRR,PGE,MCA,CMOV,PAT,PSE36,CFLUSH,DS,ACPI,MMX,FXSR,SSE,SSE2,SS,HTT,TM,PBE,SSE3,PCLMUL,DTES64,MWAIT,DS-CPL,VMX,EST,TM2,SSSE3,SDBG,CX16,xTPR,PDCM,PCID,SSE4.1,SSE4.2,x2APIC,MOVBE,POPCNT,DEADLINE,AES,XSAVE,RDRAND,NXE,PAGE1GB,RDTSCP,LONG,LAHF,ABM,3DNOWP,PERF,ITSC,FSGSBASE,TSC_ADJUST,SGX,SMEP,ERMS,INVPCID,MPX,RDSEED,SMAP,CLFLUSHOPT,PT,SRBDS_CTRL,MD_CLEAR,IBRS,IBPB,STIBP,L1DF,SSBD,SENSOR,ARAT,IBRS_ALL,SKIP_L1DFL,MDS_NO,MISC_PKG_CT,ENERGY_FILT,FB_CLEAR,RRSBA,GDS_CTRL,XSAVEOPT,XSAVEC,XGETBV1,XSAVES
> cpu0: 32KB 64b/line 8-way D-cache, 32KB 64b/line 8-way I-cache, 256KB 
> 64b/line 4-way L2 cache, 2MB 64b/line 8-way L3 cache
> cpu0: smt 0, core 0, package 0
> mtrr: Pentium Pro MTRR support, 10 var ranges, 88 fixed ranges
> cpu0: apic clock running at 24MHz
> cpu0: mwait min=64, max=64, C-substates=0.2.1.2.4.1.1.1, IBE
> cpu1 at mainbus0: apid 2 (application processor)
> cpu1: Intel(R) Celeron(R) CPU 5305U @ 2.30GHz, 2194.90 MHz, 06-8e-0c, patch 
> 000000f8
> cpu1: 
> FPU,VME,DE,PSE,TSC,MSR,PAE,MCE,CX8,APIC,SEP,MTRR,PGE,MCA,CMOV,PAT,PSE36,CFLUSH,DS,ACPI,MMX,FXSR,SSE,SSE2,SS,HTT,TM,PBE,SSE3,PCLMUL,DTES64,MWAIT,DS-CPL,VMX,EST,TM2,SSSE3,SDBG,CX16,xTPR,PDCM,PCID,SSE4.1,SSE4.2,x2APIC,MOVBE,POPCNT,DEADLINE,AES,XSAVE,RDRAND,NXE,PAGE1GB,RDTSCP,LONG,LAHF,ABM,3DNOWP,PERF,ITSC,FSGSBASE,TSC_ADJUST,SGX,SMEP,ERMS,INVPCID,MPX,RDSEED,SMAP,CLFLUSHOPT,PT,SRBDS_CTRL,MD_CLEAR,IBRS,IBPB,STIBP,L1DF,SSBD,SENSOR,ARAT,IBRS_ALL,SKIP_L1DFL,MDS_NO,MISC_PKG_CT,ENERGY_FILT,FB_CLEAR,RRSBA,GDS_CTRL,XSAVEOPT,XSAVEC,XGETBV1,XSAVES
> cpu1: 32KB 64b/line 8-way D-cache, 32KB 64b/line 8-way I-cache, 256KB 
> 64b/line 4-way L2 cache, 2MB 64b/line 8-way L3 cache
> cpu1: smt 0, core 1, package 0
> ioapic0 at mainbus0: apid 2 pa 0xfec00000, version 20, 120 pins
> acpiprt0 at acpi0: bus 0 (PCI0)
> acpiprt1 at acpi0: bus -1 (PEG0)
> acpiprt2 at acpi0: bus -1 (PEG1)
> acpiprt3 at acpi0: bus -1 (PEG2)
> acpiprt4 at acpi0: bus -1 (RP01)
> acpiprt5 at acpi0: bus -1 (RP02)
> acpiprt6 at acpi0: bus -1 (RP03)
> acpiprt7 at acpi0: bus -1 (RP04)
> acpiprt8 at acpi0: bus -1 (RP05)
> acpiprt9 at acpi0: bus -1 (RP06)
> acpiprt10 at acpi0: bus 1 (RP07)
> acpiprt11 at acpi0: bus 2 (RP08)
> acpiprt12 at acpi0: bus -1 (RP09)
> acpiprt13 at acpi0: bus -1 (RP10)
> acpiprt14 at acpi0: bus -1 (RP11)
> acpiprt15 at acpi0: bus -1 (RP12)
> acpiprt16 at acpi0: bus 3 (RP13)
> acpiprt17 at acpi0: bus 4 (RP14)
> acpiprt18 at acpi0: bus 5 (RP15)
> acpiprt19 at acpi0: bus 6 (RP16)
> acpiprt20 at acpi0: bus -1 (RP17)
> acpiprt21 at acpi0: bus -1 (RP18)
> acpiprt22 at acpi0: bus -1 (RP19)
> acpiprt23 at acpi0: bus -1 (RP20)
> acpiprt24 at acpi0: bus -1 (RP21)
> acpiprt25 at acpi0: bus -1 (RP22)
> acpiprt26 at acpi0: bus -1 (RP23)
> acpiprt27 at acpi0: bus -1 (RP24)
> acpiec0 at acpi0: not present
> acpipci0 at acpi0 PCI0: 0x00000000 0x00000011 0x00000001
> com0 at acpi0 UAR1 addr 0x3f8/0x8 irq 4: ns16550a, 16 byte fifo
> pchgpio0 at acpi0 GPI0 addr 0xfd6e0000/0x10000 0xfd6d0000/0x10000 
> 0xfd6a0000/0x10000 irq 14, 320 pins
> "PNP0C14" at acpi0 not configured
> acpibtn0 at acpi0: SLPB
> "PNP0C14" at acpi0 not configured
> "PNP0C14" at acpi0 not configured
> "INT33A1" at acpi0 not configured
> "ACPI000E" at acpi0 not configured
> acpibtn1 at acpi0: PWRB
> "PNP0C0B" at acpi0 not configured
> "PNP0C0B" at acpi0 not configured
> "PNP0C0B" at acpi0 not configured
> "PNP0C0B" at acpi0 not configured
> "PNP0C0B" at acpi0 not configured
> acpipwrres0 at acpi0: PXP_, resource for RP05
> acpipwrres1 at acpi0: PXP_, resource for RP09
> acpipwrres2 at acpi0: USBC, resource for XDCI
> acpipwrres3 at acpi0: V0PR
> acpipwrres4 at acpi0: V1PR
> acpipwrres5 at acpi0: V2PR
> acpipwrres6 at acpi0: PXTC
> acpipwrres7 at acpi0: WRST
> acpipwrres8 at acpi0: PAUD
> acpicpu0 at acpi0: C3(200@1034 mwait.1@0x60), C2(200@151 mwait.1@0x33), 
> C1(1000@1 mwait.1), PSS
> acpicpu1 at acpi0: C3(200@1034 mwait.1@0x60), C2(200@151 mwait.1@0x33), 
> C1(1000@1 mwait.1), PSS
> acpipwrres9 at acpi0: FN00, resource for FAN0
> acpipwrres10 at acpi0: FN01, resource for FAN1
> acpipwrres11 at acpi0: FN02, resource for FAN2
> acpipwrres12 at acpi0: FN03, resource for FAN3
> acpipwrres13 at acpi0: FN04, resource for FAN4
> acpitz0 at acpi0: critical temperature is 119 degC
> acpipwrres14 at acpi0: PIN_
> acpivideo0 at acpi0: GFX0
> acpivout0 at acpivideo0: DD1F
> cpu0: Enhanced SpeedStep 2194 MHz: speeds: 2301, 2300, 2200, 2000, 1900, 
> 1800, 1600, 1500, 1300, 1200, 1100, 900, 800, 700, 500, 400 MHz
> pci0 at mainbus0 bus 0
> pchb0 at pci0 dev 0 function 0 "Intel Core 10G Host" rev 0x0c
> inteldrm0 at pci0 dev 2 function 0 "Intel UHD Graphics" rev 0x02
> drm0 at inteldrm0
> inteldrm0: msi, COMETLAKE, gen 9
> "Intel Core GMM" rev 0x00 at pci0 dev 8 function 0 not configured
> pchtemp0 at pci0 dev 18 function 0 "Intel 400 Series Thermal" rev 0x00
> xhci0 at pci0 dev 20 function 0 "Intel 400 Series xHCI" rev 0x00: msi, xHCI 
> 1.10
> usb0 at xhci0: USB revision 3.0
> uhub0 at usb0 configuration 1 interface 0 "Intel xHCI root hub" rev 3.00/1.00 
> addr 1
> "Intel 400 Series Shared SRAM" rev 0x00 at pci0 dev 20 function 2 not 
> configured
> sdhc0 at pci0 dev 20 function 5 "Intel 400 Series SDXC" rev 0x00: apic 2 int 
> 19
> sdhc0: SDHC 3.0, 200 MHz base clock
> sdmmc0 at sdhc0: 4-bit, sd high-speed, mmc high-speed, ddr52, dma
> dwiic0 at pci0 dev 21 function 0 "Intel 400 Series I2C" rev 0x00: apic 2 int 
> 16
> iic0 at dwiic0
> dwiic1 at pci0 dev 21 function 1 "Intel 400 Series I2C" rev 0x00: apic 2 int 
> 17
> iic1 at dwiic1
> "Intel 400 Series MEI" rev 0x00 at pci0 dev 22 function 0 not configured
> ahci0 at pci0 dev 23 function 0 "Intel 400 Series AHCI" rev 0x00: msi, AHCI 
> 1.3.1
> ahci0: port 0: 6.0Gb/s
> ahci0: PHY offline on port 1
> scsibus1 at ahci0: 32 targets
> sd0 at scsibus1 targ 0 lun 0: <ATA, DOGFISH MSATA 64, W022> 
> t10.ATA_DOGFISH_MSATA_64GB_GV230530L000000734_
> sd0: 61057MB, 512 bytes/sector, 125045424 sectors, thin
> dwiic2 at pci0 dev 25 function 0 "Intel 400 Series I2C" rev 0x00: apic 2 int 
> 32
> iic2 at dwiic2
> sdhc1 at pci0 dev 26 function 0 "Intel 400 Series eMMC" rev 0x00: apic 2 int 
> 16
> sdhc1: SDHC 3.0, 200 MHz base clock
> sdmmc1 at sdhc1: 8-bit, sd high-speed, mmc high-speed, ddr52, dma
> ppb0 at pci0 dev 28 function 0 "Intel 400 Series PCIE" rev 0xf0: msi
> pci1 at ppb0 bus 1
> em0 at pci1 dev 0 function 0 "Intel I210" rev 0x03: msi, address 
> 00:f1:f3:1d:5d:4a
> ppb1 at pci0 dev 28 function 7 "Intel 400 Series PCIE" rev 0xf0: msi
> pci2 at ppb1 bus 2
> em1 at pci2 dev 0 function 0 "Intel I210" rev 0x03: msi, address 
> 00:f1:f3:1d:5d:4b
> ppb2 at pci0 dev 29 function 0 "Intel 400 Series PCIE" rev 0xf0: msi
> pci3 at ppb2 bus 3
> em2 at pci3 dev 0 function 0 "Intel I210" rev 0x03: msi, address 
> 00:f1:f3:1d:5d:4c
> ppb3 at pci0 dev 29 function 5 "Intel 400 Series PCIE" rev 0xf0: msi
> pci4 at ppb3 bus 4
> em3 at pci4 dev 0 function 0 "Intel I210" rev 0x03: msi, address 
> 00:f1:f3:1d:5d:4d
> ppb4 at pci0 dev 29 function 6 "Intel 400 Series PCIE" rev 0xf0: msi
> pci5 at ppb4 bus 5
> em4 at pci5 dev 0 function 0 "Intel I210" rev 0x03: msi, address 
> 00:f1:f3:1d:5d:4e
> ppb5 at pci0 dev 29 function 7 "Intel 400 Series PCIE" rev 0xf0: msi
> pci6 at ppb5 bus 6
> em5 at pci6 dev 0 function 0 "Intel I210" rev 0x03: msi, address 
> 00:f1:f3:1d:5d:4f
> pcib0 at pci0 dev 31 function 0 "Intel 400 Series LPC" rev 0x00
> ichiic0 at pci0 dev 31 function 4 "Intel 400 Series SMBus" rev 0x00: apic 2 
> int 16
> iic3 at ichiic0
> spdmem0 at iic3 addr 0x50: 4GB DDR4 SDRAM PC4-21300 SO-DIMM
> "Intel 400 Series SPI" rev 0x00 at pci0 dev 31 function 5 not configured
> isa0 at pcib0
> isadma0 at isa0
> pckbc0 at isa0 port 0x60/5 irq 1 irq 12
> pckbd0 at pckbc0 (kbd slot)
> wskbd0 at pckbd0: console keyboard
> pcppi0 at isa0 port 0x61
> spkr0 at pcppi0
> it0 at isa0 port 0x2e/2: IT8772F rev 1, EC port 0xa40
> vmm0 at mainbus0: VMX/EPT
> efifb at mainbus0 not configured
> sdmmc1: can't enable card
> uhidev0 at uhub0 port 3 configuration 1 interface 0 "Logitech USB Keyboard" 
> rev 1.10/64.00 addr 2
> uhidev0: iclass 3/1
> ukbd0 at uhidev0: 8 variable keys, 6 key codes
> wskbd1 at ukbd0 mux 1
> uhidev1 at uhub0 port 3 configuration 1 interface 1 "Logitech USB Keyboard" 
> rev 1.10/64.00 addr 2
> uhidev1: iclass 3/0, 3 report ids
> ucc0 at uhidev1 reportid 1: 2 usages, 3 keys, enum
> wskbd2 at ucc0 mux 1
> uhid0 at uhidev1 reportid 2: input=1, output=0, feature=0
> ucc1 at uhidev1 reportid 3: 21 usages, 14 keys, enum
> wskbd3 at ucc1 mux 1
> vscsi0 at root
> scsibus2 at vscsi0: 256 targets
> softraid0 at root
> scsibus3 at softraid0: 256 targets
> root on sd0a (1c9afc71f1ee6dbb.a) swap on sd0b dump on sd0b
> drm:pid0:drm_dp_dual_mode_detect *ERROR* [drm] *ERROR* Unexpected DP dual 
> mode adaptor ID 7f
> inteldrm0: 1920x1080, 32bpp
> wsdisplay0 at inteldrm0 mux 1: console (std, vt100 emulation), using wskbd0
> wskbd1: connecting to wsdisplay0
> wskbd2: connecting to wsdisplay0
> wskbd3: connecting to wsdisplay0
> wsdisplay0: screen 1-5 added (std, vt100 emulation)
> 

-- 
andrew

Beta. Software undergoes beta testing shortly before it's released.
                           Beta is Latin for "still doesn't work."

Reply via email to