[PATCH 3/3] test: slab_arena -- Verify madvise

Vladimir Davydov vdavydov.dev at gmail.com
Mon May 6 13:55:52 MSK 2019


On Mon, May 06, 2019 at 01:48:04PM +0300, Cyrill Gorcunov wrote:
> On Mon, May 06, 2019 at 01:45:20PM +0300, Vladimir Davydov wrote:
> > [Cc += Alexander re tests]
> > 
> > On Wed, May 01, 2019 at 06:50:06PM +0300, Cyrill Gorcunov wrote:
> > > Since madvise support depends on sys libraries
> > > and kernel version we print error if only small
> > > supports it and /proc/self/smaps provide more
> > > less decent VmFlags.
> > 
> > TBO I wouldn't bother testing this feature at all, because it's way too
> > platform dependent.
> 
> Test does check for /proc/self/smaps file present and for VmFlags entry
> present, if neither of it found we just exit silently, without errors.

Okay, I'm fine with it.

To be prudent though, we'd have to check that the flag is set not only
for the preallocated memory region, but also for slabs allocated
afterwards (see slab_map -> mmap_checked). Not sure if it's worth it
though.



More information about the Tarantool-patches mailing list