From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from galois.linutronix.de (Galois.linutronix.de [193.142.43.55]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B9DB53A6B91; Tue, 17 Mar 2026 11:05:38 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=193.142.43.55 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773745539; cv=none; b=sTSw9J5hzuKSlgjrmpg1gQK0KoS7tb+TmUgaCUjgJUhV5OKZU2D7wuRGkJ1h6VDhA8uELFXmZhA7H1w/u/x0HMVJC7i+Ni7mjNIqFDDbt7iSHCcFUvYXLp91ovrpazoGJXNMy4bS28lW4BHguvIYC3jZ3DZioTfo0zycACTuT/0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773745539; c=relaxed/simple; bh=XbeIt3ZOAl7dYmoRenwbbISnD93cW82NdG4pwZZFXJY=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=G8rGZikVD0JSPtkFar4hIl8/8xfAivdh7OfaUVx53j37xSzUDBV/NN/eohMUO0NonBnZELppttNKfXBaGcd7w+Otfg7p+TIRpJZWNRJENtvUbWdn1dPo71q7UURkR6iMPaLvyD6T+5NIBe41iKph3M1+WsnCMmrq5xlnykUNQzw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linutronix.de; spf=pass smtp.mailfrom=linutronix.de; dkim=pass (2048-bit key) header.d=linutronix.de header.i=@linutronix.de header.b=PFxkJ9H0; dkim=permerror (0-bit key) header.d=linutronix.de header.i=@linutronix.de header.b=/e2/9Ltg; arc=none smtp.client-ip=193.142.43.55 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linutronix.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linutronix.de Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linutronix.de header.i=@linutronix.de header.b="PFxkJ9H0"; dkim=permerror (0-bit key) header.d=linutronix.de header.i=@linutronix.de header.b="/e2/9Ltg" Date: Tue, 17 Mar 2026 12:05:35 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1773745536; h=from:from:reply-to:subject:subject: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=/JN1SD/V1awQNnrLIW7XWJMZue616SYt+kBa3fB6sCY=; b=PFxkJ9H0y6nJMt58mCQSmAumodne11b1d5Y5Ku843irL4qFbsWZWAuCAwFj5r5RTcJQo0g IvSgT/zGWAakvNW7HdjT/5Jy0oKThJ11bUzdbi+6zYz92dIpbflZWp3YJRnpA/OMD33RVD 4cSB+x8L4hbrG2+x0Gp/upSBs1My9T5TC/kY0B2GS4LEFZ7xJ0U8qVTL0XZCJGtLIP4fcy xgFylrsRAb6+T1IyGnxoMJRqPQlINRTpt3GsD7mcYDMeGFVJLg3kGuOQLv68JztWNQm8H8 SPK/ha/6ODZugMaTACgpYDzhw0Khv8N071dcGw2fNO1hTWxXxEg8helsowpUng== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1773745536; h=from:from:reply-to:subject:subject: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=/JN1SD/V1awQNnrLIW7XWJMZue616SYt+kBa3fB6sCY=; b=/e2/9LtgIepdygo4bacGdSwUGRuwmGgamLcP71bRVWcqfOjy2Np5AwnSwUw0vJi2UoMkim J86tICLNKuYCmACg== From: Sebastian Andrzej Siewior To: Jan Kiszka Cc: "K. Y. Srinivasan" , Haiyang Zhang , Wei Liu , Dexuan Cui , Long Li , "linux-hyperv@vger.kernel.org" , Linux Kernel Mailing List , Florian Bezdeka Subject: Re: [PATCH] Drivers: hv: vmbus: Move add_interrupt_randomness back to real interrupt Message-ID: <20260317110535.Smn9viQ7@linutronix.de> References: <1b53653a-98a5-402a-a224-996b26edaa97@siemens.com> Precedence: bulk X-Mailing-List: linux-hyperv@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <1b53653a-98a5-402a-a224-996b26edaa97@siemens.com> On 2026-03-17 09:09:27 [+0100], Jan Kiszka wrote: > From: Jan Kiszka > > Sebastian Siewior wrote: > "This is feeding entropy and would like to see interrupt registers. But > since this is invoked from a thread it won't." > > So move it back to where it is always in interrupt context. > > Fixes: f8e6343b7a89 ("Drivers: hv: vmbus: Simplify allocation of vmbus_evt") > Signed-off-by: Jan Kiszka > --- > drivers/hv/vmbus_drv.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/hv/vmbus_drv.c b/drivers/hv/vmbus_drv.c > index bc4fc1951ae1..28025a264861 100644 > --- a/drivers/hv/vmbus_drv.c > +++ b/drivers/hv/vmbus_drv.c > @@ -1361,8 +1361,6 @@ static void __vmbus_isr(void) > > vmbus_message_sched(hv_cpu, hv_cpu->hyp_synic_message_page); > vmbus_message_sched(hv_cpu, hv_cpu->para_synic_message_page); > - > - add_interrupt_randomness(vmbus_interrupt); > } > > static DEFINE_PER_CPU(bool, vmbus_irq_pending); > @@ -1410,6 +1408,8 @@ void vmbus_isr(void) > lockdep_hardirq_threaded(); > __vmbus_isr(); > } > + > + add_interrupt_randomness(vmbus_interrupt); > } > EXPORT_SYMBOL_FOR_MODULES(vmbus_isr, "mshv_vtl"); Why not sysvec_hyperv_callback()? Sebastian