- User Since
- Mar 11 2014, 8:46 PM (240 w, 4 d)
Wed, Oct 17
My last note is a suggestion, it's also fine as-is. I will probably work on changing the pmap to honor VM_PROT_EXECUTE next which will alter this a bit anyway and might use that approach then.
Tue, Oct 16
I think I would perhaps reword the commit message as "2nd Scheme" isn't very descriptive. Maybe something like:
Mon, Oct 15
Thu, Oct 11
- Drop commented out fence at the end of sfence_vma.
- Use static instead of PMAP_INLINE for pmap_invalidate_*().
- Use the SBI interface for remote fence.i for pmap_sync_icache.
- Flush the i-cache on all harts before writing a new PTE for an executable mapping.
- Don't use SFENCE.VMA after changing a mapping from invalid to valid.
Wed, Oct 10
Tue, Oct 9
- Fix whitespace in sbi_call().
- Drop fence() before single-threaded fence_i().
- Trim parens.
Hmm, so this should actually be part of the commit to libntpevent if we do that independent of the openssl merge (which I think we should)
Mon, Oct 8
My only suggestion might be to consider an explicit "disabled" tunable. For SMP we have 'kern.smp.disabled=1'. Having 'vm.numa.disabled=1' or 'kern.numa.disabled=1' might be more intuitive than setting ndomains to 1. (The implementation might be that you set ndomains to 1 if the tunable is set, but it's the UI I'm thinking of).
Looks good apart from a sorting nit.
Fri, Oct 5
Hmm, I think for this we want to actually wait to set PTE_A and PTE_D for user mappings until we get a fault? Presumably we'd want to check in the page fault handler if f the PTE was already valid before calling vm_fault and if it was set PTE_A (and PTE_D for a write) and then retry?
Thu, Oct 4
Actually, my plan is to just change them on powerpc and have them unified across the board. They should then really just live in sys/elf_common.h. I think if we want to move them now we should just move them and maybe add some #ifdef powerpc that we eventually remove when renumbering powerpc's vectors. I think the actual renumbering change is probably too big of a change during code freeze.
Wed, Oct 3
- Fix build on mips (32-bit and using GCC instead of clang)
Just commit already. openssh isn't buildable without this. :-P Note that if we merge this to head before openssl is updated we'd need a different config.h in head.
Tue, Oct 2
- More bjk feedback.
Use "mips64el" instead of "mips64le" in the commit message. Good sleuthing!
Mon, Oct 1
Fri, Sep 28
Thu, Sep 27
Agree with kib on the general point that all of freebsd32_ioctl is probably busted and should be moved to the actual ioctl handlers.
Tue, Sep 25
- Various fixes from bjk@
Mon, Sep 24
Sat, Sep 22
I did some very simple testing which seemed to work:
Fri, Sep 21
Fixed the various issues I noted earlier.
Sep 21 2018
So the fact that some things were updated for the new name and that it clashed with the old name made just reverting to the old name messy. I already had done the union change as a followup to remove the compat macros, and it occurred to me that since I had it, I could actually go back to using kf_sa_local/peer in both places which would let both the old and new names work at the same time. This in theory shouldn't require reverting any of the ports changes or upstream port changes while still preserving the existing API. We could even go back and fix the kf_sock_ variants to remove the trailing '0' now if we wished to as well.
- Use an anonymous union and struct for FreeBSD 11 compat fields.
- Rename the kf_un.kf_sock variants back to their original names.
Sep 20 2018
- Use a regular 'fence' in pmap_sync_icache() before the IPI.
exp-run requested at https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=231525
- Use kf_sock_local/peer instead of kf_sa_local0/peer0.
Originally it was useful to disable this when a bug was encountered, but the window where that was useful probably ended several years ago.