In D43711#998275, @kib wrote:Do you have somebody to commit this for you?
If not, please mail me git-format-patches.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed Advanced Search
Advanced Search
Advanced Search
Feb 7 2024
Feb 7 2024
alfredo.mazzinghi_cl.cam.ac.uk added a comment to D43711: Abstract UIO allocation and deallocation..
Feb 6 2024
Feb 6 2024
alfredo.mazzinghi_cl.cam.ac.uk updated the diff for D43711: Abstract UIO allocation and deallocation..
Fix additional freeuio() calls in compat/freebsd32.
Feb 2 2024
Feb 2 2024
alfredo.mazzinghi_cl.cam.ac.uk added a comment to D43711: Abstract UIO allocation and deallocation..
In D43711#996954, @kib wrote:I do not have much objections against this change, but I wonder what do you mean by 'changing the sub-allocation layout'. Note that uio->uio_iov is often modified during uio life, resulting in the pointer no longer pointing to the initial iov vector at the time of uio allocation.
alfredo.mazzinghi_cl.cam.ac.uk added reviewers for D43711: Abstract UIO allocation and deallocation.: kib, jhb.
alfredo.mazzinghi_cl.cam.ac.uk added reviewers for D43710: Fix subr_uio.c style(9) with uses of sizeof.: kib, jhb.
alfredo.mazzinghi_cl.cam.ac.uk requested review of D43711: Abstract UIO allocation and deallocation..
alfredo.mazzinghi_cl.cam.ac.uk requested review of D43710: Fix subr_uio.c style(9) with uses of sizeof..
Jun 27 2023
Jun 27 2023
alfredo.mazzinghi_cl.cam.ac.uk updated the diff for D40244: efi: Update efi_phys_to_kva to accept the size of the object being mapped..
Dropped #ifdef PHYS_HAS_DMAP for the PHYS_SZ_IN_DMAP macro definition.
Jun 20 2023
Jun 20 2023
Jun 19 2023
Jun 19 2023
alfredo.mazzinghi_cl.cam.ac.uk updated the diff for D40244: efi: Update efi_phys_to_kva to accept the size of the object being mapped..
Move PHYS_SZ_IN_DMAP to the machine-independent vm/vm_param.h
Jun 2 2023
Jun 2 2023
alfredo.mazzinghi_cl.cam.ac.uk updated the diff for D40244: efi: Update efi_phys_to_kva to accept the size of the object being mapped..
Fixed off-by-one check for the DMAP limit.
alfredo.mazzinghi_cl.cam.ac.uk updated the diff for D40244: efi: Update efi_phys_to_kva to accept the size of the object being mapped..
Should now be fixed. I misinterpreted the desired behaviour for PHYS_SZ_IN_DMAP.
Jun 1 2023
Jun 1 2023
alfredo.mazzinghi_cl.cam.ac.uk updated the diff for D40244: efi: Update efi_phys_to_kva to accept the size of the object being mapped..
Added the PHYS_SZ_IN_DMAP macro as requested.
May 31 2023
May 31 2023
alfredo.mazzinghi_cl.cam.ac.uk updated subscribers of D40244: efi: Update efi_phys_to_kva to accept the size of the object being mapped..
@kib Do you have any further comment?
May 25 2023
May 25 2023
alfredo.mazzinghi_cl.cam.ac.uk added a comment to D40110: riscv: Fix size reserved for the devmap region in pmap_bootstrap..
In D40110#916824, @mhorne wrote:LGTM, thanks. I can push this to the main branch.
Can you update the title+summary to reflect the final diff? (We didn't end up changing the devmap size.) Even easier would be if you could just send the git format-patch output to me over email: mhorne@freebsd.org.
May 23 2023
May 23 2023
May 22 2023
May 22 2023
alfredo.mazzinghi_cl.cam.ac.uk updated the diff for D40110: riscv: Fix size reserved for the devmap region in pmap_bootstrap..
Shrink the size of PMAP_MAPDEV_EARLY_SIZE to a single L2_SIZE.
This should be acceptable given the fact that it does not seem that we were
trampling the DTB mapping before.
While this solution is less robust, it avoids adding complexity to asm code
unless it is absolutely required.
Added assertions that verify that both the DTB and early DEVMAP are aligned to
L2 boundary and that the DTB mapping does not overlap the DEVMAP.
This check assumes that the DTB only consumes a single L2 entry.
May 19 2023
May 19 2023
alfredo.mazzinghi_cl.cam.ac.uk added a comment to D40110: riscv: Fix size reserved for the devmap region in pmap_bootstrap..
@mhorne I'm not sure how this currently is supposed to work, because the PMAP_MAPDEV_EARLY_SIZE constant is already larger than L2_SIZE in vmparam.h. So what we really want is also the DTB to be mapped at VM_EARLY_DTB_ADDRESS, which currently seems to be assumed to correspond to the second-to-last L2 entry in the devmap table. Is the PMAP_MAPDEV_EARLY_SIZE constant supposed to contain VM_EARLY_DTB_ADDRESS?
May 15 2023
May 15 2023
May 20 2020
May 20 2020