From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id F15BACCF9E5 for ; Sun, 26 Oct 2025 18:59:56 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 88BC210E0A8; Sun, 26 Oct 2025 18:59:56 +0000 (UTC) Authentication-Results: gabe.freedesktop.org; dkim=pass (2048-bit key; unprotected) header.d=intel.com header.i=@intel.com header.b="K2olQ1fN"; dkim-atps=neutral Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.14]) by gabe.freedesktop.org (Postfix) with ESMTPS id 5518F10E0A8 for ; Sun, 26 Oct 2025 18:59:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1761505196; x=1793041196; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=hBqsqnd6a3iauduSqWPjwT7a7bSvuTINi3/JsZDiKtg=; b=K2olQ1fNmdX+W3RsI39hZQTJ1aAkll4ZbupEVvOOT63qwLjBTsIGkRYE DVBSAzyDqVaus+pUPWN6BgBCxuSoiwjtuhORUTtFcEYyQC/aqfOzpA7fm 7S5W4ACMl75AUy6MmdJbEtHUhhxhd9VYoC/4ov0+YoQiPN9pS4k118lVu WRRIeJj+Ffjq3G5KJN2YeyJV0fIvUqxDOjmQBSCcCLDz5ZhzlifLOOk1Q Hmc64AzoTomk8njEtuXv1OaPZ3WRMquZ1fdldy9a8Qyf5XKNH6cMc8cH2 x+JowQxbaLrbN3qjpxodKFo3gH2PQ67D8Hd8ZPfWS+JClB7VFHGYn51lO Q==; X-CSE-ConnectionGUID: TaqOuP0WTmWipINOOjKJnQ== X-CSE-MsgGUID: wrE/NrnISCivWB1fK9jXpA== X-IronPort-AV: E=McAfee;i="6800,10657,11586"; a="63631645" X-IronPort-AV: E=Sophos;i="6.19,257,1754982000"; d="scan'208";a="63631645" Received: from orviesa003.jf.intel.com ([10.64.159.143]) by fmvoesa108.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Oct 2025 11:59:55 -0700 X-CSE-ConnectionGUID: 9dEK9KYCTlCpLgnc/xiR1g== X-CSE-MsgGUID: XDebQBwWQS+FkLs8VisUyQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.19,257,1754982000"; d="scan'208";a="188917868" Received: from black.igk.intel.com ([10.91.253.5]) by orviesa003.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Oct 2025 11:59:53 -0700 Date: Sun, 26 Oct 2025 19:59:50 +0100 From: Raag Jadav To: Rodrigo Vivi Cc: Riana Tauro , intel-xe@lists.freedesktop.org, anshuman.gupta@intel.com, lucas.demarchi@intel.com Subject: Re: [PATCH] drm/xe/xe_survivability: Add support for survivability mode v2 Message-ID: References: <20251014053257.3417575-2-riana.tauro@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-BeenThere: intel-xe@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel Xe graphics driver List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-xe-bounces@lists.freedesktop.org Sender: "Intel-xe" On Sun, Oct 19, 2025 at 05:55:17PM +0200, Raag Jadav wrote: > On Thu, Oct 16, 2025 at 03:17:54PM -0400, Rodrigo Vivi wrote: > > On Tue, Oct 14, 2025 at 11:02:58AM +0530, Riana Tauro wrote: > > > v2 survivability breadcrumbs introduces a new mode called > > > SPI Flash Descriptor Override mode (FDO). This is enabled by > > > PCODE when MEI itself fails and firmware cannot be updated via > > > MEI using igsc. This mode provides the ability to update > > > the firmware directly via SPI driver. > > > > > > Xe KMD initializes the nvm aux driver if FDO mode is enabled. > > > > > > Userspace should check FDO mode entry in survivability sysfs before > > > using the SPI driver to update firmware. > > > > > > v2 also supports survivability mode for critical boot errors. > > > > > > cat /sys/bus/pci/devices/0000\:03\:00.0/survivability_mode > > > > > > Capability Info: 0x138320 - 0x2001ae06 > > > Postcode Info: 0x138324 - 0x0 > > > Overflow Info: 0x138328 - 0x0 > > > Auxiliary Info 0: 0x13832c - 0x0 > > > > I am truly sorry here, but although I was the one that designed this, > > looking it now, I realized that this is breaking the sysfs rules > > of one value per file and no fancy format. This is only allowed in > > the debugfs. > > > > We need to change this asap, and with help from any tool that > > might be already consuming this. > > > > > FDO Mode: enabled > > > > After we fix that we can come and add this. > > > > About our options: I don't believe that debugfs is an option > > without the drm card right? > > > > Perhaps what we need is to transform survivability_mode in > > the directory. Each entry becomes a file in this directory. > > > > Sorry, > > Rodrigo. > > Tried discussing in this direction[1], but not very convincingly I guess. > > [1] https://lore.kernel.org/intel-xe/aIDrCMRK2gPGmIs6@black.fi.intel.com/ Documentation/filesystems/configfs.rst +61 * Normal attributes, which similar to sysfs attributes, are small ASCII text files, with a maximum size of one page (PAGE_SIZE, 4096 on i386). Preferably only one value per file should be used, and the same caveats from sysfs apply. So this also puts some of our configfs attributes in the same position, but I'll let the maintainers have final call on this. Raag > > Raag