On Oct 16, 2013, at 7:24 PM, "Steven Hartland" <[email protected]> wrote:
> ----- Original Message ----- From: "Saso Kiselkov" <[email protected]> > To: <[email protected]>; "illumos-zfs" <[email protected]> > Sent: Thursday, October 17, 2013 12:09 AM > Subject: [OpenZFS Developer] [Review] Tunable ARC buf hash sizing > > >> Okay, I've reworked and simplified the ARC buf hash changes to just >> change the way we size the hash table and bump up the amount of hash >> tables we allocate to 1MB/1GB. >> Webrev: http://cr.illumos.org/~webrev/skiselkov/new_buf_hash/ >> Issue: https://www.illumos.org/issues/4218 > > Did you mean "This index is then masked with ht_mask" (remove > the word "by"?) in the following: > + * which spits out a 64-bit hash index. This index is then masked > + * by with ht_mask to obtain the final index into the hash table: > > It may be a silly question but should the hash sizing not be based > around zfs_arc_max instead of physmem? No, because zfs_arc_max does not include l2arc. > > Regards > Steve > > ================================================ > This e.mail is private and confidential between Multiplay (UK) Ltd. and the > person or entity to whom it is addressed. In the event of misdirection, the > recipient is prohibited from using, copying, printing or otherwise > disseminating it or any information contained in it. > In the event of misdirection, illegible or incomplete transmission please > telephone +44 845 868 1337 > or return the E.mail to [email protected]. > > _______________________________________________ > developer mailing list > [email protected] > http://lists.open-zfs.org/mailman/listinfo/developer _______________________________________________ developer mailing list [email protected] http://lists.open-zfs.org/mailman/listinfo/developer
