Re: PR/50441 CVS commit: src/lib/libc/db/hash

[Manuel Bouyer <bouyer%antioche.eu.org@localhost>]

On Tue, Nov 17, 2015 at 07:24:44PM -0500, Christos Zoulas wrote:
> On Nov 17, 11:10pm, bouyer%antioche.eu.org@localhost (Manuel Bouyer) wrote:
> -- Subject: Re: PR/50441 CVS commit: src/lib/libc/db/hash
> 
> |  thanks for looking at this.
> |  But there's still something wrong. I think the db file is much larger than it
> |  should be (24k for 4k pages, 128k for 32k pages, and 76M for 64k pages).
> 
> Heh, I know. I just fixed this.

sorry for bein impatient :)

> 
> |  Also, with spamassassin, I still get a corrupted/unreadable db file.
> |  
> |  I could'nt reproduce the corruption without spamassassin (maybe this requires
> |  non-ascii keys, which I can't do with a simple shell script).
> 
> I don't know how to debug that! test please?

With
src/lib/libc/db/hash/hash_page.c 1.27 and
src/lib/libc/db/hash/hash.c 1.36
pulled up to netbsd-7, the problem seems to be fixed.
With my shell script test, the db file is 256k instead of 76M for the 64k page
test, and spamassassin also produces a file of reasonable size, witout
complains.

I've reenabled the bayes filter on my mail server, let see how it works after
a few hours.

-- 
Manuel Bouyer <bouyer%antioche.eu.org@localhost>
     NetBSD: 26 ans d'experience feront toujours la difference
--