From: "Luis R. Rodriguez" <mcgrof@do-not-panic.com>
To: backports@vger.kernel.org
Cc: ben@decadent.org.uk, noelamac@gmail.com,
linux-kernel@vger.kernel.org, rusty@rustcorp.com.au,
andi@firstfloor.org,
"Luis R. Rodriguez" <mcgrof@do-not-panic.com>
Subject: [RFC 04/30] compat: embrace LINUX_BACKPORT() for v2.6.25
Date: Wed, 20 Mar 2013 02:22:29 -0700 [thread overview]
Message-ID: <1363771375-31715-5-git-send-email-mcgrof@do-not-panic.com> (raw)
In-Reply-To: <1363771375-31715-1-git-send-email-mcgrof@do-not-panic.com>
From: "Luis R. Rodriguez" <mcgrof@do-not-panic.com>
There is one change needed here to get compilation
working on v2.6.24, strict_strtoull is now being
redefined and because of a change that went into
v2.6.38.4 kstrtoul() was added there and the old
strict_strtoul was made a define from it. To help
aid the older kernels such as 2.6.24 that need
the old strict_strtoul we simply check if strict_strtoull
is defined, then we know you're on >= 2.6.38.4 and
don't need kstrtoul() and friends.
If strict_strtoull ever needs to be backported for
older kernels we'll have to revisit this check.
It is worth documenting here why this change went into
v2.6.38.4 and not the orignal v2.6.38 release, so
the commit is provided.
commit 280a1c38c907ab1e2617bdcef66de6bc70897253
Author: Alexey Dobriyan <adobriyan@gmail.com>
kstrto*: converting strings to integers done (hopefully) right
commit 33ee3b2e2eb9b4b6c64dcf9ed66e2ac3124e748c upstream.
1. simple_strto*() do not contain overflow checks and crufty,
libc way to indicate failure.
2. strict_strto*() also do not have overflow checks but the name and
comments pretend they do.
3. Both families have only "long long" and "long" variants,
but users want strtou8()
4. Both "simple" and "strict" prefixes are wrong:
Simple doesn't exactly say what's so simple, strict should not exist
because conversion should be strict by default.
The solution is to use "k" prefix and add convertors for more types.
Enter
kstrtoull()
kstrtoll()
kstrtoul()
kstrtol()
kstrtouint()
kstrtoint()
kstrtou64()
kstrtos64()
kstrtou32()
kstrtos32()
kstrtou16()
kstrtos16()
kstrtou8()
kstrtos8()
Include runtime testsuite (somewhat incomplete) as well.
strict_strto*() become deprecated, stubbed to kstrto*() and
eventually will be removed altogether.
Use kstrto*() in code today!
Note: on some archs _kstrtoul() and _kstrtol() are left in tree, even if
they'll be unused at runtime. This is temporarily solution,
because I don't want to hardcode list of archs where these
functions aren't needed. Current solution with sizeof() and
__alignof__ at least always works.
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Tested with ckmake:
1 2.6.24 [ OK ]
2 2.6.25 [ OK ]
3 2.6.26 [ OK ]
4 2.6.27 [ OK ]
5 2.6.28 [ OK ]
6 2.6.29 [ OK ]
7 2.6.30 [ OK ]
8 2.6.31 [ OK ]
9 2.6.32 [ OK ]
10 2.6.33 [ OK ]
11 2.6.34 [ OK ]
12 2.6.35 [ OK ]
13 2.6.36 [ OK ]
14 2.6.37 [ OK ]
15 2.6.38 [ OK ]
16 2.6.39 [ OK ]
17 3.0.65 [ OK ]
18 3.1.10 [ OK ]
19 3.2.38 [ OK ]
20 3.3.8 [ OK ]
21 3.4.32 [ OK ]
22 3.5.7 [ OK ]
23 3.6.11 [ OK ]
24 3.7.9 [ OK ]
25 3.8.0 [ OK ]
26 3.9-rc1 [ OK ]
Signed-off-by: Luis R. Rodriguez <mcgrof@do-not-panic.com>
---
include/linux/compat-2.6.25.h | 5 +++++
include/linux/compat-2.6.39.h | 2 +-
2 files changed, 6 insertions(+), 1 deletion(-)
diff --git a/include/linux/compat-2.6.25.h b/include/linux/compat-2.6.25.h
index 5ca6783..5d4fe38 100644
--- a/include/linux/compat-2.6.25.h
+++ b/include/linux/compat-2.6.25.h
@@ -68,6 +68,7 @@ hit:
}
/* Backports b718989da7 */
+#define pci_enable_device_mem LINUX_BACKPORT(pci_enable_device_mem)
int __must_check pci_enable_device_mem(struct pci_dev *dev);
/*
@@ -221,7 +222,9 @@ int compat_pm_qos_power_deinit(void);
#define dev_crit(dev, format, arg...) \
dev_printk(KERN_CRIT , dev , format , ## arg)
+#define __dev_addr_sync LINUX_BACKPORT(__dev_addr_sync)
extern int __dev_addr_sync(struct dev_addr_list **to, int *to_count, struct dev_addr_list **from, int *from_count);
+#define __dev_addr_unsync LINUX_BACKPORT(__dev_addr_unsync)
extern void __dev_addr_unsync(struct dev_addr_list **to, int *to_count, struct dev_addr_list **from, int *from_count);
#define seq_file_net &init_net;
@@ -301,7 +304,9 @@ static inline void led_classdev_unregister_suspended(struct led_classdev *lcd)
* The following things are out of ./include/linux/kernel.h
* The new iwlwifi driver is using them.
*/
+#define strict_strtoul LINUX_BACKPORT(strict_strtoul)
extern int strict_strtoul(const char *, unsigned int, unsigned long *);
+#define strict_strtol LINUX_BACKPORT(strict_strtol)
extern int strict_strtol(const char *, unsigned int, long *);
#else
diff --git a/include/linux/compat-2.6.39.h b/include/linux/compat-2.6.39.h
index 8ff08ba..33db220 100644
--- a/include/linux/compat-2.6.39.h
+++ b/include/linux/compat-2.6.39.h
@@ -113,7 +113,7 @@ static inline struct msi_desc *irq_desc_get_msi_desc(struct irq_desc *desc)
* version included in compat-drivers. We use strict_strtol to check if
* kstrto* is already available.
*/
-#ifndef strict_strtol
+#ifndef strict_strtoull
/* Internal, do not use. */
int __must_check _kstrtoul(const char *s, unsigned int base, unsigned long *res);
int __must_check _kstrtol(const char *s, unsigned int base, long *res);
--
1.7.10.4
next prev parent reply other threads:[~2013-03-20 9:23 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-03-20 9:22 [RFC 00/30] compat: deal with kernel backport namespace Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 01/30] compat: add LINUX_BACKPORT() for prefixing symbols Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 02/30] compat: embrace LINUX_BACKPORT() for v2.6.23 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 03/30] compat: embrace LINUX_BACKPORT() for v2.6.24 Luis R. Rodriguez
2013-03-20 9:22 ` Luis R. Rodriguez [this message]
2013-03-20 9:22 ` [RFC 05/30] compat: embrace LINUX_BACKPORT() for v2.6.26 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 06/30] compat: embrace LINUX_BACKPORT() for v2.6.27 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 07/30] compat: embrace LINUX_BACKPORT() for v2.6.28 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 08/30] compat: embrace LINUX_BACKPORT() for v2.6.29 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 09/30] compat: embrace LINUX_BACKPORT() for v2.6.31 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 10/30] compat: embrace LINUX_BACKPORT() for v2.6.32 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 11/30] compat: embrace LINUX_BACKPORT() for v2.6.33 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 12/30] compat: embrace LINUX_BACKPORT() for v2.6.34 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 13/30] compat: embrace LINUX_BACKPORT() for v2.6.35 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 14/30] compat: embrace LINUX_BACKPORT() for v2.6.36 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 15/30] compat: embrace LINUX_BACKPORT() for v2.6.37 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 16/30] compat: embrace LINUX_BACKPORT() for v2.6.38 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 17/30] compat: embrace LINUX_BACKPORT() for v2.6.39 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 18/30] compat: embrace LINUX_BACKPORT() for v3.0 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 19/30] compat: embrace LINUX_BACKPORT() for v3.1 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 20/30] compat: embrace LINUX_BACKPORT() for v3.2 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 21/30] compat: embrace LINUX_BACKPORT() for v3.3 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 22/30] compat: embrace LINUX_BACKPORT() for v3.4 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 23/30] compat: embrace LINUX_BACKPORT() for v3.7 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 24/30] compat: embrace LINUX_BACKPORT() for v3.8 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 25/30] compat: embrace LINUX_BACKPORT() for cordic Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 26/30] compat: embrace LINUX_BACKPORT() for crc8 Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 27/30] compat: embrace LINUX_BACKPORT() for kfifo Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 28/30] compat: embrace LINUX_BACKPORT() for pm_qos_params Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 29/30] compat: embrace LINUX_BACKPORT() for uidgid Luis R. Rodriguez
2013-03-20 9:22 ` [RFC 30/30] compat: use backport_ prefix for main compat module calls Luis R. Rodriguez
2013-03-21 1:51 ` [RFC 00/30] compat: deal with kernel backport namespace Luis R. Rodriguez
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1363771375-31715-5-git-send-email-mcgrof@do-not-panic.com \
--to=mcgrof@do-not-panic.com \
--cc=andi@firstfloor.org \
--cc=backports@vger.kernel.org \
--cc=ben@decadent.org.uk \
--cc=linux-kernel@vger.kernel.org \
--cc=noelamac@gmail.com \
--cc=rusty@rustcorp.com.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome