hat.com; Eric Dumazet
> Subject: Re: [net] fq_codel: fix NULL pointer deref in fq_codel_reset
>
> On Mon, Jun 11, 2018 at 12:57 PM, Keller, Jacob E
> wrote:
> >
> > I'm open to alternative suggestinos for fixing this, I think Eric suggested
> > that
> maybe
redhat.com; Eric Dumazet
> Subject: Re: [net] fq_codel: fix NULL pointer deref in fq_codel_reset
>
> Making q->flows_cnt 0 is simpler and easier to understand.
Feel free to propose such a patch :)
Thanks,
Jake
On Mon, Jun 11, 2018 at 12:57 PM, Keller, Jacob E
wrote:
>
> I'm open to alternative suggestinos for fixing this, I think Eric suggested
> that maybe we should just remove the ->reset() call from qdisc_destroy..?
You can't remove ->reset() for non-failure call path.
For failure path, yeah, but
On Mon, Jun 11, 2018 at 10:00 AM, Jeff Kirsher
wrote:
>
> We could mitigate some of these issues by changing fq_codel_init to more
> explicitly cleanup after itself when failing. For example, we could
> ensure that q->flowcnt was set to 0 so that the loop over each flow in
> fq_codel_reset would n
> -Original Message-
> From: Kirsher, Jeffrey T
> Sent: Monday, June 11, 2018 10:00 AM
> To: da...@davemloft.net
> Cc: Keller, Jacob E ; netdev@vger.kernel.org;
> nhor...@redhat.com; sassm...@redhat.com; jogre...@redhat.com; Eric
> Dumazet ; Kirsher, Jeffrey T
>
> Subject: [net] fq_codel: