Message ID | 20210426054513.132980-2-edgenfv@gmail.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | make vfio and DAX cache work together by skipping virtio fs cache section during vfio memory region add | expand |
Patchew URL: https://patchew.org/QEMU/20210426054513.132980-2-edgenfv@gmail.com/ Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20210426054513.132980-2-edgenfv@gmail.com Subject: [PATCH] make vfio and DAX cache work together by skipping virtio fs cache section during vfio memory region add === TEST SCRIPT BEGIN === #!/bin/bash git rev-parse base > /dev/null || exit 0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu * [new tag] patchew/20210426054513.132980-2-edgenfv@gmail.com -> patchew/20210426054513.132980-2-edgenfv@gmail.com Switched to a new branch 'test' 32bddb3 make vfio and DAX cache work together by skipping virtio fs cache section during vfio memory region add === OUTPUT BEGIN === ERROR: trailing whitespace #21: FILE: hw/vfio/common.c:671: + $ ERROR: trailing whitespace #22: FILE: hw/vfio/common.c:672: + /* Do not add virtio fs cache section */ $ ERROR: trailing whitespace #29: FILE: hw/vfio/common.c:679: + } $ total: 3 errors, 0 warnings, 15 lines checked Commit 32bddb3c2253 (make vfio and DAX cache work together by skipping virtio fs cache section during vfio memory region add) has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20210426054513.132980-2-edgenfv@gmail.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-devel@redhat.com
On 26/04/2021 07:45, Edge NFV wrote: > Signed-off-by: Edge NFV <edgenfv@gmail.com> You must use your real name for a patch submission: https://wiki.qemu.org/Contribute/SubmitAPatch#Patch_emails_must_include_a_Signed-off-by:_line https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/SubmittingPatches?id=f6f94e2ab1b33f0082ac22d71f66385a60d8157f#n297 And CC: maintainer of the file: $ ./scripts/get_maintainer.pl -f hw/vfio/common.c Alex Williamson <alex.williamson@redhat.com> (supporter:VFIO) qemu-devel@nongnu.org (open list:All patches CC here) If you want to submit it to qemu-trivial, don't send a separate mail, but cc: also qemu-trivial. Thanks, Laurent > --- > hw/vfio/common.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/hw/vfio/common.c b/hw/vfio/common.c > index ae5654fcdb..83e15bf7a3 100644 > --- a/hw/vfio/common.c > +++ b/hw/vfio/common.c > @@ -668,6 +668,15 @@ static void vfio_listener_region_add(MemoryListener *listener, > int128_get64(int128_sub(section->size, int128_one()))); > return; > } > + > + /* Do not add virtio fs cache section */ > + if (!strcmp(memory_region_name(section->mr), "virtio-fs-cache")) { > + trace_vfio_listener_region_add_skip( > + section->offset_within_address_space, > + section->offset_within_address_space + > + int128_get64(int128_sub(section->size, int128_one()))); > + return; > + } > > if (unlikely((section->offset_within_address_space & > ~qemu_real_host_page_mask) != >
diff --git a/hw/vfio/common.c b/hw/vfio/common.c index ae5654fcdb..83e15bf7a3 100644 --- a/hw/vfio/common.c +++ b/hw/vfio/common.c @@ -668,6 +668,15 @@ static void vfio_listener_region_add(MemoryListener *listener, int128_get64(int128_sub(section->size, int128_one()))); return; } + + /* Do not add virtio fs cache section */ + if (!strcmp(memory_region_name(section->mr), "virtio-fs-cache")) { + trace_vfio_listener_region_add_skip( + section->offset_within_address_space, + section->offset_within_address_space + + int128_get64(int128_sub(section->size, int128_one()))); + return; + } if (unlikely((section->offset_within_address_space & ~qemu_real_host_page_mask) !=
Signed-off-by: Edge NFV <edgenfv@gmail.com> --- hw/vfio/common.c | 9 +++++++++ 1 file changed, 9 insertions(+)