LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Brent Casavant <bcasavan@sgi.com>
To: Herbert Xu <herbert@gondor.apana.org.au>
Cc: penguin-kernel@i-love.sakura.ne.jp, netdev@vger.kernel.org,
davem@davemloft.net, linux-kernel@vger.kernel.org
Subject: Re: AF_UNIX MSG_PEEK bug?
Date: Wed, 9 Jan 2008 19:19:04 -0600 (CST) [thread overview]
Message-ID: <alpine.BSF.1.00.0801091914220.35527@pkunk.americas.sgi.com> (raw)
In-Reply-To: <E1JCkrp-0000vX-00@gondolin.me.apana.org.au>
On Thu, 10 Jan 2008, Herbert Xu wrote:
> The POSIX text for MSG_WAITALL actually says that when used in
> conjunction with MSG_PEEK it may not return the full data.
That's fine. The problem is that the peek operation returns less
data than requested even when sufficient data is available on the
receive queue.
> However, if you really want this to
> happen it would help if you had attached a patch :)
A patch is definitely in progress. I'm a little confused as
to the difference between unix_detach_fds() and scm_fp_dup()
in the MSG_PEEK versus !MSG_PEEK paths in unix_stream_recvmsg(),
however once I figure that out a patch should be forthcoming.
Brent
--
Brent Casavant All music is folk music. I ain't
bcasavan@sgi.com never heard a horse sing a song.
Silicon Graphics, Inc. -- Louis Armstrong
next prev parent reply other threads:[~2008-01-10 1:19 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-08 22:27 Brent Casavant
2008-01-08 22:40 ` Rick Jones
2008-01-08 22:53 ` Tom Spink
2008-01-08 23:18 ` Brent Casavant
2008-01-08 23:39 ` Tom Spink
2008-01-08 23:46 ` Tom Spink
2008-01-09 0:08 ` Brent Casavant
2008-01-08 23:20 ` Brent Casavant
2008-01-09 11:04 ` Tetsuo Handa
2008-01-09 18:01 ` Brent Casavant
2008-01-10 0:01 ` Herbert Xu
2008-01-10 1:19 ` Brent Casavant [this message]
2008-01-10 2:50 ` Brent Casavant
2008-01-10 22:35 ` Brent Casavant
2008-01-10 22:36 ` Alan Cox
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.BSF.1.00.0801091914220.35527@pkunk.americas.sgi.com \
--to=bcasavan@sgi.com \
--cc=davem@davemloft.net \
--cc=herbert@gondor.apana.org.au \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=penguin-kernel@i-love.sakura.ne.jp \
--subject='Re: AF_UNIX MSG_PEEK bug?' \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).