> +static void _bdi_wakeup_flusher(struct backing_dev_info *bdi)
Remove the leading underscore, please.
> void bdi_start_background_writeback(struct backing_dev_info *bdi)
> {
> - __bdi_start_writeback(bdi, LONG_MAX, true, true);
> + /*
> + * We just wake up the flusher thread. It will perform background
> + * writeback as soon as there is no other work to do.
> + */
> + spin_lock_bh(&bdi->wb_lock);
> + _bdi_wakeup_flusher(bdi);
> + spin_unlock_bh(&bdi->wb_lock);
We probably want a trace point here, too.
Otherwise the patch looks good to me. Thanks for bringing it up again.
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
