From owner-freebsd-acpi@FreeBSD.ORG  Wed Jul 13 16:03:00 2011
Return-Path: <owner-freebsd-acpi@FreeBSD.ORG>
Delivered-To: freebsd-acpi@freebsd.org
Received: from [127.0.0.1] (freefall.freebsd.org [IPv6:2001:4f8:fff6::28])
	by hub.freebsd.org (Postfix) with ESMTP id 522851065670;
	Wed, 13 Jul 2011 16:03:00 +0000 (UTC)
	(envelope-from jkim@FreeBSD.org)
From: Jung-uk Kim <jkim@FreeBSD.org>
To: Vitaly Magerya <vmagerya@gmail.com>
Date: Wed, 13 Jul 2011 12:02:50 -0400
User-Agent: KMail/1.6.2
References: <BANLkTim+1UwquMJ32WP8wZBGkYxPv78MLA@mail.gmail.com>
	<20110713114521.9f684b01.taku@tackymt.homeip.net>
	<CAL409KzJwinYQU4aqqg4khL4EmwXfFfft0=S7KwzSTyxyzue6Q@mail.gmail.com>
In-Reply-To: <CAL409KzJwinYQU4aqqg4khL4EmwXfFfft0=S7KwzSTyxyzue6Q@mail.gmail.com>
MIME-Version: 1.0
Content-Disposition: inline
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Message-Id: <201107131202.53344.jkim@FreeBSD.org>
Cc: freebsd-acpi@freebsd.org, Andriy Gapon <avg@freebsd.org>
Subject: Re: (Missing) power states of an Atom N455-based netbook
X-BeenThere: freebsd-acpi@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: ACPI and power management development <freebsd-acpi.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-acpi>,
	<mailto:freebsd-acpi-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-acpi>
List-Post: <mailto:freebsd-acpi@freebsd.org>
List-Help: <mailto:freebsd-acpi-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-acpi>,
	<mailto:freebsd-acpi-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Wed, 13 Jul 2011 16:03:00 -0000

On Wednesday 13 July 2011 10:32 am, Vitaly Magerya wrote:
> Taku YAMAMOTO <taku@tackymt.homeip.net> wrote:
> > Ah, the line in the acpi_cpu_mwait function should be read as:
> >
> > 		cpu_monitor(state, 0, 0);
> >
> > Vitaly, could you test the jkim's patch again with the above line
> > modified?

I fixed that bug in the later patches, e.g.,

http://people.freebsd.org/~jkim/acpi_cx_native.diff

However, it doesn't work yet.  Please see below.

> Sure. With the above change it hangs during the boot.
>
> Here are the last two lines before the hang with verbose boot:
>
> acpi_acad0: acline initialization start
> acpi_battery0: battery initialization start

It is a known problem and the above patch hangs, too. :-(

Actually, I abandoned the patches and I am thinking about rewriting it 
from scratch, e.g., refactoring MI/MD support code 
(dev/acpica/acpi_cpu.c -> machine/machdep.c & acpica/acpi_machdep.c).  
Unfortunately, I don't have much time nor hardware to test, so it 
won't happen any time soon.  Sorry.

If anyone wants to pick it up from here, please feel free.

Jung-uk Kim