ReplicatedPG: delete mark_all_unfound_lost transactions after completion

This was a minor memory leak.

Signed-off-by: Greg Farnum <greg@inktank.com>
MOSDOp: drop ops vector in clear_data()

Otherwise, clear_data on MOSDOp will leave essentially
all of the buffers intact.  This is a problem since the
OpTracker mechanism relies on being able to keep the mesage
around without keeping around the data.

Signed-off-by: Samuel Just <sam.just@inktank.com>
OpTracker: clarify that unregister_inflight_op is only called if enabled

The !tracking_enabled branch actually had a leak which was unreachable
since the caller does the check for tracking_enabled.

Signed-off-by: Samuel Just <sam.just@inktank.com>
TrackedOp: clear_payload as well in unregister_inflight_op

We want to minimize the cost of maintaining the historic ops.

Signed-off-by: Samuel Just <sam.just@inktank.com>
MOSDECSubOpWrite: drop transaction, log_entries in clear_buffers

Signed-off-by: Samuel Just <sam.just@inktank.com>
osd_types,PG: trim mod_desc for log entries to min size

In the event that mod_desc.bl contains pointers into a large
message buffer, we'd otherwise end up keeping around the entire
MOSDECSubOpWrite which created each log entry.

Fixes: #7539
Signed-off-by: Samuel Just <sam.just@inktank.com>
Merge pull request #1344 from ceph/wip-7539

Wip 7539

Reviewed-by: Sage Weil <sage@inktank.com>