<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 9, 2015 at 6:35 PM, Loganaden Velvindron <span dir="ltr"><<a href="mailto:loganaden@gmail.com" target="_blank">loganaden@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi guys,<br><br></div>Here's a trivial patch to fix a warning with gcc 5.2, as 1 isn't casted before being compared with an unsigned integer. Diff below:<br><br><pre>diff --git a/sch_cake.c b/sch_cake.c
index 3ed18d7..aab0fe3 100644
--- a/sch_cake.c
+++ b/sch_cake.c
@@ -436,7 +436,7 @@ static int cake_enqueue(struct sk_buff *skb, struct Qdisc *sch)
         * Split GSO aggregates if they're likely to impair flow isolation
         * or if we need to know individual packet sizes for framing overhead.
         */
-       if(unlikely((len * max(fqcd->flow_count, 1) > q->peel_threshold && skb_is_gso(skb))))
+       if(unlikely((len * max(fqcd->flow_count, (u32)1) > q->peel_threshold && skb_is_gso(skb))))
        {
                struct sk_buff *segs, *nskb;
                netdev_features_t features = netif_skb_features(skb);
</pre><br></div>
</blockquote></div><br></div><div class="gmail_extra">Hi Chromi@,<br><br></div><div class="gmail_extra">Thanks for fixing that, but in a different way :)<br><br><br><br><br></div></div>