11:03:04 the bad thing is, we actually do have not one but two whole disk device nodes, GPT label will add second one (minor 'u'). IMO it would be nice to have just one (minor 'u'), and p0 as symlink for transition time (deprecated). 18:41:08 I agree it would be good to have a separate explicit "whole disk" public link that works regardless of which label (or if no label) is present etc 18:42:47 I suspect we could do a better job at documenting this, too; e.g., there are notes in disks(8) which is a deprecated program 18:45:07 Also, TIL devinfo(8), what even is this! 18:51:50 I did remember wrong, its not 'u' conflicting minort node, the actual problem is that for GPT, we do create minor node with name "wd", but its numeric value is 7 and it is conflicting with VTOC node 'h'. 18:53:17 Right 18:53:30 There's obviously a lot of history here 18:54:57 The bare device name, e.g., c0t0d0, points to the "wd" minor, I believe. Is that still true VTOC labels? 18:55:44 it is not, with VTOC world, there is no node c0t0d0 18:55:55 Ah 18:56:02 I guess we could fix that by just creating one then haha 18:56:18 on sparc, there has been only s2 to access whole disk, and on x86, the p0 18:56:52 Right. 18:57:14 And on x86 s2 would get you access to the whole SOLARIS2 partition I guess 19:02:46 yes 19:06:44 errm. actually because on transition from EFI to VTOC and VTOC to EFI, we do remove and create minor devices as needed, we could just re-create "wd" node with other numeric value for minor. But of course, thats not really nice and it would be better to have one value. 19:08:07 I don't think the numeric value is particularly important 19:08:11 https://src.illumos.org/source/xref/illumos-gate/usr/src/uts/common/io/cmlb.c?r=bbf21555&mo=19489&fi=685#1561 19:09:40 actually it is, because we have #define WD_NODE 7 and it is used to index cl_map array like there: https://src.illumos.org/source/xref/illumos-gate/usr/src/uts/common/io/cmlb.c?r=bbf21555&mo=19489&fi=685#1149 19:10:24 so it is software and can be reworked, but still something to keep in mind 19:11:15 Yes I understand that the driver uses it internally, I just mean outside the driver it's more important that there be a consistently named minor available etc 19:12:10 I have filed: 17043 disk device nodes and label types could be documented in disks(7) 19:12:25 This has come up a lot in the last couple of years so it's probably time to write it down 19:12:40 Is the ARM stuff just 100% GPT? 19:13:16 probably -- AFAIK it all uses UEFI for better or worse, which would mean GPT 19:13:51 GPT is probably the most acceptable part of UEFI so that seems fine haha 19:14:44 there are other parts that aren't _too_ bad, but the windows-centric bits are quite annoying (e.g. UCS2 strings) 19:35:06 It as after all MS-DOS 2097! 19:35:09 *It is 19:42:15 jclulow: we support non-gpt, because of some booting quirks 19:42:52 jclulow: (and because that's all common code, I guess) 19:43:11 I just remember that andyf made it optional what we use on the images because of something or other (hadfl's rpi rev?) 19:43:20 something weird didn't work 19:45:50 yeah my old rpi4 didn't boot other than MBR. but a firmware update fixed it. so not relying on MBR anymore 20:02:31 well, we have seen system boot implemented as read N blocks from fixed location (N from [1-16]), now we see read file from file system, and to be honest, file system is much more better option. Even if its FAT. 20:15:23 richlowe: Interesting! 20:17:37 also: writing CamelCase C just feels wrong :P 20:17:47 (other UEFI quirks) 20:17:51 :D 20:19:04 It's not great! 20:24:45 one can always just use linuxboot;) 20:49:53 [illumos-gate] 17031 ddi_intr_add_handler(9f): several issues -- Yuri Pankov 20:50:00 @yuripv ^^^ 21:24:24 [illumos-gate] 17012 ZTS: add new_fs -- Toomas Soome 21:32:19 BootLayer [~BootLayer@user/BootLayer] has quit [Quit: Leaving] 21:32:36 uhhh, unintentionally copypasted part of my irssi buffer? sorry about that 21:36:47 I thought it was a joke about us complaining about UEFI and then "BootLayer" quitting, tbh 22:36:27 [illumos-gate] 16835 mkfs_pcfs: allow sector sizes other than 512B -- Toomas Soome 22:41:35 That's usually how I hedge my bets when mine don't land 22:42:09 "oh, yeah, I was just trying to quit vi ha ha" 22:42:17 for 4kn, https://code.illumos.org/c/illumos-gate/+/3738 is left for OS itself. 22:42:18 → CODE REVIEW 3738: 16837 libzfs: pool start needs to be properly aligned with zpool_label_disk() (NEW) | https://www.illumos.org/issues/16837