From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: passt.top; dmarc=none (p=none dis=none) header.from=njh.eu Received: from mail.notjusthosting.com (mail.notjusthosting.com [IPv6:2a01:4f8:a0:516f:1::1]) by passt.top (Postfix) with ESMTPS id 3BC435A068E for ; Fri, 19 Sep 2025 02:59:30 +0200 (CEST) Received: from dynamic-2a02-3100-2f1d-5f00-7c36-44f3-0d18-800a.310.pool.telefonica.de ([2a02:3100:2f1d:5f00:7c36:44f3:d18:800a] helo=vlap) by mail.notjusthosting.com with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1uzPTF-0002c7-Lj; Fri, 19 Sep 2025 02:59:29 +0200 Date: Fri, 19 Sep 2025 02:59:29 +0200 From: Volker Diels-Grabsch To: Stefano Brivio Subject: Re: [PATCH v6 1/5] Message-ID: References: <20250916192209.3986375-1-v@njh.eu> <20250917163242.34096a32@elisabeth> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20250917163242.34096a32@elisabeth> Message-ID-Hash: AQ3TGRX7QE5VCRSUOESSUY2NO7Y3PNZA X-Message-ID-Hash: AQ3TGRX7QE5VCRSUOESSUY2NO7Y3PNZA X-MailFrom: v@njh.eu X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header CC: passt-dev@passt.top X-Mailman-Version: 3.3.8 Precedence: list List-Id: Development discussion and patches for passt Archived-At: Archived-At: List-Archive: List-Archive: List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: Stefano Brivio wrote: > On Tue, 16 Sep 2025 21:21:11 +0200 > Volker Diels-Grabsch wrote: > > > This incorporates all feedback of v5, and the commits are > > slightly reordered so the comment of the last commit makes > > more sense. > > Applied, thanks to sticking to this! I hope this will finally > make the QEMU disconnect/reconnect behaviour robust enough. As far as I can tell from my tests, locally as well as on production systems, this is pretty much the case. Thanks a lot for the opportunity to contribute some helpful improvements to passt! > A couple of notes for future changes: > > - you can add this kind of message as cover letter instead, just > git format-patch --cover-letter and git will format things as > needed, and concatenate In-Reply-To: and References: email > headers > > - reporting the version number in every subject line might make > reviewers' life marginally easier and it just takes a > --subject-prefix="PATCH v6" (in this case) or even something > like -v6 argument to git format-patch > > - carry Reviewed-by: tags if you don't... change the change. For > example, here, 4/5 was already reviewed by David. I added the > tag back (and will always do anyway) > > ...in any case, those are very minor details that don't really > cause me any trouble as a maintainer. I believe these would be great additions to the upcoming CONTRIBUTING.md, especially for people like myself who are used to workflows like Gerrit or pull requests (a.k.a. merge requests) and had no prior experience with the kernel-style email based workflow. (And moreover, the interactive interface of "git send-email" is far from self-explanatory and requires quite a lot of trial and error to get everything right, and usage mistakes are mostly impossible to fix, since the emails are then already sent.) Best regards, Volker -- .---<<<((()))>>>---. | [[||]] | '---<<<((()))>>>---'