Received: by 2002:ac0:bc90:0:0:0:0:0 with SMTP id a16csp524103img; Thu, 21 Mar 2019 03:25:55 -0700 (PDT) X-Google-Smtp-Source: APXvYqxwztOd2BoqiQGKQajHYltmkmHS0RxOz4DYwk6COeQynX4aEVcfUEEhSRt2Uts6ehvb+uxV X-Received: by 2002:a65:5289:: with SMTP id y9mr2586897pgp.404.1553163955394; Thu, 21 Mar 2019 03:25:55 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1553163955; cv=none; d=google.com; s=arc-20160816; b=pEDLa2RC8pg5HDb4fe96lLt+pTACar85D3Dap8htHntk2hDHlsKjexCixXPAKzaF0M OLhLRdqwzYg8c6yIWiZnUokddMZoGd/Or/vK8ARqlpf0OBj71WRipJDcH2Q2aWt4uOtC Ys0d5ITHRamOPTXya2OGAlQ3e5xf8yDIC/MiZRlL1sSf4w9jC+5nbdZkvYRPYg/n7Odi iX9xlde7RiadPW9k7fPVUaHv7urpqRihDK0qg+rvZDPiI+TryR4Z/xy7/DrUU41TLZ3k 7nRfcouyjFcAZpmJC9pHPgeFVhRLReKv2GmKY79wpcf1ZfYDEhJkAf7rmgtWCb5MgXDV O2pg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:to:references:message-id :content-transfer-encoding:cc:date:in-reply-to:from:subject :mime-version:dkim-signature; bh=v0TkPBWgrLmRX0kAD4/pUy041dM3Z5H22OqMdbWAwiQ=; b=zmBWjGLJUBdYwxyLhjH47rCodI7Zbkplr/vjsYeb1UdbrX7Hgov0lvJgIw1g/sD+O6 eSEgDcaqEj4LB4J//SmRCm5hVeYrgapJ70vspOCzYIkGzJnZqYmseS+7r3YZ5ORSkAzH Le1fxGADZ9ljNVj7RhZkLjjqQjgkwCZaLrS5sDhvdmoz6CAg78kdU88pROXD4xT5QyVO YqPrKlT19hf0zV4mCTl2AqDezRGJyeh+20ET3m8WVErQkblWZ5R1NN4yDHFnugWW+tH9 KNcp0bRvfQXqO1DrPDm/djM/ujTmwNgQsR3T2kQCvzkGy/w3ARJd1g86bT9JZQnEczuj UVTw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@oracle.com header.s=corp-2018-07-02 header.b=GdjYGLlt; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=oracle.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id a5si4204577plh.299.2019.03.21.03.25.37; Thu, 21 Mar 2019 03:25:55 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@oracle.com header.s=corp-2018-07-02 header.b=GdjYGLlt; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=oracle.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727969AbfCUKY4 (ORCPT + 99 others); Thu, 21 Mar 2019 06:24:56 -0400 Received: from userp2120.oracle.com ([156.151.31.85]:33874 "EHLO userp2120.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727881AbfCUKY4 (ORCPT ); Thu, 21 Mar 2019 06:24:56 -0400 Received: from pps.filterd (userp2120.oracle.com [127.0.0.1]) by userp2120.oracle.com (8.16.0.27/8.16.0.27) with SMTP id x2LANrov181609; Thu, 21 Mar 2019 10:24:39 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=content-type : mime-version : subject : from : in-reply-to : date : cc : content-transfer-encoding : message-id : references : to; s=corp-2018-07-02; bh=v0TkPBWgrLmRX0kAD4/pUy041dM3Z5H22OqMdbWAwiQ=; b=GdjYGLltvDLGMZHSRme1wNhT4CI31vSNtgLgbKBD+Mlik4G/oKKhIoa2TgAWu3d9hMeT ymdyuokeX1t24yqCTrbFsJ9F56yMQWGgAL4xEH3Cuw/Q35s/A7bzEHarazvIOMh8PRg1 c7kbR8h6JnnvC1wiJa5w7+499U6o3XatZmfRBUY/LZrf9B99wXwdRAnUW1FZB7Q6p0BW OtxQpf9Lt2pjwMDAoZ4U9bLhbsFRHyQRYvYt+gGg3vPoesQKn1mSfOs5o6I4akyqFPcg XTPogUTpurVUqn9+sx9dsoCYL1G49mPBQc6jg+Dij+omfmqF86EMCrbYr2ubGq3lwcg4 Rw== Received: from userv0021.oracle.com (userv0021.oracle.com [156.151.31.71]) by userp2120.oracle.com with ESMTP id 2r8ssrqgb3-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 21 Mar 2019 10:24:39 +0000 Received: from userv0121.oracle.com (userv0121.oracle.com [156.151.31.72]) by userv0021.oracle.com (8.14.4/8.14.4) with ESMTP id x2LAOcMu024317 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 21 Mar 2019 10:24:39 GMT Received: from abhmp0015.oracle.com (abhmp0015.oracle.com [141.146.116.21]) by userv0121.oracle.com (8.14.4/8.13.8) with ESMTP id x2LAObFu013098; Thu, 21 Mar 2019 10:24:37 GMT Received: from [192.168.0.110] (/73.243.10.6) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Thu, 21 Mar 2019 03:24:36 -0700 Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.8\)) Subject: Re: [PATCH 1/3] mm/sparse: Clean up the obsolete code comment From: William Kucharski In-Reply-To: <20190321092138.GY18740@MiWiFi-R3L-srv> Date: Thu, 21 Mar 2019 04:24:35 -0600 Cc: Matthew Wilcox , Mike Rapoport , Oscar Salvador , LKML , Andrew Morton , Pavel Tatashin , mhocko@suse.com, rppt@linux.vnet.ibm.com, richard.weiyang@gmail.com, linux-mm@kvack.org Content-Transfer-Encoding: 7bit Message-Id: <3FFF0A5F-AD27-4F31-8ECF-3B72135CF560@oracle.com> References: <20190320073540.12866-1-bhe@redhat.com> <20190320111959.GV19508@bombadil.infradead.org> <20190320122011.stuoqugpjdt3d7cd@d104.suse.de> <20190320122243.GX19508@bombadil.infradead.org> <20190320123658.GF13626@rapoport-lnx> <20190320125843.GY19508@bombadil.infradead.org> <20190321064029.GW18740@MiWiFi-R3L-srv> <20190321092138.GY18740@MiWiFi-R3L-srv> To: Baoquan He X-Mailer: Apple Mail (2.3445.104.8) X-Proofpoint-Virus-Version: vendor=nai engine=5900 definitions=9201 signatures=668685 X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1011 lowpriorityscore=0 mlxscore=0 impostorscore=0 mlxlogscore=760 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1810050000 definitions=main-1903210077 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > On Mar 21, 2019, at 3:21 AM, Baoquan He wrote: It appears as is so often the case that the usage has far outpaced the documentation and -EEXIST may be the proper code to return. The correct answer here may be to modify the documentation to note the additional semantic, though if the usage is solely within the kernel it may be sufficient to explain its use in the header comment for the routine (in this case sparse_add_one_section()).