[dm-devel] [PATCH 16/24] dm cache: log error message if dm_kcopyd_copy() fails

Alasdair G Kergon agk at redhat.com
Fri Nov 8 13:33:11 UTC 2013


On Thu, Oct 24, 2013 at 02:30:29PM -0400, Mike Snitzer wrote:
> A migration failure should be logged (albeit limited).
 
Does this *really* add anything useful?

migration_failure() already always logs a message, so you'll now get
two (throttled) messages instead of one every time.

Would it be better to log not just the type of failure but also some
details about the data block location too?

And if you really need to know "issuing migration failed" then why not
just add the string to the existing log message?

Alasdair




More information about the dm-devel mailing list