linux-kernel - [PATCH] export the queue

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <Pine.LNX.4.44L0.0608291448380.3753-100000@iolanthe.rowland.org>
Date:	Tue, 29 Aug 2006 15:37:18 -0400 (EDT)
From:	Alan Stern <stern@...land.harvard.edu>
To:	Jonathan Corbet <corbet@....net>
cc:	linux-kernel@...r.kernel.org,
	SCSI development list <linux-scsi@...r.kernel.org>,
	Jens Axboe <axboe@...e.de>, Andrew Morton <akpm@...l.org>,
	Ingo Molnar <mingo@...hat.com>
Subject: [PATCH] export the queue_work wrappers GPL-only

This patch (as777) fixes an oversight in a couple of earlier patches.  Now 
the wrapper routines:

	queue_work(), queue_delayed_work(), queue_delayed_work_on(),
	schedule_work(), schedule_delayed_work(), and
	schedule_delayed_work_on()

are exported GPL-only, just as the originals used to be.

Signed-off-by: Alan Stern <stern@...land.harvard.edu>

---

On Tue, 29 Aug 2006, Jonathan Corbet wrote:

> One little thing I just noticed.  The old queue_work() functions were
> exported GPL-only.  And the new ones are too:
> 
> > -EXPORT_SYMBOL_GPL(queue_work);
> > +EXPORT_SYMBOL_GPL(add_work_to_q);
> 
> But the new wrappers are not:
> 
> > +EXPORT_SYMBOL(queue_work);
> 
> They should probably be exported in the same mode as before.

You're right...  I don't know how I managed to miss that.

> Also, should there be an entry added to
> Documentation/feature-removal-schedule.txt?

It's a question of whether anyone feels the need to remove the legacy
routines.

Andrew, if you think that after (say) a year's time those WARN_ON()s no
longer serve any useful purpose, I could do a big search-and-replace to
get rid of those old functions entirely.  I assume there's no problem with
accepting patches that change hundreds of files.

Alan Stern


Index: mm/kernel/workqueue.c
===================================================================
--- mm.orig/kernel/workqueue.c
+++ mm/kernel/workqueue.c
@@ -501,7 +501,7 @@ void fastcall queue_work(struct workqueu
 	rc = add_work_to_q(wq, work);
 	WARN_ON(rc < 0);
 }
-EXPORT_SYMBOL(queue_work);
+EXPORT_SYMBOL_GPL(queue_work);
 
 void fastcall queue_delayed_work(struct workqueue_struct *wq,
 		struct work_struct *work, unsigned long delay)
@@ -511,7 +511,7 @@ void fastcall queue_delayed_work(struct 
 	rc = add_delayed_work_to_q(wq, work, delay);
 	WARN_ON(rc < 0);
 }
-EXPORT_SYMBOL(queue_delayed_work);
+EXPORT_SYMBOL_GPL(queue_delayed_work);
 
 void queue_delayed_work_on(int cpu, struct workqueue_struct *wq,
 		struct work_struct *work, unsigned long delay)
@@ -521,7 +521,7 @@ void queue_delayed_work_on(int cpu, stru
 	rc = add_delayed_work_to_q_on(cpu, wq, work, delay);
 	WARN_ON(rc < 0);
 }
-EXPORT_SYMBOL(queue_delayed_work_on);
+EXPORT_SYMBOL_GPL(queue_delayed_work_on);
 
 void fastcall schedule_work(struct work_struct *work)
 {
@@ -530,7 +530,7 @@ void fastcall schedule_work(struct work_
 	rc = add_work_to_q(keventd_wq, work);
 	WARN_ON(rc < 0);
 }
-EXPORT_SYMBOL(schedule_work);
+EXPORT_SYMBOL_GPL(schedule_work);
 
 void fastcall schedule_delayed_work(struct work_struct *work,
 		unsigned long delay)
@@ -540,7 +540,7 @@ void fastcall schedule_delayed_work(stru
 	rc = add_delayed_work_to_q(keventd_wq, work, delay);
 	WARN_ON(rc < 0);
 }
-EXPORT_SYMBOL(schedule_delayed_work);
+EXPORT_SYMBOL_GPL(schedule_delayed_work);
 
 void schedule_delayed_work_on(int cpu, struct work_struct *work,
 		unsigned long delay)
@@ -550,7 +550,7 @@ void schedule_delayed_work_on(int cpu, s
 	rc = add_delayed_work_to_q_on(cpu, keventd_wq, work, delay);
 	WARN_ON(rc < 0);
 }
-EXPORT_SYMBOL(schedule_delayed_work_on);
+EXPORT_SYMBOL_GPL(schedule_delayed_work_on);
 
 /**
  * schedule_on_each_cpu - call a function on each online CPU from keventd

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/