On Wed, 2017-06-07 at 14:54 -0400, David Miller wrote:
> From: Eric Dumazet <eric.duma...@gmail.com>
> Date: Tue, 06 Jun 2017 12:19:27 -0700
>
> > @@ -320,18 +320,37 @@ struct tcp_splice_state {
> > * All the __sk_mem_schedule() is of this nature: accounting
> > * is strict, actions are advisory and have some latency.
> > */
> > -int tcp_memory_pressure __read_mostly;
> > +unsigned long tcp_memory_pressure __read_mostly;
> > EXPORT_SYMBOL(tcp_memory_pressure);
>
> I know it is not strictly related to your change, but if you are
> adding new exports in this area let's do it properly.
>
> The existing tcp_memory_pressure export, as well as these two:
>
> ...
> > EXPORT_SYMBOL(tcp_enter_memory_pressure);
> ...
> > +EXPORT_SYMBOL(tcp_leave_memory_pressure);
>
> ought to be EXPORT_SYMBOL_GPL(). They are exported for the sake of
> ipv6 TCP support, and nothing else.
>
> I'll apply this with that fixed.
>
> Thanks Eric.
Sure, I will send a V2 with the two EXPORT_SYMBOL_GPL()
Thanks.
