On 17/07/2025 01:07, Randy Dunlap wrote:
>
>
> On 7/16/25 4:25 AM, Stephen Rothwell wrote:
>> Hi all,
>>
>> Changes since 20250715:
>>
>
> on ARCH=um SUBARCH=x86_64:
>
> ../drivers/vfio/cdx/intr.c: In function ‘vfio_cdx_msi_enable’:
> ../drivers/vfio/cdx/intr.c:41:15: error: implicit declaratio
On 31/03/2024 10:43, Krzysztof Kozlowski wrote:
> 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
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 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
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
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
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
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
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
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
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 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
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
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
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
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
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 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
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
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
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
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
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
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
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
this task away from the drivers to the core
virtio code, just like we did for platform_driver in
commit 9447057eaff8 ("platform_device: use a macro instead of
platform_driver_register").
Signed-off-by: Krzysztof Kozlowski
---
Documentation/driver-api/virtio/writing_virtio_drive
9447057eaff8 ("platform_device: use a macro instead of
platform_driver_register").
Best regards,
Krzysztof
---
Krzysztof Kozlowski (25):
virtio: store owner from modules with register_virtio_driver()
virtio: balloon: drop owner assignment
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
On 29/03/2024 12:42, Stefano Garzarella wrote:
>> };
>>
>> -int register_virtio_driver(struct virtio_driver *driver)
>> +int __register_virtio_driver(struct virtio_driver *driver, struct module
>> *owner)
>> {
>> /* Catch this early. */
>> BUG_ON(driver->feature_table_size && !driver->fe
On 29/03/2024 11:27, Bartosz Golaszewski wrote:
> On Wed, Mar 27, 2024 at 1:45 PM Krzysztof Kozlowski
> wrote:
>>
>> virtio core already sets the .owner, so driver does not need to.
>>
>> Signed-off-by: Krzysztof Kozlowski
>>
>> ---
>>
>> De
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
On 27/03/2024 14:34, Johannes Berg wrote:
> On Wed, 2024-03-27 at 13:40 +0100, Krzysztof Kozlowski wrote:
>> virtio core already sets the .owner, so driver does not need to.
>
>> All further patches depend on the first virtio patch, therefore please ack
>> and this should
On 27/03/2024 13:55, Kalle Valo wrote:
> Krzysztof Kozlowski writes:
>
>> virtio core already sets the .owner, so driver does not need to.
>>
>> Signed-off-by: Krzysztof Kozlowski
>
> We use "wifi:" in the title, not "wireless:". It would
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
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
virtio core already sets the .owner, so driver does not need to.
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/rpmsg/virtio_rpmsg_bus.c
index
virtio core already sets the .owner, so driver does not need to.
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.c
index
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
virtio core already sets the .owner, so driver does not need to.
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_vsock/virtio_transport.c
index
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
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
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
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
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
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
virtio core already sets the .owner, so driver does not need to.
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 b/drivers/gpio/gpio-virtio.c
index fcc5e8c08973
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 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
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
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
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
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
virtio core already sets the .owner, so driver does not need to.
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
index 97a37c062997
this task away from the drivers to the core
amba bus code, just like we did for platform_driver in
commit 9447057eaff8 ("platform_device: use a macro instead of
platform_driver_register").
Signed-off-by: Krzysztof Kozlowski
---
Documentation/driver-api/virtio/writing_virtio_drive
the drivers to the core
amba bus code, just like we did for platform_driver in commit
9447057eaff8 ("platform_device: use a macro instead of
platform_driver_register").
Best regards,
Krzysztof
Signed-off-by: Krzysztof Kozlowski
---
Krzysztof Kozlowski (22):
virtio: store owner fr
55 matches
Mail list logo