Page MenuHomeFreeBSD

alfredo.mazzinghi_cl.cam.ac.uk (Alfredo Mazzinghi)
User

Projects

User does not belong to any projects.

User Details

User Since
May 20 2020, 5:27 PM (204 w, 22 h)

Recent Activity

Feb 7 2024

alfredo.mazzinghi_cl.cam.ac.uk added a comment to D43711: Abstract UIO allocation and deallocation..
In D43711#998275, @kib wrote:

Do you have somebody to commit this for you?

If not, please mail me git-format-patches.

Feb 7 2024, 11:15 AM

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 6 2024, 5:42 PM

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.

Feb 2 2024, 5:41 PM
alfredo.mazzinghi_cl.cam.ac.uk added reviewers for D43711: Abstract UIO allocation and deallocation.: kib, jhb.
Feb 2 2024, 3:46 PM
alfredo.mazzinghi_cl.cam.ac.uk added reviewers for D43710: Fix subr_uio.c style(9) with uses of sizeof.: kib, jhb.
Feb 2 2024, 3:46 PM
alfredo.mazzinghi_cl.cam.ac.uk requested review of D43711: Abstract UIO allocation and deallocation..
Feb 2 2024, 3:11 PM
alfredo.mazzinghi_cl.cam.ac.uk requested review of D43710: Fix subr_uio.c style(9) with uses of sizeof..
Feb 2 2024, 3:11 PM

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 27 2023, 3:27 PM

Jun 20 2023

alfredo.mazzinghi_cl.cam.ac.uk added inline comments to D40244: efi: Update efi_phys_to_kva to accept the size of the object being mapped..
Jun 20 2023, 10:06 AM

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 19 2023, 8:01 PM

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.

Jun 2 2023, 1:46 PM
alfredo.mazzinghi_cl.cam.ac.uk added inline comments to D40244: efi: Update efi_phys_to_kva to accept the size of the object being mapped..
Jun 2 2023, 11:02 AM
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 2 2023, 10:05 AM

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.

Jun 1 2023, 2:48 PM

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 31 2023, 5:15 PM

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..

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 25 2023, 3:33 PM

May 23 2023

alfredo.mazzinghi_cl.cam.ac.uk requested review of D40244: efi: Update efi_phys_to_kva to accept the size of the object being mapped..
May 23 2023, 5:55 PM

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 22 2023, 8:09 PM

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 19 2023, 1:31 PM

May 15 2023

alfredo.mazzinghi_cl.cam.ac.uk requested review of D40110: riscv: Fix size reserved for the devmap region in pmap_bootstrap..
May 15 2023, 3:31 PM

May 20 2020

alfredo.mazzinghi_cl.cam.ac.uk accepted D24931: Simplify hot-patching cpu_switch() for lack of UserLocal register..

Looks good to me.

May 20 2020, 5:53 PM