[PATCH v2 00/25] virtio: store owner from modules with register_virtio_driver()

Changes in v2: - Three new patches: virtio mem+input+balloon - Minor commit msg adjustments - Add tags - Link to v1: https://lore.kernel.org/r/20240327-module-owner-virtio-v1-0-0feffab77...@linaro.org Merging === All further patches depend on the first virtio patch, therefore please ack and t

[PATCH v2 03/25] virtio: input: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Changes in v2: 1. New patch --- drivers/virtio/virtio_input.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/virtio/virtio_input.c b/drivers/virtio/virtio_input.c index 3aa467038

[PATCH v2 01/25] virtio: store owner from modules with register_virtio_driver()

Modules registering driver with register_virtio_driver() might forget to set .owner field. i2c-virtio.c for example has it missing. The field is used by some of other kernel parts for reference counting (try_module_get()), so it is expected that drivers will set it. Solve the problem by moving t

[PATCH v2 02/25] virtio: balloon: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Changes in v2: 1. New patch --- drivers/virtio/virtio_balloon.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/virtio/virtio_balloon.c b/drivers/virtio/virtio_balloon.c index 1f5

[PATCH v2 04/25] virtio: mem: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Changes in v2: 1. New patch --- drivers/virtio/virtio_mem.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/virtio/virtio_mem.c b/drivers/virtio/virtio_mem.c index e8355f55a8f7..e

[PATCH v2 05/25] um: virt-pci: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Acked-by: Johannes Berg Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- arch/um/drivers/virt-pci.c | 1 - 1 file changed, 1 deletion(-) diff --git a/arch/um/drivers/virt-pci.c b/arch/um/drivers/virt-pci.c

[PATCH v2 07/25] bluetooth: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/bluetooth/virtio_bt.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/bluetooth/virtio_bt.c b/drivers/bluetooth/virtio_bt.c index 2ac70b560

[PATCH v2 08/25] hwrng: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/char/hw_random/virtio-rng.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/char/hw_random/virtio-rng.c b/drivers/char/hw_random/virtio-rn

[PATCH v2 09/25] virtio_console: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/char/virtio_console.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/char/virtio_console.c b/drivers/char/virtio_console.c index 035f89f

[PATCH v2 06/25] virtio_blk: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/block/virtio_blk.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/block/virtio_blk.c b/drivers/block/virtio_blk.c index 42dea7601d87..46bd

[PATCH v2 10/25] crypto: virtio - drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/crypto/virtio/virtio_crypto_core.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/crypto/virtio/virtio_crypto_core.c b/drivers/crypto/vir

[PATCH v2 11/25] firmware: arm_scmi: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/firmware/arm_scmi/virtio.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/firmware/arm_scmi/virtio.c b/drivers/firmware/arm_scmi/virtio.c

[PATCH v2 13/25] drm/virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/gpu/drm/virtio/virtgpu_drv.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/virtio/virtgpu_drv.c b/drivers/gpu/drm/virtio/virtgpu

[PATCH v2 15/25] misc: nsm: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/misc/nsm.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/misc/nsm.c b/drivers/misc/nsm.c index 0eaa3b4484bd..ef7b32742340 100644 --- a/dr

[PATCH v2 16/25] net: caif: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/net/caif/caif_virtio.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/caif/caif_virtio.c b/drivers/net/caif/caif_virtio.c index 0b0f23

[PATCH v2 17/25] net: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/net/virtio_net.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index c22d1118a133..61f6806996

[PATCH v2 18/25] net: 9p: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- net/9p/trans_virtio.c | 1 - 1 file changed, 1 deletion(-) diff --git a/net/9p/trans_virtio.c b/net/9p/trans_virtio.c index e305071eb7b8..0b8086f58ad5 100644

[PATCH v2 21/25] nvdimm: virtio_pmem: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Acked-by: Dave Jiang Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/nvdimm/virtio_pmem.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/nvdimm/virtio_pmem.c b/drivers/nvdimm/virtio_pmem

[PATCH v2 19/25] vsock/virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Acked-by: Stefano Garzarella Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- net/vmw_vsock/virtio_transport.c | 1 - 1 file changed, 1 deletion(-) diff --git a/net/vmw_vsock/virtio_transport.c b/net/vmw_vs

[PATCH v2 20/25] wifi: mac80211_hwsim: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/net/wireless/virtual/mac80211_hwsim.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/wireless/virtual/mac80211_hwsim.c b/drivers/net/

[PATCH v2 22/25] rpmsg: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Reviewed-by: Mathieu Poirier Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/rpmsg/virtio_rpmsg_bus.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/rpmsg/virtio_rpmsg_bus.c b/drivers/rp

[PATCH v2 23/25] scsi: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/scsi/virtio_scsi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/scsi/virtio_scsi.c b/drivers/scsi/virtio_scsi.c index 617eb892f4ad..89ca

[PATCH v2 24/25] fuse: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- fs/fuse/virtio_fs.c | 1 - 1 file changed, 1 deletion(-) diff --git a/fs/fuse/virtio_fs.c b/fs/fuse/virtio_fs.c index 322af827a232..ca7b64f9c3c7 100644 --- a

[PATCH v2 25/25] sound: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- sound/virtio/virtio_card.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sound/virtio/virtio_card.c b/sound/virtio/virtio_card.c index 2da20c625247..7805

[PATCH v2 14/25] iommu: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/iommu/virtio-iommu.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/iommu/virtio-iommu.c b/drivers/iommu/virtio-iommu.c index 04048f64a2c0

[PATCH v2 12/25] gpio: virtio: drop owner assignment

virtio core already sets the .owner, so driver does not need to. Acked-by: Bartosz Golaszewski Acked-by: Viresh Kumar Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- drivers/gpio/gpio-virtio.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpio/gpio-virtio.c

Re: [PATCH 01/22] virtio: store owner from modules with register_virtio_driver()

On Wed, Mar 27, 2024 at 01:40:54PM +0100, Krzysztof Kozlowski wrote: > Modules registering driver with register_virtio_driver() might forget to > set .owner field. i2c-virtio.c for example has it missing. The field > is used by some of other kernel parts for reference counting > (try_module_get()

Re: [PATCH v2 11/25] firmware: arm_scmi: virtio: drop owner assignment

On Sun, Mar 31, 2024 at 10:43:58AM +0200, Krzysztof Kozlowski wrote: > virtio core already sets the .owner, so driver does not need to. > Acked-by: Sudeep Holla -- Regards, Sudeep