From mboxrd@z Thu Jan 1 00:00:00 1970 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 46F49CAC59A for ; Thu, 18 Sep 2025 13:48:39 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1uzEzU-0003r7-3J; Thu, 18 Sep 2025 09:48:04 -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 1uzEzO-0003om-GN for qemu-devel@nongnu.org; Thu, 18 Sep 2025 09:47:58 -0400 Received: from smtp-out1.suse.de ([195.135.223.130]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1uzEzJ-0005EH-PI for qemu-devel@nongnu.org; Thu, 18 Sep 2025 09:47:55 -0400 Received: from imap1.dmz-prg2.suse.org (unknown [10.150.64.97]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id 02D073368A; Thu, 18 Sep 2025 13:47:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1758203272; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=kP8XdKzQ0b2vmyRGHlyJZDIvlvkpI8LH6Hp9KoTIuBk=; b=Ier0illpb/7MTRH+rEcDi20eO+SvAwBIaVM2CgWCXqBiBWBCnHZgOQPYLAOXDyaYks1NSr mOc64tJBHgQ7vFO6B+vGbEjYfpt/FibAAUufB0txLoYPeefTpFqu78QakCZN9iNiXMo5YP soWt4/QutMFkaeiMSsTRTCrSIO6g6yI= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1758203272; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=kP8XdKzQ0b2vmyRGHlyJZDIvlvkpI8LH6Hp9KoTIuBk=; b=7685oxMccnfnsuRr81skWy/Bmv7TB2snjO74eWBtbhd9RplPVgz4hWOOmX3130sBSmuiZP /b2toJQeQh09G/CA== Authentication-Results: smtp-out1.suse.de; none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1758203272; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=kP8XdKzQ0b2vmyRGHlyJZDIvlvkpI8LH6Hp9KoTIuBk=; b=Ier0illpb/7MTRH+rEcDi20eO+SvAwBIaVM2CgWCXqBiBWBCnHZgOQPYLAOXDyaYks1NSr mOc64tJBHgQ7vFO6B+vGbEjYfpt/FibAAUufB0txLoYPeefTpFqu78QakCZN9iNiXMo5YP soWt4/QutMFkaeiMSsTRTCrSIO6g6yI= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1758203272; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=kP8XdKzQ0b2vmyRGHlyJZDIvlvkpI8LH6Hp9KoTIuBk=; b=7685oxMccnfnsuRr81skWy/Bmv7TB2snjO74eWBtbhd9RplPVgz4hWOOmX3130sBSmuiZP /b2toJQeQh09G/CA== Received: from imap1.dmz-prg2.suse.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by imap1.dmz-prg2.suse.org (Postfix) with ESMTPS id 6F61013A51; Thu, 18 Sep 2025 13:47:51 +0000 (UTC) Received: from dovecot-director2.suse.de ([2a07:de40:b281:106:10:150:64:167]) by imap1.dmz-prg2.suse.org with ESMTPSA id omtYC4cNzGimNgAAD6G6ig (envelope-from ); Thu, 18 Sep 2025 13:47:51 +0000 From: Fabiano Rosas To: Peter Xu Cc: qemu-devel@nongnu.org, Juraj Marcin , =?utf-8?Q?Da?= =?utf-8?Q?niel_P_=2E_Berrang=C3=A9?= Subject: Re: [PATCH 0/3] migration/tls: Graceful shutdowns for main and postcopy channels In-Reply-To: References: <20250910160144.1762894-1-peterx@redhat.com> <87tt10vlvb.fsf@suse.de> Date: Thu, 18 Sep 2025 10:47:48 -0300 Message-ID: <87ikhfvpmj.fsf@suse.de> MIME-Version: 1.0 Content-Type: text/plain X-Spamd-Result: default: False [-4.30 / 50.00]; BAYES_HAM(-3.00)[100.00%]; NEURAL_HAM_LONG(-1.00)[-1.000]; NEURAL_HAM_SHORT(-0.20)[-1.000]; MIME_GOOD(-0.10)[text/plain]; ARC_NA(0.00)[]; MISSING_XM_UA(0.00)[]; RCVD_TLS_ALL(0.00)[]; MIME_TRACE(0.00)[0:+]; RCVD_VIA_SMTP_AUTH(0.00)[]; TO_DN_SOME(0.00)[]; FUZZY_RATELIMITED(0.00)[rspamd.com]; MID_RHS_MATCH_FROM(0.00)[]; DKIM_SIGNED(0.00)[suse.de:s=susede2_rsa,suse.de:s=susede2_ed25519]; FROM_HAS_DN(0.00)[]; RCPT_COUNT_THREE(0.00)[4]; FROM_EQ_ENVFROM(0.00)[]; TO_MATCH_ENVRCPT_ALL(0.00)[]; RCVD_COUNT_TWO(0.00)[2]; DBL_BLOCKED_OPENRESOLVER(0.00)[suse.de:mid] Received-SPF: pass client-ip=195.135.223.130; envelope-from=farosas@suse.de; helo=smtp-out1.suse.de X-Spam_score_int: -43 X-Spam_score: -4.4 X-Spam_bar: ---- X-Spam_report: (-4.4 / 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_MED=-2.3, RCVD_IN_VALIDITY_CERTIFIED_BLOCKED=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 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 Peter Xu writes: > On Wed, Sep 17, 2025 at 05:56:40PM -0300, Fabiano Rosas wrote: >> Peter Xu writes: >> >> > On Wed, Sep 10, 2025 at 12:01:41PM -0400, Peter Xu wrote: >> >> Fabiano fixed graceful shutdowns for multifd channels previously: >> >> >> >> https://lore.kernel.org/qemu-devel/20250206175824.22664-1-farosas@suse.de/ >> >> >> >> However we likely forgot the rest channels. Do it the same for the main >> >> and postcopy channels. This fixes a warning message when running unit test >> >> /ARCH/migration/postcopy/preempt/tls/psk. >> >> >> >> Thanks, >> >> >> >> Peter Xu (3): >> >> migration/tls: Gracefully shutdown main and preempt channels >> >> migration: Make migration_has_failed() work even for CANCELLING >> >> migration/multifd: Use the new graceful termination helper >> > >> > Please hold off the review on this one. Juraj reported the issue wasn't >> > resolved by the changes, and I can also reproduce. I'll have a look and >> > repost.. >> >> I'm wondering if the assumption that only succeeded migrations should >> gracefully exit is correct. My understanding is that we need to always >> exit gracefully, but after failure, the channel might not be there, so >> we ignore failures. But that does not seem to mean a failed migration >> can simply not exit gracefully. > > Currently tls channels will ignore premature terminations whenever there's > a shutdown() on READ. So when failed / cancelled and whenever there's a > shutdown(), iochannel already does the bypass no matter what migration does. > I'm thinking if it's possible for a premature termination to be detected by TLS before we did the shutdown(). So my suggestion was to always bye() before shutdown(), not matter the state migration is in. But maybe your way is ok, I'm not sure now. Let me read the other versions of the series... > Or do you mean we should remove that, and still try to do graceful > shutdowns even if the channel was shutdown()? > > Feel free to have a look at v2 of this series, especially patch 1. v3 will > come soon, but just to say, v2 is hugely different from v1, and should be > fairly close to upcoming v3, at least on patch 1 which is the real fix. > Ah, sorry, I didn't see the v2 on my list. But it's there. > I should have mentioned that earlier.. :(