Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

UVMOptions.SCSIControllerCount should be uint #397

Merged
merged 1 commit into from
Nov 30, 2018
Merged

Conversation

jterry75
Copy link
Contributor

Signed-off-by: Justin Terry (VM) juterry@microsoft.com

@jterry75 jterry75 requested a review from lowenna November 29, 2018 18:39
@jstarks
Copy link
Member

jstarks commented Nov 29, 2018

Why?

Signed-off-by: Justin Terry (VM) <juterry@microsoft.com>
@jterry75
Copy link
Contributor Author

Its all internal code anyways. Seems to me we should use the type system. So either:

SCSIControllerCount int and <0 is "not set" 0-4 is a valid value. The problem is the 0 value for the struct is not the same as the zero value for the platform.

SCSIControllerCount *uint where nil is default and means platform default. Values 0-4 are user specified and mean exactly that number. Seems to match the best.

@jterry75
Copy link
Contributor Author

@jhowardmsft - Do you care about this? Its internal code should we close it or take it?

@lowenna
Copy link
Contributor

lowenna commented Nov 30, 2018

I'm OK taking it. It makes it consistent with pmem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants