[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5386563F.8000901@windriver.com>
Date: Wed, 28 May 2014 15:33:51 -0600
From: Chris Friesen <chris.friesen@...driver.com>
To: "'Arnaldo Carvalho de Melo'" <acme@...nel.org>,
David Laight <David.Laight@...LAB.COM>
CC: "Michael Kerrisk (man-pages)" <mtk.manpages@...il.com>,
lkml <linux-kernel@...r.kernel.org>,
"linux-man@...r.kernel.org" <linux-man@...r.kernel.org>,
netdev <netdev@...r.kernel.org>,
Ondrej Bílka <neleai@...nam.cz>,
Caitlin Bestler <caitlin.bestler@...il.com>,
Neil Horman <nhorman@...driver.com>,
Elie De Brauwer <eliedebrauwer@...il.com>,
David Miller <davem@...emloft.net>,
Steven Whitehouse <steve@...gwyn.com>,
Rémi Denis-Courmont
<remi.denis-courmont@...ia.com>, Paul Moore <paul@...l-moore.com>
Subject: Re: [PATCH/RFC] Re: recvmmsg() timeout behavior strangeness [RESEND]
On 05/28/2014 01:50 PM, 'Arnaldo Carvalho de Melo' wrote:
> What is being discussed here is how to return the EFAULT that may happen
> _after_ datagram processing, be it interrupted by an EFAULT, signal, or
> plain returning all that was requested, with no errors.
>
> This EFAULT _after_ datagram processing may happen when updating the
> remaining timeout, because then how can userspace both receive the
> number of successfully copied datagrams (in any of the cases mentioned
> in the previous paragraph) and know that that timeout can't be used
> because there was a problem while trying to copy it to userspace
> (EFAULT)?
How does select() handle this problem? It updates the timeout and also
modifies other data.
Could we just check whether the timeout pointer is valid before doing
anything else? Of course we could still fault the page out while
waiting for messages and then fail to fault it back in later, but that
seems like a not-very-likely scenario.
Chris
--
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