diff options
| author | Suriyan Ramasami <suriyan.r@gmail.com> | 2013-10-16 09:54:24 -0700 |
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2013-11-08 15:25:14 -0500 |
| commit | efd9bb9c0286dace970397e3336cf0399dc7b31d (patch) | |
| tree | d1543b86a7153bfc4367406459bdf43fd7359ae0 /drivers | |
| parent | a81630e0bfbf08ceb49d29bb56e9b6da84538b51 (diff) | |
netconsole loses 2nd character of input
Netconsole loses the second character when used as input by
either setenv stdin nc or setenv stdin serial,nc if using CONSOLE_CONSOLE_MUX
Before a nc_send_packet() to echo the input, a check is done to see if
nc_ether is valid. If its not, it waits for an arp request and then sends
the packet (which contains the first character of line to be displayed as
output). As part of reaping the arp request, the second character is consumed.
We protect this by making the call to NetLoop(NETCONS) between
input_recursion.
Signed-off-by: Suriyan Ramasami <suriyan.r@gmail.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/netconsole.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/netconsole.c b/drivers/net/netconsole.c index df8ab07b94..65c747e14b 100644 --- a/drivers/net/netconsole.c +++ b/drivers/net/netconsole.c @@ -184,7 +184,9 @@ static void nc_send_packet(const char *buf, int len) return; /* inside net loop */ output_packet = buf; output_packet_len = len; + input_recursion = 1; NetLoop(NETCONS); /* wait for arp reply and send packet */ + input_recursion = 0; output_packet_len = 0; return; } |