From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f45.google.com (mail-ej1-f45.google.com [209.85.218.45]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 7422439B498 for ; Thu, 9 Jul 2026 16:59:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783616361; cv=none; b=p5Cy/QfQ4z6W/IqlfSF+E+sCXd3LBu2b4vDiW8wpeW9zrrm0ftGFIs5zZxCz8w+zlzWGJq5NLZ6n0EZps+LDHpBNaebF0dFY4bXOydzh72P3OjcuVdhgpdugB49dy84xYep+WPfz+ytJXlfQCb3lzZOvM97Ci/vWUE+PFO2bkLo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783616361; c=relaxed/simple; bh=pGq6Ho7akFAbOMMkEMP4bgv7HVaDVNSGjgc1WcBOf1o=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=j5P4DiAJODd21Hy5u9ZCsu2VsQgpj0rLBAlTed9KnC1y2FNTIOQuEVe5lK/UpgN49avyayWkYzIlsea/uoTnxdS3v4OOuQZmXAUbbVGpAcZOdGRUCMYlU1BMlSxZC+4blwbLapZlaBUr8lvmkwba1ifAyBeda641Od3PuSHjS+U= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=Z/VFXBqu; arc=none smtp.client-ip=209.85.218.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Z/VFXBqu" Received: by mail-ej1-f45.google.com with SMTP id a640c23a62f3a-c15ba3a2b4bso13913966b.1 for ; Thu, 09 Jul 2026 09:59:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783616359; x=1784221159; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=FcqmO2qOzufGvSZVtWMC1+200YXwxI3/0WI8kXhX1g4=; b=Z/VFXBquoOVgqI9jKhtNefNpdSQGrQyz2SVHh2E9pdylEA9OJFPOUTsg5e+A4XVOtR IZ7LTVmPXfnEZtGr8DLDEe4vUzntA6YEBN51zW/Y414/MgGltVlutgdJnv3qrhXkLukF JxBcHVLImUikKuQjaNgoOravmmsVoSrLKrW1bsiojCIlfPkvxhtrC65N968vEYjSeLdf fzjsNRJ/J7g7l+8vlEttIO7rN5oQ0FYxp20nzW9QUaZ+nSYu+3qa0of3NA3UTKkN+bu1 Auc3/Ug8il6CEKus06AF3q+ikYa+iSu7bvCihZKs+6aGLPqOKlqt7aJhee4Zi2LtaLbl CBlA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783616359; x=1784221159; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=FcqmO2qOzufGvSZVtWMC1+200YXwxI3/0WI8kXhX1g4=; b=eBVoh+a612D0lf/+nLjtEretYR2KG8IvlOR8cUqm08Hxt9HvNq7ByvXpgJU0rDTwGG x57S2qB8aE/PA6FdZjNPvDM1hhhC/uLd22K1mUVwSEKw7RAEtwCXpeRRSNEfV9yujftK kASdLlt+5hPlOzFTkvrxtw28Vr7fCD8hWvTGEz+fXZIdTi9HLa2gbgwUwXTQ7X3DF0aj ljsEZbN5ov1ifn64XxyxmLKB0yP/soi52Kc8EmqiWzp2JmbEIg7CnBQ8Z7DG8PmtCS6N FY4KV31Fs3kryi6N1jIzAB7GIKnSDbbrTBroe0agDWJ7cf0iCjvQ4FItoc+zBo5r/Jf7 wVOg== X-Forwarded-Encrypted: i=1; AHgh+Rqe5IGCUJcG6dMSgTtnQIaAipWk7mfXakpuBpRURgFpT+Ta9GB0cnVwS1zxojKHPYvUtHTDxkLkSKfw+mE=@vger.kernel.org X-Gm-Message-State: AOJu0YxUvVvCejYRl4ehs5Vl0uIXN8QAe3bQcOFHlGwkc5y+BA7nD/zQ UCQGHMqkbn3itfGpY2evKp017ghEva8gw8IlB9zX9kNHzUB1vJFlyTZd X-Gm-Gg: AfdE7clch4Ss+l8WtjKz1TxR6nQ2fo0rXabf3NrHfK3CTFOoa8KUiLxs4CzzUjTGQXe UWgZMXpaSSVPj7cAvMRC1v3UF54YMmc6Yz3dC2xWOfJFd4vlchhdIie2uzhXkFwpLqNihFAu2Pf h7QulX4Czq3kfuqg8dZVyBoRT4teBi/dOkS+0+kcRdPAF7ctd0z/C6pQ8+2TRPKARDB60MaL5Rp qFxOE1h660y6IRIpa+/FwV7NZ+iR3G7ruzc7GfM2sdSFuDdAQQlKd+Hyg6UpgrzIMVNMUVz5jFx Ym/yBiWf5pssk1FXaoTU8ZeVGGqGumOx5OIpID1XONr2ZP3cSNSRaXooL0wDhf86fL9+lpZonH5 imoTmno0K5fQb3wJTxuFUNYBA8ZyTbZuGJuq7m/Xr5QQQsBH8uR73VLiKun3xeHBLAusFl16WwN Y1uQBIpYwupdx1anZiyn/3NWuZJfxXGpMivKuuhl/9cEBInvKdfBPrU0oXWGc5JK4= X-Received: by 2002:a17:907:d15:b0:c15:9350:dfa6 with SMTP id a640c23a62f3a-c15ce21e09dmr319755066b.60.1783616358766; Thu, 09 Jul 2026 09:59:18 -0700 (PDT) Received: from node ([202.47.63.86]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-c15c79f2a3fsm329902666b.49.2026.07.09.09.59.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 Jul 2026 09:59:18 -0700 (PDT) From: Muhammad Bilal To: Hans de Goede , =?UTF-8?q?Ilpo=20J=C3=A4rvinen?= , Jorge Lopez , =?UTF-8?q?Thomas=20Wei=C3=9Fschuh?= , platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org Cc: stable@vger.kernel.org, Mario Limonciello , Armin Wolf , Muhammad Bilal Subject: [PATCH v5 4/4] platform/x86: hp-bioscfg: warn on element type mismatch instead of failing Date: Thu, 9 Jul 2026 21:58:59 +0500 Message-ID: <20260709165900.30615-5-meatuni001@gmail.com> X-Mailer: git-send-email 2.55.0 In-Reply-To: <20260709165900.30615-1-meatuni001@gmail.com> References: <20260709165900.30615-1-meatuni001@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit hp_populate_enumeration_elements_from_package() returns -EIO and aborts enumeration of the entire attribute when any single element has an unexpected ACPI type. This is observed on HP EliteBook 840 G2 when the BIOS returns malformed ACPI data following a failed WMI query: ACPI BIOS Error (bug): AE_AML_BUFFER_LIMIT, Index (0x000000032) is beyond end of object (length 0x32) ACPI Error: Aborting method \_SB.WMID.WQBE due to previous error Error expected type 2 for elem 13, but got type 1 instead hp_bioscfg: Returned error 0x3, "Invalid command value/Feature not supported" Aborting immediately discards the attribute entirely. Warn about the unexpected element type, free the temporary string, skip the offending element, and continue parsing the remaining package instead of failing the whole attribute. Fixes: a34fc329b189 ("platform/x86: hp-bioscfg: bioscfg") Cc: stable@vger.kernel.org Signed-off-by: Muhammad Bilal --- drivers/platform/x86/hp/hp-bioscfg/enum-attributes.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/platform/x86/hp/hp-bioscfg/enum-attributes.c b/drivers/platform/x86/hp/hp-bioscfg/enum-attributes.c index de156a9f88a18..21077d17113b9 100644 --- a/drivers/platform/x86/hp/hp-bioscfg/enum-attributes.c +++ b/drivers/platform/x86/hp/hp-bioscfg/enum-attributes.c @@ -163,10 +163,11 @@ static int hp_populate_enumeration_elements_from_package(union acpi_object *enum /* Check that both expected and read object type match */ if (expected_enum_types[eloc] != enum_obj[elem].type) { - pr_err("Error expected type %d for elem %d, but got type %d instead\n", - expected_enum_types[eloc], elem, enum_obj[elem].type); + pr_warn("Unexpected element type at elem %d: expected %d, got %d, skipping\n", + elem, expected_enum_types[eloc], enum_obj[elem].type); kfree(str_value); - return -EIO; + str_value = NULL; + continue; } /* Assign appropriate element value to corresponding field */ -- 2.55.0