"input_name", and a new key 'K_SWITCH_INPUT_DATA' will be
passed back to the built-in command which will perform the switch.
This initial version only enables it for 'perf report'.
v2: rebase to latest 'perf/core' branch (6e1d4dd) of acme's perf tree
Signed-o
This is for tui browser only. This patch will check the returned
key of tui hists browser, if it's K_SWITH_INPUT_DATA, then recreate
a session for the new selected data file.
V2: Move the setup_brower() before the "repeat" jump point.
Signed-off-by: Feng Tang
---
tools/perf/b
On Tue, Feb 26, 2013 at 04:02:03PM +1100, Michael Ellerman wrote:
> Commit ad0de09 "Enable the runtime switching of perf data file" broke
> the build with NO_NEWT=1:
>
> CC builtin-report.o
> builtin-report.c: In function '__cmd_report':
> builtin-report.c:479:15: error: 'K_SWITCH_INPUT_DATA'
On Tue, Mar 12, 2013 at 02:09:08PM -0700, John Stultz wrote:
> On 03/11/2013 08:56 PM, Feng Tang wrote:
> }
> >-/* re-base the last cycle value */
> >-tk->clock->cycle_last = tk->clock->read(tk->clock);
&
>From 8e2fe258906fc83869be10a1b4a40ef9acc4a2a6 Mon Sep 17 00:00:00 2001
From: Feng Tang
Date: Mon, 27 Aug 2012 14:01:33 +0800
Subject: [PATCH 1/3] perf tools: Fix a compiling error in trace-event-perl.c
for 32 bits machine
On my x86_32 mahcine, there is a compile error:
CC u
Delvare wrote:
> Hi guys,
>
> Sorry for joining the discussion a little late, I was on vacation.
>
> On Thu, 23 Aug 2012 16:28:52 +0800, Feng Tang wrote:
> > On Wed, 22 Aug 2012 11:17:51 +0200
> > Alexander Stein wrote:
> > > Am Mittwoch, 22. August 2012, 16:04
Hi Alexander,
On Thu, 30 Aug 2012 09:49:52 +0200
Alexander Stein wrote:
> >
> > Whenever you call i2c_register_board_info(), every I2C bus number
> > referenced in the I2C device list passed as a parameter is reserved for
> > static I2C bus numbers, dynamic I2C bus numbers will never overlap.
>
Hi Alexander,
On Thu, 30 Aug 2012 13:08:09 +0200
Alexander Stein wrote:
> On Thursday 30 August 2012 17:19:15, Feng Tang wrote:
> > > IMO the i2c_register_board_info only works in quite static setups.
> > > Especially with I2C-Busses attached to hotplugable PCI devices th
Since perf script no longer only handle the trace points, we can
add the symbol filter option so that scripts can handle specified
samples.
Signed-off-by: Feng Tang
---
tools/perf/builtin-script.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/tools/perf/builtin
So that other perf commands/browser has a way to dig out the available
scripts info in system, this is a preparation for the script browser.
Signed-off-by: Feng Tang
---
tools/perf/builtin-script.c | 46 +++
tools/perf/builtin.h|1 +
2 files
and not supported by this static browser. You can run
the event_analyzing_sample.py for example.
Signed-off-by: Feng Tang
---
tools/perf/Makefile |5 +
tools/perf/ui/browsers/scripts.c | 159 ++
tools/perf/ui/browsers/scripts.h |5 +
annotation browser
The patches are on top of current perf/core branch of your git tree.
Please help to review, thanks
Feng
---
Feng Tang (7):
perf symbols: Filter samples with unresolved symbol when "--symbols"
option is used
perf scripts: Add --symbols option
With the patch:
===
100.00% 48 swapper [kernel.kallsyms]
Signed-off-by: Feng Tang
---
tools/perf/util/event.c |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/tools/perf/util/event.c b/tools/perf/util/event.c
index 3a0f1a5..08910f0 100644
--- a/tools/perf/uti
So that event_analyzing_sample.py can be shown by "perf script -l"
Signed-off-by: Feng Tang
---
.../python/bin/event_analyzing_sample-record |8
.../python/bin/event_analyzing_sample-report |3 +++
2 files changed, 11 insertions(+), 0 deletions(-)
c
Integrate the script browser into annotation, users can press function
key 'r' to list all perf scripts and select one of them to run that
script, the output will be shown in a separate browser.
Signed-off-by: Feng Tang
---
tools/perf/ui/browsers/annotate.c |8
1 files
Integrate the script browser into "perf report" framework, users can
use function key 'r' or the drop down menu to list all perf scripts
and select one of them, just like they did for the annotation.
Signed-off-by: Feng Tang
---
tools/perf/ui/bro
Hi Namhyung,
Thanks for your kind and thorough reviews.
On Tue, 4 Sep 2012 10:57:35 +0900
Namhyung Kim wrote:
> On Mon, 3 Sep 2012 16:14:31 +0800, Feng Tang wrote:
> > Create a script browser, so that user can check all the available
> > scripts and run them inside the main
As 'perf top' has no data files to run scripts against. Also add
a is_report_browser() helper function to judge whether the running
browser is for 'perf report'.
Signed-off-by: Feng Tang
---
tools/perf/ui/browsers/hists.c | 12 ++--
1 file changed, 10 insert
"input_name", and a new key 'K_SWITCH_INPUT_DATA' will be
passed back to the built-in command which will perform the switch.
This initial version only enables it for 'perf report'.
Signed-off-by: Feng Tang
---
tools/perf/ui/browsers/hists.c | 113 +
This is for tui browser only. This patch will check the returned
key of tui hists browser, if it's K_SWITH_INPUT_DATA, then recreate
a session for the new selected data file.
Signed-off-by: Feng Tang
---
tools/perf/builtin-report.c | 18 --
1 file changed, 16 insertions(
Hi All,
[seems the email from my intel email address didn't hit the list,
resend with gmail, sorry for the noise]
When I debugged a suspend/resume bug, I found that tick_broadcast_mask is not
restored for a CPU after it is offline/onlined since kernel 3.4, while it's
fine for 3.3.
Further check
Hi Paul,
On Mon, 30 Jul 2012 06:39:13 -0700
"Paul E. McKenney" wrote:
> On Mon, Jul 30, 2012 at 03:15:59PM +0800, Feng Tang wrote:
> > Hi All,
> >
> > When I debugged a suspend/resume bug, I found that tick_broadcast_mask is
> > not restored for a CPU aft
In commit 99b725084
ACPI processor hotplug: Delay acpi_processor_start() call for
hotplugged cores
acpi_processor_hotplug(pr) was wrongly replaced by
acpi_processor_cst_has_changed,
this patch will just restore it back.
Cc: Thomas Renninger
Cc: Deepthi Dharwar
Signed-off-by: Feng
Hi Paul,
On Mon, 30 Jul 2012 10:42:18 -0700
"Paul E. McKenney" wrote:
> On Mon, Jul 30, 2012 at 10:08:47AM -0700, Paul E. McKenney wrote:
> > On Mon, Jul 30, 2012 at 11:07:47PM +0800, Feng Tang wrote:
> > > Hi Paul,
> > >
> > > On Mon, 30 Jul 2012
dcast_mask regression:
https://lkml.org/lkml/2012/7/30/169
[Resend to ACPI mail list and Cc stable kernel]
Signed-off-by: Feng Tang
Cc: Thomas Renninger
Cc: Deepthi Dharwar
Cc: # 3.3+
---
drivers/acpi/processor_driver.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git
Next step should be to remove those cases that sends the BROADCAST
ON/OFF/FORCE msg on one cpu for another CPU, if this patch is reviewed
to be ok.
Signed-off-by: Feng Tang
Cc: Suresh Siddha
Cc: Thomas Gleixner
---
kernel/time/tick-broadcast.c | 15 +--
kernel/time/tick-common.c
#x27;t cover 'record'
in commit log
Since v1:
* Add filter for scripts can't be run in script browser
* Fix some bugs about buffer handling and error check
Feng
--
Feng Tang (9):
perf hists: Move hists_init() from ut
Integrate the script browser into "perf report" framework, users can
use function key 'r' or the drop down menu to list all perf scripts
and select one of them, just like they did for the annotation.
Signed-off-by: Feng Tang
---
tools/perf/ui/bro
er of perf_evsel__hists_browser() equals NULL.
Signed-off-by: Feng Tang
---
tools/perf/ui/browsers/hists.c | 75 +++-
tools/perf/ui/keysyms.h|1 +
2 files changed, 75 insertions(+), 1 deletions(-)
diff --git a/tools/perf/ui/browsers/hists.c b/tool
for it, also it can some other benefits:
1. When calling script browser inside hists/annotation browser, it
needs to know the perf data file name to run that script.
2. For further feature like runtime switching to another perf data
file, this variable can also help.
Signed-off-by: Feng Tang
---
all the output
lines are put into one ui browser, pressing 'q' or left arrow key will
make it return to previous browser.
Signed-off-by: Feng Tang
---
tools/perf/Makefile |4 +
tools/perf/ui/browsers/scripts.c | 182 ++
tools/perf/uti
This is for tui browser only. This patch will check the returned
key of tui hists browser, if it's K_SWITH_INPUT_DATA, then recreate
a session for the new selected data file.
Signed-off-by: Feng Tang
---
tools/perf/builtin-report.c | 18 --
1 files changed, 16 insertions(
With this func, other modules can basically check whether a file
is a legal perf data file by checking its first 8 bytes aginst
all possible perf magic nunbers.
Signed-off-by: Feng Tang
---
tools/perf/util/header.c | 11 +++
tools/perf/util/header.h |1 +
2 files changed, 12
Integrate the script browser into annotation, users can press function
key 'r' to list all perf scripts and select one of them to run that
script, the output will be shown in a separate browser.
Signed-off-by: Feng Tang
---
tools/perf/ui/browsers/annotate.c |6 ++
1 files
-by: Feng Tang
---
tools/perf/builtin-script.c | 76 +--
1 files changed, 73 insertions(+), 3 deletions(-)
diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
index 4e2f10f..37a0df8 100644
--- a/tools/perf/builtin-script.c
+++ b/tools
Which looks more natural
Signed-off-by: Feng Tang
---
tools/perf/util/evsel.c | 10 --
tools/perf/util/evsel.h |2 --
tools/perf/util/hist.c | 10 ++
tools/perf/util/hist.h |1 +
4 files changed, 11 insertions(+), 12 deletions(-)
diff --git a/tools/perf/util
On Mon, 24 Sep 2012 13:02:39 -0300
Arnaldo Carvalho de Melo wrote:
> Em Mon, Sep 24, 2012 at 11:24:03PM +0800, Feng Tang escreveu:
> > Which looks more natural
>
> It is there to avoid dragging the hist code into the python binding :-\
Didn't notice this, please ignore thi
On Tue, 25 Sep 2012 11:11:21 +0900
Namhyung Kim wrote:
> On Mon, 24 Sep 2012 23:24:10 +0800, Feng Tang wrote:
> [snip]
> > + if (!check_perf_magic(magic)) {
> > + options[nr_options] = strdup(name);
> > + abs_path[nr_
On Tue, 25 Sep 2012 11:07:15 +0900
Namhyung Kim wrote:
> On Mon, 24 Sep 2012 23:24:09 +0800, Feng Tang wrote:
> [snip]
> > +/* Return 0 if matched */
> > +int check_perf_magic(u64 magic)
> > +{
> > + if (!memcmp(&magic, __perf_magic1, sizeof(magic))
> &g
On Tue, Sep 25, 2012 at 08:17:03AM -0300, Arnaldo Carvalho de Melo wrote:
> Em Tue, Sep 25, 2012 at 04:20:53PM +0800, Feng Tang escreveu:
> > On Tue, 25 Sep 2012 11:11:21 +0900
> > Namhyung Kim wrote:
> > > Ditto. Plus it might leak previous input_name.
> >
> &g
On Tue, 25 Sep 2012 10:47:03 +0900
Namhyung Kim wrote:
> Hi Feng,
>
> On Mon, 24 Sep 2012 23:24:05 +0800, Feng Tang wrote:
> > As suggested by Arnaldo, many scripts have their own usages and need
> > capture specific events or tracepoints, so only those scripts whose
>
can't be run in script browser
* Fix some bugs about buffer handling and error check
Thanks,
Feng
--
Feng Tang (8):
perf tool: Add a global variable "const char *input_name"
perf script: Add more filter to find_scripts()
perf ui/browser: Add a browser
With this function, other modules can basically check whether a
file is a legal perf data file by checking its first 8 bytes
against all possible perf magic numbers.
Change the function name from check_perf_magic to more meaningful
is_perf_magic as suggested by acme.
Signed-off-by: Feng Tang
This is for tui browser only. This patch will check the returned
key of tui hists browser, if it's K_SWITH_INPUT_DATA, then recreate
a session for the new selected data file.
Signed-off-by: Feng Tang
---
tools/perf/builtin-report.c | 18 --
1 file changed, 16 insertions(
"xxx-record"
script to cherry pick out all events name and comparing them with
those inside the perf data file.
Signed-off-by: Feng Tang
---
tools/perf/builtin-script.c | 82 +--
1 file changed, 79 insertions(+), 3 deletions(-)
diff --git a/
for it, also it can some other benefits:
1. When calling script browser inside hists/annotation browser, it
needs to know the perf data file name to run that script.
2. For further feature like runtime switching to another perf data
file, this variable can also help.
Signed-off-by: Feng Tang
---
"input_name", and a new key 'K_SWITCH_INPUT_DATA' will be
passed back to the built-in command which will perform the switch.
This initial version only enables it for 'perf report', by checking
the "timer" parameter of perf_evsel__hists_browser() equals NULL.
Sign
Integrate the script browser into "perf report" framework, users can
use function key 'r' or the drop down menu to list all perf scripts
and select one of them, just like they did for the annotation.
Signed-off-by: Feng Tang
---
tools/perf/ui/bro
Integrate the script browser into annotation, users can press function
key 'r' to list all perf scripts and select one of them to run that
script, the output will be shown in a separate browser.
Signed-off-by: Feng Tang
---
tools/perf/ui/browsers/annotate.c |6 ++
1 file
all the output
lines are put into one ui browser, pressing 'q' or left arrow key will
make it return to previous browser.
Signed-off-by: Feng Tang
---
tools/perf/Makefile |4 +
tools/perf/ui/browsers/scripts.c | 189 ++
tools/perf/uti
Hi Arnaldo,
Thanks for the review.
On Mon, Oct 29, 2012 at 12:06:20PM -0200, Arnaldo Carvalho de Melo wrote:
> Em Tue, Oct 30, 2012 at 11:56:08AM +0800, Feng Tang escreveu:
> > + pwd_dir = opendir(pwd);
> > + if (!pwd_dir)
> > + return ret;
> > +
> &
On Mon, Oct 29, 2012 at 12:54:23PM -0200, Arnaldo Carvalho de Melo wrote:
> Em Tue, Oct 30, 2012 at 11:56:06AM +0800, Feng Tang escreveu:
> > Integrate the script browser into "perf report" framework, users can
> > use function key 'r' or the drop down menu to l
Since perf script no longer only handle the trace points, we can
add the symbol filter option so that scripts can handle specified
samples.
Signed-off-by: Feng Tang
---
tools/perf/builtin-script.c |3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/tools/perf/builtin
So that other perf commands/browser has a way to dig out the available
scripts info in system, this is a preparation for the script browser.
Signed-off-by: Feng Tang
---
tools/perf/builtin-script.c | 55 +++
tools/perf/builtin.h|1 +
2 files
chose one, which will be executed with
popen("perf script -s xxx.xx",) and all the output lines are put into
one ui browser, pressing 'q' or left arrow key will make it return
to previous browser.
Signed-off-by: Feng Tang
---
tools/perf/Makefile |4 +
to
Integrate the script browser into "perf report" framework, users can
use function key 'r' or the drop down menu to list all perf scripts
and select one of them, just like they did for the annotation.
Signed-off-by: Feng Tang
---
tools/perf/ui/bro
uot;, I'll handle it later, and would post the
v2 first for review.
Changelog:
Since v1:
* Add filter for scripts can't be run in script browser
* Fix some bugs about buffer handling and error check
Feng
---
Feng Tang (7):
perf symbols:
Integrate the script browser into annotation, users can press function
key 'r' to list all perf scripts and select one of them to run that
script, the output will be shown in a separate browser.
Signed-off-by: Feng Tang
---
tools/perf/ui/browsers/annotate.c |6 ++
1 files
So that event_analyzing_sample.py can be shown by "perf script -l"
Signed-off-by: Feng Tang
---
.../python/bin/event_analyzing_sample-record |8
.../python/bin/event_analyzing_sample-report |3 +++
2 files changed, 11 insertions(+), 0 deletions(-)
c
With the patch:
===
100.00% 48 swapper [kernel.kallsyms]
Signed-off-by: Feng Tang
---
tools/perf/util/event.c |5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/tools/perf/util/event.c b/tools/perf/util/event.c
index 3a0f1a5..08910f0 100644
--- a/tools/perf/uti
0:00:01.0
> > > pcilib: Cannot open /sys/bus/pci/devices/:00:00.0/config
> > > lspci: Unable to read the standard configuration space header of
> > > device :00:00.0
> > > -[:00]-
> > >
> > > bisected to this commit
> > >
>
Hi Jonathan,
On Tue, 9 Oct 2012 14:16:33 +0800
Jonathan Nieder wrote:
> Len Brown wrote:
>
> > By enlarging the GPE storm threshold back to 20, that laptop's
> > EC works fine with interrupt mode instead of polling mode.
>
> What would go wrong if the threshold were just increased to 20 on all
On Mon, 8 Oct 2012 23:59:07 -0700
Jonathan Nieder wrote:
> Feng Tang wrote:
> > Jonathan Nieder wrote:
>
> >> What would go wrong if the threshold were just increased to 20 on all
> >> models?
> >
> > Then some other platform will stop to work.
> &
On Thu, Nov 08, 2012 at 05:49:40AM +0800, Rafael J. Wysocki wrote:
> On Tuesday, November 06, 2012 01:48:26 PM Greg KH wrote:
> > On Tue, Nov 06, 2012 at 04:42:24PM +0400, Azat Khuzhin wrote:
> > > I'v also have such errors on my macbook pro.
> > >
> > > $ dmesg | tail
> > > [17056.008564] ACPI Er
On Fri, Nov 09, 2012 at 10:30:43PM +0800, Moore, Robert wrote:
> The ACPI Global Lock is in fact intended to provide exclusion between the
> BIOS and the OS.
> Bob
Thanks for the info.
And per my check, most of ACPI FW don't implement this lock, say
after driver probe, the ec->global_lock will b
On Mon, Dec 17, 2012 at 11:22:02AM -0700, Jason Gunthorpe wrote:
> On Tue, Dec 18, 2012 at 12:14:33AM +0800, Feng Tang wrote:
>
> > > Sure, but my view on this is that it has nothing to do with
> > > read_persistent_clock. If the RTC driver can run with IRQs off is a
&
Hi Jason,
On Thu, Dec 13, 2012 at 07:38:26PM -0700, Jason Gunthorpe wrote:
>
> > make the HCTOSYS option be dependent on !HAS_PERSISTENT_CLOCK. This
> > way we avoid having configs where there are conflicting paths that
> > we chose from.
>
> On ARM the read_presistent_clock is used to access a
From: Namhyung Kim
The perf session environment information was saved (so allocated)
during perf_session__open, but was not freed. As free(3) handles NULL
pointer input properly it won't cause a issue for writing modes -
e.g. perf record.
Signed-off-by: Namhyung Kim
---
tools/perf/util/sessio
This is for tui browser only. This patch will check the returned
key of tui hists browser, if it's K_SWITH_INPUT_DATA, then recreate
a session for the new selected data file.
Signed-off-by: Feng Tang
---
tools/perf/builtin-report.c | 22 +++---
1 file changed, 19 inser
"input_name", and a new key 'K_SWITCH_INPUT_DATA' will be
passed back to the built-in command which will perform the switch.
This initial version only enables it for 'perf report'.
Signed-off-by: Feng Tang
---
tools/perf/ui/browsers/hists.c | 113 +
Hi Arnaldo,
On Tue, Nov 20, 2012 at 12:16:44PM -0300, Arnaldo Carvalho de Melo wrote:
> Em Tue, Nov 20, 2012 at 05:31:16PM +0800, Feng Tang escreveu:
> > +static int switch_data_file(void)
> > +{
> > + char *pwd, *options[32], *abs_path[32], *tmp;
> > + DIR *pwd_
Sorry, just repost the old patch by mistake. should not do patch
work in midnight :), here is real updated one.
>From 0045aab3beeb39f119e44a4f5980cca71b599468 Mon Sep 17 00:00:00 2001
From: Feng Tang
Date: Tue, 20 Nov 2012 16:21:10 +0800
Subject: [PATCH 2/3] perf hists browser: Add option
= sqlite3.connect("/dev/shm/perf.db")
> -con.isolation_level = None
> +
>
> def trace_begin():
> print "In trace_begin:\n"
Thanks for the root causing the slowness of SQLite3 operation.
Acked-by: Feng Tang
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
s suggested by John to not hurt accuracy
* Modify the CPU feature flag name to be consistent with
other flags
* Solve the problem of judging S3/S4, as the clocksource
counter will be reset after coming out S4.
- Feng
Feng Tang (4):
x8
Signed-off-by: Feng Tang
---
arch/x86/kernel/tsc.c |6 +-
1 files changed, 5 insertions(+), 1 deletions(-)
diff --git a/arch/x86/kernel/tsc.c b/arch/x86/kernel/tsc.c
index 4b9ea10..fb581c6 100644
--- a/arch/x86/kernel/tsc.c
+++ b/arch/x86/kernel/tsc.c
@@ -768,7 +768,8 @@ static cycle_t
Some x86 processors have a TSC clocksource, which continue to work when
system is suspend. Add a feature flag so that it could be utilized.
Signed-off-by: Feng Tang
---
include/linux/clocksource.h |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/include/linux
ill change the trying order to:
suspend-nonstop clocksource -> persistent clock -> rtc
Signed-off-by: Feng Tang
---
kernel/time/timekeeping.c | 57 ++--
1 files changed, 49 insertions(+), 8 deletions(-)
diff --git a/kernel/time/timekeeping
cycle, so add a flag for it.
Signed-off-by: Feng Tang
---
arch/x86/include/asm/cpufeature.h |1 +
arch/x86/kernel/cpu/intel.c | 12
2 files changed, 13 insertions(+), 0 deletions(-)
diff --git a/arch/x86/include/asm/cpufeature.h
b/arch/x86/include/asm/cpufeature.h
ind
Hi Jason,
Thanks for the review and suggestions!
On Mon, Mar 04, 2013 at 08:20:36PM -0700, Jason Gunthorpe wrote:
> On Tue, Mar 05, 2013 at 02:34:25AM +, Tang, Feng wrote:
> > Hi Jason,
> >
> > Sorry, I forgot to add you in cc list in the first place. Please
> > help to review the patch seri
On Mon, Mar 04, 2013 at 09:32:03PM -0700, Jason Gunthorpe wrote:
> On Tue, Mar 05, 2013 at 11:53:02AM +0800, Feng Tang wrote:
> > > You may want to also CC the maintainers of all the ARM subsystems that
> > > use read_persistent_clock and check with them to ensure this new
On Tue, Mar 05, 2013 at 02:27:34PM +0800, John Stultz wrote:
> On 03/05/2013 10:27 AM, Feng Tang wrote:
> >There are some new processors whose TSC clocksource won't stop during
> >suspend. Currently, after system resumes, kernel will use persistent
> >clock or RTC to comp
On Tue, Mar 05, 2013 at 02:47:27PM +0800, John Stultz wrote:
> On 03/05/2013 02:38 PM, Feng Tang wrote:
> >On Tue, Mar 05, 2013 at 02:27:34PM +0800, John Stultz wrote:
> >
> >>
> >>So this might be ok for an initial implementation, as on the
> >>
On Tue, Mar 05, 2013 at 02:17:59PM +0800, John Stultz wrote:
> On 03/05/2013 12:32 PM, Jason Gunthorpe wrote:
> >On Tue, Mar 05, 2013 at 11:53:02AM +0800, Feng Tang wrote:
> >
> >>>// Drops some small precision along the way but is simple..
> >>>static i
Signed-off-by: Feng Tang
---
include/linux/clocksource.h | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/include/linux/clocksource.h b/include/linux/clocksource.h
index aa7032c..1ecc872 100644
--- a/include/linux/clocksource.h
+++ b/include/linux/clocksource.h
ill change the trying order to:
suspend-nonstop clocksource -> persistent clock -> rtc
Signed-off-by: Feng Tang
---
kernel/time/timekeeping.c | 42 ++
1 file changed, 34 insertions(+), 8 deletions(-)
diff --git a/kernel/time/timekeeping
with
other flags
* Solve the problem of judging S3/S4, as the clocksource
counter will be reset after coming out S4.
- Feng
-
Feng Tang (5):
x86: Add cpu capability flag X86_FEATURE_NONSTOP_TSC_S3
clocksource: Add new feature flag CLOCK_SOURCE_SUSPEND_
cycle, so add a flag for it.
Signed-off-by: Feng Tang
---
arch/x86/include/asm/cpufeature.h |1 +
arch/x86/kernel/cpu/intel.c | 12
2 files changed, 13 insertions(+)
diff --git a/arch/x86/include/asm/cpufeature.h
b/arch/x86/include/asm/cpufeature.h
index 93fe929..a846
Signed-off-by: Feng Tang
---
arch/x86/kernel/tsc.c |6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/arch/x86/kernel/tsc.c b/arch/x86/kernel/tsc.c
index 4b9ea10..098b3cf 100644
--- a/arch/x86/kernel/tsc.c
+++ b/arch/x86/kernel/tsc.c
@@ -768,7 +768,8 @@ static cycle_t
Some x86 processors have a TSC clocksource, which continues to run
even when system is suspended. Also most OMAP platforms have a
32 KHz timer which has similar capability. Add a feature flag so that
it could be utilized.
Signed-off-by: Feng Tang
---
include/linux/clocksource.h |1 +
1 file
On Wed, Mar 06, 2013 at 03:15:49PM +0100, Thomas Gleixner wrote:
> On Wed, 6 Mar 2013, Feng Tang wrote:
>
> > There are some new processors whose TSC clocksource won't stop during
> > suspend. Currently, after system resumes, kernel will use persistent
> > clock o
Hi Thomas,
Thanks for the reviews.
On Wed, Mar 06, 2013 at 03:09:26PM +0100, Thomas Gleixner wrote:
> On Wed, 6 Mar 2013, Feng Tang wrote:
>
> > Current clocksource_cyc2ns() has a implicit limit that the (cycles * mult)
> > can not exceed 64 bits limit. Jason Gunthorpe
Bcc:
Subject: Re: [PATCH 2/3] timekeeping: sync persistent clock and RTC on system
time step changes
Reply-To:
Hi David,
> From: David Vrabel
> Date: 2013/5/14
> Subject: Re: [PATCH 2/3] timekeeping: sync persistent clock and RTC on
> system time step changes
> To: John Stultz
> Cc: "xen-de.
Hi Pavel,
On Tue, May 07, 2013 at 09:10:42AM +0800, Feng Tang wrote:
> -- Forwarded message --
> From: Pavel Machek
> Date: 2013/5/7
> Subject: Re: [GIT PULL] timer changes for v3.10
> To: Ingo Molnar
> Cc: Linus Torvalds ,
> linux-kernel@vger.kernel.org, Th
Hi Ingo,
On Tue, May 07, 2013 at 08:53:48AM +0200, Ingo Molnar wrote:
>
> * Feng Tang wrote:
>
> > > is even worse than that. Machine can stay is s2ram for weeks (for a
> > > lot more if it is desktop and you do s2ram for powersaving). Also
> > > tem
Hi Pavel,
On Wed, May 08, 2013 at 12:55:42PM +0200, Pavel Machek wrote:
> Hi!
>
> > > > Sorry. You seem to not like the merged change, but I guess I'm not
> > > > quite sure what exactly your objection is here.
> > >
> > > I'm not exactly sure what my objections are.
> > >
> > > TSC was not des
gt; being optimized is likely to be reworked in the future, so
> lets revert this change.
>
> I suspect this will miss 3.9-final, so marking it for 3.9-stable.
thanks
Acked-by: Feng Tang
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body
On Wed, Mar 06, 2013 at 05:10:53PM +0100, Thomas Gleixner wrote:
> On Wed, 6 Mar 2013, Feng Tang wrote:
> > Hi Thomas,
> >
> > Thanks for the reviews.
> >
> > On Wed, Mar 06, 2013 at 03:09:26PM +0100, Thomas Gleixner wrote:
> > > On Wed, 6 Mar 201
of judging S3/S4, as the clocksource
counter will be reset after coming out S4.
- Feng
-
Feng Tang (4):
x86: Add cpu capability flag X86_FEATURE_NONSTOP_TSC_S3
clocksource: Add new feature flag CLOCK_SOURCE_SUSPEND_NONSTOP
x86: tsc: Add support for new S3_NONSTOP featur
Signed-off-by: Feng Tang
---
arch/x86/kernel/tsc.c |6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/arch/x86/kernel/tsc.c b/arch/x86/kernel/tsc.c
index 4b9ea10..098b3cf 100644
--- a/arch/x86/kernel/tsc.c
+++ b/arch/x86/kernel/tsc.c
@@ -768,7 +768,8 @@ static cycle_t
Some x86 processors have a TSC clocksource, which continues to run
even when system is suspended. Also most OMAP platforms have a
32 KHz timer which has similar capability. Add a feature flag so that
it could be utilized.
Signed-off-by: Feng Tang
---
include/linux/clocksource.h |1 +
1 file
1 - 100 of 563 matches
Mail list logo
