* [PATCH] virt: vmgenid: drop redundant .owner
@ 2024-03-26 7:53 Krzysztof Kozlowski
2024-03-26 8:07 ` Krzysztof Kozlowski
0 siblings, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-26 7:53 UTC (permalink / raw
To: Theodore Ts'o, Jason A. Donenfeld, linux-kernel; +Cc: Krzysztof Kozlowski
Core already sets .owner in acpi_bus_register_driver().
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
drivers/virt/vmgenid.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/virt/vmgenid.c b/drivers/virt/vmgenid.c
index b67a28da4702..8f6880c3a87f 100644
--- a/drivers/virt/vmgenid.c
+++ b/drivers/virt/vmgenid.c
@@ -88,7 +88,6 @@ static const struct acpi_device_id vmgenid_ids[] = {
static struct acpi_driver vmgenid_driver = {
.name = "vmgenid",
.ids = vmgenid_ids,
- .owner = THIS_MODULE,
.ops = {
.add = vmgenid_add,
.notify = vmgenid_notify
--
2.34.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] virt: vmgenid: drop redundant .owner
2024-03-26 7:53 [PATCH] virt: vmgenid: drop redundant .owner Krzysztof Kozlowski
@ 2024-03-26 8:07 ` Krzysztof Kozlowski
2024-03-26 14:17 ` Jason A. Donenfeld
0 siblings, 1 reply; 4+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-26 8:07 UTC (permalink / raw
To: Theodore Ts'o, Jason A. Donenfeld, linux-kernel
On 26/03/2024 08:53, Krzysztof Kozlowski wrote:
> Core already sets .owner in acpi_bus_register_driver().
>
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
> drivers/virt/vmgenid.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/virt/vmgenid.c b/drivers/virt/vmgenid.c
> index b67a28da4702..8f6880c3a87f 100644
> --- a/drivers/virt/vmgenid.c
> +++ b/drivers/virt/vmgenid.c
> @@ -88,7 +88,6 @@ static const struct acpi_device_id vmgenid_ids[] = {
> static struct acpi_driver vmgenid_driver = {
> .name = "vmgenid",
> .ids = vmgenid_ids,
> - .owner = THIS_MODULE,
This does not make sense and is not correct. I need to fix
acpi_bus_register_driver first(). Please ignore.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] virt: vmgenid: drop redundant .owner
2024-03-26 8:07 ` Krzysztof Kozlowski
@ 2024-03-26 14:17 ` Jason A. Donenfeld
2024-03-26 16:10 ` Krzysztof Kozlowski
0 siblings, 1 reply; 4+ messages in thread
From: Jason A. Donenfeld @ 2024-03-26 14:17 UTC (permalink / raw
To: Krzysztof Kozlowski; +Cc: Theodore Ts'o, linux-kernel
On Tue, Mar 26, 2024 at 09:07:52AM +0100, Krzysztof Kozlowski wrote:
> On 26/03/2024 08:53, Krzysztof Kozlowski wrote:
> > Core already sets .owner in acpi_bus_register_driver().
> >
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> > ---
> > drivers/virt/vmgenid.c | 1 -
> > 1 file changed, 1 deletion(-)
> >
> > diff --git a/drivers/virt/vmgenid.c b/drivers/virt/vmgenid.c
> > index b67a28da4702..8f6880c3a87f 100644
> > --- a/drivers/virt/vmgenid.c
> > +++ b/drivers/virt/vmgenid.c
> > @@ -88,7 +88,6 @@ static const struct acpi_device_id vmgenid_ids[] = {
> > static struct acpi_driver vmgenid_driver = {
> > .name = "vmgenid",
> > .ids = vmgenid_ids,
> > - .owner = THIS_MODULE,
>
> This does not make sense and is not correct. I need to fix
> acpi_bus_register_driver first(). Please ignore.
Oh, okay. Does that mean this will resurface as part of a series that is
core fix + individual driver cleanups subsequently?
You made some comment on the other thread indicating that this
meant the driver is "clearly abandoned" because of "all this owner
crap". Wondering if this driver here hits some weird edge case that
isn't covered by changes made "many years ago" or if something else is
up.
Jason
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] virt: vmgenid: drop redundant .owner
2024-03-26 14:17 ` Jason A. Donenfeld
@ 2024-03-26 16:10 ` Krzysztof Kozlowski
0 siblings, 0 replies; 4+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-26 16:10 UTC (permalink / raw
To: Jason A. Donenfeld; +Cc: Theodore Ts'o, linux-kernel
On 26/03/2024 15:17, Jason A. Donenfeld wrote:
> On Tue, Mar 26, 2024 at 09:07:52AM +0100, Krzysztof Kozlowski wrote:
>> On 26/03/2024 08:53, Krzysztof Kozlowski wrote:
>>> Core already sets .owner in acpi_bus_register_driver().
>>>
>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>> ---
>>> drivers/virt/vmgenid.c | 1 -
>>> 1 file changed, 1 deletion(-)
>>>
>>> diff --git a/drivers/virt/vmgenid.c b/drivers/virt/vmgenid.c
>>> index b67a28da4702..8f6880c3a87f 100644
>>> --- a/drivers/virt/vmgenid.c
>>> +++ b/drivers/virt/vmgenid.c
>>> @@ -88,7 +88,6 @@ static const struct acpi_device_id vmgenid_ids[] = {
>>> static struct acpi_driver vmgenid_driver = {
>>> .name = "vmgenid",
>>> .ids = vmgenid_ids,
>>> - .owner = THIS_MODULE,
>>
>> This does not make sense and is not correct. I need to fix
>> acpi_bus_register_driver first(). Please ignore.
>
> Oh, okay. Does that mean this will resurface as part of a series that is
> core fix + individual driver cleanups subsequently?
Yes.
>
> You made some comment on the other thread indicating that this
> meant the driver is "clearly abandoned" because of "all this owner
> crap". Wondering if this driver here hits some weird edge case that
> isn't covered by changes made "many years ago" or if something else is
> up.
No, it was just my mistake. Driver is fine, sorry for too fast judging.
It turns out only some xxx_driver set .owner field. Which I think leads
to actual problems, but that's a bit bigger patchset I need to prepare.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-03-26 16:11 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-26 7:53 [PATCH] virt: vmgenid: drop redundant .owner Krzysztof Kozlowski
2024-03-26 8:07 ` Krzysztof Kozlowski
2024-03-26 14:17 ` Jason A. Donenfeld
2024-03-26 16:10 ` Krzysztof Kozlowski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).