lkml.org 
[lkml]   [2013]   [May]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: workqueue: inline trivial wrappers
From
On Mon, Apr 29, 2013 at 10:31 PM, Linux Kernel Mailing List
<linux-kernel@vger.kernel.org> wrote:
> Gitweb: http://git.kernel.org/linus/;a=commit;h=8425e3d5bdbe8e741d2c73cf3189ed59b4038b84
> Commit: 8425e3d5bdbe8e741d2c73cf3189ed59b4038b84
> Parent: 611c92a0203091bb022edec7e2d8b765fe148622
> Author: Tejun Heo <tj@kernel.org>
> AuthorDate: Wed Mar 13 16:51:36 2013 -0700
> Committer: Tejun Heo <tj@kernel.org>
> CommitDate: Wed Mar 13 16:51:36 2013 -0700
>
> workqueue: inline trivial wrappers
>
> There's no reason to make these trivial wrappers full (exported)
> functions. Inline the followings.
>
> queue_work()
> queue_delayed_work()
> mod_delayed_work()
> schedule_work_on()
> schedule_work()
> schedule_delayed_work_on()
> schedule_delayed_work()
> keventd_up()
>
> Signed-off-by: Tejun Heo <tj@kernel.org>

Hi Tejun,

This commit has the side effect of making some functions
EXPORT_SYMBOL_GPL that had been previously exported with EXPORT_SYMBOL
for a long time, such as schedule_work(). This is used by OpenAFS and
probably other out of tree modules. Past exchanges on this list (ex:
http://marc.info/?l=linux-fsdevel&m=123550631318910) suggest the
license requirements for existing interfaces shouldn't change, at
least without providing an alternate means of getting the same
functionality. Any thoughts on whether this can be adjusted within
the current release cycle?

Thanks,
Marc


\
 
 \ /
  Last update: 2013-05-06 17:21    [from the cache]
©2003-2011 Jasper Spaans