Re: [PATCH v2] net: sctp: Don't use 64 kilobyte lookup table for four elements

2015-09-28 Thread David Miller
From: Denys Vlasenko Date: Mon, 28 Sep 2015 14:34:04 +0200 > Seemingly innocuous sctp_trans_state_to_prio_map[] array > is way bigger than it looks, since > "[SCTP_UNKNOWN] = 2" expands into "[0x] = 2" ! > > This patch replaces it with switch() statement. > >

Re: [PATCH v2] net: sctp: Don't use 64 kilobyte lookup table for four elements

2015-09-28 Thread Denys Vlasenko
On 09/28/2015 05:32 PM, David Laight wrote: > From: Eric Dumazet >> Sent: 28 September 2015 15:27 >> On Mon, 2015-09-28 at 14:12 +, David Laight wrote: >>> From: Neil Horman Sent: 28 September 2015 14:51 On Mon, Sep 28, 2015 at 02:34:04PM +0200, Denys Vlasenko wrote: > Seemingly

RE: [PATCH v2] net: sctp: Don't use 64 kilobyte lookup table for four elements

2015-09-28 Thread David Laight
From: Eric Dumazet > Sent: 28 September 2015 15:27 > On Mon, 2015-09-28 at 14:12 +, David Laight wrote: > > From: Neil Horman > > > Sent: 28 September 2015 14:51 > > > On Mon, Sep 28, 2015 at 02:34:04PM +0200, Denys Vlasenko wrote: > > > > Seemingly innocuous sctp_trans_state_to_prio_map[]

RE: [PATCH v2] net: sctp: Don't use 64 kilobyte lookup table for four elements

2015-09-28 Thread David Laight
From: Neil Horman > Sent: 28 September 2015 14:51 > On Mon, Sep 28, 2015 at 02:34:04PM +0200, Denys Vlasenko wrote: > > Seemingly innocuous sctp_trans_state_to_prio_map[] array > > is way bigger than it looks, since > > "[SCTP_UNKNOWN] = 2" expands into "[0x] = 2" ! > > > > This patch replaces

Re: [PATCH v2] net: sctp: Don't use 64 kilobyte lookup table for four elements

2015-09-28 Thread Eric Dumazet
On Mon, 2015-09-28 at 14:12 +, David Laight wrote: > From: Neil Horman > > Sent: 28 September 2015 14:51 > > On Mon, Sep 28, 2015 at 02:34:04PM +0200, Denys Vlasenko wrote: > > > Seemingly innocuous sctp_trans_state_to_prio_map[] array > > > is way bigger than it looks, since > > >

Re: [PATCH v2] net: sctp: Don't use 64 kilobyte lookup table for four elements

2015-09-28 Thread Marcelo Ricardo Leitner
On Mon, Sep 28, 2015 at 02:34:04PM +0200, Denys Vlasenko wrote: > Seemingly innocuous sctp_trans_state_to_prio_map[] array > is way bigger than it looks, since > "[SCTP_UNKNOWN] = 2" expands into "[0x] = 2" ! > > This patch replaces it with switch() statement. > > Signed-off-by: Denys

Re: [PATCH v2] net: sctp: Don't use 64 kilobyte lookup table for four elements

2015-09-28 Thread Neil Horman
On Mon, Sep 28, 2015 at 02:34:04PM +0200, Denys Vlasenko wrote: > Seemingly innocuous sctp_trans_state_to_prio_map[] array > is way bigger than it looks, since > "[SCTP_UNKNOWN] = 2" expands into "[0x] = 2" ! > > This patch replaces it with switch() statement. > > Signed-off-by: Denys