Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Fri, 15 Jul 2022 17:38:36 -0400
From:   Rodrigo Vivi <rodrigo.vivi@intel.com>
To:     Mauro Carvalho Chehab <mchehab@kernel.org>
CC:     Alan Previn <alan.previn.teres.alexis@intel.com>,
        David Airlie <airlied@linux.ie>,
        <intel-gfx@lists.freedesktop.org>,
        Lucas De Marchi <lucas.demarchi@intel.com>,
        <linux-kernel@vger.kernel.org>, <dri-devel@lists.freedesktop.org>
Subject: Re: [Intel-gfx] [PATCH v2 13/39] drm/i915: i915_gpu_error.c:
 document dump_flags
Message-ID: <YtHeXF2lG9kyQAQP@intel.com>
References: <cover.1657699522.git.mchehab@kernel.org>
 <bbebefd789a7038661783ceb13fad8fdbd440f54.1657699522.git.mchehab@kernel.org>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <bbebefd789a7038661783ceb13fad8fdbd440f54.1657699522.git.mchehab@kernel.org>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?2Sw1i7FQP0rZs0IX77yUZsY9AQkwDoS1JI0XgYDbmk8XNRc3Pxor/UyWALKb?=
 =?us-ascii?Q?i3qxjTzLbtDhbWZsEGtOMco712Xs64+Vtlesp9pQrRf3n60C8MEcMksedJVO?=
 =?us-ascii?Q?vT4z45WARkiGEnXCRsYGBp+QOfRZ48XF0cV9VC0XBnOKaTYuVn7hYChR0fxK?=
 =?us-ascii?Q?EogML8fSrNLt0dn7xi6Wk0fdLvOV/pEn5LemCNmrf2UdZ3EiLfkm0WVRCFQt?=
 =?us-ascii?Q?0REtrXf2oE/u27nrd0Bzk/96pxHpOPYDUOrTRtgJRr3pd3LByhLlgmku6ne3?=
 =?us-ascii?Q?11SvZpxLiz9ptxHFnDAAnaQja7DTFsn48veP27qP6yZDh+n5GbtqU5rl3JNE?=
 =?us-ascii?Q?dAeQ0M4IAXOqvHPRqdK7KgrjzSoGQeYiXvItDxevjka4SQJk7T3X68p4G/f9?=
 =?us-ascii?Q?jPfRycb/t6uS8P7qj7BVRjNZRi/ywFfdeDBQrrsyQ7e8tepJJp1+oaBoyb9q?=
 =?us-ascii?Q?INswu24an/RXSQ10OFVmJ1QEoEZNzb3PPi3EPAyv3S2C8Hlq7AYbW8XWaDec?=
 =?us-ascii?Q?NaFhlH8Ss4DWXXBiD3D5Wt1xM2J7pnowFjb62/fEoNGljvo3bxpMThZMPhXe?=
 =?us-ascii?Q?ijckU/SQb47c7KDlK9t/bfjadynI7SkMtXp+ODx5/KtGN0WUVY9r9vv+OJbD?=
 =?us-ascii?Q?cgl/XBY3bFM7RoevLc9nYjSYPrGjgOzjI5FgoCHZ7sDG6k0E7tO4Qk0QrgjL?=
 =?us-ascii?Q?r9xt2kVt7KbCUNSSTiBL29/jhiOmIMK6IllOHD7J3BdnBwmi1IG9jBr1SC9M?=
 =?us-ascii?Q?cXC5pAyen0k9i3E/ox4BvMra7lZI6hoXi+gNM4lvc37XviTVclfu/DzWpm4e?=
 =?us-ascii?Q?SnbI7w3uKNaolpULIYKMj8YVCoQFn6q+gYK6U6dOScWfh6d9yBz00E7Tr633?=
 =?us-ascii?Q?GM7Bo41W4Foiu7t4NXSU6hijVg5a67H6JfN7vUgMg4B+4kP89zzdlCeo8oPL?=
 =?us-ascii?Q?56LJl6cy+u6txMlJe2XIabWwlWdPtOoDV8TSUfm2Xvya0YZZI9jBgZ6LJiFJ?=
 =?us-ascii?Q?iWweAwn1kn/7XAOOdoPNUaR12A1FB7F0WlATMEaPlzUI1hRlrm7E6fsFTxEh?=
 =?us-ascii?Q?m+sUw+MbU7k3asyVJ2vcuLFlPyDb75gbV59OMwQ6zN1QFN9QN0tqYn+jQM9a?=
 =?us-ascii?Q?D4WIbqz6BgO6kdlnmfFPM4dRKj8mLqb2LsxSxgwYZxrdnNEXVfhEU5T4kVci?=
 =?us-ascii?Q?tD9X9jDlRRVmiEP64CTRNFKMnfukgTsGZBaAaUKiSqYuQJqFz170DmmFKMHE?=
 =?us-ascii?Q?SfiUwvO0ZH+B3U9VuxdfbAerEW2R7unKnW60hiO6uzACxRU09C94MKHn09vE?=
 =?us-ascii?Q?bUvxtHqgQ18PozCn38jpagYezggZF/pJbsdyyJp/Xqobap6lbuy3P6PoEAop?=
 =?us-ascii?Q?n7L9kvVQZVdqLxUQSZaK+woPEwblsH60W9DpPl7tQ9usPfmjffGYkj77guPU?=
 =?us-ascii?Q?mBOjZovV3fpAYa+HRLyoVkEpiw5XO7KXjCKAIx3N/OfoDIZ0OuzGuPzw1L3l?=
 =?us-ascii?Q?hrGazf0hZa33y11gW+2MaGnJoun3ebwqmwRLXgr+wWhCpr3QPDa3lvzTlnAc?=
 =?us-ascii?Q?qsdEDw8D6IQfYnt2u1eWdiqDrdmpBnLaGCVUTENnetPY0HjoSwlZquPWGSQZ?=
 =?us-ascii?Q?cA=3D=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: bc07a4b7-e819-48c7-ed5d-08da66aa6271
X-MS-Exchange-CrossTenant-AuthSource: MN0PR11MB6059.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Jul 2022 21:38:41.0640
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: BOJPJSVvISkhpCFi9LdxU/hdxMV36NLVcDWPb14vlvNqV+GFcH3afz35ximaM2EAMcBWOf8gVVEsWZ31PLAcfw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB4707
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-5.0 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_MED,
        SPF_HELO_NONE,SPF_NONE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, Jul 13, 2022 at 09:12:01AM +0100, Mauro Carvalho Chehab wrote:
> Kernel-doc dump_flags parameter is missing at i915_capture_error_state().
> Document it.
> 
> Fixes: a6f0f9cf330a ("drm/i915/guc: Plumb GuC-capture into gpu_coredump")

Why the fix here and not in the i915_vma_destroy_locked one?
That was also a new introduction. Should we bother with the Fixes
anyway since there were many broken already? And if we do, shouldn't
we add to the others?

But anyway, just trying to understand the differences and reasons,
because the patch is correct:


Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>


> Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
> ---
> 
> To avoid mailbombing on a large number of people, only mailing lists were C/C on the cover.
> See [PATCH v2 00/39] at: https://lore.kernel.org/all/cover.1657699522.git.mchehab@kernel.org/
> 
>  drivers/gpu/drm/i915/i915_gpu_error.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_gpu_error.c b/drivers/gpu/drm/i915/i915_gpu_error.c
> index 32e92651ef7c..7790f97b26db 100644
> --- a/drivers/gpu/drm/i915/i915_gpu_error.c
> +++ b/drivers/gpu/drm/i915/i915_gpu_error.c
> @@ -2096,7 +2096,8 @@ void i915_error_state_store(struct i915_gpu_coredump *error)
>   * i915_capture_error_state - capture an error record for later analysis
>   * @gt: intel_gt which originated the hang
>   * @engine_mask: hung engines
> - *
> + * @dump_flags: bitmap flags. When %CORE_DUMP_FLAG_IS_GUC_CAPTURE is used,
> + *	dump engine record registers and execlists.
>   *
>   * Should be called when an error is detected (either a hang or an error
>   * interrupt) to capture error state from the time of the error.  Fills
> -- 
> 2.36.1
>