Wed, Sep 18
This is very specific to AIM64 HPT. Can you rename it to kvm_..._hpt.c, or something, and add another key to the header to probe against for the particular pmap implementation? I'll want to add Book-E support, as well as AIM Radix when we do finally get radix stabilized.
Tue, Sep 17
Sat, Sep 14
Fri, Sep 13
Thu, Sep 12
This should be able to go in before flag day, there's nothing preventing it from working (we don't need linker support just for this, it's all in code).
Tue, Sep 10
Mon, Sep 9
Fri, Sep 6
Only a trivial review so far.
I like it, except for the typo :)
@i_maskray.me, you're right. It should just be
Thu, Sep 5
Can you instead add something like the following to crt1.c?
Address luporl's comments.
Wed, Sep 4
Mon, Sep 2
On Book-E it's already allocated out of KVA, as part of the bootstrap carving. Looks like on AIM I could just adjust virtual_avail as you mention in your comment in D21491, there would be up to 32MB (less 2 pages) slop, (15MB less 1 page on either side) which is probably fine on NUMA machines. But we do only have 32GB KVA available currently, and if the vm_page_array is ~3% of total memory, on a 256GB machine that's ~8GB, which is pretty big. I think I'd rather hold off on it until we either increase KVA or get actual working dump support.
@mjg glib does use fdwalk() if available, and emulates it where needed. This work I started specifically because I saw this same code flow in multiple ports.
Sun, Sep 1
Address feedback. I think this is better overall, simplifies things.
Sat, Aug 31
This change appears to give me a 15-20% improvement with building llvm on my Talos (144 thread POWER9). No change at all with buildworld.
Wed, Aug 28
Tue, Aug 27
Mon, Aug 26
You're right, I don't know what I was thinking. The page_range change is the only thing needed from this diff.
Sun, Aug 25
Aug 21 2019
Aug 20 2019
Aug 18 2019
Fix 32-bit build.
Aug 17 2019
It shouldn't be built for powerpc32, only powerpc64. However, it doesn't hurt anything, and is a good belt-and-suspenders change.
Aug 16 2019
Aug 15 2019
Address @kib's feedback. The man page will probably need more iterating, but I
think the code is correct. It may not be optimal, though.
Aug 14 2019
Don't call getpid(), we can pass 0 now.
Aug 13 2019
Add a new sysctl, kern.proc.fdmap.<proc>, to provide the bitmap. Walk the bitmap instead of the file descriptor list.
Aug 12 2019
Aug 11 2019
It's not hard to add such a sysctl. If you want I can hack it up for you. I think it should just export the fd bitmap, then iteration over it is trivial.
Aug 10 2019
Address @kib's feedback. Man page will probably require several iterations
to get right. It's still a little muddy.
Aug 9 2019
Aug 8 2019
Looks fine overall, just the one question. I just want to be sure this is thoroughly tested before it goes in, on the corner cases.
Looks good now.
Aug 6 2019
Aug 5 2019
Aug 4 2019
Aug 3 2019
Looks good. Can the same change apply to powerpc?
Aug 2 2019
Address @hselasky's comments.
Aug 1 2019
Jul 31 2019
Thanks for checking. Looks good. I assume you tested on bare metal, too?
@hselasky yes, I'm working on it, but got sidetracked doing other things while working on it. I'll have an update either tonight or tomorrow.
Jul 30 2019
Jul 29 2019
Jul 28 2019
I'll have a closer look tomorrow. Most of this looks good.
Is this patch also needed for powerpc?
Jul 25 2019
Jul 24 2019
Have you checked performance on this latest version?