Page MenuHomeFreeBSD
Differential D2512

SCSI disk hot add and remove on Hyper-V
ClosedPublic
Actions

Authored by whu on May 11 2015, 7:15 AM.
Tags
None
Referenced Files
Unknown Object (File)
Oct 6 2024, 6:09 PM
Unknown Object (File)
Sep 28 2024, 1:35 AM
Unknown Object (File)
Sep 28 2024, 1:33 AM
Unknown Object (File)
Sep 28 2024, 1:33 AM
Unknown Object (File)
Sep 28 2024, 1:27 AM
Unknown Object (File)
Sep 26 2024, 3:26 PM
Unknown Object (File)
Sep 19 2024, 8:24 AM
Unknown Object (File)
Sep 8 2024, 8:48 AM
View All 96 Files
Subscribers
gibbs
imp

Details

Reviewers
royger
Commits
rS283053: Add support for SCSI disk hot add and remove. Also add padding according to
Summary

Add support for SCSI disk hot add and remove. Also add padding
according to requirement of different hypervisors.

Test Plan

Tested by OSTC test team.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

whu updated this revision to Diff 5320.May 11 2015, 7:15 AM
whu retitled this revision from to SCSI disk hot add and remove on Hyper-V.
whu updated this object.
whu edited the test plan for this revision. (Show Details)
whu added a reviewer: royger.
whu added a subscriber: gibbs.
Herald added a subscriber: imp. · View Herald TranscriptMay 11 2015, 7:15 AM
royger edited edge metadata.May 16 2015, 9:36 AM

Only one minor comment.

sys/dev/hyperv/storvsc/hv_storvsc_drv_freebsd.c
400 ↗(On Diff #5320)

Would it make sense to have a macro that calculates the size? I see you are using this in a bunch of places:

#define VSTOR_PKT_SIZE (sizeof(struct vstor_packet) - vmscsi_size_delta)

whu updated this revision to Diff 5448.May 18 2015, 9:37 AM
whu edited edge metadata.
  • Incorporate review comments from rogyer.
royger accepted this revision.May 18 2015, 10:10 AM
royger edited edge metadata.

Looks fine to me.

This revision is now accepted and ready to land.May 18 2015, 10:10 AM
Closed by commit rS283053: Add support for SCSI disk hot add and remove. Also add padding according to (authored by whu). · Explain WhyMay 18 2015, 10:31 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
hyperv/
storvsc/
95 lines
25 lines

Diff 5449

View Options

head/sys/dev/hyperv/storvsc/hv_storvsc_drv_freebsd.c

Loading...
View Options

head/sys/dev/hyperv/storvsc/hv_vstorage.h

Loading...