From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [PATCH 1/2] writeback: eliminate work item allocation in bd_start_writeback() To: Jan Kara Cc: linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, hannes@cmpxchg.org References: <1507043781-2874-1-git-send-email-axboe@kernel.dk> <1507043781-2874-2-git-send-email-axboe@kernel.dk> <20171004072635.GP11879@quack2.suse.cz> From: Jens Axboe Message-ID: Date: Wed, 4 Oct 2017 08:42:12 -0600 MIME-Version: 1.0 In-Reply-To: <20171004072635.GP11879@quack2.suse.cz> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: On 10/04/2017 01:26 AM, Jan Kara wrote: >> diff --git a/include/linux/backing-dev-defs.h b/include/linux/backing-dev-defs.h >> index 420de5c7c7f9..f0f1df29d6b8 100644 >> --- a/include/linux/backing-dev-defs.h >> +++ b/include/linux/backing-dev-defs.h >> @@ -116,6 +116,7 @@ struct bdi_writeback { >> >> struct fprop_local_percpu completions; >> int dirty_exceeded; >> + int start_all_reason; > > This should be 'enum wb_reason' instead of 'int'. Yes good point, I'll move the enum and change 'start_all_reason' to be of type enum wb_reason. Thanks for the review! -- Jens Axboe