Re: [ovs-dev] [PATCH 1/2] docs/install/afxdp: Fix wrapping in QEMU CMDs

2021-09-15 Thread Ilya Maximets
On 9/9/21 16:57, Kevin Traynor wrote: > On 27/08/2021 16:15, Cian Ferriter wrote: >> Directly copying the CMD from either the .rst file or the docs on the >> web caused errors in the QEMU command because of how it was wrapped. >> >> Signed-off-by: Cian Ferriter >> > > Acked-by: Kevin Traynor

Re: [ovs-dev] [PATCH 1/2] docs/install/afxdp: Fix wrapping in QEMU CMDs

2021-09-09 Thread Kevin Traynor
On 27/08/2021 16:15, Cian Ferriter wrote: > Directly copying the CMD from either the .rst file or the docs on the > web caused errors in the QEMU command because of how it was wrapped. > > Signed-off-by: Cian Ferriter > Acked-by: Kevin Traynor In general, it would be nice to replace all the

Re: [ovs-dev] [PATCH 1/2] docs/install/afxdp: Fix wrapping in QEMU CMDs

2021-08-27 Thread 0-day Robot
Bleep bloop. Greetings Cian Ferriter, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 111 characters long (recommended limit is 79) #24 FILE:

[ovs-dev] [PATCH 1/2] docs/install/afxdp: Fix wrapping in QEMU CMDs

2021-08-27 Thread Cian Ferriter
Directly copying the CMD from either the .rst file or the docs on the web caused errors in the QEMU command because of how it was wrapped. Signed-off-by: Cian Ferriter --- The CMD could be wrapped, but without the indentation to look like this: qemu-system-x86_64 -hda ubuntu1810.qcow \ -m