Received: by 2002:ac0:a594:0:0:0:0:0 with SMTP id m20-v6csp6936039imm; Sun, 20 May 2018 14:18:13 -0700 (PDT) X-Google-Smtp-Source: AB8JxZrW0S5XCk9sDm4z3JobaJ9yH0qmetMAMP/FKWmExUdN//83TR64fHEIaeZM5YMAZSAvHroX X-Received: by 2002:a17:902:1025:: with SMTP id b34-v6mr17661785pla.207.1526851093681; Sun, 20 May 2018 14:18:13 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1526851093; cv=none; d=google.com; s=arc-20160816; b=xJ3y8f9pzpR4He9i2fU2LVYwgyNiKRUzpr6pzOhg0bX6grpqyeVXblSHZeXaizrIk1 caSaS3XY4+6ZGYhR59gZY2pYjRCCI2xYCSFGOGNt7TUW9nboPCnKfh6PGSMqJnXdKXQD 5sKXRDzlqm1TjTMjZ7aVK+AvZVO3W2KAetreAifjZdecaR2fgnkXpCMeZL8kY+6UnKVP Myv8eZJQ8DDUVyeFmuC2GUjw+4ZdfoO16i25ZB2IdxeZZR35OtugX3p0ZshSe7h21H9y L4Xfu1AiiJ33ELksOdO/kV1Qa72x/uWf1mYihVsLQWNcZrB69rNoJGT+2j1/6A6cSIHk t1kQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :references:in-reply-to:mime-version:dkim-signature :arc-authentication-results; bh=JUEzC+nlG09OGIuoR85XB/5B2U5MEx/P8Stz9ncNXrU=; b=wYLqrR1rJhFqwGKybZMNAcuZAw89nnTWm0KfDNk0LOh0J21+lQzJot/tb7FP4IOXuA 9Y7yDzmcPzn1p+cjK9bUHr9b/hZsCCdh2Ob5EqPE69PfL28dFz82JCfHbX8RToLUEzes 3VJyyJ+c8WRlkPa3aKktKnsTMbOQlTSi2ptT4RAqPtnXe2/rB2iOiK1kIThZqm32mWeX /XiIX1sq5VbKuD6CIc5pH2lj/zIKbsBlDWs6vc8+yMUgsfccEaaZbUFyV1qwz6LAHLja PlJfxhlZk7WypADAGckVnu0xlMtA2A0FTMBlz0KxM/QtrlhAEtmAvHwFm6LebngosJtG iQHA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=Zs/m1jLU; 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=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id p2-v6si5360168pgn.187.2018.05.20.14.17.23; Sun, 20 May 2018 14:18:13 -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=@gmail.com header.s=20161025 header.b=Zs/m1jLU; 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=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752600AbeETVNL (ORCPT + 99 others); Sun, 20 May 2018 17:13:11 -0400 Received: from mail-ot0-f195.google.com ([74.125.82.195]:42173 "EHLO mail-ot0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751725AbeETVNI (ORCPT ); Sun, 20 May 2018 17:13:08 -0400 Received: by mail-ot0-f195.google.com with SMTP id l13-v6so14735870otk.9; Sun, 20 May 2018 14:13:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=JUEzC+nlG09OGIuoR85XB/5B2U5MEx/P8Stz9ncNXrU=; b=Zs/m1jLU/szMAXT2NBorMVkf+X71NKvpT62zG1ojA0/RLttbxCe/8RyE77UKVhidj6 /WmX6kBvrpOWgyQksfiPohimEGke+b9ISq2EzeRQjHPN4H5BsBMtjXGiXYpWbcxvixZK EUbFyV4peU/gL0IgDYoL1OfTv1fnUdJEGmDrGGC29q81Ns9kGxel4x+TR+5aWMaT0Hcc 4HVTOqNqodVRFQRsteIzsAWcWoFx80wTD3b2a2T6V6m+C8kuyz/nTSrGydmCFYBVrE2m APNDUnbn3EpcsxkbiL8+FdrKW7AZGeQlVfBQzmElGOwAS0GRw4GrkHeyBDiM+Qv+lG3v dijw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=JUEzC+nlG09OGIuoR85XB/5B2U5MEx/P8Stz9ncNXrU=; b=BeAYHZ12Qx7Z213GrD0cfJf1/TnxDdm3q0gExk3YcAC9G5d4W0ek5jZrWAhGz2QMHz UDDuG99ETEQixk18rSFrZQrmUTmCT1QFn+ret7f9xgp1OZnosMRm+as8S0bDAiEn4LYZ JpGtl6etsKzm1p/CfkKhvq3YjDloyWtuPazU6HARksUH4QADD2/cJwyIIVieN5XEKS6U 702OwBg3nv2zI+q0RH3U9hOjt2BcE4fVlPkUgEK8MAJ7BDFAs6YOhuHeqLBJnE2hFuuo q7htExCkLIwAHGuQVOjl6OZceUexiLxqOc0MJxoQ2zzcgKrvSmgwya5VFJ0xBIBPsqOe O4Xw== X-Gm-Message-State: ALKqPwcZ3AL7N+xB33tktqM0yf6pIoI6aL9yeakl8tzYTYpjavNIQZMr YdKdebJhlTedx9xMijtcjLg3U+CTr0jvOSs3vjo= X-Received: by 2002:a9d:2b72:: with SMTP id f47-v6mr11196372otd.69.1526850787443; Sun, 20 May 2018 14:13:07 -0700 (PDT) MIME-Version: 1.0 Received: by 2002:aca:ab4b:0:0:0:0:0 with HTTP; Sun, 20 May 2018 14:13:06 -0700 (PDT) In-Reply-To: <20180519222028.GF5488@localhost.localdomain> References: <1526308035-12484-1-git-send-email-vladbu@mellanox.com> <1526308035-12484-14-git-send-email-vladbu@mellanox.com> <20180519222028.GF5488@localhost.localdomain> From: Or Gerlitz Date: Mon, 21 May 2018 00:13:06 +0300 Message-ID: Subject: Re: [PATCH 13/14] net: sched: use unique idr insert function in unlocked actions To: Marcelo Ricardo Leitner Cc: Vlad Buslov , Linux Netdev List , David Miller , Jamal Hadi Salim , Cong Wang , Jiri Pirko , Pablo Neira Ayuso , kadlec@blackhole.kfki.hu, Florian Westphal , Alexei Starovoitov , Daniel Borkmann , Eric Dumazet , keescook@chromium.org, Linux Kernel , netfilter-devel@vger.kernel.org, coreteam@netfilter.org, Yevgeny Kliteynik Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, May 20, 2018 at 1:20 AM, Marcelo Ricardo Leitner wrote: > On Mon, May 14, 2018 at 05:27:14PM +0300, Vlad Buslov wrote: >> Substitute calls to action insert function with calls to action insert >> unique function that warns if insertion overwrites index in idr. > > I know this patch may be gone on V2, but a general comment, please use > the function names themselves instead of a textualized version. I.e., > s/action insert unique/tcf_idr_insert_unique/ disagree. While doing reviews I found out that if I ask the developer to use higher level / descriptive language and specifically to avoid putting variable / function names in patch titles and change logs, the quality gets ++ big time, vs if the developer is allowed to say net/mlx5: Changed add_vovo_bobo() Added variable do_it_right to add_vovo_bobo(), now we are terribly good.