From owner-freebsd-i386@FreeBSD.ORG Tue Jan 24 06:20:18 2006 Return-Path: X-Original-To: freebsd-i386@hub.freebsd.org Delivered-To: freebsd-i386@hub.freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 175D516A41F for ; Tue, 24 Jan 2006 06:20:18 +0000 (GMT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by mx1.FreeBSD.org (Postfix) with ESMTP id A9C0B43D49 for ; Tue, 24 Jan 2006 06:20:17 +0000 (GMT) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.13.4/8.13.4) with ESMTP id k0O6KHKZ003254 for ; Tue, 24 Jan 2006 06:20:17 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.13.4/8.13.4/Submit) id k0O6KHqn003253; Tue, 24 Jan 2006 06:20:17 GMT (envelope-from gnats) Date: Tue, 24 Jan 2006 06:20:17 GMT Message-Id: <200601240620.k0O6KHqn003253@freefall.freebsd.org> To: freebsd-i386@FreeBSD.org From: Arthur Hartwig Cc: Subject: Re: i386/92238: Spurious "atapci1: failed to enable memory mapping!" message on startup X-BeenThere: freebsd-i386@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Arthur Hartwig List-Id: I386-specific issues for FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 24 Jan 2006 06:20:18 -0000 The following reply was made to PR i386/92238; it has been noted by GNATS. From: Arthur Hartwig To: bug-followup@FreeBSD.org, arthur.hartwig@nokia.com Cc: Subject: Re: i386/92238: Spurious "atapci1: failed to enable memory mapping!" message on startup Date: Tue, 24 Jan 2006 16:18:37 +1000 The change proposed in the original problem report is insufficient; the system failed to detect the CD drive at ata1-slave. The AHCI in the cfg1 field of the Intel ids array is unfortunately also used in ata_intel_reset to indicate the channel may have a slave device. As well as the previously described patch to get rid of the memory mapping failure message I changed ata_intel_reset(), the first test from if (ctlr->chip->cfg1) { to if (ctlr->chip->chipid >= ATA_I82801FB_S1) { (to cover both ICH6 and ICH7) and the new kernel recognised the CD drive. There are obviously a lot of combinations involving the ICHx series of IDE controllers but the existing code doesn't seem to cope well with all the possibilities. For example, in ata_intel_chipinit() there is no test made to see if an AHCI capable chipset is in AHCI mode. The ICH7 specification suggests (section 12.1.33) the MAP register can be used to see if an AHCI capable chipset is actually in AHCI mode. If you don't have access to an ICH7 board or want to test on additional motherboards I'd be happy for you to send me diffs to try out revised code.