RE: [PATCH] btrfs-progs: Modify confuse error message in scrub

2015-08-05 Thread Zhao Lei
Hi, David Sterba Thanks for review this patch. > -Original Message- > From: David Sterba [mailto:dste...@suse.com] > Sent: Thursday, August 06, 2015 12:51 AM > To: Zhao Lei > Cc: linux-btrfs@vger.kernel.org > Subject: Re: [PATCH] btrfs-progs: Modify confuse error messag

Re: [PATCH] btrfs-progs: Modify confuse error message in scrub

2015-08-05 Thread David Sterba
On Wed, Aug 05, 2015 at 04:32:26PM +0800, Zhao Lei wrote: > Scrub output following error message in my test: > ERROR: scrubbing /var/ltf/tester/scratch_mnt failed for device id 5 > (Success) > > It is caused by a broken kernel and fs, In what way is it broken? Can we turn it into tests? > but

[PATCH] btrfs-progs: Modify confuse error message in scrub

2015-08-05 Thread Zhao Lei
Scrub output following error message in my test: ERROR: scrubbing /var/ltf/tester/scratch_mnt failed for device id 5 (Success) It is caused by a broken kernel and fs, but the we need to avoid outputting both "error and success" in oneline message as above. This patch modified above message to: