ldb 2.1.2: test_get_size failure on ppc64el
Andrew Bartlett
abartlet at samba.org
Thu May 7 22:43:27 UTC 2020
On Fri, 2020-05-08 at 10:08 +1200, Gary via samba-technical wrote:
>
> On 8/05/20 6:16 am, Andreas Hasenack via samba-technical wrote:
> > Hi,
> >
> > I'm building ldb 2.1.2 on ubuntu groovy 20.10, along with all the
> > other dependencies for samba 4.12.2, and I'm getting this test
> > error
> > on ppc64el only:
> >
> > [ RUN ] test_get_size
> > [ ERROR ] --- 13369 is not within the range 2500-5000
> > [ LINE ] --- ../../tests/ldb_kv_ops_test.c:1721: error:
> > Failure!
> > [ FAILED ] test_get_size
> > [==========] 13 test(s) run.
> > [ PASSED ] 12 test(s).
> > [ FAILED ] 1 test(s), listed below:
> > [ FAILED ] test_get_size
> >
> > That assert use to be
> >
> > assert_true( size > 2500);
> >
> > but was changed to
> >
> > assert_in_range(size, 2500, 5000);
> >
> > in commit
> >
> > commit e464e40c977cd6592240763c7dbb9c45ff16470f
> > Author: Andreas Schneider <asn at samba.org>
> > Date: Thu Dec 5 17:14:31 2019 +0100
> >
> > ldb:tests: Use assert_in_range() in test_get_size()
> >
> > Signed-off-by: Andreas Schneider <asn at samba.org>
> > Reviewed-by: Gary Lockyer <gary at samba.org>
> >
> >
> > Is that 5000 ceiling limit just a guess, or is this failure
> > indicating
> > a real bug somewhere? The value 13369 I got on ppc64el seems a bit
>
> It's just a guess. for tdb we estimate the number of records with
> #define RECORD_SIZE 500
>
>
> So changing the test to assert_in_range(size, 2500, 20000);
> Should be ok.
PPC64 is probably doing something sensible (as in not optimised for a
1980s home PC) with pages and so consuming more space.
Andrew Bartlett
--
Andrew Bartlett https://samba.org/~abartlet/
Authentication Developer, Samba Team https://samba.org
Samba Developer, Catalyst IT
https://catalyst.net.nz/services/samba
More information about the samba-technical
mailing list