lotto | 1 May 15:35 2011
Picon

Re:


Send your Names*Address*Phone* to claim your 1,000,000 GBP awarded to
you.Reply to lotto_agent1 <at> ymail.com for more info

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Karthigan Srinivasan | 3 May 00:16 2011
Picon

Re: [PATCH] drivers/acpi/utils.c: Fixed coding style issues

> On Thu, 2011-04-21 at 11:08 -0500, Karthigan Srinivasan wrote:

Len - I submitted this patch on 4/21 and have not received any
acknowledgement. I am re-submitting it again. Please review and approve.

Best Regards,
Karthigan Srinivasan.

====

Fixed brace and comment coding style issues is code.

Signed-off-by: Karthigan Srinivasan <karthigan.srinivasan <at> hp.com>
---
 drivers/acpi/utils.c |   15 ++++++---------
 1 files changed, 6 insertions(+), 9 deletions(-)

diff --git a/drivers/acpi/utils.c b/drivers/acpi/utils.c
index b002a47..76eaac2 100644
--- a/drivers/acpi/utils.c
+++ b/drivers/acpi/utils.c
 <at>  <at>  -99,9 +99,8  <at>  <at>  acpi_extract_package(union acpi_object *package,

 		union acpi_object *element = &(package->package.elements[i]);

-		if (!element) {
+		if (!element)
 			return AE_BAD_DATA;
-		}

(Continue reading)

Joe Perches | 3 May 02:26 2011

Re: [PATCH] drivers/acpi/utils.c: Fixed coding style issues

On Mon, 2011-05-02 at 17:16 -0500, Karthigan Srinivasan wrote:
>  drivers/acpi/utils.c |   15 ++++++---------
>  <at>  <at>  -373,7 +370,7  <at>  <at>  acpi_evaluate_reference(acpi_handle handle,
>        end:
>  	if (ACPI_FAILURE(status)) {
>  		list->count = 0;
> -		//kfree(list->handles);
> +		/* kfree(list->handles); */

Just delete the line instead.

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

lan,Tianyu | 3 May 08:10 2011
Picon

[PATCH] Replace acpi_gbl_FADT.sci_interrupt with two functions in some places.

     Some places use the acpi_gbl_FADT.sci_interrupt directly to get the ACPI SCI.ACPI SCI will not only be
stored in the FADT. 
So this patch replaces acpi_gbl_FADT.sci_interrupt with two functions for getting and setting.

Signed-off-by: Lan Tianyu <tianyu.lan <at> intel.com>
---
 arch/x86/kernel/acpi/boot.c                  |    8 ++---
 drivers/acpi/acpica/evgpeinit.c              |    5 +--
 drivers/acpi/acpica/evgpeutil.c              |    4 +-
 drivers/acpi/acpica/evsci.c                  |    4 +-
 drivers/acpi/acpica/evxface.c                |   38 +++++++++++++++++++++++++++
 drivers/acpi/bus.c                           |    4 +-
 drivers/acpi/osl.c                           |    6 ++--
 drivers/acpi/pci_link.c                      |    2 -
 drivers/staging/olpc_dcon/olpc_dcon_xo_1_5.c |    2 -
 include/acpi/acpixf.h                        |    4 ++
 10 files changed, 59 insertions(+), 18 deletions(-)

Index: linux-2.6/arch/x86/kernel/acpi/boot.c
===================================================================
--- linux-2.6.orig/arch/x86/kernel/acpi/boot.c	2011-05-03 10:19:00.163029998 +0800
+++ linux-2.6/arch/x86/kernel/acpi/boot.c	2011-05-03 10:20:50.833030004 +0800
 <at>  <at>  -408,7 +408,7  <at>  <at> 

 	acpi_table_print_madt_entry(header);

-	if (intsrc->source_irq == acpi_gbl_FADT.sci_interrupt) {
+	if (intsrc->source_irq == acpi_get_sci_interrupt()) {
 		acpi_sci_ioapic_setup(intsrc->source_irq,
 				      intsrc->inti_flags & ACPI_MADT_POLARITY_MASK,
(Continue reading)

Sebastian Färber | 3 May 12:24 2011
Picon

Hard LOCKUP, maybe acpi related (present in 2.6.32 to 2.6.37, fixed in 2.6.38)

Hi,

I'm trying to track down a problem i have with newer hardware which seems
fixed in 2.6.38.4 but which i can reproduce from 2.6.32 to 2.6.37
I'm running a DEBUG Kernel with nmi_watchdog=1 and attached a backtrace
of the lockup that happened on 2.6.37.6. I can also provide backtraces for
2.6.32 to 2.6.35 if necessary, they look very similar.

To my untrained eye this looks like a problem with ACPI/CPU C States
(CPU in Question
is a Quadcore Core i5 750). It's strange that the watchdog only shows
backtraces for 3 CPUs/Cores, one is still fine?
The servers on which this happens are all completely idle and the lockup
occurs every 1-2 days.
Would be great if someone more experienced could have a look and see
which change in 2.6.38 fixed this.
Unfortunately i can't upgrade all servers to 2.6.38 right now so I'm hoping
it's possible to backport this to the stable kernel series.

My .config is here: http://pastebin.com/iC1udT93
Backtrace: http://pastebin.com/yR1YseM7

If someone needs more information just let me know.

Regards,

Sebastian

---
BUG: NMI Watchdog detected LOCKUP on CPU1, ip c100a556, registers:
(Continue reading)

Valdis.Kletnieks | 3 May 16:15 2011
Picon

2.6.39-rc5-mmotm0429 - lockdep splat in fsck/acpi

No, I have no idea how a path invoked by fsck and a path invoked by acpi
managed to cross paths. Lot of scheduler and acpi function names in the
tracebacks, so, tossing it to everybody plausible in MAINTAINERS.

Saw this during boot this morning:

[   87.294863] =======================================================
[   87.295177] [ INFO: possible circular locking dependency detected ]
[   87.295402] 2.6.39-rc5-mmotm0429 #1
[   87.295531] -------------------------------------------------------
[   87.295727] fsck/3150 is trying to acquire lock:
[   87.295727]  (rcu_node_level_0){..-...}, at: [<ffffffff810904a8>] rcu_read_unlock_special+0x8c/0x1d5
[   87.295727] 
[   87.295727] but task is already holding lock:
[   87.295727]  (&rq->lock){-.-.-.}, at: [<ffffffff81031dfb>] scheduler_ipi+0x34/0x5d
[   87.295727] 
[   87.295727] which lock already depends on the new lock.
[   87.295727] 
[   87.295727] 
[   87.295727] the existing dependency chain (in reverse order) is:
[   87.295727] 
[   87.295727] -> #2 (&rq->lock){-.-.-.}:
[   87.295727]        [<ffffffff810670e1>] check_prevs_add+0x8b/0x104
[   87.295727]        [<ffffffff810674c9>] validate_chain+0x36f/0x3ab
[   87.295727]        [<ffffffff81067b93>] __lock_acquire+0x369/0x3e2
[   87.295727]        [<ffffffff81068137>] lock_acquire+0xfc/0x14c
[   87.295727]        [<ffffffff81567951>] _raw_spin_lock+0x36/0x45
[   87.295727]        [<ffffffff81027df9>] __task_rq_lock+0x8b/0xd3
[   87.295727]        [<ffffffff81032fd1>] wake_up_new_task+0x41/0x108
[   87.295727]        [<ffffffff81037714>] do_fork+0x265/0x33f
(Continue reading)

Karthigan Srinivasan | 3 May 17:09 2011
Picon

Re: [PATCH] drivers/acpi/utils.c: Fixed coding style issues

> > -		//kfree(list->handles);
> > +		/* kfree(list->handles); */

> Just delete the line instead.

Len, Joe, Nicolas - re-submitting patch with above recommended change. 

Best Regards,
Karthigan.

=========

Fixed brace, comment, and label coding style issues.

Signed-off-by: Karthigan Srinivasan <karthigan.srinivasan <at> hp.com>
---
 drivers/acpi/utils.c |   19 +++++++------------
 1 files changed, 7 insertions(+), 12 deletions(-)

diff --git a/drivers/acpi/utils.c b/drivers/acpi/utils.c
index b002a47..91289d4 100644
--- a/drivers/acpi/utils.c
+++ b/drivers/acpi/utils.c
 <at>  <at>  -99,9 +99,8  <at>  <at>  acpi_extract_package(union acpi_object *package,

 		union acpi_object *element = &(package->package.elements[i]);

-		if (!element) {
+		if (!element)
 			return AE_BAD_DATA;
(Continue reading)

ken leo | 3 May 18:05 2011
Picon

(unknown)

Gera diena

Mes siūlome paskolas atskiroms bendrovėms ir pigūs
3% norma, asmeninių ir investicijas, mes siūlome paskola
$ 5,000.00 iki $ 100,000,000.00 dolerių mus grįžti prie mūsų su tikslios sumos
jums reikia.
Prašome užpildyti ir grąžinti šią formą norėdami tęsti.

Vardas, pavardė: _
Kredito paskirtis: _
Amžius: _
Lytis: _
Adresas: _
Šalis: _
Įmonės pavadinimas: _
Pareigos: _
Telefonas: _
reikalinga suma, kaip paskola: _
Trukmė: _

Jei esate interested.contact su mumis žemiau
E-mail: Brucefastfunds09 <at> gmail.com
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo <at> vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Yinghai Lu | 3 May 21:02 2011

[PATCH -v2 RESENDING] x86, acpi: Handle xapic/x2apic entries in MADT at same time


One system have mixing xapic and x2apic entries in MADT and SRAT.
BIOS guys insist that ACPI 4.0 SPEC said so, if apic id < 255, even
the cpus are with x2apic mode pre-enabled, still need to use xapic entries
instead of x2apic entries.

on 8 socket system with x2apic pre-enabled, will get out of order sequence:
CPU0: socket0, core0, thread0.
CPU1 - CPU 40: socket 4 - socket 7, thread 0
CPU41 - CPU 80: socket 4 - socket 7, thread 1
CPU81 - CPU 119: socket 0 - socket 3, thread 0
CPU120 - CPU 159: socket 0 - socket 3, thread 1

so max_cpus=80 will not get all thread0 now.

Need to handle every entry in MADT at same time with xapic and x2apic.
so we can honor sequence in MADT.

We can use max_cpus= command line to use thread0 in every core,
because recent MADT always have all thread0 at first.
Also it could make the cpu to node mapping more sane.

after patch will get
CPU0 - CPU 79: socket 0 - socket 7, thread 0
CPU80 - CPU 159: socket 0 - socket 7, thread 1

-v2: update some comments, and change to pass array pointer.

Signed-off-by: Yinghai Lu <yinghai <at> kernel.org>

(Continue reading)

Lin Ming | 4 May 03:48 2011
Picon

Re: [PATCH] Replace acpi_gbl_FADT.sci_interrupt with two functions in some places.

On Tue, 2011-05-03 at 14:10 +0800, Lan, Tianyu wrote:
> Some places use the acpi_gbl_FADT.sci_interrupt directly to get the ACPI SCI.ACPI SCI will not only be
stored in the FADT. 
> So this patch replaces acpi_gbl_FADT.sci_interrupt with two functions for getting and setting.

Some trivial patch style comments,

We usually add "acpi:" prefix to the subject, like

[PATCH] acpi: xxxxx

And if acpica files are also touched, we add "acpi/acpica:" prefix, like

[PATCH] acpi/acpica: xxxx

Another thing, the change logs seems too long for each line, you'd
better split them short for each line. So that makes logs easy to read.

Thanks,
Lin Ming

> 
> Signed-off-by: Lan Tianyu <tianyu.lan <at> intel.com>
> ---
>  arch/x86/kernel/acpi/boot.c                  |    8 ++---
>  drivers/acpi/acpica/evgpeinit.c              |    5 +--
>  drivers/acpi/acpica/evgpeutil.c              |    4 +-
>  drivers/acpi/acpica/evsci.c                  |    4 +-
>  drivers/acpi/acpica/evxface.c                |   38 +++++++++++++++++++++++++++
>  drivers/acpi/bus.c                           |    4 +-
(Continue reading)


Gmane