MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8;
 format=flowed
Content-Transfer-Encoding: 7bit
Date: Tue, 07 Mar 2017 12:29:32 +0100
From: Harald Geyer <harald@ccbib.org>
To: Tejun Heo <tj@kernel.org>
Cc: Mark Brown <broonie@kernel.org>, Liam Girdwood <lgirdwood@gmail.com>,
        Lai Jiangshan <jiangshanlai@gmail.com>, <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/2] workqueue: Add new function
 =?UTF-8?Q?mod=5Ffwd=5Fdelayed=5Fwork=28=29?=
In-Reply-To: <20170306222212.GM26127@htj.duckdns.org>
References: <1487785285-3567-1-git-send-email-harald@ccbib.org>
 <20170222182111.4jajk2ed52okx323@sirena.org.uk>
 <E1cgdBA-0000LS-VU@stardust.g4.wien.funkfeuer.at>
 <20170223173449.c747nrfr3oxrjrr7@sirena.org.uk>
 <20170306222212.GM26127@htj.duckdns.org>
Message-ID: <ed1aca3f6ed8386a366b8dc6386a225a@ccbib.org>
User-Agent: RoundCube Webmail/0.7.2
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1100
Lines: 29

On 06.03.2017 23:22, Tejun Heo wrote:
> I don't think it's a matter of "fixing" the existing
> mod_delayed_work().  What the new function is implementing wouldn't
> fit use cases where the timeout should only be shortened (IIRC,
> writeback code does that).
>
> I'm not against adding new interface to handle it better but I think
> it makes more sense to add both directions.  How about adding
> expedite_delayed_work_on() and postpone_delayed_work_on()?

I think such a function should only be added if there is actually
code using it. So I'd wait for the survey of existing 
mod_delayed_work()
users Mark has promised to actually find some bugs that would be fixed
by the function before adding it.

The names you are proposing feel less clear to me then mod_fwd/mod_bwd,
but english is not my native tongue, so my feeling is probably no
strong evidence ... :)

Otherwise I agree with your comments.

Harald

-- 
If you want to support my work:
see http://friends.ccbib.org/harald/supporting/
or donate via peercoin to P98LRdhit3gZbHDBe7ta5jtXrMJUms4p7w
or CLAM xASPBtezLNqj4cUe8MT5nZjthRSEjrRQXN