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

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

The following reply was made to PR lib/50441; it has been noted by GNATS.

From: Manuel Bouyer <bouyer%antioche.eu.org@localhost>
To: Christos Zoulas <christos%zoulas.com@localhost>
Cc: gnats-bugs%NetBSD.org@localhost, lib-bug-people%netbsd.org@localhost, gnats-admin%netbsd.org@localhost,
        netbsd-bugs%netbsd.org@localhost
Subject: Re: PR/50441 CVS commit: src/lib/libc/db/hash
Date: Wed, 18 Nov 2015 12:30:33 +0100

 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
 --