Re: CVS commit: src/tests/net

[Ryota Ozaki <ozaki-r%netbsd.org@localhost>]

On Fri, Aug 23, 2019 at 3:51 PM Martin Husemann <martin%duskware.de@localhost> wrote:
>
> On Fri, Aug 23, 2019 at 01:41:58PM +0900, Ryota Ozaki wrote:
> > A workaround for the issue is:
> >   cp /usr/bin/vmstat ./vmstat
> >   $HIJACKING ./vmstat
> >   rm -f ./vmstat
> >
> > It's awkward but it's reasonable for now.  A proper fix would
> > be to stop using kvm(3) for vmstat and drop the sgid bit from
> > the binary.
>
> Wow, good catch!
>
> > I guess on most anita environments leak checks pass just in luck
> > because the environment normally doesn't communicate with outside
> > and there are no L2 caches.  OTOH on baremetal environments there
> > can be active L2 caches, which makes the leak checks fail.
>
> Sounds like a good explanation.

https://gist.github.com/ozaki-r/37bec56f93f5a57dc98ed618e8db42a3

Could you try the patch?  Running just one failed ATF test is enough
to confirm.

Thanks,
  ozaki-r