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

[Jason King]

IKEv2 was the intended consumer (for me at least).

Just doing a grep for it's use I see idmap and smbsrv both use them
(haven't looked closer yet), so those might be candidates.  I can look
a bit closer in the next few days to see if anything they're putting
into nvlists would be something that might be appropriate to be erased
before releasing the memory.

On Thu, Feb 10, 2011 at 10:20 PM, Garrett D'Amore <garrett at damore.org> wrote:
> 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
>
>
>