[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/4] xen/scripts: add cppcheck tool to the xen-analysis.py script
- To: Stefano Stabellini <sstabellini@xxxxxxxxxx>, Luca Fancellu <Luca.Fancellu@xxxxxxx>
- From: Jan Beulich <jbeulich@xxxxxxxx>
- Date: Thu, 1 Dec 2022 09:33:45 +0100
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=qeQ8/Wy2RogStjqmjqkyo67HpC4bAaSGIO5p4rpEzd8=; b=WTCV6IGCT7cC+7AgqQA9fklQMpCCjJaQE9IpvOZJxIPWCIyHCvvi+mDdlSLWT/ed0NrZY63a3go2RIbY+XuK6CEa3Z35FqbIMgsvxaSegoUu0yf+PasIMi5UWvq//KnMhMs7/ZD42dimWVveQ6Nx1b4yDoI5+SID4roNkcMgkxV+kA6YyIzB/f4hmG6VYJCcF37TtNJ4r0wY5Id0zSeK92UUFGA5F87ST+jijeX396yMJhquRfH2TC3BNqBBum8SSZ5+xxTYvWcoqdgR/+6Aql7NTyA7t5dr3dt6w6BeCwzOHmgstOicYtgf03sWIK9tKiHv4rudfWFWyS8HgrgOZg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=fdTeV6eLRqBzumJYMzjhJGGmNySKy++UW4zJKV4px1SVnSUd9N5UMKBRJOgZ/fZzXJ0bEliM6mW4bLyxl7/nEID9lbxKKDRAAARLEi0SrmcnW8gWM3FhJfj/bnwoOTD8/9yGekBRitztAiuniINaAzcK6MlJnF6Qg6ybmEOqaNNNvwSHH16rJXxnDDEuLbenDvmezpf9gHsOs9bcwFcyGUqlo/wYxmHKmUGin3oZOwAV2O+Ua72HTGvYidiRPW2QPYX68DRyITbFbtLd9AqetonnuOLX5iuvNvPTyUNd8MiUkoBOTZGQqPBYY1/tBZNFJQO4slJ2LssXvqlDB3p3nw==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
- Cc: Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Bertrand Marquis <Bertrand.Marquis@xxxxxxx>, Wei Chen <Wei.Chen@xxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Wei Liu <wl@xxxxxxx>
- Delivery-date: Thu, 01 Dec 2022 08:34:06 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 30.11.2022 21:26, Stefano Stabellini wrote:
> On Wed, 30 Nov 2022, Luca Fancellu wrote:
>>> I think the revert of the cppcheck integration in xen/Makefile and
>>> xen/tools/merge_cppcheck_reports.py could be a separate patch. There is
>>> no need to make sure cppcheck support in the xen Makefile is
>>> "bisectable". That patch could have my acked-by already.
>>
>> Ok I will split these changes in a following patch
>>
>>>
>>> Also the document changes introduced in this patch have my reviewed-by:
>>> - docs/misra/cppcheck.txt
>>> - docs/misra/documenting-violations.rst
>>> - docs/misra/false-positive-cppcheck.json
>>> - docs/misra/xen-static-analysis.rst
>>
>> Thank you, should I put those files in a separate patch with your rev-by
>> before
>> this patch or this is just a comment for you to remember which file you
>> already
>> reviewed?
>
> If Jan and the other reviewers are OK, I think you could split them out
> in a separate patch and add my reviewed-by. If Jan prefers to keep it
> all together in one patch, then I wrote it down so that I remember what
> I have already acked :-)
Docs changes being split off and going in first is okay as long as what
is being documented is present behavior. If other changes are needed to
make (parts of) new documentation actually correct, then it should imo
go together. If new documentation describes future behavior (e.g.
design docs), then of course it's fine as well to go in early, as then
there simply is no code anywhere which this would (temporarily) not
describe correctly.
Jan
|
