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."