> WRT to merging the functions, I'm a little concerned about the result > being too large. What do you think about folding in _vextent_type() but > at the same time factoring out the rmap/counter/resv post alloc bits > into an xfs_alloc_ag_vextent_accounting() helper or some such? Sounds good to me. I've looked at the function and another nice thing to do would be to not pass the ret bno to xfs_alloc_ag_vextent_agfl, but let that function fill out the args structure return value itself. Also for the trace piints that still say near in them - maybe we should change that near to ag?
