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.

Sign up for GitHub

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

Jump to bottom

usbip driver install from command line #86

Closed
wants to merge 4 commits into from
Closed

usbip driver install from command line #86

wants to merge 4 commits into from

Conversation

koudis
Copy link
Collaborator

@koudis koudis commented Dec 13, 2019 •
edited
Loading

First, working version of USBIP driver install from command line - usbip install

History is not referencable. I will squash all commits into one after PR approve.

@koudis
Copy link
Collaborator Author

koudis commented Dec 13, 2019 •
edited
Loading

Not final version. Do not make code review please. Just for now.
I will repair some coding standard constraints etc.

koudis
koudis commented Dec 13, 2019
View reviewed changes
userspace/src/usbip/usbip.c
@@ -50,47 +50,53 @@ struct command {

static const struct command cmds[] = {
{
.name = "help",
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do not known why are these changes here. I created my branch from updatream master.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed now. It's Visual Studio feature. I will repair.

userspace/src/usbip/usbip_install.c
.devdesc_inf_section = "Strings",
.devdesc_inf_key = "DeviceDesc",
.dev_instance_path = "ROOT\\USBIP\\0000"
}
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am not sure if values can stay here. Maybe move to some header file?

userspace/src/usbip/usbip.vcxproj
@@ -109,7 +109,7 @@
</ClCompile>
<Link>
<GenerateDebugInformation>true</GenerateDebugInformation>
<AdditionalDependencies>setupapi.lib;advapi32.lib;ws2_32.lib;wintrust.lib;crypt32.lib;newdev.lib</AdditionalDependencies>
<AdditionalDependencies>Shlwapi.lib;setupapi.lib;advapi32.lib;ws2_32.lib;wintrust.lib;crypt32.lib;newdev.lib</AdditionalDependencies>
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I use this for path parsing and manipulation.

@koudis koudis mentioned this pull request Dec 13, 2019
Closed
@xinheng0816
Copy link

Hello all
Sorry for interupt!
Someone can guide me how to install compile and install usbip driver in RedHat system ?
BTW, I downlod all source code with verison = Linux version 3.10.0-514.el7.x86_64. When I tied to build usbip module, it seems always nothing happen and the expected kos (usbip-core.ko & usbip-host.ko) can't be generated.
Thank you all!

@xinheng0816
Copy link

Let me describe my quesiton more clearly.
Precondition:
OS: RedHat 7.3
Linux kenel: Linux version 3.10.0-514.el7.x86_64

Background:
As required, i need to setup usbip env in my Linux station, so I'm considerring to build usbip source code locally.

Problem:
Until now, I downloaded the specified linux kernel source code (3.10.0-514.el7). Following https://wiki.centos.org/HowTos/BuildingKernelModules, when I execute below command under root directory of linux kernel source, I can't build out the expected Ko files.
$ make
$ make M=drivers/staging
Expected result:
usbip-host.ko and usbip-core.ko can be generated
Actual result:
From build log, I saw drivers/staging/usbip wasn't built at all.

Thanks for you time and expect your instruction!!

@koudis
Copy link
Collaborator Author

koudis commented Jan 4, 2020

@xinheng0816 Sorry but i am not able to help you. There are many tutorial how to build specific dirvers for linux kernel. Maybe RedHat official doc?

I close the PR because I merge my chages to #90

@koudis koudis closed this Jan 4, 2020
@koudis koudis deleted the jan_kubalek/usbip_driver_install branch March 13, 2020 21:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@koudis @xinheng0816