* v4.19.221 breaks qdisc modules
@ 2022-03-27 21:39 Thorsten Glaser
2022-03-27 21:44 ` Thorsten Glaser
2022-03-28 5:49 ` Greg Kroah-Hartman
0 siblings, 2 replies; 6+ messages in thread
From: Thorsten Glaser @ 2022-03-27 21:39 UTC (permalink / raw)
To: Greg Kroah-Hartman, Vlad Buslov, stable
Cc: Jiri Pirko, David S. Miller, Lee Jones
Hi,
commit 92833e8b5db6c209e9311ac8c6a44d3bf1856659 breaks the
build of sch_* modules in stable.
I already have:
#if LINUX_VERSION_CODE < KERNEL_VERSION(4, 20, 0)
qdisc_destroy(cl->leaf.q);
#else
qdisc_put(cl->leaf.q);
#endif
But this makes it more tricky… or can I “just” change this
to KERNEL_VERSION(4, 19, 221) ?
Nevertheless, renaming functions isn’t something I’d expect
to happen in stable. At least add a #define or so redirecting
from the old/stable name…
bye,
//mirabilos
--
Infrastrukturexperte • tarent solutions GmbH
Am Dickobskreuz 10, D-53121 Bonn • http://www.tarent.de/
Telephon +49 228 54881-393 • Fax: +49 228 54881-235
HRB AG Bonn 5168 • USt-ID (VAT): DE122264941
Geschäftsführer: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg
****************************************************
/⁀\ The UTF-8 Ribbon
╲ ╱ Campaign against Mit dem tarent-Newsletter nichts mehr verpassen:
╳ HTML eMail! Also, https://www.tarent.de/newsletter
╱ ╲ header encryption!
****************************************************
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: v4.19.221 breaks qdisc modules
2022-03-27 21:39 v4.19.221 breaks qdisc modules Thorsten Glaser
@ 2022-03-27 21:44 ` Thorsten Glaser
2022-03-28 5:50 ` Greg Kroah-Hartman
2022-03-28 5:49 ` Greg Kroah-Hartman
1 sibling, 1 reply; 6+ messages in thread
From: Thorsten Glaser @ 2022-03-27 21:44 UTC (permalink / raw)
To: Greg Kroah-Hartman, Vlad Buslov, stable
Cc: Jiri Pirko, David S. Miller, Lee Jones, Daniel.Fleischer,
Marcel.Krause, christian.hampe, haye.haehne, keith.lloyd
On Sun, 27 Mar 2022, Thorsten Glaser wrote:
> But this makes it more tricky… or can I “just” change this
> to KERNEL_VERSION(4, 19, 221) ?
Well, of course not:
$ cat /usr/src/linux-headers-4.19.0-19-amd64/include/generated/uapi/linux/version.h
#define LINUX_VERSION_CODE 267263
#define KERNEL_VERSION(a,b,c) (((a) << 16) + ((b) << 8) + (c))
$ print $(((267263 >> 16) & 0xFF)).$(((267263 >> 8) & 0xFF)).$((267263 & 0xFF))
4.19.255
Whose bright idea was *that*?
How can I make this module compatible with *both* 4.19 variants?
Thanks in advance,
//mirabilos
--
Infrastrukturexperte • tarent solutions GmbH
Am Dickobskreuz 10, D-53121 Bonn • http://www.tarent.de/
Telephon +49 228 54881-393 • Fax: +49 228 54881-235
HRB AG Bonn 5168 • USt-ID (VAT): DE122264941
Geschäftsführer: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg
****************************************************
/⁀\ The UTF-8 Ribbon
╲ ╱ Campaign against Mit dem tarent-Newsletter nichts mehr verpassen:
╳ HTML eMail! Also, https://www.tarent.de/newsletter
╱ ╲ header encryption!
****************************************************
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: v4.19.221 breaks qdisc modules
2022-03-27 21:44 ` Thorsten Glaser
@ 2022-03-28 5:50 ` Greg Kroah-Hartman
2022-03-28 20:23 ` Thorsten Glaser
0 siblings, 1 reply; 6+ messages in thread
From: Greg Kroah-Hartman @ 2022-03-28 5:50 UTC (permalink / raw)
To: Thorsten Glaser
Cc: Vlad Buslov, stable, Jiri Pirko, David S. Miller, Lee Jones,
Daniel.Fleischer, Marcel.Krause, christian.hampe, haye.haehne,
keith.lloyd
On Sun, Mar 27, 2022 at 11:44:59PM +0200, Thorsten Glaser wrote:
> On Sun, 27 Mar 2022, Thorsten Glaser wrote:
>
> > But this makes it more tricky… or can I “just” change this
> > to KERNEL_VERSION(4, 19, 221) ?
>
> Well, of course not:
>
> $ cat /usr/src/linux-headers-4.19.0-19-amd64/include/generated/uapi/linux/version.h
> #define LINUX_VERSION_CODE 267263
> #define KERNEL_VERSION(a,b,c) (((a) << 16) + ((b) << 8) + (c))
> $ print $(((267263 >> 16) & 0xFF)).$(((267263 >> 8) & 0xFF)).$((267263 & 0xFF))
> 4.19.255
>
> Whose bright idea was *that*?
>
> How can I make this module compatible with *both* 4.19 variants?
Ah, external code, sorry, you are on your own.
As for how to test for larger numbers, see the answer in the email
archives, others have done this successfully.
good luck!
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: v4.19.221 breaks qdisc modules
2022-03-28 5:50 ` Greg Kroah-Hartman
@ 2022-03-28 20:23 ` Thorsten Glaser
2022-03-28 21:37 ` Thorsten Glaser
0 siblings, 1 reply; 6+ messages in thread
From: Thorsten Glaser @ 2022-03-28 20:23 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Vlad Buslov, stable, Jiri Pirko, David S. Miller, Lee Jones,
Daniel.Fleischer, Marcel.Krause, christian.hampe, haye.haehne,
keith.lloyd
On Mon, 28 Mar 2022, Greg Kroah-Hartman wrote:
> > How can I make this module compatible with *both* 4.19 variants?
>
> Ah, external code, sorry, you are on your own.
This is… no, I’m going to have to censor my thoughts on this response.
(Besides, these are patched version of in-tree modules to experiment
with new network features.)
> As for how to test for larger numbers, see the answer in the email
> archives, others have done this successfully.
>
> good luck!
Search engine fodder would have been welcome. All I found so far is
https://lwn.net/ml/linux-kernel/20210208145805.898658055@linuxfoundation.org/
in which you speak of mysterious “different ways of extracting the
version number” that “out of tree modules have”, I’d love to see them.
bye,
//mirabilos
--
Infrastrukturexperte • tarent solutions GmbH
Am Dickobskreuz 10, D-53121 Bonn • http://www.tarent.de/
Telephon +49 228 54881-393 • Fax: +49 228 54881-235
HRB AG Bonn 5168 • USt-ID (VAT): DE122264941
Geschäftsführer: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg
****************************************************
/⁀\ The UTF-8 Ribbon
╲ ╱ Campaign against Mit dem tarent-Newsletter nichts mehr verpassen:
╳ HTML eMail! Also, https://www.tarent.de/newsletter
╱ ╲ header encryption!
****************************************************
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: v4.19.221 breaks qdisc modules
2022-03-28 20:23 ` Thorsten Glaser
@ 2022-03-28 21:37 ` Thorsten Glaser
0 siblings, 0 replies; 6+ messages in thread
From: Thorsten Glaser @ 2022-03-28 21:37 UTC (permalink / raw)
To: Greg Kroah-Hartman
Cc: Vlad Buslov, stable, Jiri Pirko, David S. Miller, Lee Jones,
Daniel.Fleischer, Marcel.Krause, christian.hampe, haye.haehne,
keith.lloyd
Dixi quod…
> On Mon, 28 Mar 2022, Greg Kroah-Hartman wrote:
> > As for how to test for larger numbers, see the answer in the email
> > archives, others have done this successfully.
> Search engine fodder would have been welcome. All I found so far is
I’ve figured out enough Makefile and cpp hackery… after finally finding
out that I need to take the values on Makefile level, not from a header
file… so, for the archives (and for the benefit of others):
https://github.com/tarent/sch_jens/commit/19c8e499fba8cef65a0b301dc158168707b5689f
bye,
//mirabilos
--
Infrastrukturexperte • tarent solutions GmbH
Am Dickobskreuz 10, D-53121 Bonn • http://www.tarent.de/
Telephon +49 228 54881-393 • Fax: +49 228 54881-235
HRB AG Bonn 5168 • USt-ID (VAT): DE122264941
Geschäftsführer: Dr. Stefan Barth, Kai Ebenrett, Boris Esser, Alexander Steeg
****************************************************
/⁀\ The UTF-8 Ribbon
╲ ╱ Campaign against Mit dem tarent-Newsletter nichts mehr verpassen:
╳ HTML eMail! Also, https://www.tarent.de/newsletter
╱ ╲ header encryption!
****************************************************
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: v4.19.221 breaks qdisc modules
2022-03-27 21:39 v4.19.221 breaks qdisc modules Thorsten Glaser
2022-03-27 21:44 ` Thorsten Glaser
@ 2022-03-28 5:49 ` Greg Kroah-Hartman
1 sibling, 0 replies; 6+ messages in thread
From: Greg Kroah-Hartman @ 2022-03-28 5:49 UTC (permalink / raw)
To: Thorsten Glaser
Cc: Vlad Buslov, stable, Jiri Pirko, David S. Miller, Lee Jones
On Sun, Mar 27, 2022 at 11:39:10PM +0200, Thorsten Glaser wrote:
> Hi,
>
> commit 92833e8b5db6c209e9311ac8c6a44d3bf1856659 breaks the
> build of sch_* modules in stable.
Where do these sch_* modules live? In the kernel tree itself? I have
not gotten any failed build errors here, what config option do I need to
enable?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-03-28 22:16 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-27 21:39 v4.19.221 breaks qdisc modules Thorsten Glaser
2022-03-27 21:44 ` Thorsten Glaser
2022-03-28 5:50 ` Greg Kroah-Hartman
2022-03-28 20:23 ` Thorsten Glaser
2022-03-28 21:37 ` Thorsten Glaser
2022-03-28 5:49 ` Greg Kroah-Hartman
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.