hyperv/hn: Remove unnecessary zeroing out the netvsc_packet
ClosedPublic
Actions

Authored by sepherosa_gmail.com on Jan 18 2016, 9:24 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Dec 11, 10:08 AM

	Unknown Object (File)
	Fri, Dec 6, 10:27 AM

	Unknown Object (File)
	Oct 4 2024, 3:52 PM

	Unknown Object (File)
	Sep 29 2024, 11:25 PM

	Unknown Object (File)
	Sep 28 2024, 4:51 AM

	Unknown Object (File)
	Sep 26 2024, 8:21 PM

	Unknown Object (File)
	Sep 25 2024, 3:17 AM

	Unknown Object (File)
	Sep 18 2024, 2:33 AM

View All 52 Files

Subscribers

freebsd-net-list

Details

Reviewers

howard0su_gmail.com
honzhan_microsoft.com
decui_microsoft.com
delphij
royger
adrian

Group Reviewers

network

Commits

rS294702: hyperv/hn: Remove unnecessary zeroing out the netvsc_packet

Summary

All used fields are setup on by one, so there is no need to zero
out this large struct.

While I'm here, move the stack variable near its usage.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

If no objection comes, it will be committed two days later

howard0su_gmail.com accepted this revision.Jan 18 2016, 2:08 PM

howard0su_gmail.com edited edge metadata.

This revision is now accepted and ready to land.Jan 18 2016, 2:08 PM

Looks good to me.

adrian accepted this revision.Jan 22 2016, 3:52 AM

adrian edited edge metadata.

Closed by commit rS294702: hyperv/hn: Remove unnecessary zeroing out the netvsc_packet (authored by sephe). · Explain WhyJan 25 2016, 5:19 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

dev/

hyperv/

netvsc/

hv_netvsc_drv_freebsd.c

6 lines

Diff 12671

View Options

head/sys/dev/hyperv/netvsc/hv_netvsc_drv_freebsd.c

hyperv/hn: Remove unnecessary zeroing out the netvsc_packetClosedPublicActions