svn_fs_base__retry_txn() is a while(1) loop that calls
begin_trail(pool) in every iteration.  The latter function callocates
a struct trail_t out of POOL.

Is there a memory growth issue here?  Will the number of iterations of
the while(1) loop be small or bounded?  If not, should we avoid repeated
allocations out of POOL?  (e.g., by re-using a single trail_t struct, or
by letting the struct live on stack until a successful txn)

Reply via email to