From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 4621A2D030; Wed, 24 Jul 2024 14:35:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1721831746; cv=none; b=MgjKt39Pyz0/4GxfLiVZiWcg2Aq53p01pYEeR80J9hStZXP9TbuBO7w6JuGBfJpajF/Cc1YN6dloRydSyZAk+wTR5G51AX4WXK7DfFwhvBUWw00ihq5EFqPbvYwPk89v3WBoBCJmfAA8NVlFlw8iF/tDALoD2w1hlwneLJxtLXM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1721831746; c=relaxed/simple; bh=LwwOw1AlNJTo8GqoEQmTYOtHMBJNfExRitQaWurXlm4=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=J2n9LcQqNN2pwdXl3Mic2lfaB/DEG5TodV/yflG6yMWURVQqf4tGoVBIviPfnM8x/bZIRXqwFk2zfOHen84zJHgU/HtRvHHCLIRugdNAvDxwv6gov7AArAvsfQS8gazAkZpHX9cnYwJQ4amJlyXAfpFIbnMfx3xBTYd/wfhm/5A= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 16F73106F; Wed, 24 Jul 2024 07:36:10 -0700 (PDT) Received: from [10.57.44.253] (unknown [10.57.44.253]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 0B2E63F5A1; Wed, 24 Jul 2024 07:35:41 -0700 (PDT) Message-ID: Date: Wed, 24 Jul 2024 15:35:39 +0100 Precedence: bulk X-Mailing-List: rust-for-linux@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [RFC PATCH] drm: panthor: add dev_coredumpv support To: Daniel Almeida Cc: Rob Herring , Boris Brezillon , Wedson Almeida Filho , Miguel Ojeda , Danilo Krummrich , Lyude Paul , Asahi Lina , =?UTF-8?Q?Ma=C3=ADra_Canal?= , David Airlie , rust-for-linux@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org References: <20240710225011.275153-1-daniel.almeida@collabora.com> <4344B22F-D859-4C64-A351-69FFB5208362@collabora.com> <20240723180642.73502856@collabora.com> <6ce8fd12-b175-4a8f-8ea9-6221a555b69c@arm.com> <6b60a83d-8416-4cf8-b373-dc3c697c788a@arm.com> <1CF0978E-344C-4EC9-A221-4F19EBA4B988@collabora.com> From: Steven Price Content-Language: en-GB In-Reply-To: <1CF0978E-344C-4EC9-A221-4F19EBA4B988@collabora.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit On 24/07/2024 15:27, Daniel Almeida wrote: > Hi Steven! > >> On 24 Jul 2024, at 10:54, Steven Price wrote: >> >> [1] Although I have to admit for a debugging feature like devcoredump >> there might well be pressure to implement this in C as well purely so >> that customer issues can be debugged… > > FYI: I picked devcoredump because it was self-contained enough that I > could make a proof-of-concept and get the discussion started. I think > that, at least from this point of view, it has been successful, even if we > decide against a partial Rust driver! :) Indeed, thanks for posting this! It's provoked a good discussion. > I was informed early on that delaying a debugging feature until the > abstractions were merged would be a problem. Don’t worry: I can rewrite > the kernel part in C, that would indeed be a very small patch. I'll leave that for you to decide. There's definitely nothing blocking a patch like this in C, but equally I'm not aware of anyone desperate for this support yet. Thanks, Steve