From patchwork Tue May 2 09:23:27 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mads Ynddal X-Patchwork-Id: 13228616 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E10B5C77B7E for ; Tue, 2 May 2023 09:24:57 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1ptmEk-0004zL-Fq; Tue, 02 May 2023 05:23:54 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1ptmEi-0004yj-V6 for qemu-devel@nongnu.org; Tue, 02 May 2023 05:23:52 -0400 Received: from mr85p00im-ztdg06021701.me.com ([17.58.23.196]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1ptmEg-0003yw-Vj for qemu-devel@nongnu.org; Tue, 02 May 2023 05:23:52 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ynddal.dk; s=sig1; t=1683019426; bh=FJhwtHJ3YSQC3owUOiP3HBOSdyBnuMHJV0J7ilgQUxo=; h=From:To:Subject:Date:Message-Id:MIME-Version:Content-Type; b=Xz/SrfB6rjfOEJWy+MWpnuSh8GZid1nPpQdgtYtcgIYETi93sB6WCRgzTrauztSPM K0g4Oo2/HmrEA662ssp4UmpQ9GAaJ5EbmUoI/VzyKHpgR9Dk7MEffm/er1Ceid51tN QQl/8loS2zmXmqKzKv8aFJwYXYasalSS64QWqUKRQeah0EfIFcg7lil52zgu73WQOC QZJUN6w5pfrNvVlms5FWXMd9lq+jKOBafiRApjRuUqSQXuppDPbmeo+4USKhZdH/gK D4PdMT5MUS+zMya6cWMUgFBwSpm8nh2naFi1S8Lw9cI7KpYJN7yTWOgoCFeLqv2Gbf SzEB2zhJpYxeg== Received: from localhost.localdomain (mr38p00im-dlb-asmtp-mailmevip.me.com [17.57.152.18]) by mr85p00im-ztdg06021701.me.com (Postfix) with ESMTPSA id EB92526338CB; Tue, 2 May 2023 09:23:43 +0000 (UTC) From: Mads Ynddal To: qemu-devel@nongnu.org Cc: Cleber Rosa , Stefan Hajnoczi , John Snow , Mads Ynddal , Mads Ynddal Subject: [PATCH v2 00/12] simpletrace: refactor and general improvements Date: Tue, 2 May 2023 11:23:27 +0200 Message-Id: <20230502092339.27341-1-mads@ynddal.dk> X-Mailer: git-send-email 2.38.1 MIME-Version: 1.0 X-Proofpoint-GUID: kSBKuisdwSznXxf2Fwb6Q72A5h9zD8_m X-Proofpoint-ORIG-GUID: kSBKuisdwSznXxf2Fwb6Q72A5h9zD8_m X-Proofpoint-Virus-Version: =?utf-8?q?vendor=3Dfsecure_engine=3D1=2E1=2E170-?= =?utf-8?q?22c6f66c430a71ce266a39bfe25bc2903e8d5c8f=3A6=2E0=2E425=2C18=2E0?= =?utf-8?q?=2E816=2C17=2E11=2E62=2E513=2E0000000_definitions=3D2022-01-18=5F?= =?utf-8?q?01=3A2022-01-14=5F01=2C2022-01-18=5F01=2C2021-12-02=5F01_signatur?= =?utf-8?q?es=3D0?= X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 adultscore=0 spamscore=0 phishscore=0 clxscore=1030 mlxlogscore=369 bulkscore=0 malwarescore=0 mlxscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2212070000 definitions=main-2305020080 Received-SPF: pass client-ip=17.58.23.196; envelope-from=mads@ynddal.dk; helo=mr85p00im-ztdg06021701.me.com X-Spam_score_int: -27 X-Spam_score: -2.8 X-Spam_bar: -- X-Spam_report: (-2.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_MSPIKE_H2=-0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Mads Ynddal I wanted to use simpletrace.py for an internal project, so I tried to update and polish the code. Some of the commits resolve specific issues, while some are more subjective. I've tried to divide it into commits so we can discuss the individual changes, and I'm ready to pull things out, if it isn't needed. v2: * Added myself as maintainer of simpletrace.py * Improve docstring on `process` * Changed call to `process` in scripts/analyse-locks-simpletrace.py to reflect new argument types * Replaced `iteritems()` with `items()` in scripts/analyse-locks-simpletrace.py to support Python 3 Mads Ynddal (12): simpletrace: Improve parsing of sys.argv; fix files never closed. simpletrace: Annotate magic constants from QEMU code simpletrace: changed naming of edict and idtoname to improve readability simpletrace: update code for Python 3.11 simpletrace: Changed Analyzer class to become context-manager simpletrace: Simplify construction of tracing methods simpletrace: Improved error handling on struct unpack simpletrace: define exception and add handling simpletrace: Refactor to separate responsibilities MAINTAINERS: add maintainer of simpletrace.py scripts/analyse-locks-simpletrace.py: changed iteritems() to items() scripts/analyse-locks-simpletrace.py: reflect changes to process in simpletrace.py MAINTAINERS | 8 +- scripts/analyse-locks-simpletrace.py | 5 +- scripts/simpletrace.py | 307 ++++++++++++--------------- 3 files changed, 150 insertions(+), 170 deletions(-)