Re: [PATCH] lpc_ich: Fix a 3.5 kernel regression for iTCO_wdt driver

2012-08-22 Thread Feng Tang
On Wed, 22 Aug 2012 22:55:43 +0100 Matthew Garrett wrote: > On Wed, Aug 22, 2012 at 09:55:12PM +0200, Wim Van Sebroeck wrote: > > > Any idea why the acpi_check_resource_conflict() check gives a conflict? > > Because the resource range is declared in ACPI and we assume that that > means the

Re: i2c-eg20t: regression since i2c_add_numbered_adapter change

2012-08-22 Thread Feng Tang
Hi Alexander, On Wed, 22 Aug 2012 09:57:07 +0200 Alexander Stein wrote: > Hello, > > On Wednesday 22 August 2012 15:29:18, Feng Tang wrote: > > On Wed, 22 Aug 2012 08:30:35 +0200 > > Alexander Stein wrote: > > > > > Hello, > > > > > > I

Re: pci_get_subsys: GFP_KERNEL allocations with IRQs disabled

2012-08-22 Thread Feng Tang
> > if (DEBUG_LOCKS_WARN_ON(irqs_disabled_flags(flags))) > > return; > > It's bisected down to this commit: > > commit 55c844a4dd16a4d1fdc0cf2a283ec631a02ec448 > Author: Feng Tang > AuthorDate: Wed May 30 23:15:41 2012 +0800 > Commit:

Re: i2c-eg20t: regression since i2c_add_numbered_adapter change

2012-08-22 Thread Feng Tang
On Wed, 22 Aug 2012 08:30:35 +0200 Alexander Stein wrote: > Hello, > > I just noticed the 3.4 linux kernel fails to sucessfully probe the i2c-eg20t > driver. I returns with EBUSY error. It worked on the 3.0 kernel. To my view > it > is caused the commit

Re: i2c-eg20t: regression since i2c_add_numbered_adapter change

2012-08-22 Thread Feng Tang
On Wed, 22 Aug 2012 08:30:35 +0200 Alexander Stein alexander.st...@systec-electronic.com wrote: Hello, I just noticed the 3.4 linux kernel fails to sucessfully probe the i2c-eg20t driver. I returns with EBUSY error. It worked on the 3.0 kernel. To my view it is caused the commit

Re: pci_get_subsys: GFP_KERNEL allocations with IRQs disabled

2012-08-22 Thread Feng Tang
(irqs_disabled_flags(flags))) return; It's bisected down to this commit: commit 55c844a4dd16a4d1fdc0cf2a283ec631a02ec448 Author: Feng Tang feng.t...@intel.com AuthorDate: Wed May 30 23:15:41 2012 +0800 Commit: Ingo Molnar mi...@kernel.org CommitDate: Wed Jun 6 12:03

Re: i2c-eg20t: regression since i2c_add_numbered_adapter change

2012-08-22 Thread Feng Tang
Hi Alexander, On Wed, 22 Aug 2012 09:57:07 +0200 Alexander Stein alexander.st...@systec-electronic.com wrote: Hello, On Wednesday 22 August 2012 15:29:18, Feng Tang wrote: On Wed, 22 Aug 2012 08:30:35 +0200 Alexander Stein alexander.st...@systec-electronic.com wrote: Hello

Re: [PATCH] lpc_ich: Fix a 3.5 kernel regression for iTCO_wdt driver

2012-08-22 Thread Feng Tang
On Wed, 22 Aug 2012 22:55:43 +0100 Matthew Garrett mj...@srcf.ucam.org wrote: On Wed, Aug 22, 2012 at 09:55:12PM +0200, Wim Van Sebroeck wrote: Any idea why the acpi_check_resource_conflict() check gives a conflict? Because the resource range is declared in ACPI and we assume that that

Re: pci_get_subsys: GFP_KERNEL allocations with IRQs disabled

2012-08-22 Thread Feng Tang
Hi Bjorn, On Wed, 22 Aug 2012 11:02:52 -0700 Bjorn Helgaas bhelg...@google.com wrote: On Wed, Aug 22, 2012 at 12:49 AM, Feng Tang feng.t...@intel.com wrote: Hi Fengguang, On Wed, 22 Aug 2012 10:50:08 +0800 Fengguang Wu fengguang...@intel.com wrote: Feng, I think it's

[tip:perf/core] perf script python: Correct handler check and spelling errors

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: 87b6a3ad40ba304ec468b972e979e7e410852476 Gitweb: http://git.kernel.org/tip/87b6a3ad40ba304ec468b972e979e7e410852476 Author: Feng Tang AuthorDate: Thu, 9 Aug 2012 13:46:13 +0800 Committer: Arnaldo Carvalho de Melo CommitDate: Thu, 9 Aug 2012 13:26:19 -0300 perf script python

[tip:perf/core] perf scripts python: Add event_analyzing_sample. py as a sample for general event handling

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: 0076d546b4f9b5c15121c6959d108a83fe43fa9a Gitweb: http://git.kernel.org/tip/0076d546b4f9b5c15121c6959d108a83fe43fa9a Author: Feng Tang AuthorDate: Wed, 8 Aug 2012 17:57:55 +0800 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 8 Aug 2012 12:55:38 -0300 perf scripts

[tip:perf/core] perf scripts python: Add a python library EventClass.py

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: 02f1c33f7d630183518ea42d45a6acf275541b08 Gitweb: http://git.kernel.org/tip/02f1c33f7d630183518ea42d45a6acf275541b08 Author: Feng Tang AuthorDate: Wed, 8 Aug 2012 17:57:54 +0800 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 8 Aug 2012 12:53:08 -0300 perf scripts

[tip:perf/core] perf scripts python: Pass event/thread/ dso name and symbol info to event handler in python

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: fd6b858a1e110c76e701cd9972a284ed2a7bcc33 Gitweb: http://git.kernel.org/tip/fd6b858a1e110c76e701cd9972a284ed2a7bcc33 Author: Feng Tang AuthorDate: Wed, 8 Aug 2012 17:57:53 +0800 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 8 Aug 2012 12:48:02 -0300 perf scripts

[tip:perf/core] perf script: Replace "struct thread" with " struct addr_location" as a parameter for "process_event()"

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: 73994dc158a24df4af77d0a76c9702f120f7a6ad Gitweb: http://git.kernel.org/tip/73994dc158a24df4af77d0a76c9702f120f7a6ad Author: Feng Tang AuthorDate: Wed, 8 Aug 2012 17:57:52 +0800 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 8 Aug 2012 12:46:40 -0300 perf script

[tip:perf/core] perf script: Add general python handler to process non-tracepoint events

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: 6a6daec2ae9f097703c1da4925367f1c198c9492 Gitweb: http://git.kernel.org/tip/6a6daec2ae9f097703c1da4925367f1c198c9492 Author: Feng Tang AuthorDate: Wed, 8 Aug 2012 17:57:51 +0800 Committer: Arnaldo Carvalho de Melo CommitDate: Wed, 8 Aug 2012 12:45:33 -0300 perf script: Add

[tip:perf/core] perf script: Add general python handler to process non-tracepoint events

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: 6a6daec2ae9f097703c1da4925367f1c198c9492 Gitweb: http://git.kernel.org/tip/6a6daec2ae9f097703c1da4925367f1c198c9492 Author: Feng Tang feng.t...@intel.com AuthorDate: Wed, 8 Aug 2012 17:57:51 +0800 Committer: Arnaldo Carvalho de Melo a...@redhat.com CommitDate: Wed, 8 Aug 2012

[tip:perf/core] perf script: Replace struct thread with struct addr_location as a parameter for process_event()

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: 73994dc158a24df4af77d0a76c9702f120f7a6ad Gitweb: http://git.kernel.org/tip/73994dc158a24df4af77d0a76c9702f120f7a6ad Author: Feng Tang feng.t...@intel.com AuthorDate: Wed, 8 Aug 2012 17:57:52 +0800 Committer: Arnaldo Carvalho de Melo a...@redhat.com CommitDate: Wed, 8 Aug 2012

[tip:perf/core] perf scripts python: Pass event/thread/ dso name and symbol info to event handler in python

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: fd6b858a1e110c76e701cd9972a284ed2a7bcc33 Gitweb: http://git.kernel.org/tip/fd6b858a1e110c76e701cd9972a284ed2a7bcc33 Author: Feng Tang feng.t...@intel.com AuthorDate: Wed, 8 Aug 2012 17:57:53 +0800 Committer: Arnaldo Carvalho de Melo a...@redhat.com CommitDate: Wed, 8 Aug 2012

[tip:perf/core] perf scripts python: Add a python library EventClass.py

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: 02f1c33f7d630183518ea42d45a6acf275541b08 Gitweb: http://git.kernel.org/tip/02f1c33f7d630183518ea42d45a6acf275541b08 Author: Feng Tang feng.t...@intel.com AuthorDate: Wed, 8 Aug 2012 17:57:54 +0800 Committer: Arnaldo Carvalho de Melo a...@redhat.com CommitDate: Wed, 8 Aug 2012

[tip:perf/core] perf scripts python: Add event_analyzing_sample. py as a sample for general event handling

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: 0076d546b4f9b5c15121c6959d108a83fe43fa9a Gitweb: http://git.kernel.org/tip/0076d546b4f9b5c15121c6959d108a83fe43fa9a Author: Feng Tang feng.t...@intel.com AuthorDate: Wed, 8 Aug 2012 17:57:55 +0800 Committer: Arnaldo Carvalho de Melo a...@redhat.com CommitDate: Wed, 8 Aug 2012

[tip:perf/core] perf script python: Correct handler check and spelling errors

2012-08-21 Thread tip-bot for Feng Tang
Commit-ID: 87b6a3ad40ba304ec468b972e979e7e410852476 Gitweb: http://git.kernel.org/tip/87b6a3ad40ba304ec468b972e979e7e410852476 Author: Feng Tang feng.t...@intel.com AuthorDate: Thu, 9 Aug 2012 13:46:13 +0800 Committer: Arnaldo Carvalho de Melo a...@redhat.com CommitDate: Thu, 9 Aug 2012

[PATCH] lpc_ich: Fix a 3.5 kernel regression for iTCO_wdt driver

2012-08-14 Thread Feng Tang
check could be removed for the gpio-ich in lpc_ich.c, but I'm not sure if it will cause problems. Signed-off-by: Feng Tang Cc: Aaron Sierra Cc: Wim Van Sebroeck Cc: Len Brown Cc: Bob Moore --- drivers/mfd/lpc_ich.c | 20 +--- 1 files changed, 1 insertions(+), 19 deletions

Re: [PATCH RFC] tick: Emphasize tck_broadcast_on_off could only be called for host cpu

2012-08-14 Thread Feng Tang
On Tue, 31 Jul 2012 15:57:21 +0800 Feng Tang wrote: > With current tick_do_broadcast_on_off() and tick_broadcast_on_off(), > it only cares host cpu, and doesn't really support to make the on/off > for another target CPU as it seems to be. So remove the unneeded > online check an

Re: [PATCH RFC] tick: Emphasize tck_broadcast_on_off could only be called for host cpu

2012-08-14 Thread Feng Tang
On Tue, 31 Jul 2012 15:57:21 +0800 Feng Tang feng.t...@intel.com wrote: With current tick_do_broadcast_on_off() and tick_broadcast_on_off(), it only cares host cpu, and doesn't really support to make the on/off for another target CPU as it seems to be. So remove the unneeded online check

[PATCH] lpc_ich: Fix a 3.5 kernel regression for iTCO_wdt driver

2012-08-14 Thread Feng Tang
check could be removed for the gpio-ich in lpc_ich.c, but I'm not sure if it will cause problems. Signed-off-by: Feng Tang feng.t...@intel.com Cc: Aaron Sierra asie...@xes-inc.com Cc: Wim Van Sebroeck w...@iguana.be Cc: Len Brown len.br...@intel.com Cc: Bob Moore robert.mo...@intel.com --- drivers

Re: [PATCH RFT 1/2] gpio: gpio-ml-ioh: Use spinlock for register access protection

2012-08-13 Thread Feng Tang
On Mon, 13 Aug 2012 14:01:31 +0200 Linus Walleij wrote: > On Sun, Jul 29, 2012 at 4:54 AM, Axel Lin wrote: > > > gpio_chip.can_sleep is 0, but current code uses mutex in ioh_gpio_set, > > ioh_gpio_get and ioh_gpio_direction_input functions. > > Thus those functions are not callable from

Re: [PATCH RFT 1/2] gpio: gpio-ml-ioh: Use spinlock for register access protection

2012-08-13 Thread Feng Tang
On Mon, 13 Aug 2012 14:01:31 +0200 Linus Walleij linus.wall...@linaro.org wrote: On Sun, Jul 29, 2012 at 4:54 AM, Axel Lin axel@gmail.com wrote: gpio_chip.can_sleep is 0, but current code uses mutex in ioh_gpio_set, ioh_gpio_get and ioh_gpio_direction_input functions. Thus those

[PATCH] wm831x-dcdc: fix coccinelle warnings of missing IRQF_ONESHOT

2012-08-09 Thread Feng Tang
Generated by: scripts/coccinelle/misc/irqf_oneshot.cocci Make sure threaded IRQs without a primary handler are always request with IRQF_ONESHOT Signed-off-by: Fengguang Wu Signed-off-by: Feng Tang --- drivers/regulator/wm831x-dcdc.c |8 1 files changed, 4 insertions(+), 4

[PATCH] wm831x-dcdc: fix coccinelle warnings of missing IRQF_ONESHOT

2012-08-09 Thread Feng Tang
without IRQF_ONESHOT Generated by: scripts/coccinelle/misc/irqf_oneshot.cocci Make sure threaded IRQs without a primary handler are always request with IRQF_ONESHOT Signed-off-by: Fengguang Wu fengguang...@intel.com Signed-off-by: Feng Tang feng.t...@intel.com --- drivers/regulator/wm831x

[PATCH v2] perf script python: Correct handler check and spelling errors

2012-08-08 Thread Feng Tang
>From efbf00591514aa1d6134a3b940dc627d1a7cc8f8 Mon Sep 17 00:00:00 2001 From: Feng Tang Date: Thu, 9 Aug 2012 12:50:56 +0800 Subject: [PATCH] perf script python: Correct handler check and spelling errors Correct the checking for handler returned by PyDict_GetItemString(), also fix some spell

Re: [PATCH v5 5/5] perf script: Add event_analyzing_sample.py as a sample for general event handling

2012-08-08 Thread Feng Tang
On Thu, 9 Aug 2012 14:19:29 +0900 Namhyung Kim wrote: > Hi, > > On Thu, 9 Aug 2012 12:30:07 +0800, Feng Tang wrote: > > Hi Namhyung, > > > > Many thanks for your valuable comments, I'll fold the fixes into an > > incremental patch. > > > > On Th

[PATCH] perf script python: Correct handler check and spelling errors

2012-08-08 Thread Feng Tang
>From 3332f03c6e8f641e26430a25704c364c2f30d833 Mon Sep 17 00:00:00 2001 From: Feng Tang Date: Thu, 9 Aug 2012 12:50:56 +0800 Subject: [PATCH] perf script python: Correct handler check and spelling errors Correct the checking for handler returned by PyDict_GetItemString(), also fix some spell

Re: [PATCH v5 5/5] perf script: Add event_analyzing_sample.py as a sample for general event handling

2012-08-08 Thread Feng Tang
Hi Namhyung, Many thanks for your valuable comments, I'll fold the fixes into an incremental patch. On Thu, 9 Aug 2012 10:30:34 +0900 Namhyung Kim wrote: > On Wed, 8 Aug 2012 17:57:55 +0800, Feng Tang wrote: > > Currently only trace point events are supported in perf/pyth

[PATCH v5 1/5] perf script: Add general python handler to process non-tracepoint events

2012-08-08 Thread Feng Tang
tion named "process_event" in the python script, and pass the event attribute, perf_sample, raw_data in format of raw string. And the python script can use "struct" module's unpack function to disasemble the needed info and process. Signed-off-by: Feng Tang Cc: Andi Kleen Cc: David

[PATCH v5 3/5] perf script/python: Pass event/thread/dso name and symbol info to event handler in python

2012-08-08 Thread Feng Tang
Also as suggested by Arnaldo, pack all these parameters to a dictionary, which is more expandable for adding new parameters while keeping the compatibility for old scripts. Signed-off-by: Feng Tang Cc: David Ahern Cc: Peter Zijlstra Cc: Robert Richter Cc: Andi Kleen Cc: Stephane Eranian

[PATCH v5 5/5] perf script: Add event_analyzing_sample.py as a sample for general event handling

2012-08-08 Thread Feng Tang
-2.12.1.so5 ### Signed-off-by: Feng Tang --- .../perf/scripts/python/event_analyzing_sample.py | 193 1 files changed, 193 insertions(+), 0 deletions(-) create mode 100644 tools/perf/scripts/python/event_analyzing_sample.py diff --git a/tools/perf/scripts

[PATCH v5 2/5] perf script: Replace "struct thread" with "struct addr_location" as a parameter for "process_event()"

2012-08-08 Thread Feng Tang
the thread info, while providing additional symbol and dso info if exist, so that the script itself doesn't need to bother to do the symbol resolving and dso searching work. Signed-off-by: Feng Tang Acked-by: David Ahern Tested-by: David Ahern Cc: David Ahern Cc: Peter Zijlstra Cc: Robert Richter

[PATCH v5 4/5] perf script: Add a python library EventClass.py

2012-08-08 Thread Feng Tang
on requriements. Signed-off-by: Feng Tang --- .../Perf-Trace-Util/lib/Perf/Trace/EventClass.py | 95 1 files changed, 95 insertions(+), 0 deletions(-) create mode 100755 tools/perf/scripts/python/Perf-Trace-Util/lib/Perf/Trace/EventClass.py diff --git a/tools/perf

[PATCH v5 0/5] perf script: Add general event support to event handler of python script

2012-08-08 Thread Feng Tang
or whether the dso info exist Since v2: * Correct some __unused usage * Add a new parameter: event name to event handler Since v1: * Use dictory as the paramter passwd from c to python * Use "process_event" name to comply with

[PATCH v5 0/5] perf script: Add general event support to event handler of python script

2012-08-08 Thread Feng Tang
the dso info exist Since v2: * Correct some __unused usage * Add a new parameter: event name to event handler Since v1: * Use dictory as the paramter passwd from c to python * Use process_event name to comply with perl Feng Tang (5): perf

[PATCH v5 4/5] perf script: Add a python library EventClass.py

2012-08-08 Thread Feng Tang
on requriements. Signed-off-by: Feng Tang feng.t...@intel.com --- .../Perf-Trace-Util/lib/Perf/Trace/EventClass.py | 95 1 files changed, 95 insertions(+), 0 deletions(-) create mode 100755 tools/perf/scripts/python/Perf-Trace-Util/lib/Perf/Trace/EventClass.py diff --git

[PATCH v5 2/5] perf script: Replace struct thread with struct addr_location as a parameter for process_event()

2012-08-08 Thread Feng Tang
the thread info, while providing additional symbol and dso info if exist, so that the script itself doesn't need to bother to do the symbol resolving and dso searching work. Signed-off-by: Feng Tang feng.t...@intel.com Acked-by: David Ahern dsah...@gmail.com Tested-by: David Ahern dsah...@gmail.com Cc

[PATCH v5 5/5] perf script: Add event_analyzing_sample.py as a sample for general event handling

2012-08-08 Thread Feng Tang
-2.12.1.so5 ### Signed-off-by: Feng Tang feng.t...@intel.com --- .../perf/scripts/python/event_analyzing_sample.py | 193 1 files changed, 193 insertions(+), 0 deletions(-) create mode 100644 tools/perf/scripts/python/event_analyzing_sample.py diff --git

[PATCH v5 3/5] perf script/python: Pass event/thread/dso name and symbol info to event handler in python

2012-08-08 Thread Feng Tang
Also as suggested by Arnaldo, pack all these parameters to a dictionary, which is more expandable for adding new parameters while keeping the compatibility for old scripts. Signed-off-by: Feng Tang feng.t...@intel.com Cc: David Ahern dsah...@gmail.com Cc: Peter Zijlstra pet...@infradead.org Cc

[PATCH v5 1/5] perf script: Add general python handler to process non-tracepoint events

2012-08-08 Thread Feng Tang
in the python script, and pass the event attribute, perf_sample, raw_data in format of raw string. And the python script can use struct module's unpack function to disasemble the needed info and process. Signed-off-by: Feng Tang feng.t...@intel.com Cc: Andi Kleen a...@firstfloor.org Cc: David

Re: [PATCH v5 5/5] perf script: Add event_analyzing_sample.py as a sample for general event handling

2012-08-08 Thread Feng Tang
Hi Namhyung, Many thanks for your valuable comments, I'll fold the fixes into an incremental patch. On Thu, 9 Aug 2012 10:30:34 +0900 Namhyung Kim namhy...@kernel.org wrote: On Wed, 8 Aug 2012 17:57:55 +0800, Feng Tang wrote: Currently only trace point events are supported in perf/python

[PATCH] perf script python: Correct handler check and spelling errors

2012-08-08 Thread Feng Tang
From 3332f03c6e8f641e26430a25704c364c2f30d833 Mon Sep 17 00:00:00 2001 From: Feng Tang feng.t...@intel.com Date: Thu, 9 Aug 2012 12:50:56 +0800 Subject: [PATCH] perf script python: Correct handler check and spelling errors Correct the checking for handler returned by PyDict_GetItemString(), also

Re: [PATCH v5 5/5] perf script: Add event_analyzing_sample.py as a sample for general event handling

2012-08-08 Thread Feng Tang
On Thu, 9 Aug 2012 14:19:29 +0900 Namhyung Kim namhy...@kernel.org wrote: Hi, On Thu, 9 Aug 2012 12:30:07 +0800, Feng Tang wrote: Hi Namhyung, Many thanks for your valuable comments, I'll fold the fixes into an incremental patch. On Thu, 9 Aug 2012 10:30:34 +0900 Namhyung Kim

[PATCH v2] perf script python: Correct handler check and spelling errors

2012-08-08 Thread Feng Tang
From efbf00591514aa1d6134a3b940dc627d1a7cc8f8 Mon Sep 17 00:00:00 2001 From: Feng Tang feng.t...@intel.com Date: Thu, 9 Aug 2012 12:50:56 +0800 Subject: [PATCH] perf script python: Correct handler check and spelling errors Correct the checking for handler returned by PyDict_GetItemString(), also

Re: [PATCH v2] ACPI processor: Fix a typo error in commit 99b725084

2012-08-03 Thread Feng Tang
On Fri, 3 Aug 2012 11:26:04 +0530 "Srivatsa S. Bhat" wrote: > On 08/03/2012 09:52 AM, Len Brown wrote: > > Applied. > > > > thanks for the refresh. > > > > This patch needs to be CC'ed to stable also right? Yes, I've sent the patch directly to stable 3 days ago. thanks for the note. - Feng

Re: [PATCH v2] ACPI processor: Fix a typo error in commit 99b725084

2012-08-03 Thread Feng Tang
On Fri, 3 Aug 2012 11:26:04 +0530 Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com wrote: On 08/03/2012 09:52 AM, Len Brown wrote: Applied. thanks for the refresh. This patch needs to be CC'ed to stable also right? Yes, I've sent the patch directly to stable 3 days ago. thanks for

[PATCH v3 1/2] ACPI: Replace acpi_get_table_with_size() with acpi_get_table()

2012-08-01 Thread Feng Tang
of the acpi_get_table_with_size(). v3: Remove a not-needed cast found by Don Dutile. Signed-off-by: Feng Tang --- arch/x86/kernel/apic/es7000_32.c |7 +++ drivers/acpi/tables.c| 21 + drivers/iommu/amd_iommu_init.c | 10 -- drivers/iommu/dmar.c

Re: [PATCH v2 1/2] ACPI: Replace acpi_get_table_with_size() with acpi_get_table()

2012-08-01 Thread Feng Tang
On Wed, 1 Aug 2012 16:59:39 -0400 Don Dutile wrote: > On 08/01/2012 02:37 AM, Feng Tang wrote: > > This is a preparation for removing the acpi_get_table_with_size(), as this > > function could be well covered by acpi_get_table(), and there is no need > > to have both of them

Re: linux-next: build failure after merge of the acpi tree

2012-08-01 Thread Feng Tang
On Thu, 2 Aug 2012 11:20:33 +1000 Stephen Rothwell wrote: > Hi Len, > > After merging the acpi tree, today's linux-next build (x86_64 > allmodconfig) failed like this: > > drivers/iommu/amd_iommu_init.c: In function 'early_amd_iommu_init': > drivers/iommu/amd_iommu_init.c:1535:2: error:

[PATCH v2 1/2] ACPI: Replace acpi_get_table_with_size() with acpi_get_table()

2012-08-01 Thread Feng Tang
of the acpi_get_table_with_size(). Signed-off-by: Feng Tang --- arch/x86/kernel/apic/es7000_32.c |7 +++ drivers/acpi/tables.c| 21 + drivers/iommu/amd_iommu_init.c | 10 -- drivers/iommu/dmar.c | 14 +++--- 4 files changed, 23

[PATCH v2 1/2] ACPI: Replace acpi_get_table_with_size() with acpi_get_table()

2012-08-01 Thread Feng Tang
of the acpi_get_table_with_size(). Signed-off-by: Feng Tang feng.t...@intel.com --- arch/x86/kernel/apic/es7000_32.c |7 +++ drivers/acpi/tables.c| 21 + drivers/iommu/amd_iommu_init.c | 10 -- drivers/iommu/dmar.c | 14 +++--- 4 files

Re: linux-next: build failure after merge of the acpi tree

2012-08-01 Thread Feng Tang
On Thu, 2 Aug 2012 11:20:33 +1000 Stephen Rothwell s...@canb.auug.org.au wrote: Hi Len, After merging the acpi tree, today's linux-next build (x86_64 allmodconfig) failed like this: drivers/iommu/amd_iommu_init.c: In function 'early_amd_iommu_init': drivers/iommu/amd_iommu_init.c:1535:2:

Re: [PATCH v2 1/2] ACPI: Replace acpi_get_table_with_size() with acpi_get_table()

2012-08-01 Thread Feng Tang
On Wed, 1 Aug 2012 16:59:39 -0400 Don Dutile ddut...@redhat.com wrote: On 08/01/2012 02:37 AM, Feng Tang wrote: This is a preparation for removing the acpi_get_table_with_size(), as this function could be well covered by acpi_get_table(), and there is no need to have both of them to exist

[PATCH v3 1/2] ACPI: Replace acpi_get_table_with_size() with acpi_get_table()

2012-08-01 Thread Feng Tang
of the acpi_get_table_with_size(). v3: Remove a not-needed cast found by Don Dutile. Signed-off-by: Feng Tang feng.t...@intel.com --- arch/x86/kernel/apic/es7000_32.c |7 +++ drivers/acpi/tables.c| 21 + drivers/iommu/amd_iommu_init.c | 10 -- drivers/iommu

Re: [PATCH 1/2] ACPI: replace all acpi_get_table_with_size with acpi_get_table

2012-07-31 Thread Feng Tang
On Tue, 31 Jul 2012 13:56:58 -0700 Yinghai Lu wrote: > On Mon, Jul 30, 2012 at 8:20 PM, Len Brown wrote: > > Applied. > > > > thanks, > > Len Brown, Intel Open Source Technology Center > > > > ps. next time, please send to linux-a...@vger.kernel.org list > > also, put comments not for the

[PATCH RFC] tick: Emphasize tck_broadcast_on_off could only be called for host cpu

2012-07-31 Thread Feng Tang
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|6

[PATCH RFC] tick: Emphasize tck_broadcast_on_off could only be called for host cpu

2012-07-31 Thread Feng Tang
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 feng.t...@intel.com Cc: Suresh Siddha suresh.b.sid...@intel.com Cc: Thomas Gleixner t...@linutronix.de --- kernel/time/tick-broadcast.c

Re: [PATCH 1/2] ACPI: replace all acpi_get_table_with_size with acpi_get_table

2012-07-31 Thread Feng Tang
On Tue, 31 Jul 2012 13:56:58 -0700 Yinghai Lu ying...@kernel.org wrote: On Mon, Jul 30, 2012 at 8:20 PM, Len Brown l...@kernel.org wrote: Applied. thanks, Len Brown, Intel Open Source Technology Center ps. next time, please send to linux-a...@vger.kernel.org list also, put comments

[PATCH v2] ACPI processor: Fix a typo error in commit 99b725084

2012-07-30 Thread Feng Tang
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

Re: [Regression 3.4] tick_broadcast_mask is not restored after a CPU has been offline/onlined

2012-07-30 Thread Feng Tang
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

[PATCH] ACPI processor: Fix a typo error in commit 99b725084

2012-07-30 Thread Feng Tang
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

Re: [Regression 3.4] tick_broadcast_mask is not restored after a CPU has been offline/onlined

2012-07-30 Thread Feng Tang
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

[Regression][Bisected] tick_broadcast_mask is not restored after a CPU is offline/onlined

2012-07-30 Thread Feng Tang
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

[Regression][Bisected] tick_broadcast_mask is not restored after a CPU is offline/onlined

2012-07-30 Thread Feng Tang
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

Re: [Regression 3.4] tick_broadcast_mask is not restored after a CPU has been offline/onlined

2012-07-30 Thread Feng Tang
Hi Paul, On Mon, 30 Jul 2012 06:39:13 -0700 Paul E. McKenney paul...@linux.vnet.ibm.com 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 after it is offline/onlined

[PATCH] ACPI processor: Fix a typo error in commit 99b725084

2012-07-30 Thread Feng Tang
...@linux.vnet.ibm.com Signed-off-by: Feng Tang feng.t...@intel.com --- drivers/acpi/processor_driver.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/acpi/processor_driver.c b/drivers/acpi/processor_driver.c index ff8e04f..bfc31cb 100644 --- a/drivers/acpi

Re: [Regression 3.4] tick_broadcast_mask is not restored after a CPU has been offline/onlined

2012-07-30 Thread Feng Tang
Hi Paul, On Mon, 30 Jul 2012 10:42:18 -0700 Paul E. McKenney paul...@linux.vnet.ibm.com 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 06:39:13 -0700 Paul E. McKenney

[PATCH v2] ACPI processor: Fix a typo error in commit 99b725084

2012-07-30 Thread Feng Tang
regression: https://lkml.org/lkml/2012/7/30/169 [Resend to ACPI mail list and Cc stable kernel] Signed-off-by: Feng Tang feng.t...@intel.com Cc: Thomas Renninger tr...@suse.de Cc: Deepthi Dharwar deep...@linux.vnet.ibm.com Cc: sta...@vger.kernel.org # 3.3+ --- drivers/acpi/processor_driver.c

Re: [PATCH 2/2] ACPI: remove acpi_get_table_with_size() fucntion

2012-07-19 Thread Feng Tang
Update the commit log. Thanks, Feng >From 61e3349625843bd9e4c353598ec50a2333409986 Mon Sep 17 00:00:00 2001 From: Feng Tang Date: Wed, 18 Jul 2012 15:01:18 +0800 Subject: [PATCH 2/2] ACPI: Remove acpi_get_table_with_size() fucntion Remove the unnecessary acpi_get_table_with_s

Re: [PATCH 1/2] ACPI: replace all acpi_get_table_with_size with acpi_get_table

2012-07-19 Thread Feng Tang
Update the commit log. Thanks, Feng >From 633ba8625e9ae1999b94edac514a11ddbc4176ef Mon Sep 17 00:00:00 2001 From: Feng Tang Date: Thu, 19 Jul 2012 15:10:02 +0800 Subject: [PATCH 1/2] ACPI: Replace acpi_get_table_with_size() with acpi_get_table() This is a preparation for remov

[PATCH 2/2] ACPI: remove acpi_get_table_with_size() fucntion

2012-07-19 Thread Feng Tang
--- drivers/acpi/acpica/tbxface.c | 18 +++--- include/acpi/acpixf.h |4 2 files changed, 3 insertions(+), 19 deletions(-) diff --git a/drivers/acpi/acpica/tbxface.c b/drivers/acpi/acpica/tbxface.c index abcc641..3daf93d 100644 --- a/drivers/acpi/acpica/tbxface.c

[PATCH 1/2] ACPI: replace all acpi_get_table_with_size with acpi_get_table

2012-07-19 Thread Feng Tang
--- arch/x86/kernel/apic/es7000_32.c |7 +++ drivers/acpi/tables.c| 21 + drivers/iommu/dmar.c | 14 +++--- 3 files changed, 19 insertions(+), 23 deletions(-) diff --git a/arch/x86/kernel/apic/es7000_32.c

[PATCH 1/2] ACPI: replace all acpi_get_table_with_size with acpi_get_table

2012-07-19 Thread Feng Tang
--- arch/x86/kernel/apic/es7000_32.c |7 +++ drivers/acpi/tables.c| 21 + drivers/iommu/dmar.c | 14 +++--- 3 files changed, 19 insertions(+), 23 deletions(-) diff --git a/arch/x86/kernel/apic/es7000_32.c

[PATCH 2/2] ACPI: remove acpi_get_table_with_size() fucntion

2012-07-19 Thread Feng Tang
--- drivers/acpi/acpica/tbxface.c | 18 +++--- include/acpi/acpixf.h |4 2 files changed, 3 insertions(+), 19 deletions(-) diff --git a/drivers/acpi/acpica/tbxface.c b/drivers/acpi/acpica/tbxface.c index abcc641..3daf93d 100644 --- a/drivers/acpi/acpica/tbxface.c

Re: [PATCH 1/2] ACPI: replace all acpi_get_table_with_size with acpi_get_table

2012-07-19 Thread Feng Tang
Update the commit log. Thanks, Feng From 633ba8625e9ae1999b94edac514a11ddbc4176ef Mon Sep 17 00:00:00 2001 From: Feng Tang feng.t...@intel.com Date: Thu, 19 Jul 2012 15:10:02 +0800 Subject: [PATCH 1/2] ACPI: Replace acpi_get_table_with_size() with acpi_get_table() This is a preparation

Re: [PATCH 2/2] ACPI: remove acpi_get_table_with_size() fucntion

2012-07-19 Thread Feng Tang
Update the commit log. Thanks, Feng From 61e3349625843bd9e4c353598ec50a2333409986 Mon Sep 17 00:00:00 2001 From: Feng Tang feng.t...@intel.com Date: Wed, 18 Jul 2012 15:01:18 +0800 Subject: [PATCH 2/2] ACPI: Remove acpi_get_table_with_size() fucntion Remove the unnecessary

<    4   5   6   7   8   9