lkml.org 
[lkml]   [2022]   [Apr]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
SubjectRe: [PATCH] openvswitch: meter: Remove unnecessary int
From
Date
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 <solomonbstoner@protonmail.ch>
> ---
> 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

\
 
 \ /
  Last update: 2022-04-21 10:48    [W:0.045 / U:0.120 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site