I think this is all good stuff. For my 2c I think when this is committed we may as well get rid of the unused ledger files. We will still have them in cvs history if we want to refer back.

I think your latest patch still has one stray
"#ifdef TROVE_HANDLE_LEDGER_ENABLED" running around in it :)

-Phil

Sam Lang wrote:

Here's the latest with a number of important changes:

* removed all reference to ledger code. The current ledger files are still in place but never get invoked by the rest of the code. * added a hash for recently removed handles with an expiration to prevent reusing handles too quickly. This is mostly a failsafe -- generating a handle value from a 64 bit space isn't going to collide with any of the recently deleted ones.
* removed all references to handles_available in statfs
* fallback to srand if urandom not supported.

-sam
_______________________________________________
Pvfs2-developers mailing list
[email protected]
http://www.beowulf-underground.org/mailman/listinfo/pvfs2-developers

Reply via email to