svn commit: r298580 - in head/sys/riscv: include riscv

Konstantin Belousov kostikbel at gmail.com
Mon Apr 25 15:50:56 UTC 2016


On Mon, Apr 25, 2016 at 02:47:51PM +0000, Ruslan Bukin wrote:
> Author: br
> Date: Mon Apr 25 14:47:51 2016
> New Revision: 298580
> URL: https://svnweb.freebsd.org/changeset/base/298580
> 
> Log:
>   o Implement shared pagetables and switch from 4 to 3 levels page
>   memory system.

> +/* An entry in the list of all pmaps */
> +struct pmap_list_entry {
> +	SLIST_ENTRY(pmap_list_entry) pmap_link;
> +	struct pmap *pmap;
> +};

This is weird. Why do you need separate structure to track the all
pmaps list, instead of embedding the list link into pmap itself ? In
particular, the pmap_list_entry.pmap pointing to the pmap looks strange.

And why do you use single-linked list for the container where you need
to remove elements ?  The cost is the iteration over the whole container
on removal, vs. additional pointer in each pmap.


More information about the svn-src-head mailing list