[illumos-Developer] Review: 668 libnvpair could use an allocator that zeros memory on free

Garrett D'Amore garrett at damore.org
Thu Feb 10 20:20:47 PST 2011


Btw, I'd feel better about integrating this code if we had a consumer
for it in place at the same time.  Do you have plans for a specific
consumer in ON?  I know you talked about using it for your IKEv2 work,
but perhaps there is already one that could benefit from this?

	- Garrett

On Thu, 2011-02-10 at 20:18 -0800, Garrett D'Amore wrote:
> Looks ok to me.  Did we decide that we didn't want to lock pages used by
> this allocation to prevent them from being swapped to disk?  (It seemed
> like a good idea to me at the time.)
> 
> 	- Garrett
> 
> On Thu, 2011-02-10 at 21:44 -0600, Jason King wrote:
> > Webrev at http://cr.illumos.org/view/njgj7cyj/
> > 
> > I'm running nightly against the lastest snapshot of illumos-gate, but
> > earlier nightly's built without issue a few weeks ago (so just a
> > sanity check).
> > 
> > I've attached the test case I used to verify the changes.
> > _______________________________________________
> > Developer mailing list
> > Developer at lists.illumos.org
> > http://lists.illumos.org/m/listinfo/developer
> 
> 
> 
> _______________________________________________
> Developer mailing list
> Developer at lists.illumos.org
> http://lists.illumos.org/m/listinfo/developer





More information about the Developer mailing list