From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f176.google.com (mail-pg1-f176.google.com [209.85.215.176]) (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 EAFC03242D0 for ; Thu, 20 Nov 2025 17:51:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.176 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763661069; cv=none; b=BJR4I3zSHqOdVxZ/m3IWE4RE78a2QHo50NUg3zZOlePL6KYSuZULuVrteQZxG2hLBs4nbkT/+DIjyd2xrJjDWfBxYFD5iiAUCttC82ahzAyAEsUYWw7nV29Cd/tFygasZcj4zgNb2OM1NHv9MNRznOJJHaUV6o1zO56fivKVQtw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763661069; c=relaxed/simple; bh=zD7NgsDEUXwSzRf4wlPfCqiqHrm9X7LwU1FRDzbje5o=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=P1QsIu3Elqkk+fKrrQ+eAz5l1fN3CtqnK1wI3lFrysumYOcLKUv7j6MR2DhfqaDmNt9i2gwO+M/VMH4TJN0mqCtfeItguDt4boI+MzHt5c8fjVh/3X2gdnaIimgXdKunS+7hjmyA4FAEhkvIA+DSgkUHkaLC6L/ClUw3ikU2AmA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=hfFmInOp; arc=none smtp.client-ip=209.85.215.176 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="hfFmInOp" Received: by mail-pg1-f176.google.com with SMTP id 41be03b00d2f7-bcfd82f55ebso1251934a12.1 for ; Thu, 20 Nov 2025 09:51:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1763661061; x=1764265861; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=SvyQOcIQQ86C542mLC2IQK954GH3/Pv3kj39BklXd04=; b=hfFmInOpbCBMKVDGugkn6iMnE13/FpUqFH9aDkOZMKL18kT82X0s+R2RdXJi3ha6Xk buHuyyG7xm0Uv0rXYJQmrbjxN0eF5Fu22t2VCjbf0awNObS3zK4ydtOxQUH7HIbTsxLk 8b2FPZgJKa5FBKXFjQ38XWqorj3wo6c5vTHak8+7bnrXeNRFwPQtDuapIU9s9iaxUc7+ UdpagSCjLlUqHFGxIMdRYZJoWBPX2hkj2So3miDSj48n8XuFkXiI8WH0JIc/owRGHcjf GfWna8x0+Mjq1G9B2K9CgjXNjZwhYp2IlLG9AZGmZvRi6LOnKAqRrfbbJ/tqgv5d2Y39 U+ow== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763661061; x=1764265861; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=SvyQOcIQQ86C542mLC2IQK954GH3/Pv3kj39BklXd04=; b=AgOOe88K7Un0SsEHqUCwnB90j8jQm2hYiBDrQtI28IHZf/joYFz4rrQ6s/WpYnfbER ICS4MwzKovoch6sejkS0st6lFQlNZi2mUx4G+NGlAo/+n06OWL3Dj15+t+as9CVvYDQV bq6pLkcLtiU6Yj2xVZgP3YKOhy0QwtPM23B8B002JkSbzKlPGMNCmfCxt3gSBOsBtg6e HsbFOhETb4Z7cLwTbRz1UTtGJ1MxFNK4oFed0dujaV8UGket0nS50ATQUINLd5+Fy2d2 Y9eENyU0qhqeo0CysW4YpRA3Hn0qR34lOKxIwMFG/Am+hkH7i8rYL+EYX214uvGUMUGX u7mw== X-Forwarded-Encrypted: i=1; AJvYcCW9PtMVtu1DTQHFYxJDInxWlNRJFwL32hm0B8LdJbh5gFO6bWB+vfXvCWuBMst3ffE4c0kq2Txei6aIEcs=@vger.kernel.org X-Gm-Message-State: AOJu0YyruXsgQfdn3jJmKwK9voy63ZHLzK4CVOqKdnDOsiPlfuzOPuNg kuUSK2JRhF0Jc7TjVUXlKp9ZOVTwPBcEImPEmnz3cWxuDaAlGHPCzHTmZT+KVO83FGU= X-Gm-Gg: ASbGnct6ovR2h4G3MmzSf9tkDjqesHS274hEXT9EMIwNzJC+PcxTGAIbQ2rhG0sbLPT IZ7aubaQKAybgloMm/CEvpKvMSmphpPiOF3MX5oakOPqMV9QQStG59z+XBgolqN3MMv7NYR4BP6 chpvWrgXuHbvIR7azY+hn3CopJzLjFZgX7O2FY0plgDJfX8urTAXKzwSdWPcbkW3CSJWJtACEha RZDctu8iPU9RxnTw3GXDUPJ0DrSnO0eRxptzQHucS8XNXCH7IVfPu9gy3dKdaz4Ll76sYrbk8kK q+HCEcXY92Sfz4jwqP70QPIIrRYJTMEVSAbNhHbLIjInGjz3672WtKF+y/mqykBWa0gavlxmLK2 GTQptWToDT0Q5x47CCFlVO1Ec2tac/7P0DCcKpc1FE3ZvTvfqkkK8p8NQbzPICKRwQqtNgwbFfT 5lFr1240Zh0bjc3g== X-Google-Smtp-Source: AGHT+IG5nfGUp6w8q9EcQONf815YWnnfwp5npwGR40rOPHk9oLxe0qBxjPkC/6purKVN5bANUXYjjw== X-Received: by 2002:a17:90b:5144:b0:343:e692:f8d7 with SMTP id 98e67ed59e1d1-34729867d6dmr3904935a91.11.1763661061313; Thu, 20 Nov 2025 09:51:01 -0800 (PST) Received: from p14s ([2604:3d09:148c:c800:17b0:cde9:c3a1:870b]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-34726696bb2sm3242515a91.3.2025.11.20.09.51.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 20 Nov 2025 09:51:00 -0800 (PST) Date: Thu, 20 Nov 2025 10:50:58 -0700 From: Mathieu Poirier To: Tanmay Shah Cc: andersson@kernel.org, linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2 1/3] remoteproc: xlnx: enable boot recovery Message-ID: References: <20251113154403.2454319-1-tanmay.shah@amd.com> <20251113154403.2454319-2-tanmay.shah@amd.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20251113154403.2454319-2-tanmay.shah@amd.com> Good morning, On Thu, Nov 13, 2025 at 07:44:02AM -0800, Tanmay Shah wrote: > This is the default method to recover the remote processor from crash. > During this recovery the Linux will stop the remote, load the same > firmware again and start the remote processor. As of now, coredump > callback does not contain any useful implementation, but this can be > changed as required. > > Signed-off-by: Tanmay Shah > --- > drivers/remoteproc/xlnx_r5_remoteproc.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/remoteproc/xlnx_r5_remoteproc.c b/drivers/remoteproc/xlnx_r5_remoteproc.c > index 0b7b173d0d26..8677b732ad14 100644 > --- a/drivers/remoteproc/xlnx_r5_remoteproc.c > +++ b/drivers/remoteproc/xlnx_r5_remoteproc.c > @@ -891,6 +891,11 @@ static int zynqmp_r5_detach(struct rproc *rproc) > return 0; > } > > +static void zynqmp_r5_coredump(struct rproc *rproc) > +{ > + (void)rproc; > +} > + Function rproc_coredump(), which is set by default in rproc_alloc_ops(), won't work? If not please indicate why this is the case, otherwise this patch can be dropped. > static const struct rproc_ops zynqmp_r5_rproc_ops = { > .prepare = zynqmp_r5_rproc_prepare, > .unprepare = zynqmp_r5_rproc_unprepare, > @@ -905,6 +910,7 @@ static const struct rproc_ops zynqmp_r5_rproc_ops = { > .get_loaded_rsc_table = zynqmp_r5_get_loaded_rsc_table, > .attach = zynqmp_r5_attach, > .detach = zynqmp_r5_detach, > + .coredump = zynqmp_r5_coredump, > }; > > /** > @@ -938,7 +944,7 @@ static struct zynqmp_r5_core *zynqmp_r5_add_rproc_core(struct device *cdev) > > rproc_coredump_set_elf_info(r5_rproc, ELFCLASS32, EM_ARM); > > - r5_rproc->recovery_disabled = true; > + r5_rproc->recovery_disabled = false; > r5_rproc->has_iommu = false; > r5_rproc->auto_boot = false; > r5_core = r5_rproc->priv; > -- > 2.34.1 >