From: Carsten Andrich <carsten.andrich-hs6bpBdVsEZfm0AUMx9V0g@public.gmane.org>
To: "Michael Kerrisk (man-pages)"
<mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: 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>
Subject: Re: [patch] packet.7: PACKET_LOSS has inverse meaning
Date: Thu, 24 Apr 2014 11:39:31 +0200 [thread overview]
Message-ID: <1da011719b1828d43232fd8d91b609ff@localhost> (raw)
In-Reply-To: <5358C957.8080402-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
"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.
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'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 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 :)
Cheers,
Carsten
--
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
next prev parent reply other threads:[~2014-04-24 9:39 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 [this message]
2014-04-24 10:21 ` Michael Kerrisk (man-pages)
[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=1da011719b1828d43232fd8d91b609ff@localhost \
--to=carsten.andrich-hs6bpbdvsezfm0aumx9v0g@public.gmane.org \
--cc=dborkman-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org \
--cc=linux-man-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mtk.manpages-Re5JQEeQqe8AvxtiuMwx3w@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).