From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f50.google.com (mail-wm1-f50.google.com [209.85.128.50]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A0857335091 for ; Tue, 16 Dec 2025 19:59:16 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.50 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1765915158; cv=none; b=ZUFUqoYx5UcaQELdNSXPQ7xmQoU0FluAb4oifVvWOqoVbgn+YkUGCsomPTO0QnyukWSAhsfbJJduFh316XEpqaQOZvaSSF+q+UWKPH1DRJUDgCQAMkPm8gUKC4JUUEEaSrPWNIPmMqWyz1t7+BPmPsq3LbAzJ1GfzDQfRvQnCJo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1765915158; c=relaxed/simple; bh=vz6zEIxg/+wl/MDvP+RvMv0qeimFFpWWQmNPdyV4ahQ=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=oVajPJ09Lk+4aEqHlzV5YVWhzxrYBdi/XGhnHn4PQX4yBXhtnYogV/WT5rFwAd5z4l2Vh/fh+dY2BxrbXrqvKQb4O01/gnOf2ViAxxYcrUcd/sRkBqNFl+IbCSLzpPpOqOuKX0jhdn4Sblz5OTs9TrpFbo+Lt6QcMd/qu2ZMuOY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=mQkHgQ8r; arc=none smtp.client-ip=209.85.128.50 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="mQkHgQ8r" Received: by mail-wm1-f50.google.com with SMTP id 5b1f17b1804b1-47755de027eso33265275e9.0 for ; Tue, 16 Dec 2025 11:59:16 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1765915155; x=1766519955; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:from:to:cc:subject:date :message-id:reply-to; bh=GjniS35+gFl94KgDPsESP+GSizMKZS5/qxbShY9gflU=; b=mQkHgQ8rCUqYWNQeWJT+spqbmbXpQXZBVbGCJeUWSHpIIPcLjCD4WKKtC8gq5X3V1h uPiB7fkh1u26HvOc8vP0tjZDG+gQ1gR9wTMPHbNV4zBRAbei19yhIG0FspKIuG3edPK0 72/VM1hTUDRmIWXpjPpznytNY4UqOpmQIzRnuJTNADpvdQnHJgs6SgXGQuJsA6Sgqhxs zXqpHAPK7R5P+tZpGQWEYNg3Sz3YMI1g9UpKuOJMk7yQEFOBsLJQvZaCUvjWGEcJ4Are Fg2kSpbK3lzL89h7g+ICvVPL0tR3UUaUaT1nr57OdgNuQDNj8K+OUu9RuKEqu4L+p/RI XCFA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1765915155; x=1766519955; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:subject:cc:to:from:date:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to; bh=GjniS35+gFl94KgDPsESP+GSizMKZS5/qxbShY9gflU=; b=Xv2DbVBBTbetqGkjZqXWHjra1w2aPUAZWoautWgJTXMElVSLUa+UQ1hUVnEgl3JIRl EOw9cmVu38G4Bcd371PIHbaD3eyHChuxoqIUuox/sPjTyBsChDbtxWNkqf0iIJM2QV6S pydWJJ9Zlyv+f7s0llpd5j24FeTJxPKBgRcGw3m/G+L9ImCT57qlAuZ3maXG8kj5hGIq TROfNH/+QknCjD6DTG4lIgt+TdtbHVFrxnDgEZMlCBaM8ch/6AymmdKxvZ7f5bLTXWjR O63SpK2fYl+HSrcmkxk+An/4XxaEJKHvALWSY1Ysu7D8nfe52bnkbldbcKZcXFhdYVpH r5gg== X-Forwarded-Encrypted: i=1; AJvYcCV1QyivlLbFFD+uASnatCMSTBPZneeVjuGg6u9hSr2GZnX94Qe+syhSLV7Ty70EHe1oxKMDaqiwAR5vyp0=@vger.kernel.org X-Gm-Message-State: AOJu0YzOA/xMTgQTGjC8PPTzGw4HwJMPDC2ucy8SDl67XMEW/yTqc84J xm2+SpDK/x6CGvndy5sLB4Xw2afebN74AIWHRJFaDY4lu83WylQ57hcQ X-Gm-Gg: AY/fxX6VisBC8HESB7OVzgBoaZiL98v+6x0I2/nP8fM1QFfPtfo4L/N6MMsjdPmaoS4 l06hLbGisD/VFrxD4iN2jv5YAxl8HhwyhurF4MzNc+LVtO3DR1T/cwqBydMZ0n+nV7pMUbVYrVu crHdeTP4ayFAttsZD7ToR8Xysq9Qqur8zpotG1A1jGUq96DsgAMEjoFZse3savl+HKLWTLUro9N FCITCFnp0ZwDRc+Sp02ZNrQkBgNGgUbtw5ifEmJXD87F9Tj9QzeR7hUAE6dSfWag+iVv/nrQf9p jdw410z+CbCz0brZ398+A8OoDglRzbteh7y9UTQ6uOUWhb5HCk/yBOISOC1bwVbRYXQJffV1o1V mtxENcHfImpWRlZmNJtEvjcVdjN3ZGdLk0sKG3fSYzpiCIGJXUDMj4FIv6++3FBSXSXbcInw381 G71i87HaZnXAmY5EcEVkjODv3oGEEUSCb6LoIur6yio8xdmKYsqJUv4U/VzZc7OYg= X-Google-Smtp-Source: AGHT+IGkMCP0kONGgG89SYNUeIIN6orwCVUUHEl6uObTm3wydLwUSISgvgOvaVGUZQxtmSBHQ8CSsQ== X-Received: by 2002:a05:600c:4595:b0:479:3a87:2092 with SMTP id 5b1f17b1804b1-47a8f9145edmr152180075e9.36.1765915154784; Tue, 16 Dec 2025 11:59:14 -0800 (PST) Received: from pumpkin (82-69-66-36.dsl.in-addr.zen.co.uk. [82.69.66.36]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-4310ade7c3fsm779077f8f.26.2025.12.16.11.59.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 16 Dec 2025 11:59:14 -0800 (PST) Date: Tue, 16 Dec 2025 19:59:12 +0000 From: David Laight To: "H. Peter Anvin" Cc: =?UTF-8?B?SsO8cmdlbiBHcm/Dnw==?= , Ingo Molnar , linux-kernel@vger.kernel.org, x86@kernel.org, virtualization@lists.linux.dev, kvm@vger.kernel.org, linux-hwmon@vger.kernel.org, linux-block@vger.kernel.org, Thomas Gleixner , Ingo Molnar , Borislav Petkov , Dave Hansen , Ajay Kaher , Alexey Makhalov , Broadcom internal kernel review list , Paolo Bonzini , Vitaly Kuznetsov , Boris Ostrovsky , xen-devel@lists.xenproject.org, Jean Delvare , Guenter Roeck , Denis Efremov , Jens Axboe Subject: Re: [PATCH 0/5] x86: Cleanups around slow_down_io() Message-ID: <20251216195912.0727cc0d@pumpkin> In-Reply-To: <14EF14B1-8889-4037-8E7B-C8446299B1E9@zytor.com> References: <20251126162018.5676-1-jgross@suse.com> <14EF14B1-8889-4037-8E7B-C8446299B1E9@zytor.com> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.38; arm-unknown-linux-gnueabihf) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Tue, 16 Dec 2025 07:32:09 -0800 "H. Peter Anvin" wrote: > On December 16, 2025 5:55:54 AM PST, "J=C3=BCrgen Gro=C3=9F" wrote: > >On 16.12.25 14:48, Ingo Molnar wrote: =20 > >>=20 > >> * J=C3=BCrgen Gro=C3=9F wrote: > >> =20 > >>>> CPUs anymore. Should it cause any regressions, it's easy to bisect t= o. > >>>> There's been enough changes around all these facilities that the > >>>> original timings are probably way off already, so we've just been > >>>> cargo-cult porting these to newer kernels essentially. =20 > >>>=20 > >>> Fine with me. > >>>=20 > >>> Which path to removal of io_delay would you (and others) prefer? > >>>=20 > >>> 1. Ripping it out immediately. =20 > >>=20 > >> I'd just rip it out immediately, and see who complains. :-) =20 > > > >I figured this might be a little bit too evil. :-) > > > >I've just sent V2 defaulting to have no delay, so anyone hit by that > >can still fix it by applying the "io_delay" boot parameter. > > > >I'll do the ripping out for kernel 6.21 (or whatever it will be called). > > > > > >Juergen =20 >=20 > Ok, I'm going to veto ripping it out from the real-mode init code, > because I actually know why it is there :) ... Pray tell. One thing I can think of is the delay allows time for a level-sensitive IRQ line to de-assert before an ISR exits. Or, maybe more obscure, to avoid back to back accesses to some register breaking the 'inter-cycle recovery time' for the device. That was a good way to 'break' the Zilog SCC and the 8259 interrupt controller (eg on any reference board with a '286 cpu). David > and that code is pre-UEFI legacy these days anyway. >=20 > Other places... I don't care :) >=20