* Re: [PATCH v2] staging: rtl8192u: Remove broken driver
[not found] <20231014211051.GA29518@matrix-ESPRIMO-P710>
@ 2023-10-17 5:33 ` Thorsten Leemhuis
2023-10-17 7:20 ` Greg Kroah-Hartman
0 siblings, 1 reply; 4+ messages in thread
From: Thorsten Leemhuis @ 2023-10-17 5:33 UTC (permalink / raw)
To: Philipp Hortmann, Greg Kroah-Hartman, linux-staging, linux-kernel
Cc: Arnd Bergmann, Kalle Valo, Larry Finger, Linux Next Mailing List
On 14.10.23 23:10, Philipp Hortmann wrote:
> Tests on rtl8192u hardware have shown that this driver is broken since
> 2016. Remove broken driver. Find fix for two bugs in second link.
>
> Link: https://lore.kernel.org/lkml/db98d9ac-7650-4a72-8eb9-4def1f17ea0d@app.fastmail.com/
> Link: https://lore.kernel.org/lkml/cover.1697089416.git.philipp.g.hortmann@gmail.com/
> Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
> ---
> V2: Added 2x links and adapted description.
>
> Did not find any artifacts of rtl8192u in MAINTAINERS.
> ---
> drivers/staging/Kconfig | 2 -
> drivers/staging/rtl8192u/Kconfig | 12 -
> [...]
My daily linux-next builds for Fedora (based on Fedora rawhide config)
failed today with this error msg:
"""
+ /usr/bin/make -s 'HOSTCFLAGS=-O2 -fexceptions -g
-grecord-gcc-switches -pipe -Wall -Werror=format-security
-Werror=implicit-function-declaration -Werror=implicit-int
-Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS
-specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong
-specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mcpu=power8
-mtune=power8 -fasynchronous-unwind-tables -fstack-clash-protection '
'HOSTLDFLAGS=-Wl,-z,relro -Wl,--as-needed -Wl,-z,now
-specs=/usr/lib/rpm/redhat/redhat-hardened-ld
-specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -Wl,--build-id=sha1
-specs=/usr/lib/rpm/redhat/redhat-package-notes ' -j2 mrproper
scripts/Makefile.clean:12: drivers/staging/rtl8192u/Makefile: No such
file or directory
make[5]: *** No rule to make target 'drivers/staging/rtl8192u/Makefile'.
Stop.
make[4]: *** [scripts/Makefile.clean:61: drivers/staging/rtl8192u] Error 2
make[3]: *** [scripts/Makefile.clean:61: drivers/staging] Error 2
make[3]: *** Waiting for unfinished jobs....
make[2]: *** [scripts/Makefile.clean:61: drivers] Error 2
make[1]: ***
[/builddir/build/BUILD/kernel-next-20231017/linux-6.6.0-0.0.next.20231017.447.vanilla.fc40.ppc64le/Makefile:1920:
_clean_.] Error 2
make: *** [Makefile:236: __sub-make] Error 2
error: Bad exit status from /var/tmp/rpm-tmp.TJDlJV (%build)
"""
Full log:
https://copr-be.cloud.fedoraproject.org/results/@kernel-vanilla/next/fedora-rawhide-ppc64le/06536998-next-next-all/builder-live.log.gz
Made me wonder if this patch left something behind; a quick grep showed
something suspicious in drivers/staging/Makefile:
"""
obj-$(CONFIG_RTL8192U) += rtl8192u/
"""
Ciao, Thorsten
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] staging: rtl8192u: Remove broken driver
2023-10-17 5:33 ` [PATCH v2] staging: rtl8192u: Remove broken driver Thorsten Leemhuis
@ 2023-10-17 7:20 ` Greg Kroah-Hartman
2023-10-17 7:42 ` Thorsten Leemhuis
2023-10-17 7:44 ` Arnd Bergmann
0 siblings, 2 replies; 4+ messages in thread
From: Greg Kroah-Hartman @ 2023-10-17 7:20 UTC (permalink / raw)
To: Thorsten Leemhuis
Cc: Philipp Hortmann, linux-staging, linux-kernel, Arnd Bergmann,
Kalle Valo, Larry Finger, Linux Next Mailing List
On Tue, Oct 17, 2023 at 07:33:49AM +0200, Thorsten Leemhuis wrote:
> On 14.10.23 23:10, Philipp Hortmann wrote:
> > Tests on rtl8192u hardware have shown that this driver is broken since
> > 2016. Remove broken driver. Find fix for two bugs in second link.
> >
> > Link: https://lore.kernel.org/lkml/db98d9ac-7650-4a72-8eb9-4def1f17ea0d@app.fastmail.com/
> > Link: https://lore.kernel.org/lkml/cover.1697089416.git.philipp.g.hortmann@gmail.com/
> > Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
> > ---
> > V2: Added 2x links and adapted description.
> >
> > Did not find any artifacts of rtl8192u in MAINTAINERS.
> > ---
> > drivers/staging/Kconfig | 2 -
> > drivers/staging/rtl8192u/Kconfig | 12 -
> > [...]
>
> My daily linux-next builds for Fedora (based on Fedora rawhide config)
> failed today with this error msg:
>
> """
> + /usr/bin/make -s 'HOSTCFLAGS=-O2 -fexceptions -g
> -grecord-gcc-switches -pipe -Wall -Werror=format-security
> -Werror=implicit-function-declaration -Werror=implicit-int
> -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS
> -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong
> -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mcpu=power8
> -mtune=power8 -fasynchronous-unwind-tables -fstack-clash-protection '
> 'HOSTLDFLAGS=-Wl,-z,relro -Wl,--as-needed -Wl,-z,now
> -specs=/usr/lib/rpm/redhat/redhat-hardened-ld
> -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -Wl,--build-id=sha1
> -specs=/usr/lib/rpm/redhat/redhat-package-notes ' -j2 mrproper
> scripts/Makefile.clean:12: drivers/staging/rtl8192u/Makefile: No such
> file or directory
> make[5]: *** No rule to make target 'drivers/staging/rtl8192u/Makefile'.
> Stop.
> make[4]: *** [scripts/Makefile.clean:61: drivers/staging/rtl8192u] Error 2
> make[3]: *** [scripts/Makefile.clean:61: drivers/staging] Error 2
> make[3]: *** Waiting for unfinished jobs....
> make[2]: *** [scripts/Makefile.clean:61: drivers] Error 2
> make[1]: ***
> [/builddir/build/BUILD/kernel-next-20231017/linux-6.6.0-0.0.next.20231017.447.vanilla.fc40.ppc64le/Makefile:1920:
> _clean_.] Error 2
> make: *** [Makefile:236: __sub-make] Error 2
> error: Bad exit status from /var/tmp/rpm-tmp.TJDlJV (%build)
> """
>
> Full log:
> https://copr-be.cloud.fedoraproject.org/results/@kernel-vanilla/next/fedora-rawhide-ppc64le/06536998-next-next-all/builder-live.log.gz
>
> Made me wonder if this patch left something behind; a quick grep showed
> something suspicious in drivers/staging/Makefile:
> """
> obj-$(CONFIG_RTL8192U) += rtl8192u/
> """
If you did 'make oldconfig' the option should have been removed and then
this line wouldn't have triggered, right? Odd, but I'll go drop this
entry as well.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] staging: rtl8192u: Remove broken driver
2023-10-17 7:20 ` Greg Kroah-Hartman
@ 2023-10-17 7:42 ` Thorsten Leemhuis
2023-10-17 7:44 ` Arnd Bergmann
1 sibling, 0 replies; 4+ messages in thread
From: Thorsten Leemhuis @ 2023-10-17 7:42 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Philipp Hortmann, linux-staging, linux-kernel, Arnd Bergmann,
Kalle Valo, Larry Finger, Linux Next Mailing List
On 17.10.23 09:20, Greg Kroah-Hartman wrote:
> On Tue, Oct 17, 2023 at 07:33:49AM +0200, Thorsten Leemhuis wrote:
>> On 14.10.23 23:10, Philipp Hortmann wrote:
>>> Tests on rtl8192u hardware have shown that this driver is broken since
>>> 2016. Remove broken driver. Find fix for two bugs in second link.
>>>
>>> Link: https://lore.kernel.org/lkml/db98d9ac-7650-4a72-8eb9-4def1f17ea0d@app.fastmail.com/
>>> Link: https://lore.kernel.org/lkml/cover.1697089416.git.philipp.g.hortmann@gmail.com/
>>> Signed-off-by: Philipp Hortmann <philipp.g.hortmann@gmail.com>
>>> ---
>>> V2: Added 2x links and adapted description.
>>>
>>> Did not find any artifacts of rtl8192u in MAINTAINERS.
>>> ---
>>> drivers/staging/Kconfig | 2 -
>>> drivers/staging/rtl8192u/Kconfig | 12 -
>>> [...]
>>
>> My daily linux-next builds for Fedora (based on Fedora rawhide config)
>> failed today with this error msg:
>>
>> """
>> + /usr/bin/make -s 'HOSTCFLAGS=-O2 -fexceptions -g
>> -grecord-gcc-switches -pipe -Wall -Werror=format-security
>> -Werror=implicit-function-declaration -Werror=implicit-int
>> -Wp,-U_FORTIFY_SOURCE,-D_FORTIFY_SOURCE=3 -Wp,-D_GLIBCXX_ASSERTIONS
>> -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong
>> -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mcpu=power8
>> -mtune=power8 -fasynchronous-unwind-tables -fstack-clash-protection '
>> 'HOSTLDFLAGS=-Wl,-z,relro -Wl,--as-needed -Wl,-z,now
>> -specs=/usr/lib/rpm/redhat/redhat-hardened-ld
>> -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -Wl,--build-id=sha1
>> -specs=/usr/lib/rpm/redhat/redhat-package-notes ' -j2 mrproper
>> scripts/Makefile.clean:12: drivers/staging/rtl8192u/Makefile: No such
>> file or directory
>> make[5]: *** No rule to make target 'drivers/staging/rtl8192u/Makefile'.
>> Stop.
>> make[4]: *** [scripts/Makefile.clean:61: drivers/staging/rtl8192u] Error 2
>> make[3]: *** [scripts/Makefile.clean:61: drivers/staging] Error 2
>> make[3]: *** Waiting for unfinished jobs....
>> make[2]: *** [scripts/Makefile.clean:61: drivers] Error 2
>> make[1]: ***
>> [/builddir/build/BUILD/kernel-next-20231017/linux-6.6.0-0.0.next.20231017.447.vanilla.fc40.ppc64le/Makefile:1920:
>> _clean_.] Error 2
>> make: *** [Makefile:236: __sub-make] Error 2
>> error: Bad exit status from /var/tmp/rpm-tmp.TJDlJV (%build)
>> """
>>
>> Full log:
>> https://copr-be.cloud.fedoraproject.org/results/@kernel-vanilla/next/fedora-rawhide-ppc64le/06536998-next-next-all/builder-live.log.gz
>>
>> Made me wonder if this patch left something behind; a quick grep showed
>> something suspicious in drivers/staging/Makefile:
>> """
>> obj-$(CONFIG_RTL8192U) += rtl8192u/
>> """
>
> If you did 'make oldconfig' the option should have been removed and then
> this line wouldn't have triggered, right?
Guess so, but apparently Fedora's rpm spec file runs the failing "make
mrproper" before running 'make oldconfig'.
> Odd, but I'll go drop this entry as well.
Great, thx!
Ciao, Thorsten
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] staging: rtl8192u: Remove broken driver
2023-10-17 7:20 ` Greg Kroah-Hartman
2023-10-17 7:42 ` Thorsten Leemhuis
@ 2023-10-17 7:44 ` Arnd Bergmann
1 sibling, 0 replies; 4+ messages in thread
From: Arnd Bergmann @ 2023-10-17 7:44 UTC (permalink / raw)
To: Greg Kroah-Hartman, Thorsten Leemhuis
Cc: Philipp Hortmann, linux-staging, linux-kernel, Kalle Valo,
Larry Finger, linux-next
On Tue, Oct 17, 2023, at 09:20, Greg Kroah-Hartman wrote:
> On Tue, Oct 17, 2023 at 07:33:49AM +0200, Thorsten Leemhuis wrote:
>> On 14.10.23 23:10, Philipp Hortmann wrote:
next-next-all/builder-live.log.gz
>>
>> Made me wonder if this patch left something behind; a quick grep showed
>> something suspicious in drivers/staging/Makefile:
>> """
>> obj-$(CONFIG_RTL8192U) += rtl8192u/
>> """
>
> If you did 'make oldconfig' the option should have been removed and then
> this line wouldn't have triggered, right? Odd, but I'll go drop this
> entry as well.
It looks like "make clean" enters both the obj-y and obj-n directories,
unlike 'make vmlinux' or 'make modules', which only care about
obj-m and obj-y.
Arnd
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-10-17 7:45 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20231014211051.GA29518@matrix-ESPRIMO-P710>
2023-10-17 5:33 ` [PATCH v2] staging: rtl8192u: Remove broken driver Thorsten Leemhuis
2023-10-17 7:20 ` Greg Kroah-Hartman
2023-10-17 7:42 ` Thorsten Leemhuis
2023-10-17 7:44 ` Arnd Bergmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox