[<prev] [next>] [day] [month] [year] [list]
Message-ID: <CA+FuTSdgvgt_5HwRdY=OA+dv=nds4VELQ-ZAXdxmHJ3mr0CDnQ@mail.gmail.com>
Date: Wed, 1 Oct 2014 11:57:31 -0400
From: Willem de Bruijn <willemb@...gle.com>
To: Fengguang Wu <fengguang.wu@...el.com>
Cc: Dave Hansen <dave.hansen@...el.com>,
Network Development <netdev@...r.kernel.org>,
LKML <linux-kernel@...r.kernel.org>, lkp@...org
Subject: Re: [LKP] [net] 4ed2d765dfa: ltp.recv01.4.TFAIL
On Wed, Oct 1, 2014 at 11:29 AM, Fengguang Wu <fengguang.wu@...el.com> wrote:
> Hi Willem,
>
> FYI, we noticed the below LTP failures on commit
Thanks for the report, Fengguang. The failures are
recv01 4 TFAIL : recv01.c:142: invalid flags set ; returned -1
(expected -1), errno 11 (expected 22)
recvfrom01 4 TFAIL : recvfrom01.c:164: invalid socket length ;
returned -1 (expected -1), errno 11 (expected 22)
recvfrom01 6 TFAIL : recvfrom01.c:164: invalid flags set ;
returned -1 (expected -1), errno 11 (expected 22)
recvmsg01 9 TFAIL : recvmsg01.c:228: invalid flags set ;
returned -1 (expected -1), errno 11 (expected 22)
In other words, these functions return EAGAIN now, when they used to
return EINVAL. This will happen from this patch onwards if flags
includes MSG_ERRQUEUE. The patch introduced error queue support for
TCP sockets. Like other recvmsg implementations, the error queue code
will act on some flags and ignore others. Previously, the code
happened to check MSG_OOB first and return EINVAL. The error queue
code ignores this flag and returns EAGAIN if no data is waiting.
This was recently discussed in
LTP recv/recvmsg tests failing on 3.17
http://comments.gmane.org/gmane.linux.network/331750
where the consensus was that applications should not expect
particular error codes when passing unsupported combinations
of flags.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists