Bug 206185 - [netgraph][patch] refactoring ng_patch
Summary: [netgraph][patch] refactoring ng_patch
Status: Closed FIXED
Alias: None
Product: Base System
Classification: Unclassified
Component: kern (show other bugs)
Version: 10.2-STABLE
Hardware: Any Any
: --- Affects Some People
Assignee: Julian Elischer
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-01-13 09:20 UTC by Dmitry Wagin
Modified: 2017-12-05 15:10 UTC (History)
0 users

See Also:
julian: mfc-stable11-
julian: mfc-stable10-

Attachments
refactoring ng_patch (26.96 KB, patch)
2016-01-13 09:20 UTC, Dmitry Wagin 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Dmitry Wagin 2016-01-13 09:20:56 UTC 
Created attachment 165484 [details]
refactoring ng_patch

* Code cleanup
* Add IPv6 checksum flags
* Add setdlt for relative offset (support DLT_EN10MB and DLT_RAW)
Comment 1 commit-hook freebsd_committer freebsd_triage 2016-08-01 11:34:46 UTC 
A commit references this bug:

Author: julian
Date: Mon Aug  1 11:34:13 UTC 2016
New revision: 303611
URL: https://svnweb.freebsd.org/changeset/base/303611

Log:
  slite style changes. There is an incoming patch that rewrites a
  lot of this module and I want to get the style and whitespace changes in
  a separate commit (or maybe more).

  PR: 206185
  Submitted by:	Dmitry Vagin
  MFC after:	1 month

Changes:
  head/sys/netgraph/ng_patch.c
  head/sys/netgraph/ng_patch.h
Comment 2 commit-hook freebsd_committer freebsd_triage 2016-12-02 05:42:55 UTC 
A commit references this bug:

Author: julian
Date: Fri Dec  2 05:42:05 UTC 2016
New revision: 309389
URL: https://svnweb.freebsd.org/changeset/base/309389

Log:
  MFH: r303611

  slite style changes. There is an incoming patch that rewrites a
  lot of this module and I want to get the style and whitespace changes in
  a separate commit (or maybe more).

  PR: 206185
  Submitted by:	Dmitry Vagin

Changes:
_U  stable/10/
  stable/10/sys/netgraph/ng_patch.c
  stable/10/sys/netgraph/ng_patch.h
Comment 3 commit-hook freebsd_committer freebsd_triage 2016-12-02 05:42:57 UTC 
A commit references this bug:

Author: julian
Date: Fri Dec  2 05:42:49 UTC 2016
New revision: 309390
URL: https://svnweb.freebsd.org/changeset/base/309390

Log:
  MFH: r303611

  slite style changes. There is an incoming patch that rewrites a
  lot of this module and I want to get the style and whitespace changes in
  a separate commit (or maybe more).

  PR: 206185
  Submitted by:	Dmitry Vagin

Changes:
_U  stable/11/
  stable/11/sys/netgraph/ng_patch.c
  stable/11/sys/netgraph/ng_patch.h
Comment 4 commit-hook freebsd_committer freebsd_triage 2016-12-02 10:48:00 UTC 
A commit references this bug:

Author: julian
Date: Fri Dec  2 10:47:10 UTC 2016
New revision: 309408
URL: https://svnweb.freebsd.org/changeset/base/309408

Log:
  Changes to allow the patching of packets with an offset (and other changes.. see man page)

  PR:	206185
  Submitted by:	Dmitry Vagin <daemon-hammer@ya.ru>
  MFC after:	 1 week
  Relnotes:	yes (also ng_checksum node)

Changes:
  head/share/man/man4/ng_patch.4
  head/sys/netgraph/ng_patch.c
  head/sys/netgraph/ng_patch.h
Comment 5 commit-hook freebsd_committer freebsd_triage 2016-12-27 08:32:16 UTC 
A commit references this bug:

Author: julian
Date: Tue Dec 27 08:31:41 UTC 2016
New revision: 310625
URL: https://svnweb.freebsd.org/changeset/base/310625

Log:
  MFH: r309408

  Changes to allow the patching of packets with an offset (and other changes.. see man page)

  PR:	206185
  Submitted by:	Dmitry Vagin <daemon-hammer@ya.ru>
  MFC after:	 1 week
  Relnotes:	yes (also ng_checksum node)

Changes:
_U  stable/11/
  stable/11/share/man/man4/ng_patch.4
  stable/11/sys/netgraph/ng_patch.c
  stable/11/sys/netgraph/ng_patch.h
Comment 6 Kubilay Kocak freebsd_committer freebsd_triage 2017-12-05 02:30:22 UTC 
Mark and classify to track what occurred. Assign to committer resolving.

What left to do to resolve/close?
Comment 7 Julian Elischer freebsd_committer freebsd_triage 2017-12-05 14:52:35 UTC 
 Second change will not be MFC'd to 10 as it introduces an API change.