From: Fengguang Wu <fengguang.wu@intel.com>
To: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Christopher Li <sparse@chrisli.org>, LKML <linux-kernel@vger.kernel.org>
Subject: Re: [dynticks:3.7-nohz1 5/25] arch/x86/kernel/apm_32.c:910:38: sparse: incorrect type in argument 3 (different modifiers)
Date: Sun, 23 Dec 2012 10:58:20 +0800 [thread overview]
Message-ID: <20121223025820.GC14002@localhost> (raw)
In-Reply-To: <CAFTL4hxPW829tM=dB6T5USpkRyuHAvE3KiEj2csCR=RZ3Ab90w@mail.gmail.com>
On Sat, Dec 22, 2012 at 07:11:49PM +0100, Frederic Weisbecker wrote:
> 2012/12/21 kbuild test robot <fengguang.wu@intel.com>:
> > tree: git://git.kernel.org/pub/scm/linux/kernel/git/frederic/linux-dynticks.git 3.7-nohz1
> > head: 46209cdbb7d5346c29b2243c54b298a233684a5a
> > commit: 40d488f9fcd6840722402976f7a168e05819f526 [5/25] cputime: Use accessors to read task cputime stats
> >
> >
> > sparse warnings:
> >
> > + arch/x86/kernel/apm_32.c:910:38: sparse: incorrect type in argument 3 (different modifiers)
> > arch/x86/kernel/apm_32.c:910:38: expected unsigned long [nocast] [usertype] *stime
> > arch/x86/kernel/apm_32.c:910:38: got unsigned long *<noident>
>
> Ok I must say I don't really understand this report. Do you have any
> hint on what it means?
It's possibly due to this "__nocast" modifier in the type definition:
typedef unsigned long __nocast cputime_t;
Not sure how to fix it though.
> 40d488f9 arch/x86/kernel/apm_32.c Frederic Weisbecker 2012-11-13 902 cputime_t stime;
> ^1da177e arch/i386/kernel/apm.c Linus Torvalds 2005-04-16 903
> ^1da177e arch/i386/kernel/apm.c Linus Torvalds 2005-04-16 904 int apm_idle_done = 0;
> ^1da177e arch/i386/kernel/apm.c Linus Torvalds 2005-04-16 905 unsigned int jiffies_since_last_check = jiffies - last_jiffies;
> ^1da177e arch/i386/kernel/apm.c Linus Torvalds 2005-04-16 906 unsigned int bucket;
> ^1da177e arch/i386/kernel/apm.c Linus Torvalds 2005-04-16 907
> 99c63221 arch/x86/kernel/apm_32.c Len Brown 2011-04-01 908 WARN_ONCE(1, "deprecated apm_cpu_idle will be deleted in 2012");
> ^1da177e arch/i386/kernel/apm.c Linus Torvalds 2005-04-16 909 recalc:
> 40d488f9 arch/x86/kernel/apm_32.c Frederic Weisbecker 2012-11-13 @910 task_cputime(current, NULL, &stime);
Thanks,
Fengguang
parent reply other threads:[~2012-12-23 2:58 UTC|newest]
Thread overview: expand[flat|nested] mbox.gz Atom feed
[parent not found: <CAFTL4hxPW829tM=dB6T5USpkRyuHAvE3KiEj2csCR=RZ3Ab90w@mail.gmail.com>]
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=20121223025820.GC14002@localhost \
--to=fengguang.wu@intel.com \
--cc=fweisbec@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=sparse@chrisli.org \
/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 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.