Wed, Oct 9
Sun, Oct 6
Thu, Sep 26
while we've transitioned to teken, it looks like it eats the ESC c
Ok, that looks pretty tough to deal with. Now that you mention it, I'm not seeing colors on the EFI serial console either, and under bhyve the scroll region is broken after the loader hands off to the kernel as well. It's everything I wanted to fix, still broken! Ugh! I thought I had tested that, but I guess I only tested a video output with the EFI console.
It is the most maddening part to get right... It's why I never upstreamed my hacks that I knew didn't work.
One way of dealing with this could be to add a variable to bypass teken for users of serial EFI consoles. A more challenging but perhaps more flexible option would be to implement a driver for the EFI serial protocol and either expose it as something like "efiserial" console or maybe add special handling to the existing EFI console driver to duplicate ANSI sequences to the serial port.
I actually think that's quite hard and involved. The current code doesn't differentiate, unfortunately. It would take a little while to understand a good path forward. At the moment, I see parsing ConOut, looking up handles and using that in something complicated, but there may be something simpler...
I don't think solving that problem with the EFI console/teken is within the scope of this change, though I am certainly hoping it gets fixed. I've only learned of teken's existence a few days ago. Can someone more familiar with that piece provide their input on the issue?
This can really only work for comconsole. And even there it can only be imperfect. And people will complain and expect me to fix it, so I'm reluctant to sign off until we have a better notion of a path forward...
Wed, Sep 25
Sun, Sep 22
Fri, Sep 20
Hmm... on second thought, this might need tested with a black-on-white xterm. I seem to recall having to spit out color.default() because we couldn't cope with the different background color and kept screwing up @emaste's xterm. I'm assuming this will put us back into that state, and we'll need to have the color.default() back after the reset.
Tue, Sep 17
Mon, Sep 16
Sep 14 2019
Actually yes, this change isn't correct. See https://github.com/tianocore/edk2/blob/master/MdePkg/Library/UefiMemoryAllocationLib/MemoryAllocationLib.c for an example of how to allocate aligned memory (InternalAllocateAlignedPages etc.)
Yes, this fix is not complete. We should always use "bounce buffer" here and only use aligned memory. Of course the disk write is also broken here (unless I'm mistaken)...
Sep 12 2019
Sep 10 2019
Sep 8 2019
Sep 6 2019
Sep 5 2019
Sep 2 2019
Aug 31 2019
Aug 30 2019
Aug 22 2019
Aug 20 2019
review suggestions from illumos.
Aug 19 2019
About the EFI_HTTP_Config_Data:
Aug 9 2019
Aug 8 2019
Aug 6 2019
read vim_entries by chunks.
Aug 5 2019
love the idea, but think it would be better to create a centralized location where we wrap this.
Call me paranoid, but I fear there may be a EFI bios we need to do this for :(
But also, it will make the code easier to read, so win/win.
Aug 4 2019
Jul 2 2019
Jun 30 2019
Jun 26 2019
Jun 19 2019
also chain /boot/loader.efi does not complain about network any more. network boot seems to be all ok too.
Jun 9 2019
Jun 8 2019
Jun 6 2019
Jun 5 2019
Agreed, a sane vdev_probe() would be nice. But I have totally no idea how you want to omit cleaning up partially initialized spa/vdev objects without rewriting the whole file. Either some of the current error paths are dead code or things like vdev_init_from_nvlist() have to be replaced.
Jun 4 2019
Jun 3 2019
Yes, but not for all failures. If a label is found in the loop but later the associated data found invalid the vdev_probe() will fail even if there would be another label with better data.
Jun 2 2019
Something is off there. The for loop in vdev_probe() does continue in case of failures, but is intended to walk through all 4 labels, so why do we fail?
Jun 1 2019
May 31 2019
There is problem now. The zfs_alloc was "always" succeeding but malloc can fail, therefore we can not just replace here, but we also need to add checks.