[PATCH v3] slab_arena: Enhance slab_arena_create to support madvise hints
Cyrill Gorcunov
gorcunov at gmail.com
Wed May 15 16:22:16 MSK 2019
On Wed, May 15, 2019 at 04:08:12PM +0300, Vladimir Davydov wrote:
> >
> > You know I thought if we could use small/conf.h inside tarantool.
> > For this sake I prefixed symbols with TARANROOL_SMALL_ to not interfere
> > with any other symbols.
> >
> > Another option -- provide something like
> >
> > static const uint64_t small_features = (bitset of features);
>
> I would define HAVE_MADV_DONTNEED in tarantool's config.h.
This won't work: you may have tarantool compiled on machine
where don't need not supported, but small compiled on a system
where madv is present (note that I'm talking about system libs,
the kernel may have own features enabled).
Also I think the main idea of "small" as a library is to separate
code as much as possible: tarantool should not do any own compile
testing but rather request the library if particular feature is
implemented (either via API or via config.h shipped with library).
Lets talk f2f.
>
> It does compile on my box, which runs Linux. Note the cast: it should be
> fine to explicitly cast (char *) to intptr_t.
Weird, I'll retry.
More information about the Tarantool-patches
mailing list