linux-man.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

  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).