[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CA+V-a8ud7PJXRH-cLkj2YknkKaFy2XZOh4yPjc2yhk1+M1twTQ@mail.gmail.com>
Date: Tue, 22 Oct 2013 08:04:39 +0530
From: Prabhakar Lad <prabhakar.csengg@...il.com>
To: Mariusz Ceier <mceier+kernel@...il.com>
Cc: "David S. Miller" <davem@...emloft.net>,
Mugunthan V N <mugunthanvnm@...com>,
Jingoo Han <jg1.han@...sung.com>,
Jiri Pirko <jiri@...nulli.us>, netdev <netdev@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] davinci_emac.c: Fix IFF_ALLMULTI setup
Hi Mariusz,
On Mon, Oct 21, 2013 at 11:15 PM, Mariusz Ceier <mceier+kernel@...il.com> wrote:
> When IFF_ALLMULTI flag is set on interface and IFF_PROMISC isn't,
> emac_dev_mcast_set should only enable RX of multicasts and reset
> MACHASH registers.
>
> It does this, but afterwards it either sets up multicast MACs
> filtering or disables RX of multicasts and resets MACHASH registers
> again, rendering IFF_ALLMULTI flag useless.
>
> This patch fixes emac_dev_mcast_set, so that multicast MACs filtering and
> disabling of RX of multicasts are skipped when IFF_ALLMULTI flag is set.
>
> Tested with kernel 2.6.37.
>
It would have been better if you would have tested this on the latest kernel.
Anyway David has pulled this patch in his tree.
Regards,
--Prabhakar Lad
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists