Thank you.

Still, the mail that was sent and contained the exact instructions was useful. This patch just repeats the description from rc.conf.5. Without the exact recipes as was done in the mail, it is probably not useful.

NB: this is still being tested by Peter, I decided to go ahead and publish the review as an early exposure

I suppose this is for stable/15, right?

D54335

In D53965#1241683, @alc wrote:

This change is now similar to D43263. Abandon this one?

I have to keep sah tree rlocked much deeper, right until the if_input is called finally. Also, the same problem exists for all ESP/AH/IPCOMP processing callbacks.

Also the ports:misc/compat15x needs to be updated.

I have no idea about USB code

Can you pleas explain

1. how the list of missed symbols was obtained
2. why do we need the internallib for krb5profile now

Use next_end

In D54229#1240206, @des wrote:

Can I please get a decision on this?

This looks fine to me, but I prefer to have more eyes on it before is is engraved into the libc.

In D54258#1239897, @markj wrote:

In D54258#1239826, @kib wrote:

This is only relevant for 32bit arches, am I right? (I do not claim that it is not a bug to fix).

Hmm, now that I look at it again, I think this patch is just bogus, and there is no problem. If size < 0 and size is promoted to an unsigned type, addition with tm_ea_memory_inuse will cause wraparound and the result will be correct. It does not make much sense to perform the comparison at all if size < 0, but I think there is no problem, even on 32-bit arches. I did not test it at all yet.

Correct the article in comment.

This is only relevant for 32bit arches, am I right? (I do not claim that it is not a bug to fix).

Fix pend (and eliminate).

Ok, if somebody from the powerpc crowd confirm that the isync is redundant there, I am fine with that.
If not, I put a sketch for my proposal at D54246.

Not even compiled.

Remove pstart.
Remove clipping from swap_pager_seek_data(), so that PG_ZERO check can see all pages in the object.

swap_pager_seek_data(): check for the swap object, this is what the function assumes

In D54221#1238704, @jrtc27 wrote:

IMO the commits that added the icache sync were done due to some actual problems seen.

Probably due to a broken pmap (at least at the time) then.

in principle, non-plt relocations can modify any kind of segments. There are DT_TEXTREL binries, as well as nothing in ELF spec prevents existence of PF_W|PF_X segment with relocs.
IMO the commits that added the icache sync were done due to some actual problems seen.

In D54214#1238647, @des wrote:

@kib can you also please take a look at the PR? the originator claims this used to work as recently as 14.2 but I see no evidence that it has ever worked, the code to zero out the fsid has been there since 1997.

Indeed, p_klist initialization should not be moved into constructor, since it is cleared in exit1() (and should not be done in destructor, I believe). Then it would leak in each case where exit1() is not called, i.e. fork failures.
For mac, I have no idea what modules expect, so IMO it is better to keep them as is.

So lets split this review, hopefully to revive it.

In D54215#1238587, @kib wrote:

IMO it would be more proper to move the initialization into proc_init().

IMO it would be more proper to move the initialization into proc_init().

I think users must have ability to get back to the previous behavior. Might be add a sysctl to tweak it.

It is really eventfd data, so might be slightly better to call it eventfd_data. I do not object to proposal to rename the FreeBSD own structure,