Differential D4963

Use standard taskqueue instead of hv_work_queue
ClosedPublic
Actions

Authored by howard0su_gmail.com on Jan 17 2016, 12:12 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Jan 26, 4:39 PM

	Unknown Object (File)
	Jan 3 2025, 4:45 AM

	Unknown Object (File)
	Dec 10 2024, 12:21 AM

	Unknown Object (File)
	Nov 10 2024, 1:26 AM

	Unknown Object (File)
	Oct 20 2024, 2:46 AM

	Unknown Object (File)
	Oct 18 2024, 9:15 AM

	Unknown Object (File)
	Oct 18 2024, 6:14 AM

	Unknown Object (File)
	Oct 9 2024, 10:49 PM

Subscribers

Details

Reviewers

honzhan_microsoft.com
decui_microsoft.com
sepherosa_gmail.com
royger
adrian

Group Reviewers

Contributor Reviews (src)

Commits

rS295307: hyperv: Use standard taskqueue instead of hv_work_queue

Summary

HyperV code was porting from Linux. There is an implementation of work queue
called as hv_work_queue. In FreeBSD, taskqueue is using for the same
purpose. Convert all the consumer of hv_work_queue to using taskqueue and remove work
queue implementation.

The net impact is reduced code and make the driver more FreeBSD style.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 2280
Build 2289: arc lint + arc unit

Event Timeline

howard0su_gmail.com updated this revision to Diff 12382.Jan 17 2016, 12:12 PM

howard0su_gmail.com retitled this revision from to Use standard taskqueue instead of hv_work_queue.

howard0su_gmail.com updated this object.

howard0su_gmail.com edited the test plan for this revision. (Show Details)

Herald added a reviewer: Contributor Reviewers (ports). · View Herald TranscriptJan 17 2016, 12:12 PM

Herald added a subscriber: imp. · View Herald Transcript

Fix a error logic which can lead to panic

howard0su_gmail.com added reviewers: sepherosa_gmail.com, royger, decui_microsoft.com, honzhan_microsoft.com.Jan 17 2016, 12:58 PM

koobs edited reviewers, added: Contributor Reviews (src); removed: Contributor Reviewers (ports).Jan 20 2016, 6:27 AM

Cleanup header file, recycle unused defs.
also update one place which can use WAITOK now.

howard0su_gmail.com added a child revision: D5081: Remove NOWAIT in the places where actually can wait..Jan 27 2016, 2:35 AM

honzhan_microsoft.com added inline comments.Jan 27 2016, 4:37 AM

sys/dev/hyperv/vmbus/hv_connection.c
167	This sema can be removed. It is only used in hv_work_queue_create which has been removed by this patch.
271	remove
324	It can be removed.

rebase to HEAD

address review comments, remove control_sema

honzhan_microsoft.com accepted this revision.Jan 29 2016, 10:41 AM

honzhan_microsoft.com edited edge metadata.

This revision is now accepted and ready to land.Jan 29 2016, 10:41 AM

Remove a empty pair of lock/unlock

This revision now requires review to proceed.Feb 1 2016, 4:26 AM

howard0su_gmail.com added a reviewer: adrian.Feb 4 2016, 4:48 AM

adrian accepted this revision.Feb 4 2016, 5:48 AM

adrian edited edge metadata.

This revision is now accepted and ready to land.Feb 4 2016, 5:48 AM

Closed by commit rS295307: hyperv: Use standard taskqueue instead of hv_work_queue (authored by sephe). · Explain WhyFeb 5 2016, 7:10 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

dev/

hyperv/

include/

33 lines

utilities/

35 lines

58 lines

vmbus/

hv_channel_mgmt.c

212 lines

hv_connection.c

3 lines

hv_vmbus_drv_freebsd.c

22 lines

hv_vmbus_priv.h

6 lines

Diff 12740

sys/dev/hyperv/include/hyperv.h

Loading...

sys/dev/hyperv/utilities/hv_kvp.c

Loading...

sys/dev/hyperv/utilities/hv_util.c

Loading...

sys/dev/hyperv/vmbus/hv_channel_mgmt.c

Loading...

sys/dev/hyperv/vmbus/hv_connection.c

Loading...

sys/dev/hyperv/vmbus/hv_vmbus_drv_freebsd.c

Loading...

sys/dev/hyperv/vmbus/hv_vmbus_priv.h

Loading...