Re: [PATCH] Packet loss when 'control' messages are present with large data (sendmsg(2))

From: Alan Somers <asomers_at_freebsd.org>
Date: Tue, 26 Aug 2014 15:22:23 -0600
On Tue, Aug 26, 2014 at 1:51 PM, Mark Johnston <markj_at_freebsd.org> wrote:
> On Tue, Aug 26, 2014 at 03:15:31PM -0400, John Baldwin wrote:
>> On Tuesday, August 26, 2014 11:05:12 am Alan Somers wrote:
>> > On Mon, Aug 25, 2014 at 1:52 PM, John Baldwin <jhb_at_freebsd.org> wrote:
>> > > On Friday, August 22, 2014 01:34:28 PM Harald Schmalzbauer wrote:
>> > >>  Bezüglich Yuri's Nachricht vom 02.09.2013 06:54 (localtime):
>> > >> > Please check in this patch:
>> > >> > http://www.freebsd.org/cgi/query-pr.cgi?pr=181741
>> > >> > Please MFC into 9.X
>> > >> >
>> > >> > Description of the problem is within PR.
>> > >> >
>> > >> > Thanks,
>> > >> > Yuri
>> > >>
>> > >> Hello,
>> > >>
>> > >> I guess this fix should make it into 10.1.
>> > >> Can someone check please?
>> > >
>> > > A fix has to make into HEAD first.  I've cc'd Alan who responded to the bug.
>> > > Alan, note that glebius_at_ already committed the test case to HEAD a while ago.
>> >
>> > First, Gleb's testcase needs to be converted to ATF.  This would be a
>> > good exercise for anybody who's new to ATF and wants some practice.
>> > Anybody interested?
>>
>> While that would be nice, I don't know that it's quite fair to require the
>> test to be converted before working on a possible fix.  The existing test
>> doesn't seem that hard to run by hand:
>>
>> % cd work/freebsd/head/tools/regression/sockets/unix_passfd/
>> % make
>> ...
>> % > ./unix_passfd
>> beginning test1-simplesendfd
>> test1-simplesendfd passed
>> ...
>> beginning test8-rigths+creds+payload
>> unix_passfd: test8-rigths+creds+payload: recvmsg: 24 bytes received
>>
>> I only say this because in the bug followup you seemed to have described a
>> possible solution so it seems that you would be able to develop a fix quicker
>> than other folks since you are already familiar with the issues involved.
>> (Also, you've fixed other related issues recently.)
>
> As it happens, I went ahead and did this anyway:
> https://reviews.freebsd.org/D689

BTW, is it just me, or is arcanist insanely slow?  Usually "arc diff
--create" or "arc diff --update" take many minutes to complete.  Like,
30 minutes.  I've been trying to do "arc patch" for nearly an hour
now, but it hasn't completed yet.
Received on Tue Aug 26 2014 - 19:22:27 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:51 UTC