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

RHBZ#2075862: VirtIO-FS: make the service stoppable #757

Merged
merged 4 commits into from Apr 18, 2022

Conversation

viktor-prutyanov
Copy link
Collaborator

Now service can be stopped by Service Manager, because it handles SERVICE_CONTROL_STOP and SERVICE_CONTROL_SHUTDOWN.

Besides of that, the service submits FUSE_DESTROY request to end FUSE session at service stop and device remove.

…T_SHUTDOWN

Now service can be stopped by Service Manager, because it handles
SERVICE_CONTROL_STOP and SERVICE_CONTROL_SHUTDOWN.

Signed-off-by: Viktor Prutyanov <viktor@daynix.com>
Signed-off-by: Viktor Prutyanov <viktor@daynix.com>
…ce remove

FUSE session should be started with FUSE_INIT and ended with
FUSE_DESTROY. It resets host state, including unreferencing all inodes.

Signed-off-by: Viktor Prutyanov <viktor@daynix.com>
…FileSystem

FileSystem in Stop() may be deleted already if the device is removed.

Signed-off-by: Viktor Prutyanov <viktor@daynix.com>
@kostyanf14
Copy link
Collaborator

kostyanf14 commented Apr 15, 2022

@viktor-prutyanov HCK-CI is currently disabled. I will try to enable it ASAP.

@viktor-prutyanov
Copy link
Collaborator Author

viktor-prutyanov commented Apr 15, 2022

@viktor-prutyanov HCK-CI is currently disabled. I will try to enable it ASAP.

@kostyanf14 I think we aren't going to merge this PR until Monday. Moreover, there are no driver changes here, so for this PR no WHQL is needed, only build.

@YanVugenfirer YanVugenfirer merged commit ce3c05a into virtio-win:master Apr 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants