linux-man.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Michael Kerrisk (man-pages)" <mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Carsten Andrich
	<carsten.andrich-hs6bpBdVsEZfm0AUMx9V0g@public.gmane.org>
Cc: mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
	Willem de Bruijn
	<willemb-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>,
	Daniel Borkmann
	<dborkman-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
	linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Vince Weaver
	<vincent.weaver-e7X0jjDqjFGHXe+LvDLADg@public.gmane.org>,
	Neil Horman <nhorman-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>,
	Andi Kleen <andi-Vw/NltI1exuRpAAqCnN02g@public.gmane.org>
Subject: Re: [patch] packet.7: PACKET_LOSS has inverse meaning
Date: Thu, 24 Apr 2014 12:21:18 +0200	[thread overview]
Message-ID: <5358E59E.2000207@gmail.com> (raw)
In-Reply-To: <1da011719b1828d43232fd8d91b609ff@localhost>

On 04/24/2014 11:39 AM, Carsten Andrich wrote:
> "Michael Kerrisk (man-pages)" schrieb:
>> On 04/23/2014 09:10 PM, Carsten Andrich wrote:
>>> [...]
>>> Unfortunately my experience with Linux packet sockets is very limited. I
>>> messed with them a few times, but only really started using SOCK_RAW
>>> packet sockets (including PACKET_MMAP) a few weeks ago (as a part of my
>>> Bachelor's thesis). I haven't looked at the linux (networking) code
>>> before Monday, when I tried to figure out if either packet.7 or I was
>>> wrong regarding PACKET_LOSS. And I've never even touched SOCK_DGRAM.
>>>
>>> All this doesn't exactly qualify me for the job :)
>>
>> Maybe, maybe not. On the other hand, I have the idea (may be wrong) 
>> that Vince Weaver was a beginner with perf_event_open() when he
>> started sending me patches. He's continued doing so with a vengeance
>> (thanks again Vince!). That has been a great help, since 
>> perf_event_open() is a complex, constantly evolving API surface, and
>> I just don't have the time to track the details. The API described in 
>> packet.7 has not changed much over time, but the man page has seen
>> little love over the years. An inquiring mind, willing to test things
>> and read code are the main prerequisites. ANd it sounds like some 
>> others might chip with help and Acks.
> 
> We could give it a shot. It may be of an advantage to not have a deep
> understanding of the code driving the interface one is trying to
> document, as to not leave out aspects which appear to be self-evident.

(Nod.)

> It would make sense to limit our efforts to getting the PACKET_MMAP
> documentation in good shape, at least for now.
> I'd definitely require a reviewer (who's willing to answer some
> questions in advance) with a good understanding of at_packet.c and since
> I've already mentioned the need for synchronizing packet.7 and
> packet_mmap.txt we'd also need a decent plan how to distribute/share
> information among both.

I'd leave that plan largely to you. It sounds like Willem and
Daniel are willing to help out. 

> I'll try to come up with some ideas once I've handed in my Bachelor's
> thesis (deadline ends 3 weeks from now).
> Is there anyone else from the kernel-side we'd need to involve besides
> Willem and Daniel?

Maybe even Neil and Andi (CCed) might chip in from time to time.

>>> Maybe we should continue this discussion in private, so we don't bore
>>> everyone to death.
>>
>> Drop me a private mail if you want, but there are also benefits 
>> to public conversations ;-).
> I just don't want to annoy anyone, since we kinda force Willem, Daniel
> and now Vince to read this :)

So it goes...

Thanks, Carsten.

Cheers,

Michael



-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2014-04-24 10:21 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-22 16:01 [patch] packet.7: PACKET_LOSS has inverse meaning Carsten Andrich
2014-04-22 18:23 ` Michael Kerrisk (man-pages)
     [not found]   ` <5356B3BA.7050401-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-04-22 18:36     ` Willem de Bruijn
     [not found]       ` <CA+FuTSc=4fwH=+jX58teeQHrRse718qLBOLzAcrZJSYug2TfSw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-04-22 18:55         ` Michael Kerrisk (man-pages)
     [not found]           ` <5356BB3F.4030502-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-04-22 19:06             ` Carsten Andrich
2014-04-22 19:14             ` Willem de Bruijn
     [not found]               ` <CA+FuTScUexjc6bETYMVgPUgmN5CSR40=UrELT9N363Tvk1+waA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-04-22 20:53                 ` Carsten Andrich
     [not found]                   ` <1398200019.5031.24.camel-FQO4gtnRtnzkVFMGpb/cPg@public.gmane.org>
2014-04-23  6:06                     ` Michael Kerrisk (man-pages)
     [not found]                       ` <53575872.6080403-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-04-23  8:51                         ` Carsten Andrich
2014-04-23 10:12                           ` Michael Kerrisk (man-pages)
     [not found]                             ` <53579224.3060006-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-04-23 19:10                               ` Carsten Andrich
     [not found]                                 ` <1398280218.2416.18.camel-FQO4gtnRtnzkVFMGpb/cPg@public.gmane.org>
2014-04-24  8:20                                   ` Michael Kerrisk (man-pages)
     [not found]                                     ` <5358C957.8080402-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-04-24  9:39                                       ` Carsten Andrich
2014-04-24 10:21                                         ` Michael Kerrisk (man-pages) [this message]
     [not found]                                           ` <5358E59E.2000207-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-04-24 10:45                                             ` Daniel Borkmann
     [not found]                                               ` <CA+FuTSf6_Lvs_Nht7TXPUT973rePz9hKEqv+YvPwXSq+kOoiyQ@mail.gmail.com>
     [not found]                                                 ` <CA+FuTSf6_Lvs_Nht7TXPUT973rePz9hKEqv+YvPwXSq+kOoiyQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-04-24 19:07                                                   ` Carsten Andrich
     [not found]                                                     ` <1398366446.2647.4.camel-FQO4gtnRtnzkVFMGpb/cPg@public.gmane.org>
2014-04-25 11:20                                                       ` Neil Horman
2014-04-24 10:59                                             ` Neil Horman
2014-04-23 20:53                               ` Stefan Puiu

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=5358E59E.2000207@gmail.com \
    --to=mtk.manpages-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
    --cc=andi-Vw/NltI1exuRpAAqCnN02g@public.gmane.org \
    --cc=carsten.andrich-hs6bpBdVsEZfm0AUMx9V0g@public.gmane.org \
    --cc=dborkman-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
    --cc=linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=nhorman-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org \
    --cc=vincent.weaver-e7X0jjDqjFGHXe+LvDLADg@public.gmane.org \
    --cc=willemb-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).