From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id B0258C433EF for ; Thu, 21 Apr 2022 08:47:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1386888AbiDUIuP (ORCPT ); Thu, 21 Apr 2022 04:50:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59308 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1386872AbiDUIuJ (ORCPT ); Thu, 21 Apr 2022 04:50:09 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id C7C2F140C0 for ; Thu, 21 Apr 2022 01:47:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1650530839; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=cmuFO35zQUp1s4VhyCud3srWXPlk5AJzSrlDvha5upI=; b=ZI1d3mTwo3AKBjUbqtJhR3SD2AIzw6ppBes3+gnMnPoAlHrztCfAhf3x4PjuPPY8brrtnC Im/AYttT/5ufwIHQU/UAa9Qmm/To+gSOmxMOLsCJKdNmvjRxle1cS4tzVFFqzlDF+mJNxM woBuLxcy9//LVtBO7thaSn2HV3gigL4= Received: from mail-qv1-f72.google.com (mail-qv1-f72.google.com [209.85.219.72]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-284-cc5BfymsPYGML0bIXIjCNg-1; Thu, 21 Apr 2022 04:47:18 -0400 X-MC-Unique: cc5BfymsPYGML0bIXIjCNg-1 Received: by mail-qv1-f72.google.com with SMTP id ke23-20020a056214301700b0044bba34469eso2302467qvb.17 for ; Thu, 21 Apr 2022 01:47:18 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:subject:from:to:date:in-reply-to :references:user-agent:mime-version:content-transfer-encoding; bh=cmuFO35zQUp1s4VhyCud3srWXPlk5AJzSrlDvha5upI=; b=Kz5P/LxkPSV9WFY24DbdmLS+M6T4z+ZXgbxaBZBh9Rg8vFlzx4F+e3gx2abribWuB6 qKxYx3xaNUohutDZIgu2Xk3wwInPW2W1CcBFtER9HVCYozSlIB2Lp8BUdnSfEC6Pu/ri YBwoqai7ARBGA6b8bry5qILlOKlAyfFlZ2vefq4PJ/9GY1mzuy9t4Tvi1/joYUIvVcPS 7xlJ5KGdmLBUNMIUiX06ApjSAaqgbAqCSMiU4gFiX13zKS5CSG+VPgbnC5zMhQDfSsWs 83xjiDUmyJKp1mIHB5s0w6WBOYe2CtAhYqAZn2dDRrezfo18G/XRwPrhRSHzYhiemq10 kFsw== X-Gm-Message-State: AOAM531u6ruYEIC6ZxXDrx1ffk2cVJDLRzt3yaSY8fkbS08172OF4Tvr b8w7RhdTaYg0U/OdZUXmjWzo3AI9jOD++Obyvv6NddA16dd+l9HBUFINqaBZvblP0K3VFk58G9w ytu0F6OZks/6ZUaQsBxKiODzJ X-Received: by 2002:a05:6214:2a8d:b0:446:5a52:47f5 with SMTP id jr13-20020a0562142a8d00b004465a5247f5mr14666502qvb.131.1650530838155; Thu, 21 Apr 2022 01:47:18 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx2GJ8m4EUIE8m8G2Vb5sGhuPqmE97xodK09lR4BsEnMPjsLVXYZ96WzI2QTuXoVzXWb0tD1w== X-Received: by 2002:a05:6214:2a8d:b0:446:5a52:47f5 with SMTP id jr13-20020a0562142a8d00b004465a5247f5mr14666492qvb.131.1650530837957; Thu, 21 Apr 2022 01:47:17 -0700 (PDT) Received: from gerbillo.redhat.com (nat-pool-mxp-t.redhat.com. [149.6.153.186]) by smtp.gmail.com with ESMTPSA id e13-20020a05620a12cd00b0069e908ab48dsm2559403qkl.106.2022.04.21.01.47.16 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 21 Apr 2022 01:47:17 -0700 (PDT) Message-ID: <84310b72be223d736c8ac9fc58eb4936a98aa839.camel@redhat.com> Subject: Re: [PATCH] openvswitch: meter: Remove unnecessary int From: Paolo Abeni To: Solomon Tan , pshelar@ovn.org, davem@davemloft.net, kuba@kernel.org, netdev@vger.kernel.org, dev@openvswitch.org, linux-kernel@vger.kernel.org Date: Thu, 21 Apr 2022 10:47:14 +0200 In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.42.4 (3.42.4-2.fc35) MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2022-04-18 at 00:50 +0000, Solomon Tan wrote: > This patch addresses the checkpatch.pl warning that long long is > preferred over long long int. Please don't do that. This kind of changes cause e.g. backporting issue for any later relevant bugfix touching the same area, for no real benefit. Documentation/process/2.Process.rst explicltly states to avoid this kind of patches. > > Signed-off-by: Solomon Tan > --- > net/openvswitch/meter.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/net/openvswitch/meter.c b/net/openvswitch/meter.c > index 04a060ac7fdf..a790920c11d6 100644 > --- a/net/openvswitch/meter.c > +++ b/net/openvswitch/meter.c > @@ -592,8 +592,8 @@ static int ovs_meter_cmd_del(struct sk_buff *skb, struct genl_info *info) > bool ovs_meter_execute(struct datapath *dp, struct sk_buff *skb, > struct sw_flow_key *key, u32 meter_id) > { > - long long int now_ms = div_u64(ktime_get_ns(), 1000 * 1000); > - long long int long_delta_ms; > + long long now_ms = div_u64(ktime_get_ns(), 1000 * 1000); > + long long long_delta_ms; > struct dp_meter_band *band; > struct dp_meter *meter; > int i, band_exceeded_max = -1; Additionally the patch is mangled by non plain-text encoding. For any later submissions (regarding some other different topic) please ensure that your client/mailer send purely plain-text messages, Thanks, Paolo