kthread: convert remaining users to kthread_create_worker

Convert ivtv to use kthread_run_worker() instead of the deprecated kthread_run(kthread_worker_fn) pattern. The new API sets worker->task before the worker starts.

Signed-off-by Bradley Morgan <[email protected]>

---
drivers/media/pci/ivtv/ivtv-driver.c | 14 ++++++-------- drivers/media/pci/ivtv/ivtv-driver.h | 3 +--

2 files changed, 7 insertions(+), 10 deletions(-)

drivers/media/pci/ivtv/ivtv-driver.c +3 -5

index b9ea56ec9593..6f0e0e46deb5 100644
@@ -703,15 +703,13 @@ static int ivtv_init_struct1(struct ivtv *itv)
 	spin_lock_init(&itv->lock);
 	spin_lock_init(&itv->dma_reg_lock);
 
-	kthread_init_worker(&itv->irq_worker);
-	itv->irq_worker_task = kthread_run(kthread_worker_fn, &itv->irq_worker,
-					   "%s", itv->v4l2_dev.name);
-	if (IS_ERR(itv->irq_worker_task)) {
+	itv->irq_worker = kthread_run_worker(0, "%s", itv->v4l2_dev.name);
+	if (IS_ERR(itv->irq_worker)) {
 		IVTV_ERR("Could not create ivtv task\n");
 		return -1;
 	}
 	/* must use the FIFO scheduler as it is realtime sensitive */
-	sched_set_fifo(itv->irq_worker_task);
+	sched_set_fifo(itv->irq_worker->task);
 
 	kthread_init_work(&itv->irq_work, ivtv_irq_work_handler);
 

@@ -1232,7 +1230,7 @@ static int ivtv_probe(struct pci_dev *pdev, const struct pci_device_id *pci_id)
 	v4l2_ctrl_handler_free(&itv->cxhdl.hdl);
 	exit_ivtv_i2c(itv);

free_worker:

drivers/media/pci/ivtv/ivtv-driver.c +1 -1

-	kthread_stop(itv->irq_worker_task);
+	kthread_destroy_worker(itv->irq_worker);

err:

	if (retval == 0)
		retval = -ENODEV;

drivers/media/pci/ivtv/ivtv-driver.c +2 -2

@@ -1372,8 +1370,8 @@ static void ivtv_remove(struct pci_dev *pdev)
 	timer_shutdown_sync(&itv->dma_timer);
 
 	/* Kill irq worker */
-	kthread_flush_worker(&itv->irq_worker);
-	kthread_stop(itv->irq_worker_task);
+	kthread_flush_worker(itv->irq_worker);
+	kthread_destroy_worker(itv->irq_worker);
 
 	ivtv_streams_cleanup(itv);
 	ivtv_udma_free(itv);

drivers/media/pci/ivtv/ivtv-driver.h +0 -0

index 000e8beecc7c..091bc44f13bc 100644

Thank you for your contribution! Sashiko AI review found 2 potential
issue(s) to consider:
- [High] kthread_queue_work() receives a double pointer instead of a pointer to kthread_worker, leading to a build error or runtime memory corruption.

- [High] kthread_destroy_worker() is called before free_irq(), creating a Use-After-Free race condition with the interrupt handler.

--

commit a504d2685cd2f4c4e6b93a81c847d55367eae1c0
Author: Bradley Morgan <[email protected]>

media: ivtv: convert to kthread_run_worker

Convert ivtv to use kthread_run_worker() instead of the deprecated kthread_run(kthread_worker_fn) pattern. The new API sets worker->task before the worker starts.

Review

Quoted 1 line

(quoted diff) +0 -0

b/drivers/media/pci/ivtv/ivtv-driver.c

Quoted 3 lines

index b9ea56ec9593f..6f0e0e46deb5b 100644

drivers/media/pci/ivtv/ivtv-driver.c +0 -0

[ ... ]

drivers/media/pci/ivtv/ivtv-driver.c static void ivtv_remove(struct pci_dev *pdev) · line 1374

Quoted 8 lines

drivers/media/pci/ivtv/ivtv-driver.c +2 -2

@@ -1372,8 +1370,8 @@ static void ivtv_remove(struct pci_dev *pdev)
 	timer_shutdown_sync(&itv->dma_timer);
 
 	/* Kill irq worker */
-	kthread_flush_worker(&itv->irq_worker);
-	kthread_stop(itv->irq_worker_task);
+	kthread_flush_worker(itv->irq_worker);
+	kthread_destroy_worker(itv->irq_worker);

[Severity: High]
Does this create a use-after-free race condition with the interrupt handler?

kthread_destroy_worker() frees the dynamically allocated worker structure here,
but free_irq() is called later in ivtv_remove():

drivers/media/pci/ivtv/ivtv-driver.c:ivtv_remove() {
	...
	kthread_destroy_worker(itv->irq_worker);
	...
	free_irq(itv->pdev->irq, (void *)itv);
	...
}

Because free_irq() is the synchronization point that guarantees no interrupt
handlers are still running, an in-flight ivtv_irq_handler() on another CPU can survive past kthread_destroy_worker().

If the handler subsequently attempts to queue work, it will dereference the
already-freed irq_worker pointer, causing a kernel panic.

Review

Quoted 4 lines

drivers/media/pci/ivtv/ivtv-driver.c +0 -0

 	ivtv_streams_cleanup(itv);
 	ivtv_udma_free(itv);

(quoted diff) +0 -0

b/drivers/media/pci/ivtv/ivtv-driver.h

drivers/media/pci/ivtv/ivtv-driver.h struct ivtv { · line 671

Quoted 10 lines

index f1f18911332e8..41789a50d1ca6 100644

drivers/media/pci/ivtv/ivtv-driver.h +1 -2

@@ -668,8 +668,7 @@ struct ivtv {
 	/* Interrupts & DMA */
 	u32 irqmask;                    /* active interrupts */
 	u32 irq_rr_idx;                 /* round-robin stream index */
-	struct kthread_worker irq_worker;		/* kthread worker for PIO/YUV/VBI actions */
-	struct task_struct *irq_worker_task;		/* task for irq_worker */
+	struct kthread_worker *irq_worker;		/* kthread worker for PIO/YUV/VBI actions */

[Severity: High]
Does this change from a struct to a pointer cause a build error or memory corruption in ivtv_irq_handler()?

Because irq_worker is now a pointer, the unmodified call site in ivtv_irq_handler() now passes a double pointer:

drivers/media/pci/ivtv/ivtv-irq.c:ivtv_irq_handler() {
   ...
   kthread_queue_work(&itv->irq_worker, &itv->irq_work);
   ...
}

This violates the API contract and will cause a build failure with modern compilers due to incompatible pointer types.

If the warning is ignored or coerced, kthread_queue_work() will treat the offset within struct ivtv as a kthread_worker object, corrupting adjacent fields when it acquires the spinlock, leading to a kernel crash.

Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider:

Pre-existing issues:
- [High] Missing state cleanup for priv->tx_skb in TX timeout, interface stop, and device removal paths leads to memory leaks, data races, and potential BUG() panics.
- [High] Hardware memory structures read over SPI are not converted from little-endian to CPU endianness.

--

commit d2a821de33aec8b828ff820d968fcd13ce493790
Author: Bradley Morgan <[email protected]>

net: encx24j600: convert to kthread_run_worker

Convert encx24j600 to use kthread_run_worker() instead of the deprecated kthread_run(kthread_worker_fn) pattern. The new API sets worker->task before the worker starts.

Review

Quoted 1 line

(quoted diff) +0 -0

b/drivers/net/ethernet/microchip/encx24j600.c

drivers/net/ethernet/microchip/encx24j600.c static netdev_tx_t encx24j600_tx(struct sk_buff

Quoted 3 lines

drivers/net/ethernet/microchip/encx24j600.c +0 -0

@@ -884,7 +883,7 @@ static netdev_tx_t encx24j600_tx(struct sk_buff

*skb, struct net_device *dev)

drivers/net/ethernet/microchip/encx24j600.c static netdev_tx_t encx24j600_tx(struct sk_buff *skb, struct net_device *dev) · line 886

Quoted 5 lines

drivers/net/ethernet/microchip/encx24j600.c +1 -1

 	/* Remember the skb for deferred processing */
 	priv->tx_skb = skb;
 
-	kthread_queue_work(&priv->kworker, &priv->tx_work);
+	kthread_queue_work(priv->kworker, &priv->tx_work);

[Severity: High]
This is a pre-existing issue, but is there a missing state cleanup for priv->tx_skb in the TX timeout path?

If a TX timeout occurs, encx24j600_tx_timeout() wakes the queue without cleaning up priv->tx_skb or clearing pending TX state. The next TX here unconditionally overwrites priv->tx_skb, leaking the previous packet.

Furthermore, a delayed hardware interrupt for the previous packet can arrive
and trigger encx24j600_tx_complete(), which frees the new priv->tx_skb and sets it to NULL. When encx24j600_tx_complete() subsequently fires for the new packet, it sees priv->tx_skb is NULL and calls BUG():

drivers/net/ethernet/microchip/encx24j600.c:encx24j600_tx_complete() {
   if (!priv->tx_skb) {
       BUG();
       return;
   }

...

Alternatively, encx24j600_hw_tx() may run concurrently with the delayed interrupt, reading priv->tx_skb->len when it has been set to NULL, causing a NULL pointer dereference.

[ ... ]

drivers/net/ethernet/microchip/encx24j600.c static void encx24j600_spi_remove(struct

Quoted 1 line

drivers/net/ethernet/microchip/encx24j600.c +0 -0

@@ -1100,7 +1096,7 @@ static void encx24j600_spi_remove(struct

spi_device *spi)

drivers/net/ethernet/microchip/encx24j600.c static void encx24j600_spi_remove(struct spi_device *spi) · line 1102

Quoted 8 lines

drivers/net/ethernet/microchip/encx24j600.c +1 -1

 	struct encx24j600_priv *priv = dev_get_drvdata(&spi->dev);
 
 	unregister_netdev(priv->ndev);
-	kthread_stop(priv->kworker_task);
+	kthread_destroy_worker(priv->kworker);
 
 	free_netdev(priv->ndev);
 }

[Severity: High]
This is also a pre-existing issue, but does device removal strand the un-freed priv->tx_skb?

During driver removal in encx24j600_spi_remove(), the kworker is destroyed,
and then the driver private data is freed without cleaning up priv->tx_skb,
which results in a memory leak.

[Severity: High]
This is another pre-existing issue, but are the hardware memory structures read over SPI properly converted from little-endian to CPU endianness?

The 8-byte Receive Status Vector is read directly from the SPI interface into a struct rsv using encx24j600_raw_read() in encx24j600_rx_packets():

encx24j600_raw_read(priv, RRXDATA, (u8 *)&rsv, sizeof(rsv));

However, struct rsv in drivers/net/ethernet/microchip/encx24j600_hw.h uses native CPU types without using explicit __le16 and __le32 types:

struct rsv {
   u16 next_packet;
   u16 len;
   u32 rxstat;
};

On a big-endian system, won't the raw bytes be misinterpreted? For example,
a packet length of 60 (0x003C) would be read as 0x3C00 (15360), which exceeds
MAX_FRAMELEN and causes the packet to be dropped. The corrupted next_packet
pointer would then be written back to the device's ERXTAIL register, destroying the receive ring buffer state.

Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider:

New issues:
- [High] The explicitly allocated kthread worker is destroyed before the devm-managed threaded IRQ handler is freed, causing a Use-After-Free.

Pre-existing issues:
- [High] Closing any single port cancels the shared polling timer, starving all other active ports on the same multi-port chip. --

commit 3bc3e4920cc0fac18e8f989a3296c6b4222d065c
Author: Bradley Morgan <[email protected]>

tty: sc16is7xx: convert to kthread_run_worker

Convert sc16is7xx to use kthread_run_worker() instead of the deprecated kthread_run(kthread_worker_fn) pattern. The new API sets worker->task before the worker starts.

Review

Quoted 1 line

(quoted diff) +0 -0

b/drivers/tty/serial/sc16is7xx.c

drivers/tty/serial/sc16is7xx.c static void sc16is7xx_shutdown(struct uart_port

Quoted 4 lines

index 7107a0fb1e7bb..2cfc1f2900091 100644

drivers/tty/serial/sc16is7xx.c +0 -0

@@ -1186,7 +1185,7 @@ static void sc16is7xx_shutdown(struct uart_port

*port)

drivers/tty/serial/sc16is7xx.c static void sc16is7xx_shutdown(struct uart_port *port) · line 1189

Quoted 6 lines

drivers/tty/serial/sc16is7xx.c +1 -1

 	if (s->polling)
 		kthread_cancel_delayed_work_sync(&s->poll_work);
 
-	kthread_flush_worker(&s->kworker);
+	kthread_flush_worker(s->kworker);
 }

[Severity: High]
This is a pre-existing issue, but does closing any single port cancel the shared polling timer for all ports?

In polling mode, s->poll_work is shared across all ports on the chip. If one port is closed, sc16is7xx_shutdown() unconditionally cancels s->poll_work,
which seems to starve any other active ports on the same chip.

drivers/tty/serial/sc16is7xx.c void sc16is7xx_remove(struct device *dev) · line 1709

Quoted 11 lines

drivers/tty/serial/sc16is7xx.c +2 -2

@@ -1705,8 +1702,8 @@ void sc16is7xx_remove(struct device *dev)
 	if (s->polling)
 		kthread_cancel_delayed_work_sync(&s->poll_work);
 
-	kthread_flush_worker(&s->kworker);
-	kthread_stop(s->kworker_task);
+	kthread_flush_worker(s->kworker);
+	kthread_destroy_worker(s->kworker);
 
 	clk_disable_unprepare(s->clk);
 }

[Severity: High]
Can kthread_destroy_worker() lead to a use-after-free if the threaded IRQ handler fires concurrently?

The driver uses devm_request_threaded_irq() in sc16is7xx_probe() to manage its
interrupt, meaning the IRQ handler remains active until after sc16is7xx_remove() returns and devres cleans it up.

If an interrupt fires or an in-flight threaded IRQ executes in parallel right
after kthread_destroy_worker() frees s->kworker, the IRQ handler could call
functions like sc16is7xx_ier_clear() which enqueue work:

drivers/tty/serial/sc16is7xx.c:sc16is7xx_ier_clear() {
   ...
   kthread_queue_work(s->kworker, &one->reg_work);
}

This would dereference the freed s->kworker pointer. Previously kworker was
embedded in the sc16is7xx_port struct, so stopping the task was safe from a
use-after-free, but now kthread_destroy_worker() frees the dynamically allocated worker before the IRQ handler is disabled.

Thank you for your contribution! Sashiko AI review found 1 potential
issue(s) to consider:
- [Medium] Removing the self-assignment worker->task = current; breaks out-of-tree (OOT) users of the old kthread_run(kthread_worker_fn, ...) API, causing them to permanently deadlock due to missed wakeups.

--

commit 4e8323d40e4fe5053d4171cbca8193f9bd6080b8
Author: Bradley Morgan <[email protected]>

kthread: remove worker->task self assignment

All kthread worker users now create their workers with kthread_create_worker*(),
which sets worker->task before the worker starts. The self assignment in kthread_worker_fn() is removed, leaving a warning check for out-of-tree users.

Quoted 7 lines

kernel/kthread.c +0 -0

index 63beb59b7a3d1..643407647e79d 100644
@@ -987,12 +987,10 @@ int kthread_worker_fn(void *worker_ptr)
 	struct kthread_worker *worker = worker_ptr;
 	struct kthread_work *work;

kernel/kthread.c int kthread_worker_fn(void *worker_ptr) · line 993

Quoted 10 lines

kernel/kthread.c +3 -5

 
-	/*
-	 * FIXME: Update the check and remove the assignment when all kthread
-	 * worker users are created using kthread_create_worker*() functions.
+	/* All workers are created with worker->task set. If this fires,
+	 * the caller is broken.
 	 */
-	WARN_ON(worker->task && worker->task != current);
-	worker->task = current;
+	WARN_ON(worker->task != current);

[Severity: Medium]
Does removing this assignment cause out-of-tree users of the old kthread_run() API to hang permanently?

Because the self-assignment is removed, worker->task will remain NULL for any
callers still using the old API. When new work is inserted, the wakeup will be
silently skipped in kthread_insert_work():

kernel/kthread.c:kthread_insert_work() {
	...
	if (!worker->current_work && likely(worker->task))
		wake_up_process(worker->task);
	...
}

This means the warning intended as a sanity check turns into a silent, fatal
breakage where the worker hangs in TASK_INTERRUPTIBLE and queued work is never
processed.

Quoted 3 lines

kernel/kthread.c +0 -0

 	if (worker->flags & KTW_FREEZABLE)
 		set_freezable();

Hi Bradley,

kernel test robot noticed the following build errors:

[auto build test ERROR on tty/tty-testing]
[also build test ERROR on tty/tty-next tty/tty-linus rafael-pm/linux-next rafael-pm/bleeding-edge brauner-vfs/vfs.all linus/master v7.3-rc1 next-20260904]

[If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Bradley-Morgan/media-ivtv-convert-to-kthread_run_worker/20260904-093753
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/tty.git tty-testing

patch link:
https://lore.kernel.org/r/c925e99c73bea9e0cdab656ba09453df7e81dcb2.1788513591.git.brads%40mainlining.org
patch subject: [PATCH v2 5/5] kthread: remove worker->task self assignment
config: riscv-allmodconfig (https://download.01.org/0day-ci/archive/20260905/[email protected]/config)
compiler: clang version 24.0.0git (https://github.com/llvm/llvm-project 0edd1b088cc36b4faee80358c925a91e16006258) reproduce (this is a W=1 build):
(https://download.01.org/0day-ci/archive/20260905/[email protected]/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <[email protected]>
| Closes:
https://lore.kernel.org/oe-kbuild-all/[email protected]/

All errors (new ones prefixed by >>):

Quoted 1 line

Quoted 1 line

drivers/media/pci/ivtv/ivtv-irq.c:1053:22: error: incompatible pointer

types passing 'struct kthread_worker **' to parameter of type 'struct kthread_worker *'; remove & [-Wincompatible-pointer-types]

   1053 |                 kthread_queue_work(&itv->irq_worker, &itv->irq_work);
        |                                    ^~~~~~~~~~~~~~~~

include/linux/kthread.h:263:48: note: passing argument to parameter 'worker' here
263 | bool kthread_queue_work(struct kthread_worker *worker,
| ^
1 error generated.

vim +1053 drivers/media/pci/ivtv/ivtv-irq.c

1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil
2007-04-27 917
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil
2007-04-27 918 irqreturn_t ivtv_irq_handler(int irq, void *dev_id)
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil
2007-04-27 919 {

1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   920  	struct ivtv *itv = (struct ivtv *)dev_id;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   921  	u32 combo;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   922  	u32 stat;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   923  	int i;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   924  	u8 vsync_force = 0;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   925  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   926  	spin_lock(&itv->dma_reg_lock);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   927  	/* get contents of irq status register */
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   928  	stat = read_reg(IVTV_REG_IRQSTATUS);

1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil
2007-04-27 929
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 2007-04-27 930 combo = ~itv->irqmask & stat;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil
2007-04-27 931

1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   932  	/* Clear out IRQ */
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   933  	if (combo) write_reg(combo, IVTV_REG_IRQSTATUS);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   934  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   935  	if (0 == combo) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   936  		/* The vsync interrupt is unusual and clears itself. If we
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   937  		 * took too long, we may have missed it. Do some checks
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   938  		 */
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   939  		if (~itv->irqmask & IVTV_IRQ_DEC_VSYNC) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   940  			/* vsync is enabled, see if we're in a new field */
4e1af31aaaa8cc drivers/media/video/ivtv/ivtv-irq.c Andy Walls    2010-03-13   941  			if ((itv->last_vsync_field & 1) !=
4e1af31aaaa8cc drivers/media/video/ivtv/ivtv-irq.c Andy Walls    2010-03-13   942  			    (read_reg(IVTV_REG_DEC_LINE_FIELD) & 1)) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   943  				/* New field, looks like we missed it */
4e1af31aaaa8cc drivers/media/video/ivtv/ivtv-irq.c Andy Walls    2010-03-13   944  				IVTV_DEBUG_YUV("VSync interrupt missed %d\n",
4e1af31aaaa8cc drivers/media/video/ivtv/ivtv-irq.c Andy Walls    2010-03-13   945  				       read_reg(IVTV_REG_DEC_LINE_FIELD) >> 16);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   946  				vsync_force = 1;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   947  			}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   948  		}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   949  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   950  		if (!vsync_force) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   951  			/* No Vsync expected, wasn't for us */
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   952  			spin_unlock(&itv->dma_reg_lock);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   953  			return IRQ_NONE;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   954  		}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   955  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   956  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   957  	/* Exclude interrupts noted below from the output, otherwise the log is flooded with
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   958  	   these messages */
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   959  	if (combo & ~0xff6d0400)
bd58df6d522d5a drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-07-10   960  		IVTV_DEBUG_HI_IRQ("======= valid IRQ bits: 0x%08x ======\n", combo);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   961  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   962  	if (combo & IVTV_IRQ_DEC_DMA_COMPLETE) {
bd58df6d522d5a drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-07-10   963  		IVTV_DEBUG_HI_IRQ("DEC DMA COMPLETE\n");
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   964  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   965  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   966  	if (combo & IVTV_IRQ_DMA_READ) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   967  		ivtv_irq_dma_read(itv);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   968  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   969  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   970  	if (combo & IVTV_IRQ_ENC_DMA_COMPLETE) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   971  		ivtv_irq_enc_dma_complete(itv);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   972  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   973  
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19   974  	if (combo & IVTV_IRQ_ENC_PIO_COMPLETE) {
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19   975  		ivtv_irq_enc_pio_complete(itv);
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19   976  	}
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-05-19   977  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   978  	if (combo & IVTV_IRQ_DMA_ERR) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   979  		ivtv_irq_dma_err(itv);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   980  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   981  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   982  	if (combo & IVTV_IRQ_ENC_START_CAP) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   983  		ivtv_irq_enc_start_cap(itv);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   984  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   985  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   986  	if (combo & IVTV_IRQ_ENC_VBI_CAP) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   987  		ivtv_irq_enc_vbi_cap(itv);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   988  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   989  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   990  	if (combo & IVTV_IRQ_DEC_VBI_RE_INSERT) {
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19   991  		ivtv_irq_dec_vbi_reinsert(itv);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   992  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   993  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   994  	if (combo & IVTV_IRQ_ENC_EOS) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   995  		IVTV_DEBUG_IRQ("ENC EOS\n");
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   996  		set_bit(IVTV_F_I_EOS, &itv->i_flags);
fd8b281a2809d2 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-08-23   997  		wake_up(&itv->eos_waitq);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27   998  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27   999  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1000  	if (combo & IVTV_IRQ_DEC_DATA_REQ) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1001  		ivtv_irq_dec_data_req(itv);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1002  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27  1003  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1004  	/* Decoder Vertical Sync - We can't rely on 'combo', so check if vsync enabled */
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1005  	if (~itv->irqmask & IVTV_IRQ_DEC_VSYNC) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1006  		ivtv_irq_vsync(itv);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1007  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27  1008  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1009  	if (combo & IVTV_IRQ_ENC_VIM_RST) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1010  		IVTV_DEBUG_IRQ("VIM RST\n");
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1011  		/*ivtv_vapi(itv, CX2341X_ENC_REFRESH_INPUT, 0); */
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1012  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27  1013  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1014  	if (combo & IVTV_IRQ_DEC_AUD_MODE_CHG) {
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1015  		IVTV_DEBUG_INFO("Stereo mode changed\n");
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1016  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27  1017  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1018  	if ((combo & IVTV_IRQ_DMA) && !test_bit(IVTV_F_I_DMA, &itv->i_flags)) {
33bc4dea0ece37 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-08-18  1019  		itv->irq_rr_idx++;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1020  		for (i = 0; i < IVTV_MAX_STREAMS; i++) {
33bc4dea0ece37 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-08-18  1021  			int idx = (i + itv->irq_rr_idx) % IVTV_MAX_STREAMS;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1022  			struct ivtv_stream *s = &itv->streams[idx];
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27  1023  
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1024  			if (!test_and_clear_bit(IVTV_F_S_DMA_PENDING, &s->s_flags))
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1025  				continue;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1026  			if (s->type >= IVTV_DEC_STREAM_TYPE_MPG)
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1027  				ivtv_dma_dec_start(s);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1028  			else
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1029  				ivtv_dma_enc_start(s);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1030  			break;
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1031  		}
b6e436b263b354 drivers/media/video/ivtv/ivtv-irq.c Ian Armstrong
2009-12-21  1032  
b6e436b263b354 drivers/media/video/ivtv/ivtv-irq.c Ian Armstrong 2009-12-21  1033  		if (i == IVTV_MAX_STREAMS &&
b6e436b263b354 drivers/media/video/ivtv/ivtv-irq.c Ian Armstrong 2009-12-21  1034  		    test_bit(IVTV_F_I_UDMA_PENDING, &itv->i_flags))
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1035  			ivtv_udma_start(itv);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1036  	}
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-04-27  1037  
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19  1038  	if ((combo & IVTV_IRQ_DMA) && !test_bit(IVTV_F_I_PIO, &itv->i_flags)) {
33bc4dea0ece37 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-08-18  1039  		itv->irq_rr_idx++;
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19  1040  		for (i = 0; i < IVTV_MAX_STREAMS; i++) {
33bc4dea0ece37 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-08-18  1041  			int idx = (i + itv->irq_rr_idx) % IVTV_MAX_STREAMS;
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19  1042  			struct ivtv_stream *s = &itv->streams[idx];
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-05-19  1043  
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19  1044  			if (!test_and_clear_bit(IVTV_F_S_PIO_PENDING, &s->s_flags))
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19  1045  				continue;
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19  1046  			if (s->type == IVTV_DEC_STREAM_TYPE_VBI || s->type < IVTV_DEC_STREAM_TYPE_MPG)
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19  1047  				ivtv_dma_enc_start(s);
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19  1048  			break;
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19  1049  		}
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-05-19  1050  	}
dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 
2007-05-19  1051  
2f3a98931f51be drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-08-25  1052  	if (test_and_clear_bit(IVTV_F_I_HAVE_WORK, &itv->i_flags)) {
3989144f863ac5 drivers/media/pci/ivtv/ivtv-irq.c   Petr Mladek   2016-10-11 @1053  		kthread_queue_work(&itv->irq_worker, &itv->irq_work);
2f3a98931f51be drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-08-25  1054  	}

dc02d50a6d71cb drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 2007-05-19 1055
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 2007-04-27 1056 spin_unlock(&itv->dma_reg_lock);
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 2007-04-27 1057

1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1058  	/* If we've just handled a 'forced' vsync, it's safest to say it
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1059  	 * wasn't ours. Another device may have triggered it at just
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1060  	 * the right time.
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1061  	 */
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil  2007-04-27  1062  	return vsync_force ? IRQ_NONE : IRQ_HANDLED;

1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 2007-04-27 1063 }
1a0adaf37c30e8 drivers/media/video/ivtv/ivtv-irq.c Hans Verkuil 2007-04-27 1064

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

media: ivtv: convert to kthread_run_worker
net: encx24j600: convert to kthread_run_worker
tty: sc16is7xx: convert to kthread_run_worker
cpufreq: schedutil: convert to kthread_create_worker

kthread: remove worker->task self assignment

Quoted 4 lines · On Fri, 4 Sep 2026 09:37:21 +0000 Bradley Morgan wrote:

media: ivtv: convert to kthread_run_worker
net: encx24j600: convert to kthread_run_worker
tty: sc16is7xx: convert to kthread_run_worker
cpufreq: schedutil: convert to kthread_create_worker

Please send these 4 to appropriate subsystems

Quoted 1 line

kthread: remove worker->task self assignment

Then after the next merge window when trees converge send this one out

Quoted 6 lines · Jakub Kicinski

Quoted 4 lines · On Fri, 4 Sep 2026 09:37:21 +0000 Bradley Morgan wrote:

media: ivtv: convert to kthread_run_worker
net: encx24j600: convert to kthread_run_worker
tty: sc16is7xx: convert to kthread_run_worker
cpufreq: schedutil: convert to kthread_create_worker

Quoted 5 lines

Please send these 4 to appropriate subsystems

Quoted 1 line

kthread: remove worker->task self assignment

Then after the next merge window when trees converge send this one out

hi, I was hoping one person could merge it with acks from all subsystems

Quoted 1 line · On Fri, 04 Sep 2026 16:56:30 +0100 Bradley Morgan wrote:

On 4 September 2026 16:54:18 BST, Jakub Kicinski <[email protected]>

Quoted 8 lines · wrote:

Quoted 8 lines · On Fri, 4 Sep 2026 09:37:21 +0000 Bradley Morgan wrote:

Quoted 4 lines

media: ivtv: convert to kthread_run_worker
net: encx24j600: convert to kthread_run_worker
tty: sc16is7xx: convert to kthread_run_worker
cpufreq: schedutil: convert to kthread_create_worker

Please send these 4 to appropriate subsystems

Quoted 5 lines

Quoted 3 lines

Quoted 1 line

kthread: remove worker->task self assignment

Then after the next merge window when trees converge send this one out

hi, I was hoping one person could merge it with acks from all subsystems

Not how this works.

Quoted 11 lines · Jakub Kicinski

Quoted 1 line · On Fri, 04 Sep 2026 16:56:30 +0100 Bradley Morgan wrote:

On 4 September 2026 16:54:18 BST, Jakub Kicinski <[email protected]>

Quoted 8 lines · wrote:

Quoted 8 lines · On Fri, 4 Sep 2026 09:37:21 +0000 Bradley Morgan wrote:

Quoted 4 lines

media: ivtv: convert to kthread_run_worker
net: encx24j600: convert to kthread_run_worker
tty: sc16is7xx: convert to kthread_run_worker
cpufreq: schedutil: convert to kthread_create_worker

Please send these 4 to appropriate subsystems

Quoted 7 lines

Quoted 5 lines

Quoted 3 lines

Quoted 1 line

kthread: remove worker->task self assignment

Then after the next merge window when trees converge send this one out

hi, I was hoping one person could merge it with acks from all subsystems

Not how this works.

Ugh, I've seen it before happen.

Look, fine, I'll do a V3 soon, WITH ONLY the subsystem changes, all separate, all sent over to their respective maintainers, then when all thathas been merged and there's the next merge window, then whatever, I'll send then the kthread removal

IMHO this is a massive annoyance though.. because what if one of the driverfolks doesn't wanna respond to me!?!

Quoted 1 line · Bradley Morgan

On 4 September 2026 22:10:32 BST, Jakub Kicinski <[email protected]>

Quoted 9 lines · wrote:

Quoted 9 lines · On Fri, 04 Sep 2026 16:56:30 +0100 Bradley Morgan wrote:

Quoted 1 line

On 4 September 2026 16:54:18 BST, Jakub Kicinski <[email protected]>

Quoted 6 lines · wrote:

Quoted 6 lines · On Fri, 4 Sep 2026 09:37:21 +0000 Bradley Morgan wrote:

Quoted 4 lines

media: ivtv: convert to kthread_run_worker
net: encx24j600: convert to kthread_run_worker
tty: sc16is7xx: convert to kthread_run_worker
cpufreq: schedutil: convert to kthread_create_worker

Quoted 5 lines

Quoted 5 lines

Quoted 5 lines

Quoted 5 lines

Please send these 4 to appropriate subsystems

Quoted 1 line

kthread: remove worker->task self assignment

Then after the next merge window when trees converge send this one

out

Quoted 2 lines

Quoted 2 lines

Quoted 2 lines

hi, I was hoping one person could merge it with acks from all

subsystems

Quoted 7 lines

Quoted 2 lines

Not how this works.

Ugh, I've seen it before happen.

Look, fine, I'll do a V3 soon, WITH ONLY the subsystem changes, all separate, all sent over to their respective maintainers, then when all thathas been merged and there's the next merge window, then whatever,

I'll

Quoted 4 lines

send then the kthread removal

IMHO this is a massive annoyance though.. because what if one of the driverfolks doesn't wanna respond to me!?!

Then, after trying for the normal way, you can make the driver change at the same time. But to circumvent the "normal way" thinking it might not work, is not the best thing to do.

thanks,

greg k-h

Quoted 29 lines · Greg Kroah-Hartman

Quoted 1 line · Bradley Morgan

On 4 September 2026 22:10:32 BST, Jakub Kicinski <[email protected]>

Quoted 9 lines · wrote:

Quoted 9 lines · On Fri, 04 Sep 2026 16:56:30 +0100 Bradley Morgan wrote:

Quoted 1 line

On 4 September 2026 16:54:18 BST, Jakub Kicinski <[email protected]>

Quoted 6 lines · wrote:

Quoted 6 lines · On Fri, 4 Sep 2026 09:37:21 +0000 Bradley Morgan wrote:

Quoted 4 lines

media: ivtv: convert to kthread_run_worker
net: encx24j600: convert to kthread_run_worker
tty: sc16is7xx: convert to kthread_run_worker
cpufreq: schedutil: convert to kthread_create_worker

Quoted 5 lines

Quoted 5 lines

Quoted 5 lines

Quoted 5 lines

Please send these 4 to appropriate subsystems

Quoted 1 line

kthread: remove worker->task self assignment

Then after the next merge window when trees converge send this one

out

Quoted 2 lines

Quoted 2 lines

Quoted 2 lines

hi, I was hoping one person could merge it with acks from all

subsystems

Quoted 7 lines

Quoted 2 lines

Not how this works.

Ugh, I've seen it before happen.

Look, fine, I'll do a V3 soon, WITH ONLY the subsystem changes, all separate, all sent over to their respective maintainers, then when all thathas been merged and there's the next merge window, then whatever,

I'll

Quoted 9 lines

Quoted 4 lines

send then the kthread removal

IMHO this is a massive annoyance though.. because what if one of the driverfolks doesn't wanna respond to me!?!

Then, after trying for the normal way, you can make the driver change at the same time. But to circumvent the "normal way" thinking it might not work, is not the best thing to do.

Greg!

Right, okay, Ill tell you more.

For features being added, where many drivers from different subsystems get covered at once, what they tend to do is:

A: send all patches, rely on all maintainers to merge their own crap, which is a mess

B: Have a designated merger, who merges all the changes at once, no chance of regression because it's all bundled up.

C: send drivers first, then send the feature. So then driver maintainers can merge their crap, then the main feature gets merged in merge window

I prefer B, because it's easier that way, akpm is usually des merger hence I CCed him.

Or tip tree?

Honestly, if I had to be honest, C is a very annoying way, because of what I said above, if driver maintainer thinks I'm some newbie idiot or something, then what will I do? He won't merge it!

Quoted 14 lines · Bradley Morgan

Quoted 12 lines · Greg Kroah-Hartman

Quoted 1 line · Bradley Morgan

On 4 September 2026 22:10:32 BST, Jakub Kicinski <[email protected]>

Quoted 9 lines · wrote:

Quoted 9 lines · On Fri, 04 Sep 2026 16:56:30 +0100 Bradley Morgan wrote:

Quoted 1 line

On 4 September 2026 16:54:18 BST, Jakub Kicinski <[email protected]>

Quoted 6 lines · wrote:

Quoted 6 lines · On Fri, 4 Sep 2026 09:37:21 +0000 Bradley Morgan wrote:

Quoted 4 lines

media: ivtv: convert to kthread_run_worker
net: encx24j600: convert to kthread_run_worker
tty: sc16is7xx: convert to kthread_run_worker
cpufreq: schedutil: convert to kthread_create_worker

Quoted 5 lines

Quoted 5 lines

Quoted 5 lines

Quoted 5 lines

Quoted 5 lines

Quoted 5 lines

Please send these 4 to appropriate subsystems

Quoted 1 line

kthread: remove worker->task self assignment

Then after the next merge window when trees converge send this

one

Quoted 10 lines

Quoted 10 lines

out

Quoted 2 lines

Quoted 2 lines

Quoted 2 lines

hi, I was hoping one person could merge it with acks from all

subsystems

Quoted 6 lines

Quoted 2 lines

Not how this works.

Ugh, I've seen it before happen.

Look, fine, I'll do a V3 soon, WITH ONLY the subsystem changes, all separate, all sent over to their respective maintainers, then when

all

Quoted 8 lines

Quoted 8 lines

Quoted 1 line

thathas been merged and there's the next merge window, then whatever,

I'll

Quoted 4 lines

send then the kthread removal

IMHO this is a massive annoyance though.. because what if one of the driverfolks doesn't wanna respond to me!?!

Then, after trying for the normal way, you can make the driver change

at

Quoted 1 line

Quoted 1 line

the same time. But to circumvent the "normal way" thinking it might

not

Quoted 8 lines

Quoted 2 lines

work, is not the best thing to do.

Greg!

Right, okay, Ill tell you more.

For features being added, where many drivers from different subsystems

get

Quoted 3 lines

covered at once, what they tend to do is:

A: send all patches, rely on all maintainers to merge their own crap,

which is a mess

Quoted 2 lines

B: Have a designated merger, who merges all the changes at once, no

chance of regression because it's all bundled up.

Quoted 3 lines

C: send drivers first, then send the feature. So then driver maintainers

can merge their crap, then the main feature gets merged in merge window

Quoted 2 lines

I prefer B, because it's easier that way, akpm is usually des merger

hence

Quoted 5 lines

I CCed him.

Or tip tree?

Honestly, if I had to be honest, C is a very annoying way, because of

what

Quoted 2 lines

I said above, if driver maintainer thinks I'm some newbie idiot or something, then what will I do? He won't merge it!

To quote a longtime kernel developer years ago, "Kernel development is hard, let's go shopping."

Sorry, but yes, it can be difficult to touch cross-subsystem stuff like this, always has been. Just be patient.

good luck!

greg k-h