SLIP: Is there a maintainer for drivers/net/slip/slip.c ?

SLIP: Is there a maintainer for drivers/net/slip/slip.c ?

[Dean Jenkins]

Hi,

Using the Linux v3.9 tag, I note that

./scripts/get_maintainer.pl -f drivers/net/slip/slip.c
netdev@vger.kernel.org (open list:NETWORKING DRIVERS)
linux-kernel@vger.kernel.org (open list)

There seems to be no maintainer for drivers/net/slip/slip.c, is that true ?

I am asking because sl_encaps() and slip_write_wakeup() do not handle 
error codes from tty->ops->write() and a recursive stack overflow crash 
can occur if the tty->ops->write() fails to write all the characters. I 
have some patches to fix this but would like some feedback on an 
appropriate solution.

Thanks,

Dean Jenkins
Mentor Graphics

Re: SLIP: Is there a maintainer for drivers/net/slip/slip.c ?

[Geert Uytterhoeven]

On Mon, Jun 24, 2013 at 10:27 AM, Dean Jenkins <Dean_Jenkins@mentor.com> wrote:
> Using the Linux v3.9 tag, I note that
>
> ./scripts/get_maintainer.pl -f drivers/net/slip/slip.c
> netdev@vger.kernel.org (open list:NETWORKING DRIVERS)
> linux-kernel@vger.kernel.org (open list)
>
> There seems to be no maintainer for drivers/net/slip/slip.c, is that true ?

netdev seems to be the most appropriate mailing list, cfr. what
get_maintainer.pl reported, hence CC netdev added.

> I am asking because sl_encaps() and slip_write_wakeup() do not handle error
> codes from tty->ops->write() and a recursive stack overflow crash can occur
> if the tty->ops->write() fails to write all the characters. I have some
> patches to fix this but would like some feedback on an appropriate solution.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: SLIP: Is there a maintainer for drivers/net/slip/slip.c ?

[Peter Hurley]

On 06/24/2013 04:27 AM, Dean Jenkins wrote:
> Hi,
>
> Using the Linux v3.9 tag, I note that
>
> ./scripts/get_maintainer.pl -f drivers/net/slip/slip.c
> netdev@vger.kernel.org (open list:NETWORKING DRIVERS)
> linux-kernel@vger.kernel.org (open list)
>
> There seems to be no maintainer for drivers/net/slip/slip.c, is that true ?
>
> I am asking because sl_encaps() and slip_write_wakeup() do not handle error codes from tty->ops->write() and a recursive stack overflow crash can occur if the tty->ops->write() fails to write all the characters. I have some patches to fix this but would like some feedback on an appropriate solution.

SLIP changes would likely go through David Miller <davem@davemloft.net>.
TTY changes would go through Greg Kroah-Hartman <gregkh@linuxfoundation.org>

Regards,
Peter Hurley

Re: SLIP: Is there a maintainer for drivers/net/slip/slip.c ?

[Dean Jenkins]

Hi Peter,

On 24/06/13 14:53, Peter Hurley wrote:
> On 06/24/2013 04:27 AM, Dean Jenkins wrote:
>> Hi,
>>
>> Using the Linux v3.9 tag, I note that
>>
>> ./scripts/get_maintainer.pl -f drivers/net/slip/slip.c
>> netdev@vger.kernel.org (open list:NETWORKING DRIVERS)
>> linux-kernel@vger.kernel.org (open list)
>>
>> There seems to be no maintainer for drivers/net/slip/slip.c, is that 
>> true ?
>>
>> I am asking because sl_encaps() and slip_write_wakeup() do not handle 
>> error codes from tty->ops->write() and a recursive stack overflow 
>> crash can occur if the tty->ops->write() fails to write all the 
>> characters. I have some patches to fix this but would like some 
>> feedback on an appropriate solution.
>
> SLIP changes would likely go through David Miller <davem@davemloft.net>.
> TTY changes would go through Greg Kroah-Hartman 
> <gregkh@linuxfoundation.org>
I have already sent an E-mail to David Miller and the netdev mailing 
list but so far got not reply from David.

So I guess I have sent my patches to the correct place ;)

Thanks,

Regards,
Dean Jenkins
Mentor Graphics