From patchwork Wed Jan 31 10:57:24 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mukesh Ojha X-Patchwork-Id: 19723 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:2087:b0:106:209c:c626 with SMTP id gs7csp1808972dyb; Wed, 31 Jan 2024 03:05:08 -0800 (PST) X-Google-Smtp-Source: AGHT+IG/D7xBlv1Qje72lPTlVtodNe4KFtcRSofBtdH6dA9JLpYN/jQhrhGDNB9i3RjzNgYjDEHm X-Received: by 2002:a05:6e02:612:b0:363:8559:edf with SMTP id t18-20020a056e02061200b0036385590edfmr1342737ils.1.1706699108734; Wed, 31 Jan 2024 03:05:08 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1706699108; cv=pass; d=google.com; s=arc-20160816; b=rXfTl73agpW1y3UsAobrA8LwISmP6RVQRT1zrdeKK1BmY/UWSV8QCjwaCftYeC8mBv f3k98aPFezGCFu9T2lEeoAkSNHuR4Tv4o/wqxgw0v7fdrD6sOzgs5bMW+JpgvbpZww2y J7Ne9ju8e0lpD4NoHA8mSX7MSohwRRE3EkegufhwhVd/HJvryRkGL8ieWQW3RBRMkGZU nePZeQW5fusiN/gJD69lS4bUXtJ1ICvpcASbWKqe+3usRnRj/j7/iBtlaV2ZSX9ag6DO 4/rb26KhGX17OIrmzs0lXLz9d+ZU34ynhuSiithfPa3U4T+3iMp1f0RjOUM/eY6XDbgq 3BAw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:subject:cc:to :from:dkim-signature; bh=gF/M0cQp/Po1gWJguFknxVmvecw9rwR313j3e4N2Jkg=; fh=6RqRlS7vnONpw0BrFI48tq0rIFzwnsOULW9mAwQfhvM=; b=cOCiALWvP+lv6RiXxqNlRi8szI3dCExRwbVciVkfacNChAtEYTFDCbFxN+nCryrn1S pyoLa52uXLy1CzmpeWCmqkEYwu49G36cxd5Rrg+p+V2GmeJo11WFoA+JktBMD3/FVUyT lm35byUiFQWMZYbUBg2Txenqfwev1Qyn5EzOWm+cCcJK46+cVgw5exJRhsBQr2zD7wiC FOmVBx+cAlY9TNEExNGhcpIsW5SHmmviFnMeoBwXUwD090Gf46MSVMEwf0f3Aux5K6mA F72SgT3I1SVjrI2J/OglLKedY6oqeKb82XtdRT9aWnIITHyN4CghVmS9DDjpgLOyQCSW i1fQ==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=AqQYfp7q; arc=pass (i=1 spf=pass spfdomain=quicinc.com dkim=pass dkdomain=quicinc.com dmarc=pass fromdomain=quicinc.com); spf=pass (google.com: domain of linux-kernel+bounces-46285-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-46285-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com X-Forwarded-Encrypted: i=1; AJvYcCWfkYbaNMKb0Jhl/i75xD9DCTA2CyroPy5As0zX3jFvikPfbOkGpS1TQMTGbvc5fxvSnT1f18dNg+r2W5r/V8aGdM5wjA== Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id k2-20020a636f02000000b005d79ebbdc0asi9495216pgc.334.2024.01.31.03.05.08 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 31 Jan 2024 03:05:08 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-46285-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@quicinc.com header.s=qcppdkim1 header.b=AqQYfp7q; arc=pass (i=1 spf=pass spfdomain=quicinc.com dkim=pass dkdomain=quicinc.com dmarc=pass fromdomain=quicinc.com); spf=pass (google.com: domain of linux-kernel+bounces-46285-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-46285-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=quicinc.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 6ACC4292140 for ; Wed, 31 Jan 2024 10:59:50 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 358616EB4A; Wed, 31 Jan 2024 10:58:28 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=quicinc.com header.i=@quicinc.com header.b="AqQYfp7q" Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id BD2E869DE1; Wed, 31 Jan 2024 10:58:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706698703; cv=none; b=XnHlTWH/vHWb6ef8PcU//xyD5sF+H6ew+ydFsY7BJWvdGaKdOzY4TCCoWqCw/cizay2mYXtrSiatAwY2SmU0cSm71gjlPiZWxMIgT+tNxvm9zS72N6GD1GlBaPXL6IXu3Irc8vhGzvOIfSv9intYSIUMZtwJqRcdigeLGDunENo= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706698703; c=relaxed/simple; bh=H905wCqIWJUsyRGzBSYpaX0GfJ3CrhQMXdLI/pNFYMo=; h=From:To:CC:Subject:Date:Message-ID:MIME-Version:Content-Type; b=PcwB0icoxiAdnHfflgtfNuFM+9Fbl615UGdc24YdLDwHDAoxRac1qTdeWjyK0it8mSuErzk9+Bfj1GB2KJQzIlX+Zb2IdFVx8rYIFIES5FMcma2aSJpLwCfoe1O9I4F75uoVBOSeD9lW8ISNVXDuY7XqX8qXYq53/Re5aTwjvf8= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=quicinc.com; spf=pass smtp.mailfrom=quicinc.com; dkim=pass (2048-bit key) header.d=quicinc.com header.i=@quicinc.com header.b=AqQYfp7q; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=quicinc.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=quicinc.com Received: from pps.filterd (m0279872.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.17.1.24/8.17.1.24) with ESMTP id 40VAIf1M018247; Wed, 31 Jan 2024 10:57:58 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=quicinc.com; h= from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding:content-type; s=qcppdkim1; bh=gF/M0cQ p/Po1gWJguFknxVmvecw9rwR313j3e4N2Jkg=; b=AqQYfp7qwNmijb0Lvfhvjpa 7xSiMjB9tYO02RFEKfljLKBnbfS6h4wDqeFmPOe4kgFRBlgaWhzvt5AM9wGgRV+A oAK3jH5z2A2nunXk+Z1lm9Zxz1K8n4J9BSNUc08JoG/HpckWdru2kCrBYLyziUkt gwk3FMCqQiEJs77iISEgRlCwVQ3QeE2RG/udY5c56jm7ZzLaWAFBB6MSRyxgN1u0 zrKglrHRDCC8yU5D113ukpbPPgQSQE6cQFQtc3bj59cGEPmZne93TnY9qIU6mlmV 9fBX4y2WVitXLiHOgJC5XbaFIgY7Zo9vK0s1G69cjTgQXyUlmneVK3KHyG28Wow= = Received: from nasanppmta05.qualcomm.com (i-global254.qualcomm.com [199.106.103.254]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 3vy9nk9m6p-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 31 Jan 2024 10:57:58 +0000 (GMT) Received: from nasanex01c.na.qualcomm.com (nasanex01c.na.qualcomm.com [10.45.79.139]) by NASANPPMTA05.qualcomm.com (8.17.1.5/8.17.1.5) with ESMTPS id 40VAvuIW016952 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 31 Jan 2024 10:57:56 GMT Received: from hu-mojha-hyd.qualcomm.com (10.80.80.8) by nasanex01c.na.qualcomm.com (10.45.79.139) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1118.40; Wed, 31 Jan 2024 02:57:51 -0800 From: Mukesh Ojha To: , , , , , , , , , , , , , , , CC: , , Mukesh Ojha Subject: [PATCH v8 00/10] Add Qualcomm APSS Minidump driver related support Date: Wed, 31 Jan 2024 16:27:24 +0530 Message-ID: <20240131105734.13090-1-quic_mojha@quicinc.com> X-Mailer: git-send-email 2.43.0.254.ga26002b62827 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-ClientProxiedBy: nasanex01a.na.qualcomm.com (10.52.223.231) To nasanex01c.na.qualcomm.com (10.45.79.139) X-QCInternal: smtphost X-Proofpoint-Virus-Version: vendor=nai engine=6200 definitions=5800 signatures=585085 X-Proofpoint-GUID: PUi_W_nsurCBr3h4uii5OzuK-5sYcCLK X-Proofpoint-ORIG-GUID: PUi_W_nsurCBr3h4uii5OzuK-5sYcCLK X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26 definitions=2024-01-31_06,2024-01-30_01,2023-05-22_02 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 bulkscore=0 priorityscore=1501 impostorscore=0 lowpriorityscore=0 spamscore=0 mlxscore=0 malwarescore=0 suspectscore=0 mlxlogscore=999 clxscore=1015 adultscore=0 phishscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.19.0-2401190000 definitions=main-2401310083 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1789603724339022295 X-GMAIL-MSGID: 1789603724339022295 Abstract and PDF here: https://lpc.events/event/17/contributions/1468/ Video: https://www.youtube.com/watch?v=3vL3gtAu84s Patch 1 deals in detail documentation on minidump. Patch 2-4 refactors minidump existing layout and separate it from remoteproc files. Patch 6 is the Qualcomm APSS minidump driver. Patch 7-10 Enable support to reserve dynamic ramoops and the support to register ramoops region with minidump. Detail about Minidump is discussed in documentation patch (1/10) and also briefly discussed after below changelog. Changes in v8: - Addressed documentation comment made by Randy Dunlap. - Rebased on linux-next tag next-20240130 Changes in v7: - Addressed comment made by [Pavan.K] to use generic notifiers. - Addresses comment made on Dynamic ramoops about error handling. - Significant change minidump documentation suggested by [Bryan O'Donoghue] - Added Reviewed by from [Bagas] - Renamed ramoops notifiers. Changes in v6: https://lore.kernel.org/lkml/1700864395-1479-1-git-send-email-quic_mojha@quicinc.com/ - Accumalated the feedback received on v5 and rebase v5 versions in v6. - Removed the exported function as there is no current users of them. - Applied [Pavan.K] suggestion on caller/callee placement of dynamic ramoops reserve memory. - Addressed [krzysztof] comment on sizeof() and to have qcom_apss_md_table_exit(). - Addressed [Bagas.S] comment on minidump doc. - Tried to implement [Kees] suggestion in slight different way with callback registration with ramoops instead of pstore core. Change in rebase v5: https://lore.kernel.org/lkml/1694429639-21484-1-git-send-email-quic_mojha@quicinc.com/ - Rebased it on latest tag available on linux-next - Added missed Poovendhan sign-off on 15/17 and tested-by tag from Kathiravan. Thanks to him for testing and reminding me of missing sign-off. Changes in v5: https://lore.kernel.org/lkml/1694290578-17733-1-git-send-email-quic_mojha@quicinc.com/ - On suggestion from Pavan.k, to have single function call for minidump collection from remoteproc driver, separated the logic to have separate minidump file called qcom_rproc_minidump.c and also renamed the function from qcom_minidump() to qcom_rproc_minidump(); however, dropped his suggestion about rework on lazy deletion during region unregister in this series, will pursue it in next series. - To simplify the minidump driver, removed the complication for frontend and different backend from Greg suggestion, will pursue this once main driver gets mainlined. - Move the dynamic ramoops region allocation from Device tree approach to command line approch with the introduction command line parsing and memblock reservation during early boot up; Not added documentation about it yet, will add if it gets positive response. - Exporting linux banner from kernel to make minidump build also as module, however, minidump is a debug module and should be kernel built to get most debug information from kernel. - Tried to address comments given on dload patch series. Changes in v4: https://lore.kernel.org/lkml/1687955688-20809-1-git-send-email-quic_mojha@quicinc.com/ - Redesigned the driver and divided the driver into front end and backend (smem) so that any new backend can be attached easily to avoid code duplication. - Patch reordering as per the driver and subsystem to easier review of the code. - Removed minidump specific code from remoteproc to minidump smem based driver. - Enabled the all the driver as modules. - Address comments made on documentation and yaml and Device tree file [Krzysztof/Konrad] - Address comments made qcom_pstore_minidump driver and given its Device tree same set of properties as ramoops. [Luca/Kees] - Added patch for MAINTAINER file. - Include defconfig change as one patch as per [Krzysztof] suggestion. - Tried to remove the redundant file scope variables from the module as per [Krzysztof] suggestion. - Addressed comments made on dload mode patch v6 version https://lore.kernel.org/lkml/1680076012-10785-1-git-send-email-quic_mojha@quicinc.com/ Changes in v3: https://lore.kernel.org/lkml/1683133352-10046-1-git-send-email-quic_mojha@quicinc.com/ - Addressed most of the comments by Srini on v2 and refactored the minidump driver. - Added platform device support - Unregister region support. - Added update region for clients. - Added pending region support. - Modified the documentation guide accordingly. - Added qcom_pstore_ramdump client driver which happen to add ramoops platform device and also registers ramoops region with minidump. - Added download mode patch series with this minidump series. https://lore.kernel.org/lkml/1680076012-10785-1-git-send-email-quic_mojha@quicinc.com/ Changes in v2: https://lore.kernel.org/lkml/1679491817-2498-1-git-send-email-quic_mojha@quicinc.com/ - Addressed review comment made by [quic_tsoni/bmasney] to add documentation. - Addressed comments made by [srinivas.kandagatla] - Dropped pstore 6/6 from the last series, till i get conclusion to get pstore region in minidump. - Fixed issue reported by kernel test robot. Changes in v1: https://lore.kernel.org/lkml/1676978713-7394-1-git-send-email-quic_mojha@quicinc.com/ Minidump is a best effort mechanism to collect useful and predefined data for first level of debugging on end user devices running on Qualcomm SoCs. It is built on the premise that System on Chip (SoC) or subsystem part of SoC crashes, due to a range of hardware and software bugs. Qualcomm devices in engineering mode provides a mechanism for generating full system ramdumps for post mortem debugging. But in some cases it's however not feasible to capture the entire content of RAM. The minidump mechanism provides the means for selecting which snippets should be included in the ramdump. The core of SMEM based minidump feature is part of Qualcomm's boot firmware code. It initializes shared memory (SMEM), which is a part of DDR and allocates a small section of SMEM to minidump table i.e also called global table of content (G-ToC). Each subsystem (APSS, ADSP, ...) has their own table of segments to be included in the minidump and all get their reference from G-ToC. Each segment/region has some details like name, physical address and it's size etc. and it could be anywhere scattered in the DDR. Existing upstream Qualcomm remoteproc driver[1] already supports SMEM based minidump feature for remoteproc instances like ADSP, MODEM, ... where predefined selective segments of subsystem region can be dumped as part of coredump collection which generates smaller size artifacts compared to complete coredump of subsystem on crash. [1] https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/drivers/remoteproc/qcom_common.c#n142 In addition to managing and querying the APSS minidump description, the Linux driver maintains a ELF header in a segment. This segment gets updated with section/program header whenever a new entry gets registered. Support for Minidump enablement on Qualcomm SoCs is pursued separately and can be done via below series of changes. For testing, these patches can be applied https://lore.kernel.org/lkml/1704727654-13999-1-git-send-email-quic_mojha@quicinc.com/ https://lore.kernel.org/lkml/20240109153200.12848-12-quic_mojha@quicinc.com/ https://lore.kernel.org/lkml/20240109153200.12848-13-quic_mojha@quicinc.com/ Testing of these patches has been done on sm8450 target after enabling kernel config like CONFIG_PSTORE_RAM/CONFIG_PSTORE_CONSOLE and once the device boots up. Below command can be executed from sysfs to enable minidump in the firmware. echo mini > /sys/module/qcom_scm/parameters/download_mode Try crashing it via devmem2 0xf11c000(this is known command to create xpu violation and put the device crash dump mode) on command prompt. Default storage type is set to via USB, so Minidump would be downloaded with the help of x86_64 machine (running PCAT tool) attached to Qualcomm product which has backed Minidump boot firmware support. After that we will see a bunch of predefined registered region as binary blobs files starts with md_* downloaded on the x86 machine at configured/default location in PCAT tool from the product, more about this can be found in qualcomm minidump guide patch. Mukesh Ojha (10): docs: qcom: Add qualcomm minidump guide soc: qcom: Add qcom_rproc_minidump module remoteproc: qcom_q6v5_pas: Use qcom_rproc_minidump() remoteproc: qcom: Remove minidump related data from qcom_common.c init: export linux_banner data variable soc: qcom: Add Qualcomm APSS minidump kernel driver MAINTAINERS: Add entry for minidump related files pstore/ram: Add dynamic ramoops region support through commandline pstore/ram: Add ramoops information notifier support soc: qcom: register ramoops region with APSS minidump Documentation/admin-guide/index.rst | 1 + Documentation/admin-guide/qcom_minidump.rst | 318 +++++++++ Documentation/admin-guide/ramoops.rst | 23 +- MAINTAINERS | 10 + drivers/remoteproc/Kconfig | 1 + drivers/remoteproc/qcom_common.c | 160 ----- drivers/remoteproc/qcom_q6v5_pas.c | 3 +- drivers/soc/qcom/Kconfig | 23 + drivers/soc/qcom/Makefile | 2 + drivers/soc/qcom/qcom_minidump.c | 690 ++++++++++++++++++++ drivers/soc/qcom/qcom_minidump_internal.h | 74 +++ drivers/soc/qcom/qcom_rproc_minidump.c | 111 ++++ drivers/soc/qcom/smem.c | 20 + fs/pstore/Kconfig | 15 + fs/pstore/ram.c | 180 ++++- include/linux/init.h | 3 + include/linux/pstore_ram.h | 20 + include/linux/soc/qcom/smem.h | 2 + include/soc/qcom/qcom_minidump.h | 41 ++ init/main.c | 3 + init/version-timestamp.c | 3 + 21 files changed, 1538 insertions(+), 165 deletions(-) create mode 100644 Documentation/admin-guide/qcom_minidump.rst create mode 100644 drivers/soc/qcom/qcom_minidump.c create mode 100644 drivers/soc/qcom/qcom_minidump_internal.h create mode 100644 drivers/soc/qcom/qcom_rproc_minidump.c create mode 100644 include/soc/qcom/qcom_minidump.h base-commit: 41d66f96d0f15a0a2ad6fa2208f6bac1a66cbd52