Received: by 2002:a05:6a10:5bc5:0:0:0:0 with SMTP id os5csp343484pxb; Thu, 21 Oct 2021 00:08:17 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwoN/crZ+kSjOR1dqdlkJA+jzMx2pWhJ1yPlXTyorW8g8zxt4gTIEGVLG6NUXh8XVCjrskn X-Received: by 2002:a17:90b:1c09:: with SMTP id oc9mr4659101pjb.33.1634800097558; Thu, 21 Oct 2021 00:08:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1634800097; cv=none; d=google.com; s=arc-20160816; b=uPzCN3NY3240C7ZG/LlYv+Cf4DelKWmRJzvW0m49XRF2YM9tJim70IEBGTeSk4NYgf znfwRL5mQ9ItutNBVQxgwIxrnFfIA2yqYKfOCgD57C4i/Xb81ilfW5PUkZOAbYOUTwN8 w4fqBAu3uvbiCAVHAYTPqH110L4trWcP/nHiAdwSJdsg2lO6iiNjFYJbYO5vgRr4unis FSakB+qXe+mUd9LzlViMzt6pdfu0FFIfRhmKI9Bb/IgkvFiboLcHIvHrzNli0TOuY2uY 8vb+9m3sh+yrd/+q9wu3nHFM5FNwLFmAb3C03IOS5MdZIVcwKyuJvBW44Naqmve0Esem QMsQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=+qxpJhJimmwhzyRT6smAhm4INryhYuX3FQYl+g8fJPU=; b=q/FpQH717t5/rfUSlEoFKCNVi7j7gofFjJBPOA8epL762CCsTzoNHopVJT61PMqxjf p8Z73GvafDOGjBPkdPxclIgAVwTs32gmKYrV2IzzLBBt/9jq/2io8YFR6Yxjj0vpvnTL SwPEx69HKiLAWMgKqT52ZORbkOVOHKccVsVc21o6fMuWuSdBp9+EglyXglBcgL+OjMxj pzck0Dd7I3FiDfsLUkvLIFxI7kXQHZBlU+zYZgXnIVCDt35oGLe0nk+2r8In52Tyj46l XShXlBy/xPqZJ6SdZ8hXcQ+ldcNAnrFSIvSu4ertFS/OyHfS0/G9MXLrRwesRnJGf4Jh ABBw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=wECbXX7e; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id l72si7075032pfd.346.2021.10.21.00.07.55; Thu, 21 Oct 2021 00:08:17 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linuxfoundation.org header.s=korg header.b=wECbXX7e; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linuxfoundation.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231218AbhJUHJH (ORCPT + 99 others); Thu, 21 Oct 2021 03:09:07 -0400 Received: from mail.kernel.org ([198.145.29.99]:53544 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230050AbhJUHJF (ORCPT ); Thu, 21 Oct 2021 03:09:05 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 7D04A60FC0; Thu, 21 Oct 2021 07:06:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1634800010; bh=hs2snseJ8YgIckgi0bMl5NvKTp+MmQDXvrHOzNnxggw=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=wECbXX7eLZCGcygaG3znjjgWk0TBC6UxRvOqUsCc9efS0p7Y4NqvQAiwRoewFo//W o1xyyI5RsqaNdjsqVd7w15ePpoAggRNWF++tegglKG9DtLbH7QgS3bnBsyb5LYEVcw 7YKl01FZ0SWETmx32hSn8uxEe+6osnhe7NHmh0Zk= Date: Thu, 21 Oct 2021 09:06:47 +0200 From: Greg KH To: "Eric W. Biederman" Cc: linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org, Linus Torvalds , Oleg Nesterov , Al Viro , Kees Cook Subject: Re: [PATCH 18/20] exit/rtl8723bs: Replace the macro thread_exit with a simple return 0 Message-ID: References: <87y26nmwkb.fsf@disp2133> <20211020174406.17889-18-ebiederm@xmission.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20211020174406.17889-18-ebiederm@xmission.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 20, 2021 at 12:44:04PM -0500, Eric W. Biederman wrote: > Every place thread_exit is called is at the end of a function started > with kthread_run. The code in kthread_run has arranged things so a > kernel thread can just return and do_exit will be called. > > So just have the threads return instead of calling complete_and_exit. > > Signed-off-by: "Eric W. Biederman" > --- > drivers/staging/rtl8723bs/core/rtw_cmd.c | 2 +- > drivers/staging/rtl8723bs/core/rtw_xmit.c | 2 +- > drivers/staging/rtl8723bs/hal/rtl8723bs_xmit.c | 2 +- > drivers/staging/rtl8723bs/include/osdep_service_linux.h | 2 -- > 4 files changed, 3 insertions(+), 5 deletions(-) You "forgot" to cc: the linux-staging and the staging driver maintainer on these drivers/staging/ changes... Anyway, they look fine to me, but you will get some conflicts with some of these changes based on cleanups already in my staging-next tree (in linux-next if you want to see them). But feel free to take these all in your tree if that makes it easier: Acked-by: Greg Kroah-Hartman