From patchwork Tue Oct 27 22:54:39 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Sebastian Andrzej Siewior X-Patchwork-Id: 11862071 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 4B8876A2 for ; Tue, 27 Oct 2020 22:55:28 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 24830221F7 for ; Tue, 27 Oct 2020 22:55:28 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=linutronix.de header.i=@linutronix.de header.b="wYJh8Tec"; dkim=permerror (0-bit key) header.d=linutronix.de header.i=@linutronix.de header.b="aMRfL0Ja" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S374983AbgJ0WzZ (ORCPT ); Tue, 27 Oct 2020 18:55:25 -0400 Received: from Galois.linutronix.de ([193.142.43.55]:49820 "EHLO galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S374977AbgJ0WzY (ORCPT ); Tue, 27 Oct 2020 18:55:24 -0400 From: Sebastian Andrzej Siewior DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1603839322; 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: content-transfer-encoding:content-transfer-encoding; bh=+/ecoziKoJodJKTT7o/UwwiCBG7QroeEJmAgktvLhzY=; b=wYJh8TecCJUTXhZuSrSp5ylLddMcQOB6uBNJosQiQqu0AWxStl2sXOqoDZFSV4lopKMudj 9ev6+9/8clVjjHH1qop1zpaReraf2pNwpeOgP6Ubhfnv3Ij+7CUQNqRPLdqwtRS1XFon98 EFCHR5L8sO0qa9fv7bnt56kNL19WdB/td7ELFaxACijIqUZMTDyC4lRwYo++4+uUH3tJhZ lw9n8opRptHRVprVn9vIjuFD0wqAe3hkoBvBgmrrlQAOPt/prP81EM9YfSb8d+UGUaoliz PpzElBLfVgBYOhW/Z/iwSkna+VELswNnxRCWPMDr0Bc8ZPqaPW3Kwc4KQTLuPw== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1603839322; 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: content-transfer-encoding:content-transfer-encoding; bh=+/ecoziKoJodJKTT7o/UwwiCBG7QroeEJmAgktvLhzY=; b=aMRfL0JaulAb81YMhS3JkaJjxQnZy5Lb7ytie9xAlSd4X7i+x4QHc1shdAMexXN3aU+Eq2 iLwAP6V0gwgGNaBA== To: netdev@vger.kernel.org Cc: Aymen Sghaier , Daniel Drake , "David S. Miller" , Herbert Xu , =?utf-8?q?Horia_Geant=C4=83?= , Jakub Kicinski , Jon Mason , Jouni Malinen , Kalle Valo , Leon Romanovsky , linux-arm-kernel@lists.infradead.org, linux-crypto@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-rdma@vger.kernel.org, linux-wireless@vger.kernel.org, Li Yang , Madalin Bucur , Ping-Ke Shih , Rain River , Saeed Mahameed , Samuel Chessman , Ulrich Kunitz , Zhu Yanjun , Thomas Gleixner , Sebastian Andrzej Siewior Subject: [PATCH net-next 00/15] in_interrupt() cleanup, part 2 Date: Tue, 27 Oct 2020 23:54:39 +0100 Message-Id: <20201027225454.3492351-1-bigeasy@linutronix.de> MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org Folks, in the discussion about preempt count consistency across kernel configurations: https://lore.kernel.org/r/20200914204209.256266093@linutronix.de/ Linus clearly requested that code in drivers and libraries which changes behaviour based on execution context should either be split up so that e.g. task context invocations and BH invocations have different interfaces or if that's not possible the context information has to be provided by the caller which knows in which context it is executing. This includes conditional locking, allocation mode (GFP_*) decisions and avoidance of code paths which might sleep. In the long run, usage of 'preemptible, in_*irq etc.' should be banned from driver code completely. This is part two addressing remaining drivers except for orinoco-usb. Sebastian