Received: by 2002:ad5:4acb:0:0:0:0:0 with SMTP id n11csp5057641imw; Tue, 19 Jul 2022 20:05:39 -0700 (PDT) X-Google-Smtp-Source: AGRyM1ubkC66cDdfcEOjPh4YKRadVNh6frj7qCtI4F+cZkQeZ0RANXshI1jNZnmInaxO1X8V2QCd X-Received: by 2002:a17:902:f546:b0:16b:e4c7:671b with SMTP id h6-20020a170902f54600b0016be4c7671bmr36813329plf.129.1658286339083; Tue, 19 Jul 2022 20:05:39 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1658286339; cv=none; d=google.com; s=arc-20160816; b=bXq8mxh6XyRu2m5zMftk/v1AA69veahYDY41+NSWHNR7FJ+y78FSwtTGfztWFJjybZ S4EbqUM0bWieXIcqnGc8nuCd4qInf8Meif59rnB8ozXLWQi1HL9hyuKtc/sJdQIPwVPW nIV48qz/Qu9+sa29cv8s+uJOPe8TurJCPdrk606Osgo5d5Y9lVEv8w9A3urNwvdCz++G RrtGD9hG8yPHpuWvj5asurdqfFbftdOL4bHgGuT/n9iEe7tdrIpNOHeJq0K17GCjsjRf EnfQcn77dar23HyhC55v7dNYO9emj29BiMWBd83+bnbg+hyT1jcAz0dWX8GRDGFEfxOE HemA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to :organization:from:references:cc:to:content-language:subject :user-agent:mime-version:date:message-id:dkim-signature; bh=4Gy97tee+Piny7xG0AvgFw4ZwDYOWjS6fnh/UnSKTkM=; b=IeRaw8KjyvGXE6edGi1+0mxTsOYthsNduhlTcEDkOf0VnPfwE0EjSbaI8R727PFIoH yRC0Iry2PfQZIyXrwfO8dFppktlFN0HhhD750uiLJGelf5iyzdseG8W/r6K7UFLgD1Wk cQEzHESkyO9mpM31PRJ9AyRzqHl4TpuVPXYXVNgh3Nz32kGFHOdDu2og38ivMS0Hf+PV 2fbYPZPenRNXkIzB27GE8JAVOxmv17Ra5TXvg1iFdmlq7UTvtUX9M/wongu9gZzhrsAs eagfnKV3CFhT/FgG0rCOhmc0Hf2oNGIoYpyfBwHwS9UDMSbuBKbSt8ueZa7ah1tCqeE9 d0Ow== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@canonical.com header.s=20210705 header.b=FIDgjMwW; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=canonical.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id n10-20020a170902d2ca00b0016c8facc00asi7409265plc.435.2022.07.19.20.05.24; Tue, 19 Jul 2022 20:05:38 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-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=@canonical.com header.s=20210705 header.b=FIDgjMwW; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=canonical.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236942AbiGTCbR (ORCPT + 99 others); Tue, 19 Jul 2022 22:31:17 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60514 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229633AbiGTCa6 (ORCPT ); Tue, 19 Jul 2022 22:30:58 -0400 Received: from smtp-relay-canonical-1.canonical.com (smtp-relay-canonical-1.canonical.com [185.125.188.121]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id AB27B6E890; Tue, 19 Jul 2022 19:28:32 -0700 (PDT) Received: from [192.168.192.83] (unknown [50.126.114.69]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by smtp-relay-canonical-1.canonical.com (Postfix) with ESMTPSA id 133093F3B9; Wed, 20 Jul 2022 02:28:28 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1658284110; bh=4Gy97tee+Piny7xG0AvgFw4ZwDYOWjS6fnh/UnSKTkM=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=FIDgjMwWggLf9HaPWWjtqnFSgbaX+ul5LuJmUqDHQnRRLv9guggUo8qStLCbf2J8g iE3eAV1UuqqODL5Lj3GBRGhBKYGOBgLvzj9b1UGUWAVqz2nJC1ySgqrh91cYuDao6D PZec74E6Iq2fl9FN6KPxoMZGTumbtWpRa6NMXchk9Q7YkK+IKN7Nl2QAMw96CHmDtP KoBIBYC65wrTR7nkkb4B2ObNEjYyeVaCSLhyzXqkpbqVBV5vXmRLvkZa88p1UwhbMK 2eUC1I06spL1qrupm6EN61GHxjdbZOlykwd1bA7UPjz2nUGM/65Se0sGVH0SiilB1g OaL2NWgp9OhpQ== Message-ID: <6aca407d-9c81-b460-f03e-cea0369a6185@canonical.com> Date: Tue, 19 Jul 2022 19:28:26 -0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH -next] apparmor: Fix some kernel-doc comments Content-Language: en-US To: Yang Li Cc: paul@paul-moore.com, jmorris@namei.org, serge@hallyn.com, apparmor@lists.ubuntu.com, linux-security-module@vger.kernel.org, linux-kernel@vger.kernel.org, Abaci Robot References: <20220718063022.129998-1-yang.lee@linux.alibaba.com> From: John Johansen Organization: Canonical In-Reply-To: <20220718063022.129998-1-yang.lee@linux.alibaba.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-5.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A, RCVD_IN_DNSWL_MED,SPF_HELO_NONE,SPF_PASS 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-kernel@vger.kernel.org On 7/17/22 23:30, Yang Li wrote: > Remove warnings found by running scripts/kernel-doc, which is caused by > using 'make W=1'. > security/apparmor/policy_ns.c:65: warning: Function parameter or member 'curr' not described in 'aa_ns_name' > security/apparmor/policy_ns.c:65: warning: Function parameter or member 'view' not described in 'aa_ns_name' > security/apparmor/policy_ns.c:65: warning: Function parameter or member 'subns' not described in 'aa_ns_name' > security/apparmor/policy_ns.c:65: warning: expecting prototype for aa_na_name(). Prototype was for aa_ns_name() instead > security/apparmor/policy_ns.c:214: warning: Function parameter or member 'view' not described in '__aa_lookupn_ns' > security/apparmor/policy_ns.c:214: warning: Excess function parameter 'base' description in '__aa_lookupn_ns' > security/apparmor/policy_ns.c:297: warning: expecting prototype for aa_create_ns(). Prototype was for __aa_find_or_create_ns() instead > > Reported-by: Abaci Robot > Signed-off-by: Yang Li thanks I have pulled this into apparmor-next Acked-by: John Johansen > --- > security/apparmor/policy_ns.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/security/apparmor/policy_ns.c b/security/apparmor/policy_ns.c > index 300953a02a24..0ef268d6554d 100644 > --- a/security/apparmor/policy_ns.c > +++ b/security/apparmor/policy_ns.c > @@ -54,10 +54,10 @@ bool aa_ns_visible(struct aa_ns *curr, struct aa_ns *view, bool subns) > } > > /** > - * aa_na_name - Find the ns name to display for @view from @curr > - * @curr - current namespace (NOT NULL) > - * @view - namespace attempting to view (NOT NULL) > - * @subns - are subns visible > + * aa_ns_name - Find the ns name to display for @view from @curr > + * @curr: current namespace (NOT NULL) > + * @view: namespace attempting to view (NOT NULL) > + * @subns: are subns visible > * > * Returns: name of @view visible from @curr > */ > @@ -200,7 +200,7 @@ struct aa_ns *aa_find_ns(struct aa_ns *root, const char *name) > > /** > * __aa_lookupn_ns - lookup the namespace matching @hname > - * @base: base list to start looking up profile name from (NOT NULL) > + * @view: namespace to search in (NOT NULL) > * @hname: hierarchical ns name (NOT NULL) > * @n: length of @hname > * > @@ -285,7 +285,7 @@ static struct aa_ns *__aa_create_ns(struct aa_ns *parent, const char *name, > } > > /** > - * aa_create_ns - create an ns, fail if it already exists > + * __aa_find_or_create_ns - create an ns, fail if it already exists > * @parent: the parent of the namespace being created > * @name: the name of the namespace > * @dir: if not null the dir to put the ns entries in