Page MenuHomeFreeBSD

afedorov (Aleksandr Fedorov)
User

Projects

User Details

User Since
Jan 18 2019, 4:52 AM (174 w, 13 h)

Recent Activity

Feb 23 2022

afedorov committed R10:009a56b2e416: if_vtnet(4): Restore the ability to set promisc mode. (authored by afedorov).
if_vtnet(4): Restore the ability to set promisc mode.
Feb 23 2022, 11:10 AM
afedorov committed R10:543b492fa569: if_vxlan(4): Allow netmap_generic to intercept RX packets. (authored by afedorov).
if_vxlan(4): Allow netmap_generic to intercept RX packets.
Feb 23 2022, 11:03 AM
afedorov committed R10:f40063d32977: md(4): Add dummy support of the BIO_FLUSH command for malloc and swap (authored by afedorov).
md(4): Add dummy support of the BIO_FLUSH command for malloc and swap
Feb 23 2022, 11:00 AM

Feb 21 2022

afedorov added a comment to D34319: bridge: Don't share broadcast packets.

Looks good to me.

Feb 21 2022, 12:50 PM

Feb 19 2022

afedorov added a comment to D34319: bridge: Don't share broadcast packets.

I'm concerned about two things:

Feb 19 2022, 11:57 AM

Feb 17 2022

afedorov committed R10:cb28dfb27d12: md(4): Add dummy support of the BIO_FLUSH command for malloc and swap (authored by afedorov).
md(4): Add dummy support of the BIO_FLUSH command for malloc and swap
Feb 17 2022, 7:25 PM
afedorov closed D34260: md(4): Add dummy support of the BIO_FLUSH command for malloc and swap backend..
Feb 17 2022, 7:25 PM
afedorov added a comment to D33326: bhyve: reset event after blockif resize.

I have this problem in STABLE-13 with disk image located on NFS share.

Feb 17 2022, 9:24 AM

Feb 16 2022

afedorov added inline comments to D34298: mbuf / lro: move defines from mbuf.h to tcp_lro.h.
Feb 16 2022, 7:16 PM

Feb 15 2022

afedorov added a comment to D34260: md(4): Add dummy support of the BIO_FLUSH command for malloc and swap backend..

@vmaffione , do you have any objections?

Feb 15 2022, 5:12 PM

Feb 14 2022

afedorov added a comment to D34260: md(4): Add dummy support of the BIO_FLUSH command for malloc and swap backend..

BTW, mdstart_vnode() does not set bio_resid after handling a BIO_FLUSH. I suspect it does not matter in practice, since by convention BIO_FLUSH commands have bio_length == 0.

Feb 14 2022, 12:33 PM
afedorov updated the diff for D34260: md(4): Add dummy support of the BIO_FLUSH command for malloc and swap backend..
Feb 14 2022, 12:32 PM

Feb 13 2022

afedorov requested review of D34260: md(4): Add dummy support of the BIO_FLUSH command for malloc and swap backend..
Feb 13 2022, 1:15 PM
afedorov committed R10:85cd9f7e989b: ng pppoe(4): Add the required NET_EPOCH section to the hook (authored by afedorov).
ng pppoe(4): Add the required NET_EPOCH section to the hook
Feb 13 2022, 12:09 PM

Feb 9 2022

afedorov committed R10:b27e6e91d0ad: ng pppoe(4): Add the required NET_EPOCH section to the hook (authored by afedorov).
ng pppoe(4): Add the required NET_EPOCH section to the hook
Feb 9 2022, 7:04 PM
afedorov closed D34185: ng pppoe(4): Add the required NET_EPOCH section to the hook disconnection function. .
Feb 9 2022, 7:04 PM

Feb 8 2022

afedorov added a comment to D34185: ng pppoe(4): Add the required NET_EPOCH section to the hook disconnection function. .

Question: there are three more locations in that source file where NG_SEND_DATA_ONLY() is called and it is not apparent that we are inside NET_EPOCH.
Should we enter NET_EPOCH also there, or we are already in?

Feb 8 2022, 11:43 AM

Feb 7 2022

afedorov added a reviewer for D34185: ng pppoe(4): Add the required NET_EPOCH section to the hook disconnection function. : jhb.
Feb 7 2022, 4:47 PM

Feb 6 2022

afedorov requested review of D34185: ng pppoe(4): Add the required NET_EPOCH section to the hook disconnection function. .
Feb 6 2022, 12:49 PM
afedorov committed R10:ceaf442ff236: if_vxlan(4): Allow netmap_generic to intercept RX packets. (authored by afedorov).
if_vxlan(4): Allow netmap_generic to intercept RX packets.
Feb 6 2022, 12:32 PM
afedorov closed D30638: if_vxlan(4): Allow netmap_generic to intercept RX packets..
Feb 6 2022, 12:32 PM

Feb 5 2022

afedorov closed D30639: if_vtnet(4): Restore the ability to set promisc mode..
Feb 5 2022, 3:54 PM
afedorov committed R10:fc035df8af32: if_vtnet(4): Restore the ability to set promisc mode. (authored by afedorov).
if_vtnet(4): Restore the ability to set promisc mode.
Feb 5 2022, 3:54 PM

Dec 22 2021

afedorov added a reviewer for D33448: vmm: vlapic resume can eat 100% CPU by vlapic_callout_handler: kib.
Dec 22 2021, 5:34 PM · bhyve

Dec 16 2021

afedorov added a comment to D26268: bhyve: snapshot virtio queues can fail.

Should this be committed?

Dec 16 2021, 3:15 PM · bhyve

Dec 15 2021

afedorov added reviewers for D33448: vmm: vlapic resume can eat 100% CPU by vlapic_callout_handler: jhb, markj, elenamihailescu22_gmail.com.
Dec 15 2021, 8:28 AM · bhyve

Nov 9 2021

afedorov added a comment to D32905: Add ETHER_ALIGN support to ng_device(4)..

I don't understand what the alignment does in this case.
Does it fix a bug or is it an optimization?
What part of the code requires this alignment, is it the TCP/IP stack?
Because not only ng_eiface(4) can inject Ethernet packets into the network stack.

Nov 9 2021, 5:06 PM

Nov 8 2021

afedorov added a comment to D32879: ext2fs: extract crc16 into sys/crc16.h.

The polynomial 0x8005 is CRC-16-IBM or CRC-16-ANSI.

Nov 8 2021, 6:47 PM

Oct 21 2021

afedorov abandoned D23987: [if_vtnet] Add VIRTIO_NET_F_MTU flag support..

This is done in FreeBSD 13.

Oct 21 2021, 4:18 PM
afedorov added a comment to D32550: ng_ether: implement 'attach'.

I am also against the use of generic messages.

Oct 21 2021, 4:00 PM

Sep 13 2021

afedorov added inline comments to D31941: Cast pointer to uintptr_t to avoid alignment warnings..
Sep 13 2021, 7:25 PM

Aug 12 2021

afedorov accepted D31486: ng_bridge: Use M_NOWAIT when allocating memory in the newhook routine.

I looked at other nodes that implement the ng_newhook_t method. They already use M_NOWAIT, so I think this patch is good.

Aug 12 2021, 7:26 AM

Aug 10 2021

afedorov added a reviewer for D31486: ng_bridge: Use M_NOWAIT when allocating memory in the newhook routine: afedorov.
Aug 10 2021, 3:12 PM
afedorov added a comment to D31486: ng_bridge: Use M_NOWAIT when allocating memory in the newhook routine.

The load simply allows this bug to manifest itself. Under load, the node's queue already contains the item's, so if a message item is sent at that moment (ngtstl shutdown), it will also go to the queue and will be processed by ngthread(). Without load, the queue is mostly empty, the message item will be delivered/processed by the thread of the calling process directly: ngctl -> syscall (sandto(2)) -> ngc_send (ng_socket(4)) -> ng_eiface(4) without entering to the EPOCH section. Therefore, the bug does not manifest itself.

Aug 10 2021, 3:04 PM
afedorov added a comment to D31486: ng_bridge: Use M_NOWAIT when allocating memory in the newhook routine.

JFYI, the same situation. If you call "ngctl shutdown ngeth0:" under load:

Aug 10 2021, 9:07 AM
afedorov added a comment to D31486: ng_bridge: Use M_NOWAIT when allocating memory in the newhook routine.

I think the problem is somewhat broader. Any message item can be added to the node's queue under load or if the queue is blocked. This message item will be processed by ngthread() in the EPOCH section. There are many places in the code of the nodes that perform actions prohibited in the EPOCH section.

Aug 10 2021, 9:02 AM

Jul 22 2021

afedorov added a comment to D31263: bpf: Add an ioctl to set the VLAN Priority on packets sent by bpf.

Overall, this patch looks good to me.

Jul 22 2021, 12:19 PM

Jul 7 2021

afedorov added inline comments to D31077: if_epair: rework.
Jul 7 2021, 3:09 PM

Jun 15 2021

afedorov added inline comments to D30668: Intel I225 Driver.
Jun 15 2021, 7:21 PM
afedorov added inline comments to D30638: if_vxlan(4): Allow netmap_generic to intercept RX packets..
Jun 15 2021, 2:24 PM

Jun 5 2021

afedorov added a comment to D30638: if_vxlan(4): Allow netmap_generic to intercept RX packets..
In D30638#688320, @np wrote:

I know it's not directly related but the 'if' here is not needed. If you're tightening the len checks in the function then maybe clean this too?

out:

if (m != NULL)
        m_freem(m);
Jun 5 2021, 9:15 AM
afedorov updated the diff for D30638: if_vxlan(4): Allow netmap_generic to intercept RX packets..
  • Move ethernet packet length check to vxlan_input().
Jun 5 2021, 9:11 AM

Jun 4 2021

afedorov updated the diff for D30639: if_vtnet(4): Restore the ability to set promisc mode..
Jun 4 2021, 4:38 PM
afedorov added inline comments to D30638: if_vxlan(4): Allow netmap_generic to intercept RX packets..
Jun 4 2021, 4:31 PM
afedorov updated the summary of D30639: if_vtnet(4): Restore the ability to set promisc mode..
Jun 4 2021, 4:19 PM
afedorov requested review of D30639: if_vtnet(4): Restore the ability to set promisc mode..
Jun 4 2021, 4:16 PM
afedorov requested review of D30638: if_vxlan(4): Allow netmap_generic to intercept RX packets..
Jun 4 2021, 4:02 PM
afedorov added inline comments to D30629: tests/netgraph: Inital framework for testing libnetgraph.
Jun 4 2021, 9:52 AM

Jun 3 2021

afedorov added a comment to D30618: arm64: add snd_hda to GENERIC.

I prefer having sound loaded as a module in fact, but until we have module autoloading by PCI ID this should be available in GENERIC.

Jun 3 2021, 2:54 PM

May 18 2021

afedorov added inline comments to D30288: mmc: Add mmc-pwrseq driver.
May 18 2021, 9:48 AM

Apr 28 2021

afedorov added a comment to D30020: bhyve: add virtio-input emulation.

The capsicum support should look something like this:

Apr 28 2021, 1:39 PM · bhyve
afedorov added a comment to D30020: bhyve: add virtio-input emulation.

Have you tested this code with CAPSICUM enabled?

Apr 28 2021, 10:17 AM · bhyve

Apr 7 2021

afedorov added reviewers for D29531: Add kvmclock timecounter support to FreeBSD: kib, markj.
Apr 7 2021, 5:58 PM
afedorov added a comment to D29531: Add kvmclock timecounter support to FreeBSD.

As I mentioned previously, the other thing to consider when using kvmclock in the current patch is the need for a system call for clock_gettime(), gettimeofday(), etc.

Apr 7 2021, 5:57 PM
afedorov added a comment to D29531: Add kvmclock timecounter support to FreeBSD.

@me_freebsd_mathieu.digital, what a CPU model was used for testing?

Apr 7 2021, 4:35 PM

Mar 30 2021

afedorov accepted D29433: bhyve: change vq_getchain to return iovecs in both directions.
Mar 30 2021, 5:06 AM

Mar 28 2021

afedorov accepted D29433: bhyve: change vq_getchain to return iovecs in both directions.
Mar 28 2021, 8:59 AM

Mar 27 2021

afedorov added inline comments to D29433: bhyve: change vq_getchain to return iovecs in both directions.
Mar 27 2021, 8:10 AM

Mar 11 2021

afedorov accepted D29054: tests/sys/netgraph/ng_macfilter_test: Fix invalid TAP output.

I have no objections.

Mar 11 2021, 3:02 PM

Mar 2 2021

afedorov added inline comments to D29011: wg: Fix some mbuf leaks in the input path.
Mar 2 2021, 5:19 PM

Mar 1 2021

afedorov added a member for Jails: afedorov.
Mar 1 2021, 4:44 PM

Feb 20 2021

afedorov added a comment to D28516: netgraph/ng_bridge: learn MACs via control message.

I'm not an English expert, so you may be right.
Technically, I have no questions, I think this patch can be committed.

Feb 20 2021, 8:23 PM
afedorov added a reviewer for D28516: netgraph/ng_bridge: learn MACs via control message: bcr.
Feb 20 2021, 8:04 PM

Feb 18 2021

afedorov added a comment to D28516: netgraph/ng_bridge: learn MACs via control message.

Did you mean D28559?

Feb 18 2021, 7:18 PM
afedorov added inline comments to D28516: netgraph/ng_bridge: learn MACs via control message.
Feb 18 2021, 9:17 AM

Feb 15 2021

afedorov updated subscribers of D28679: vnode: move write cluster support data to inodes..

I have no competence in this area. Maybe you wanted to add @fsu to the reviewers? He was working on improving ext2/3/4 support.

Feb 15 2021, 2:54 PM
afedorov removed a reviewer for D28679: vnode: move write cluster support data to inodes.: afedorov.
Feb 15 2021, 2:51 PM

Feb 5 2021

afedorov added inline comments to D23963: netgraph/ng_bridge: Introduce "uplink" ports without MAC learning.
Feb 5 2021, 10:50 AM

Jan 13 2021

afedorov updated subscribers of D28123: netgraph/ng_bridge: become multithreaded.

Lutz, do you have any plans for the upcoming changes?
I also thought about getting rid of ng_bridge from NG_NODE_FORCE_WRITER. Since rcv_data is always called in the NET_EPOCH context, I think we can do it like @kp did it for if_bridge(4) (see D24250).

Jan 13 2021, 10:23 AM
afedorov accepted D28125: netgraph/ng_bridge: switch to counter framework.
Jan 13 2021, 7:44 AM
afedorov added a reviewer for D28125: netgraph/ng_bridge: switch to counter framework: afedorov.
Jan 13 2021, 7:43 AM

Dec 18 2020

afedorov added a reviewer for D27662: netgraph: Fix ng_ether's shutdown handing: glebius.
Dec 18 2020, 3:53 PM
afedorov added a comment to D27662: netgraph: Fix ng_ether's shutdown handing.

I don't really like how ng_etner(4) is implemented as a whole. But I think this patch is correct and should be committed.

Dec 18 2020, 3:53 PM
afedorov accepted D27662: netgraph: Fix ng_ether's shutdown handing.
Dec 18 2020, 3:44 PM
afedorov added a reviewer for D27662: netgraph: Fix ng_ether's shutdown handing: afedorov.
Dec 18 2020, 3:44 PM

Dec 17 2020

afedorov closed D27382: [ng_socket] Don't take the SOCKBUF_LOCK() twice in the RX data path..
Dec 17 2020, 6:15 PM
afedorov committed rS368727: [ng_socket] Don't take the SOCKBUF_LOCK() twice in the RX data path..
[ng_socket] Don't take the SOCKBUF_LOCK() twice in the RX data path.
Dec 17 2020, 6:15 PM
afedorov closed D27381: [bhyve] virtio-net: Do not allow receiving packets until features have been negotiated..
Dec 17 2020, 4:53 PM
afedorov committed rS368720: [bhyve] virtio-net: Do not allow receiving packets until features have been….
[bhyve] virtio-net: Do not allow receiving packets until features have been…
Dec 17 2020, 4:52 PM

Dec 6 2020

afedorov added a comment to D27381: [bhyve] virtio-net: Do not allow receiving packets until features have been negotiated..

@grehan do you have any objections?

Dec 6 2020, 6:28 PM

Dec 3 2020

afedorov added inline comments to D27381: [bhyve] virtio-net: Do not allow receiving packets until features have been negotiated..
Dec 3 2020, 2:00 PM

Nov 29 2020

afedorov added inline comments to D27381: [bhyve] virtio-net: Do not allow receiving packets until features have been negotiated..
Nov 29 2020, 10:31 AM
afedorov updated the diff for D27381: [bhyve] virtio-net: Do not allow receiving packets until features have been negotiated..

Eliminate issues identified by reviewers.

Nov 29 2020, 10:27 AM

Nov 27 2020

afedorov added a comment to D27381: [bhyve] virtio-net: Do not allow receiving packets until features have been negotiated..

@vmaffione, what do you think about this patch?

Nov 27 2020, 3:38 PM

Nov 26 2020

afedorov updated the diff for D27382: [ng_socket] Don't take the SOCKBUF_LOCK() twice in the RX data path..

Fix indentation.

Nov 26 2020, 2:58 PM
afedorov added inline comments to D27382: [ng_socket] Don't take the SOCKBUF_LOCK() twice in the RX data path..
Nov 26 2020, 2:37 PM
afedorov updated the diff for D27382: [ng_socket] Don't take the SOCKBUF_LOCK() twice in the RX data path..

Add a comment about locking.

Nov 26 2020, 2:34 PM
afedorov updated the diff for D27381: [bhyve] virtio-net: Do not allow receiving packets until features have been negotiated..
Nov 26 2020, 2:14 PM
afedorov added a reviewer for D27381: [bhyve] virtio-net: Do not allow receiving packets until features have been negotiated.: bryanv.
Nov 26 2020, 1:26 PM
afedorov requested review of D27382: [ng_socket] Don't take the SOCKBUF_LOCK() twice in the RX data path..
Nov 26 2020, 1:24 PM
afedorov requested review of D27381: [bhyve] virtio-net: Do not allow receiving packets until features have been negotiated..
Nov 26 2020, 12:22 PM

Nov 19 2020

afedorov added a reviewer for D22076: netgraph/ng_vlan_rotate: IEEE 802.1ad VLAN manipulation netgraph node type (new type): glebius.
Nov 19 2020, 8:19 PM
afedorov added inline comments to D22076: netgraph/ng_vlan_rotate: IEEE 802.1ad VLAN manipulation netgraph node type (new type).
Nov 19 2020, 8:04 PM
afedorov added inline comments to D27268: New netgraph node: macfilter.
Nov 19 2020, 7:55 PM

Nov 18 2020

afedorov added inline comments to D27268: New netgraph node: macfilter.
Nov 18 2020, 6:25 PM
afedorov added inline comments to D27268: New netgraph node: macfilter.
Nov 18 2020, 6:10 PM
afedorov added inline comments to D27268: New netgraph node: macfilter.
Nov 18 2020, 5:57 PM
afedorov added inline comments to D27268: New netgraph node: macfilter.
Nov 18 2020, 4:17 PM

Oct 14 2020

afedorov added a reviewer for D26755: Add hash table lookup for IPv6 raw sockets.: mav.
Oct 14 2020, 7:31 PM

Oct 13 2020

afedorov added a reviewer for D26668: Expose eventfd in the native API/ABI using a new __specialfd syscall: bcr.
Oct 13 2020, 2:43 PM · Linux Emulation