On 1/17/23 02:46, zhenwei pi wrote: > v1 -> v2: > - Rebase code against the latest commit: > 46aee2a9255adf842ab44a9292acb46189a726f7 > - Merge previous 1/7 and 2/7 into a single patch: > conf: introduce crypto device > - Add missing qemuxml2xmltest. > - Merge previous 3/7 and 4/7 into a single patch: > capabilities: introduce crypto > - Add missing formatdomaincaps.rst. > - Several fixes from Michal Prívozník' crypto_review branch. > - Add caps and test for qemu-8.0. > > Thanks to Michal for lots of suggestions! > > v1: > This series supports crypto device, also add support for QEMU. > The basic xml schema looks like: > <crypto model='virtio' type='qemu'> > <backend model='builtin' queues='1'/> > <address type='pci' domain='0x0000' bus='0x00' slot='0x0a' function='0x0'/> > </crypto> > > <crypto model='virtio' type='qemu'> > <backend model='lkcf'/> > <address type='pci' domain='0x0000' bus='0x00' slot='0x0b' function='0x0'/> > </crypto> > > Each patch has been tested by 'ninja -C build test', and launch a QEMU > with crypto devices, it works fine. > > zhenwei pi (5): > conf: introduce crypto device > capabilities: introduce crypto > qemu: alias: support crypto device > qemu: command: support crypto device > NEWS: Document 'crypto' device And we're almost there. This is way better version already. And depending on your answers, I'd merge these. Please see individual comments. No need to resend just now, it's all small nits that I can fix before pushing. To imagine changes I'm suggesting easier, I've uploaded fixup commits onto my gitlab: https://gitlab.com/MichalPrivoznik/libvirt/-/commits/review_crypto/ Please take a look and if you agree, I'll squash those commits in and push. Michal
