Re: CVS commit: src/sys/kern

[Nick Hudson <nick.hudson%gmx.co.uk@localhost>]

Sent from my iPhone

> On 30 Sep 2019, at 18:06, Tobias Nygren <tnn%netbsd.org@localhost> wrote:
> 
> On Mon, 23 Sep 2019 05:39:59 +0000
> Nick Hudson <skrll%netbsd.org@localhost> wrote:
> 
>> Modified Files:
>>    src/sys/kern: subr_pool.c
>> 
>> Log Message:
>> Enable POOL_REDZONE with DIAGNOSTIC.
>> 
>> The bug in the arm pmap was fixed long ago.
>> 
>> 
>> To generate a diff of this commit:
>> cvs rdiff -u -r1.258 -r1.259 src/sys/kern/subr_pool.c
> 
> Hi,
> 
> I see 30% packet loss on my BananaPi evbarm system with -current.
> I bisected it to the above change. It may have had unintended
> side effects.

This is maxv’s change really - feel free to revert

Sounds like it corrupts pool allocated memory in some cases

> What is the referenced pmap bug about?

sys/arch/arm/arm32/pmap.c:1.367

Sorry for the breakage

Nick