Re: [PATCH] btrfs-progs: balance: check for full-balance before background fork

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi David,

On 20/08/2019 14.32, David Sterba wrote:
On Sat, Aug 17, 2019 at 11:14:34PM +0000, Vladimir Panteleev wrote:
- Don't use grep -q, as it causes a SIGPIPE during the countdown, and
   the balance thus doesn't start.

This needs -q, otherwise the text appears in the output of make. Fixed.

What of the SIGPIPE problem mentioned in the commit message?

If using -q is preferred despite of that, then probably the note about it should be removed from the commit message, and the "cancel" afterwards should probably be removed as well (along with its note in the commit message too), as the SIGPIPE will prevent the balance from ever starting.

Perhaps redirecting the output of grep to /dev/null is a better option.

> Applied, thanks.

Not a big issue but for some reason my email address was mangled (@panteleev.md instead of @vladimir.panteleev.md). Looks fine when I look at https://patchwork.kernel.org/patch/11099359/mbox/.

--
Best regards,
 Vladimir



[Index of Archives]     [Linux Filesystem Development]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux