Received: by 2002:ad5:4acb:0:0:0:0:0 with SMTP id n11csp4529446imw; Tue, 19 Jul 2022 08:16:44 -0700 (PDT) X-Google-Smtp-Source: AGRyM1uFd8Qh3AIhHK5os0kDPmKfQodHMZx8aaCHVbo9cVR95cFAqkVzgN7VUwbkYhFSgJoZ/vl6 X-Received: by 2002:ad4:5b83:0:b0:473:8ba5:b57f with SMTP id 3-20020ad45b83000000b004738ba5b57fmr25255585qvp.74.1658243803936; Tue, 19 Jul 2022 08:16:43 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1658243803; cv=none; d=google.com; s=arc-20160816; b=umKv3Z5lu6Y5PUjcvl2kKZY7pDDxMlmc/GCLPYr8j1A6V9+dhjB8GORjbGpm5L8OHY jDPp9k10Hodb7WomP2JM8dsWRO/Gs0nV7yRu0i98+siKWpeMVS27L8a7na9cTtV7jcOG kDoPkbdpC2Oo4GR4dzLERIl/5zVZSbUYTU5PXt02QQ7FdAmIqQnp4HQ0RVdkibqwSST9 k5UDyAu5SxSyDNwBXmvLd9DU/Bvhj0AXMQlIL76C/L4+so/QBNGqkrlXRHEqXSPPXjVR 2ykdjUt0Ne0LGL/iwpFgqlQ4/nG4GY3ohqrczcR6FZcvuEcDUwCc0OfRO5m+ShPrLT7/ JoUw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:message-id:date:references :in-reply-to:subject:cc:to:from:dkim-signature; bh=DZfape1HvRcQDBJMjqCK8x7O2btj6Zg9uPaKpCKPLz4=; b=yVcHHwthk6kwNXpJC+cajpBcXDWx0OcpnFL0LpCGyfkgHPHriWMISC+HGSvWwuA6IH 44foZtOlsz9JefxPHRtRLkR6oBTi65CFQFu29P48c0iVvZZzeBMOL0Mqs0/th5fH98e+ 48Slct0HHBuLaqZxp4psshbZXqMYjM3jKG5xo15PphCWxA5dvWnjZjmykFNdQ5baGr8b sND66tFg8USWuMKItGwj8ukwmPdDtUMCMp41bjr0Tp/D1k6SLNX5y96lXQ3OlZ6GvdWQ rYnJA6BYuUsHoxe4uby6h/SgR0v3ZlgvX9efOJmEFrAVWgokBLjUxgS21FXZeHL133oE tI4A== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=jId0WW1J; spf=pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-wireless-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id p10-20020a05620a132a00b006b5c1beb2fdsi6383903qkj.415.2022.07.19.08.16.33; Tue, 19 Jul 2022 08:16:43 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=jId0WW1J; spf=pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-wireless-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238495AbiGSPPe (ORCPT + 65 others); Tue, 19 Jul 2022 11:15:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45040 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238576AbiGSPPZ (ORCPT ); Tue, 19 Jul 2022 11:15:25 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id A816F49B56 for ; Tue, 19 Jul 2022 08:15:14 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1658243707; 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=DZfape1HvRcQDBJMjqCK8x7O2btj6Zg9uPaKpCKPLz4=; b=jId0WW1JGdQxYKlQZAHEckn0uE6UlARHtx7v/Pt6PoDVf+dRxBsr7MSpKMI69Kz22Ig+fZ 40TYA/vaWJ309h/HcUBHDix+cFsDJNFKj/SddDafiEORgPwdG9P2dcsu66/K8Pw9X1aPhW H7bklgsXEt8Uzx/jP2z/u0aLGYiMOEM= Received: from mail-wm1-f71.google.com (mail-wm1-f71.google.com [209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-29-_cu7QG7tOGmSMFduuyw6AA-1; Tue, 19 Jul 2022 11:15:06 -0400 X-MC-Unique: _cu7QG7tOGmSMFduuyw6AA-1 Received: by mail-wm1-f71.google.com with SMTP id az39-20020a05600c602700b003a321d33238so1256501wmb.1 for ; Tue, 19 Jul 2022 08:15:05 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:in-reply-to:references:date :message-id:mime-version; bh=DZfape1HvRcQDBJMjqCK8x7O2btj6Zg9uPaKpCKPLz4=; b=3ItKnG+BTTl/4QZnvsbGZ7DJ4CUMfBxG7j0ZBvhHJMgAqrYDhPe2vkAHxKk86Tu6Ku YmjHQwC4GNlEkZvi6GW+wKBv9gquC6vnOJtJPT7MTJni7ha5ksZTlWi8DDo1KLCdCFuG /tauAeT+DK/Cz1ZRJsiuNiBka1pyKP8Ux3MBxEZyI7VF7Cair47C/jgxyH2d9L/rDOI0 r4Hr4OoKE/Rpp3RJiN2w4jsFTSpJ6dFFYhkUBdtwc31a72BmaXQyK+OUUdBJuLsex9e5 7Q0ipuCeDFvN8hv25IeFLHg71+n5bC341kb02jB0AR9PJbHfyNHH+kL/XagxVD5Hx4X+ iShg== X-Gm-Message-State: AJIora+a/3ULUvSmo8m7u8L63c1gi87rb6iQYmPOS99Fb052H+MZhxuD a1ulKV6wTvRou4Oqqdg3yzseqqKwz50JJaUgitzIQoDzAVxb25GEYcXoLM9ILROoH9kHOUPDJ3K EQ6PNqDTBxdmV4te/pymfNUJy4BY= X-Received: by 2002:a1c:2b05:0:b0:3a0:2ae2:5277 with SMTP id r5-20020a1c2b05000000b003a02ae25277mr32016339wmr.30.1658243704620; Tue, 19 Jul 2022 08:15:04 -0700 (PDT) X-Received: by 2002:a1c:2b05:0:b0:3a0:2ae2:5277 with SMTP id r5-20020a1c2b05000000b003a02ae25277mr32016321wmr.30.1658243704438; Tue, 19 Jul 2022 08:15:04 -0700 (PDT) Received: from vschneid.remote.csb ([185.11.37.247]) by smtp.gmail.com with ESMTPSA id bg10-20020a05600c3c8a00b003a0323463absm23061621wmb.45.2022.07.19.08.15.02 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 19 Jul 2022 08:15:03 -0700 (PDT) From: Valentin Schneider To: "Jason A. Donenfeld" Cc: Kalle Valo , Herbert Xu , linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org, Gregory Erwin , Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgense?= =?utf-8?Q?n?= , Rui Salvaterra , Ingo Molnar , Peter Zijlstra , Juri Lelli , Vincent Guittot , Dietmar Eggemann , Steven Rostedt , Ben Segall , Mel Gorman , Daniel Bristot de Oliveira , Christian Brauner , linux-crypto@vger.kernel.org Subject: Re: [PATCH v8] ath9k: let sleep be interrupted when unregistering hwrng In-Reply-To: References: <20220629114240.946411-1-Jason@zx2c4.com> <87v8s8ubws.fsf@kernel.org> Date: Tue, 19 Jul 2022 16:15:02 +0100 Message-ID: MIME-Version: 1.0 Content-Type: text/plain X-Spam-Status: No, score=-3.5 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_LOW, SPF_HELO_NONE,SPF_NONE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org On 11/07/22 13:53, Jason A. Donenfeld wrote: > Hi Valentin, > > On 7/11/22, Valentin Schneider wrote: >> Thanks for the Cc. >> >> I'm not hot on the export of wake_up_state(), IMO any wakeup with >> !(state & TASK_NORMAL) should be reserved to kernel internals. Now, here >> IIUC the problem is that the patch uses an inline invoking >> >> wake_up_state(p, TASK_INTERRUPTIBLE) >> >> so this isn't playing with any 'exotic' task state, thus it shouldn't >> actually need the export. >> >> I've been trying to figure out if this could work with just a >> wake_up_process(), but the sleeping pattern here is not very conforming >> (cf. 'wait loop' pattern in sched/core.c), AFAICT the signal is used to >> circumvent that :/ > > I don't intend to work on this patch more. If you'd like to ack the > trivial scheduler change (adding EXPORT_SYMBOL), that'd help, and then > this can move forward as planned. Otherwise, if you have particular > opinions about this patch that you want to happen, feel free to pick > up the patch and send your own revisions (though I don't intend to do > further review). Alternatively, I'll just send a patch to remove the > driver entirely. Hopefully you do find this ack-able, though. > I'm not for a blanket wake_up_state() export, however if we *really* need it then I suppose we could have a wake_up_process_interruptible() exported and used by __set_notify_signal().