Page MenuHomeFreeBSD

glebius (Gleb Smirnoff)
User

Projects (6)

User Details

User Since
May 28 2014, 2:27 PM (601 w, 5 d)

Recent Activity

Yesterday

glebius added a comment to D53872: bpf: modularize ifnet(9) part of bpf.

I'd like to push D53870 through D53877 this week. More review is appreciated.

Mon, Dec 8, 5:30 PM
glebius committed rGb986aa05a84d: lltable: use own lock (authored by glebius).
lltable: use own lock
Mon, Dec 8, 5:21 PM
glebius closed D54086: lltable: use own lock.
Mon, Dec 8, 5:21 PM
glebius committed rG607f11055d2d: linux: store Linux Ethernet interface number in struct ifnet (authored by glebius).
linux: store Linux Ethernet interface number in struct ifnet
Mon, Dec 8, 5:21 PM
glebius committed rGfbf05d2147b1: linux: separate all ifnet(9) related code into linux_ifnet.c (authored by glebius).
linux: separate all ifnet(9) related code into linux_ifnet.c
Mon, Dec 8, 5:21 PM
glebius closed D54077: linux: store Linux Ethernet interface number in struct ifnet.
Mon, Dec 8, 5:21 PM
glebius closed D54076: linux: separate all ifnet(9) related code into linux_ifnet.c.
Mon, Dec 8, 5:21 PM

Sun, Dec 7

glebius added a comment to D54109: netlink: use same cred as caller's cred similar to ifioctl.

Looks correct! Thanks. What did happen before? This command failed?

Sun, Dec 7, 5:14 PM

Sat, Dec 6

glebius committed rG6f4debc004c3: tcp: fix build with RSS (authored by glebius).
tcp: fix build with RSS
Sat, Dec 6, 4:33 PM
glebius committed rGa83191c928cd: net: fix LINT-NOIP build (authored by glebius).
net: fix LINT-NOIP build
Sat, Dec 6, 4:24 PM

Fri, Dec 5

glebius committed rGbc14dcd113ae: tests/if_bridge: mark several tests as requring vlan(4) or gif(4) (authored by glebius).
tests/if_bridge: mark several tests as requring vlan(4) or gif(4)
Fri, Dec 5, 7:31 PM
glebius requested review of D54089: net: attach IPv4 and IPv6 stacks to an interface with EVENTHANDLER(9).
Fri, Dec 5, 5:36 PM
glebius accepted D54088: ip6: Add explicit lock order information to catch boot-time LOR.
Fri, Dec 5, 5:35 PM
glebius requested review of D54086: lltable: use own lock.
Fri, Dec 5, 4:53 PM
glebius requested review of D54085: net: invoke ifnet_arrival_event handlers before linking in.
Fri, Dec 5, 4:51 PM
glebius committed rG8291ea8761c2: netinet6: use IF_ADDR_LOCK instead of IF_AFDATA_LOCK in defrtr_ipv6_only_ifp (authored by glebius).
netinet6: use IF_ADDR_LOCK instead of IF_AFDATA_LOCK in defrtr_ipv6_only_ifp
Fri, Dec 5, 4:38 PM
glebius committed rG59062fd0b2f1: netinet6: use IF_ADDR_LOCK instead of IF_AFDATA_LOCK (authored by glebius).
netinet6: use IF_ADDR_LOCK instead of IF_AFDATA_LOCK
Fri, Dec 5, 4:38 PM
glebius accepted D53787: ipfilter: Disable ipfs(8) by default.

No objection.

Fri, Dec 5, 3:44 PM
glebius updated the diff for D54077: linux: store Linux Ethernet interface number in struct ifnet.
  • Make the netlink_writer cookie an honest const ifnet pointer.
  • Refactor some code to provide the new constness constraint.
Fri, Dec 5, 5:42 AM

Thu, Dec 4

glebius requested review of D54077: linux: store Linux Ethernet interface number in struct ifnet.
Thu, Dec 4, 8:29 PM
glebius requested review of D54076: linux: separate all ifnet(9) related code into linux_ifnet.c.
Thu, Dec 4, 8:29 PM
glebius committed rGfd131b47f20d: net: remove dom_ifmtu (authored by glebius).
net: remove dom_ifmtu
Thu, Dec 4, 7:24 PM
glebius committed rGc7f05ef82478: net: routing table attach never fails (authored by glebius).
net: routing table attach never fails
Thu, Dec 4, 7:12 PM
glebius committed rG27ff90cd3d8d: tests/carp: make a 0.2 second pause before configuring second jail (authored by glebius).
tests/carp: make a 0.2 second pause before configuring second jail
Thu, Dec 4, 6:52 PM
glebius committed rGe353cbffd600: tests/carp: make sleep interval in the wait loop smaller (authored by glebius).
tests/carp: make sleep interval in the wait loop smaller
Thu, Dec 4, 6:52 PM
glebius committed rG575efcfff22f: sendfile: remove SF_SYNC in FreeBSD 16.0 (authored by glebius).
sendfile: remove SF_SYNC in FreeBSD 16.0
Thu, Dec 4, 4:45 PM
glebius committed rG215c58a1c677: pf: make unhandled_af() inline (authored by glebius).
pf: make unhandled_af() inline
Thu, Dec 4, 4:45 PM
glebius closed D54064: pf: make unhandled_af() inline.
Thu, Dec 4, 4:45 PM
glebius requested review of D54064: pf: make unhandled_af() inline.
Thu, Dec 4, 12:35 AM
glebius added a reviewer for D54063: usb: don't create ifnet(9) for usbus devices: USB.
Thu, Dec 4, 12:20 AM
glebius requested review of D54063: usb: don't create ifnet(9) for usbus devices.
Thu, Dec 4, 12:20 AM
glebius updated the diff for D53877: ipfw: create a bpf tap point for every log rule.
  • Address the fact that many rules can have the same number.
  • Support 32-bit rule numbers.
Thu, Dec 4, 12:18 AM
glebius updated the diff for D53872: bpf: modularize ifnet(9) part of bpf.
  • Rebase + unsignigicant changes.
Thu, Dec 4, 12:16 AM
glebius updated the diff for D53871: bpf: don't recursively enter the network epoch in bpf taps.
  • Address USB.
Thu, Dec 4, 12:16 AM
glebius updated the diff for D53870: bpf: convert several boolean natured fields of bpf_d to flags.

The d_promisc appears to be locked by the global BPF_LOCK(). Thus, remove it
from this changes. This addresses Mark's comments.

Thu, Dec 4, 12:14 AM

Wed, Dec 3

glebius committed rG72ca89482afc: bpf: global bpf list doesn't need CK (authored by glebius).
bpf: global bpf list doesn't need CK
Wed, Dec 3, 11:20 PM
glebius committed rG845896655815: bpf: calculate net.bpf.stats buffer size dynamically (authored by glebius).
bpf: calculate net.bpf.stats buffer size dynamically
Wed, Dec 3, 11:20 PM
glebius committed rG57673d166940: bpf: retire struct bpf_if_ext (authored by glebius).
bpf: retire struct bpf_if_ext
Wed, Dec 3, 11:20 PM
glebius closed D53869: bpf: global bpf list doesn't need CK.
Wed, Dec 3, 11:20 PM
glebius closed D53868: bpf: calculate net.bpf.stats buffer size dynamically.
Wed, Dec 3, 11:20 PM
glebius committed rG73fe85e486d2: tcp: store flowid info in syncache (authored by glebius).
tcp: store flowid info in syncache
Wed, Dec 3, 11:20 PM
glebius committed rGee18f8be2165: tcp: don't set flowid in tcp_input() (authored by glebius).
tcp: don't set flowid in tcp_input()
Wed, Dec 3, 11:20 PM
glebius closed D53867: bpf: retire struct bpf_if_ext.
Wed, Dec 3, 11:20 PM
glebius closed D53886: tcp: don't set flowid in tcp_input().
Wed, Dec 3, 11:20 PM
glebius closed D51792: tcp: store flowid info in syncache.
Wed, Dec 3, 11:20 PM
glebius accepted D54053: lacp: Sort port map by interface index.
Wed, Dec 3, 10:50 PM
glebius added a comment to D54053: lacp: Sort port map by interface index.

My understanding. First, qsort() itself never passes NULL pointer. Second, could a map[] entry within p->pm_count be NULL or not? The new code assumes it can't be. Then we also have a case with lacp_port that has NULL lp_ifp, which is covered.

Wed, Dec 3, 6:56 PM
glebius added a comment to D54053: lacp: Sort port map by interface index.

If pointers in the map[] can be NULL, then you removed too much, didn't you?

Wed, Dec 3, 6:54 PM
glebius committed R11:90f81f4af51f: minidlna: wrap up a sub-release (authored by glebius).
minidlna: wrap up a sub-release
Wed, Dec 3, 5:53 PM
glebius added inline comments to D54053: lacp: Sort port map by interface index.
Wed, Dec 3, 5:18 PM

Tue, Nov 25

glebius added inline comments to D53870: bpf: convert several boolean natured fields of bpf_d to flags.
Tue, Nov 25, 5:32 PM
glebius updated the diff for D53868: bpf: calculate net.bpf.stats buffer size dynamically.
  • Apply Mark's suggestion
Tue, Nov 25, 6:05 AM
glebius added inline comments to D53877: ipfw: create a bpf tap point for every log rule.
Tue, Nov 25, 5:58 AM

Mon, Nov 24

glebius added a comment to D53899: arp: Reduce lifetime of ARP entries.

AFAIU, when arptimer() finds entry in state ARP_LLINFO_REACHABLE it gives it arp_rexmit time (1 second) to get feedback from the forwarding path and sets its state to ARP_LLINFO_VERIFY. In 1 second if feedback was collected, it will issue arprequest(). My concern is that the jitter is bigger than 1 second. I could be wrong, and my concern is erroneous. I just wanted to make sure that this specific behavior is checked to persist. I don't think we have a regression test for it.

Mon, Nov 24, 5:58 PM
glebius added a comment to D53899: arp: Reduce lifetime of ARP entries.

In the past we used to have a logic that would renew ARP entires before they expire. The ARP request is sent before an entry is about to be expired. The goal is to avoid to have a doubled RTT on normal traffic every time we renew the entry. I did a quick look around, and failed to find the logic. Is it still there?

Mon, Nov 24, 5:21 PM

Sat, Nov 22

glebius updated the summary of D51792: tcp: store flowid info in syncache.
Sat, Nov 22, 6:53 PM
glebius requested review of D53886: tcp: don't set flowid in tcp_input().
Sat, Nov 22, 6:53 PM
glebius updated the diff for D51792: tcp: store flowid info in syncache.
  • Rebase on main.
  • Separate tcp_input() change into its own commit.
Sat, Nov 22, 6:52 PM
glebius added a reviewer for D53876: ifconfig: print warning and return success on ipfw0, ipfwlog0 cloning: lytboris_gmail.com.
Sat, Nov 22, 6:10 AM
glebius added a reviewer for D53877: ipfw: create a bpf tap point for every log rule: lytboris_gmail.com.
Sat, Nov 22, 6:10 AM
glebius added a reviewer for D53872: bpf: modularize ifnet(9) part of bpf: lytboris_gmail.com.
Sat, Nov 22, 6:10 AM
glebius added a reviewer for D53875: ipfw: create "ipfw0" and "ipfwlog0" bpf tapping points without ifnet(9): lytboris_gmail.com.
Sat, Nov 22, 6:09 AM
glebius updated the diff for D53872: bpf: modularize ifnet(9) part of bpf.
  • Rebase
  • Build with MAC
  • Address bz's comment
Sat, Nov 22, 2:58 AM
glebius committed rGe20e5724e614: bpf: remove DDB code (authored by glebius).
bpf: remove DDB code
Sat, Nov 22, 12:08 AM
glebius added a comment to D53872: bpf: modularize ifnet(9) part of bpf.
In D53872#1230746, @bz wrote:

Lastly, have you tested radiotap after this?

Sat, Nov 22, 12:03 AM

Fri, Nov 21

glebius added a reviewer for D53877: ipfw: create a bpf tap point for every log rule: ae.
Fri, Nov 21, 11:02 PM
glebius added a reviewer for D53876: ifconfig: print warning and return success on ipfw0, ipfwlog0 cloning: ae.
Fri, Nov 21, 11:01 PM
glebius added a reviewer for D53875: ipfw: create "ipfw0" and "ipfwlog0" bpf tapping points without ifnet(9): ae.
Fri, Nov 21, 11:01 PM
glebius added reviewers for D53872: bpf: modularize ifnet(9) part of bpf: ae, adrian, melifaro.
Fri, Nov 21, 11:00 PM
glebius requested review of D53877: ipfw: create a bpf tap point for every log rule.
Fri, Nov 21, 10:59 PM
glebius requested review of D53876: ifconfig: print warning and return success on ipfw0, ipfwlog0 cloning.
Fri, Nov 21, 10:59 PM
glebius requested review of D53875: ipfw: create "ipfw0" and "ipfwlog0" bpf tapping points without ifnet(9).
Fri, Nov 21, 10:58 PM
glebius requested review of D53874: libpcap: obtain list of bpf tap points with BIOCGETIFLIST.
Fri, Nov 21, 10:58 PM
glebius requested review of D53873: bpf: add BIOCGETIFLIST ioctl that returns all available tap points.
Fri, Nov 21, 10:58 PM
glebius requested review of D53872: bpf: modularize ifnet(9) part of bpf.
Fri, Nov 21, 10:58 PM
glebius requested review of D53871: bpf: don't recursively enter the network epoch in bpf taps.
Fri, Nov 21, 10:56 PM
glebius requested review of D53870: bpf: convert several boolean natured fields of bpf_d to flags.
Fri, Nov 21, 10:56 PM
glebius requested review of D53869: bpf: global bpf list doesn't need CK.
Fri, Nov 21, 10:56 PM
glebius requested review of D53868: bpf: calculate net.bpf.stats buffer size dynamically.
Fri, Nov 21, 10:56 PM
glebius requested review of D53867: bpf: retire struct bpf_if_ext.
Fri, Nov 21, 10:56 PM
glebius committed rGfd91012ebff2: bpf: leave only locked version of bpf_detachd() (authored by glebius).
bpf: leave only locked version of bpf_detachd()
Fri, Nov 21, 10:53 PM
glebius committed rGff3ccf6f1a02: bpf: refactor buffer pre-allocation for BIOCSETIF (authored by glebius).
bpf: refactor buffer pre-allocation for BIOCSETIF
Fri, Nov 21, 10:45 PM
glebius committed rG5469a3493b17: bpf: remove dead code (authored by glebius).
bpf: remove dead code
Fri, Nov 21, 10:45 PM
glebius committed rG88b38d43f557: ipfw: add extra parenthesis around ACTION_PTR() macro (authored by glebius).
ipfw: add extra parenthesis around ACTION_PTR() macro
Fri, Nov 21, 10:45 PM
glebius committed rGfd0296154d47: mbuf: allow const pointer for m_rcvif() (authored by glebius).
mbuf: allow const pointer for m_rcvif()
Fri, Nov 21, 10:45 PM
glebius committed rG1ea3eda3d016: tests/net: add some bpf(4) tests (authored by glebius).
tests/net: add some bpf(4) tests
Fri, Nov 21, 10:45 PM
glebius committed rGcf95eabb5572: netlink: in snl_init_writer() don't overwrite error in case of failure (authored by Bruno Silvestre <bruno.silvestre@gmail.com>).
netlink: in snl_init_writer() don't overwrite error in case of failure
Fri, Nov 21, 4:52 PM

Fri, Nov 14

glebius committed rG82d8a5029a80: unix/stream: fix instant panic w/o INVARIANTS (authored by glebius).
unix/stream: fix instant panic w/o INVARIANTS
Fri, Nov 14, 4:05 PM
glebius committed rG69f61cee2efb: unix/stream: fix a race with MSG_PEEK on SOCK_SEQPACKET with MSG_EOR (authored by glebius).
unix/stream: fix a race with MSG_PEEK on SOCK_SEQPACKET with MSG_EOR
Fri, Nov 14, 2:41 AM
glebius closed D53632: unix/stream: fix a race with MSG_PEEK on SOCK_SEQPACKET with MSG_EOR.
Fri, Nov 14, 2:40 AM

Thu, Nov 13

glebius accepted D53744: file: Add a fdclose method.
Thu, Nov 13, 10:48 PM
glebius accepted D53743: unix: Fix handling of listening sockets during garbage collection.
Thu, Nov 13, 10:39 PM
glebius added inline comments to D53517: Add ecn(9) manual.
Thu, Nov 13, 3:54 PM
glebius added inline comments to D53516: Update ip_ecn to RFC 6040.
Thu, Nov 13, 3:44 PM

Tue, Nov 11

glebius added inline comments to D53632: unix/stream: fix a race with MSG_PEEK on SOCK_SEQPACKET with MSG_EOR.
Tue, Nov 11, 7:44 PM
glebius updated the diff for D53632: unix/stream: fix a race with MSG_PEEK on SOCK_SEQPACKET with MSG_EOR.
  • Keep the error case freeing loop under the lock.
  • Assert before subtracting.
Tue, Nov 11, 3:28 PM

Mon, Nov 10

glebius committed rGa3cd3a4fd68c: netlink: plug lock leak in genl_register_family() (authored by Bruno Silvestre <bruno.silvestre@gmail.com>).
netlink: plug lock leak in genl_register_family()
Mon, Nov 10, 5:37 PM

Sun, Nov 9

glebius updated the diff for D53632: unix/stream: fix a race with MSG_PEEK on SOCK_SEQPACKET with MSG_EOR.

Fix pointer for partial mbuf in peek case.

Sun, Nov 9, 9:36 PM

Nov 8 2025

glebius updated the summary of D53632: unix/stream: fix a race with MSG_PEEK on SOCK_SEQPACKET with MSG_EOR.
Nov 8 2025, 11:40 PM
glebius updated the diff for D53632: unix/stream: fix a race with MSG_PEEK on SOCK_SEQPACKET with MSG_EOR.

New version.

Nov 8 2025, 11:38 PM