From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933483AbeAJPXs (ORCPT + 1 other); Wed, 10 Jan 2018 10:23:48 -0500 Received: from mga01.intel.com ([192.55.52.88]:15482 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752444AbeAJPXr (ORCPT ); Wed, 10 Jan 2018 10:23:47 -0500 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,340,1511856000"; d="scan'208";a="20506954" Date: Wed, 10 Jan 2018 17:23:43 +0200 From: Mika Westerberg To: Takashi Iwai Cc: "Rafael J. Wysocki" , linux-kernel@vger.kernel.org Subject: Re: Missing watchdog after ACPI watchdog creation failure Message-ID: <20180110152343.GO27654@lahna.fi.intel.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo User-Agent: Mutt/1.9.1 (2017-09-22) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Wed, Jan 10, 2018 at 04:13:51PM +0100, Takashi Iwai wrote: > Hi, > > on the recent kernels, i2c-i801 skips the creation of iTCO wdt when > ACPI WDAT is present. It's fine when ACPI really creates the watchdog > device. But, we've got a report showing that the watchdog is missing > on some machines because ACPI failed to create, and yet i2c-i801 still > skips because acpi_has_watchdog() returns true. > > More specifically, the machine gets an error from acpi_watchdog.c > like: > platform wdat_wdt: failed to claim resource 3: [io 0x040a-0x040c] > ACPI: watchdog: Device creation failed: -16 > > where the region was registered by pnp, > % cat /proc/ioports > .... > 0400-047f : pnp 00:01 > > > One may say that BIOS sucks, but OTOH, the complete lack of watchdog > thereafter can be seen as a regression, too. It used to work on the > older kernel as iTCO wdt was provided by i2c-i801. Hmm, if the resource is already taken I wonder how iTCO can work? Are you sure iTCO works on those systems? > Shouldn't acpi_has_watchdog() rather checks whether the watchdog > device creation succeeded or not? Yes, or rather we should first figure out what the actual problem is ;-) Are you able to get acpidump from that system with full dmesg?