Merge ~ahasenack/ubuntu/+source/samba:groovy-samba-uring into ubuntu/+source/samba:ubuntu/devel
Status: | Merged |
---|---|
Approved by: | Andreas Hasenack |
Approved revision: | bf2783b307b4ff5b14e9acc995b8c71aef629872 |
Merged at revision: | bf2783b307b4ff5b14e9acc995b8c71aef629872 |
Proposed branch: | ~ahasenack/ubuntu/+source/samba:groovy-samba-uring |
Merge into: | ubuntu/+source/samba:ubuntu/devel |
Diff against target: |
340 lines (+180/-56) 10 files modified
debian/changelog (+20/-0) debian/control (+1/-0) debian/tests/cifs-share-access (+13/-24) debian/tests/cifs-share-access-uring (+35/-0) debian/tests/control (+8/-0) debian/tests/smbclient-anonymous-share-list (+4/-1) debian/tests/smbclient-authenticated-share-list (+9/-8) debian/tests/smbclient-share-access (+12/-23) debian/tests/smbclient-share-access-uring (+29/-0) debian/tests/util (+49/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Lucas Kanashiro (community) | Approve | ||
Canonical Server Core Reviewers | Pending | ||
Review via email: mp+388683@code.launchpad.net |
Description of the change
Enable uring support in samba. This builds a new vfs module, which can then optionally be enabled.
uring support first appeared in version 4.12.0. Search for "io_uring": https:/
I also added DEP8 tests for it, basically the same tests we already had, but now on a share with this vfs module loaded. I took the opportunity to refactor these tests a bit, to avoid duplication of code. I kept the commits separate to make it easier to submit them to debian: a) the refactor; b) the new uring changes
If you are happy with these, I will then submit to debian before uploading.
Bileto, still running: https:/
I did run the tests locally before, both in a VM and in LXD, and with proposed enabled, so I don't expect trouble. Bileto is slow because of the risc64 build.
Update scan failed
At least one of the branches involved have failed to scan. You can manually schedule a rescan if required.
I am going to review this MP.