I don't know if it matters, but in modparams you prefix the avp names with "s:", and don't do it in the xlog call.
I actually copied and pasted your code in just to be sure, but no result.
--
-Barry