Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(7251)

Issue 11759044: code review 11759044: runtime: drop EV_RECEIPT support from network pollster ... (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 9 months ago by mikio
Modified:
10 years, 9 months ago
Reviewers:
dvyukov
CC:
dvyukov, minux1, golang-dev
Visibility:
Public.

Description

runtime: drop EV_RECEIPT support from network pollster on kqueue Currently Darwin and FreeBSD support and NetBSD and OpenBSD do not support EV_RECEIPT flag. We will drop use of EV_RECEIPT for now. Also enables to build runtime-integrated network pollster on freebsd/amd64,386 and openbsd/amd64,386. It just does build but never runs pollster stuff. This is in preparation for runtime-integrated network pollster for BSD variants. Update issue 5199

Patch Set 1 #

Patch Set 2 : diff -r 84cafba689b1 https://go.googlecode.com/hg/ #

Patch Set 3 : diff -r 84cafba689b1 https://go.googlecode.com/hg/ #

Patch Set 4 : diff -r e7b0d65f9e5b https://go.googlecode.com/hg/ #

Patch Set 5 : diff -r a5e5a59efd47 https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -13 lines) Patch
M src/pkg/runtime/netpoll.goc View 1 1 chunk +1 line, -1 line 0 comments Download
M src/pkg/runtime/netpoll_kqueue.c View 1 2 3 2 chunks +3 lines, -11 lines 0 comments Download
M src/pkg/runtime/netpoll_stub.c View 1 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6
mikio
Hello dvyukov@google.com, minux.ma@gmail.com (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://go.googlecode.com/hg/
10 years, 9 months ago (2013-07-24 09:04:50 UTC) #1
dvyukov
Do I understand it correctly that darwin reports errors with EV_RECEIPT and bsd systems with ...
10 years, 9 months ago (2013-07-24 09:17:23 UTC) #2
mikio
On Wed, Jul 24, 2013 at 6:17 PM, <dvyukov@google.com> wrote: > Do I understand it ...
10 years, 9 months ago (2013-07-24 10:05:00 UTC) #3
mikio
Hello dvyukov@google.com, minux.ma@gmail.com (cc: golang-dev@googlegroups.com), Please take another look.
10 years, 9 months ago (2013-07-24 15:07:37 UTC) #4
dvyukov
LGTM
10 years, 9 months ago (2013-07-24 15:17:43 UTC) #5
mikio
10 years, 9 months ago (2013-07-24 15:26:02 UTC) #6
*** Submitted as https://code.google.com/p/go/source/detail?r=acc121ea2d70 ***

runtime: drop EV_RECEIPT support from network pollster on kqueue

Currently Darwin and FreeBSD support and NetBSD and OpenBSD do not
support EV_RECEIPT flag. We will drop use of EV_RECEIPT for now.

Also enables to build runtime-integrated network pollster on
freebsd/amd64,386 and openbsd/amd64,386. It just does build but never
runs pollster stuff.

This is in preparation for runtime-integrated network pollster for BSD
variants.

Update issue 5199

R=dvyukov, minux.ma
CC=golang-dev
https://codereview.appspot.com/11759044
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b