[Qemu-devel] [PATCH v3 0/7] Code cleanup and minor fixes (for 2.11)

Jeff Cody posted 7 patches 6 years, 5 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/cover.1510093478.git.jcody@redhat.com
Test checkpatch passed
Test docker passed
Test ppc passed
Test s390x passed
block/curl.c     | 124 +++++++++++++++++++++++------------------
block/sheepdog.c | 166 +++++++++++++++++++++++++++----------------------------
block/ssh.c      |  69 ++++++++++++++---------
3 files changed, 197 insertions(+), 162 deletions(-)
[Qemu-devel] [PATCH v3 0/7] Code cleanup and minor fixes (for 2.11)
Posted by Jeff Cody 6 years, 5 months ago
Changes from v2 -> v3:

Patch 1: Return -EIO for ssh_state_init() (Thanks Eric)
Patch 4: fixed missed line (Thanks Eric)
Patch 7: fixed opaque pointer assignment (Thanks Eric)

git-backport-diff -r master.. -u 81fb084

001/7:[0007] [FC] 'block/ssh: don't call libssh2_init() in block_init()'
002/7:[----] [--] 'block/ssh: make compliant with coding guidelines'
003/7:[----] [--] 'block/sheepdog: remove spurious NULL check'
004/7:[0010] [FC] 'block/sheepdog: code beautification'
005/7:[----] [--] 'block/curl: check error return of curl_global_init()'
006/7:[----] [--] 'block/curl: fix minor memory leaks'
007/7:[0002] [FC] 'block/curl: code cleanup to comply with coding style'


Jeff Cody (7):
  block/ssh: don't call libssh2_init() in block_init()
  block/ssh: make compliant with coding guidelines
  block/sheepdog: remove spurious NULL check
  block/sheepdog: code beautification
  block/curl: check error return of curl_global_init()
  block/curl: fix minor memory leaks
  block/curl: code cleanup to comply with coding style

 block/curl.c     | 124 +++++++++++++++++++++++------------------
 block/sheepdog.c | 166 +++++++++++++++++++++++++++----------------------------
 block/ssh.c      |  69 ++++++++++++++---------
 3 files changed, 197 insertions(+), 162 deletions(-)

-- 
2.9.5


Re: [Qemu-devel] [PATCH v3 0/7] Code cleanup and minor fixes (for 2.11)
Posted by Jeff Cody 6 years, 4 months ago
On Tue, Nov 07, 2017 at 05:27:17PM -0500, Jeff Cody wrote:
> Changes from v2 -> v3:
> 
> Patch 1: Return -EIO for ssh_state_init() (Thanks Eric)
> Patch 4: fixed missed line (Thanks Eric)
> Patch 7: fixed opaque pointer assignment (Thanks Eric)
> 
> git-backport-diff -r master.. -u 81fb084
> 
> 001/7:[0007] [FC] 'block/ssh: don't call libssh2_init() in block_init()'
> 002/7:[----] [--] 'block/ssh: make compliant with coding guidelines'
> 003/7:[----] [--] 'block/sheepdog: remove spurious NULL check'
> 004/7:[0010] [FC] 'block/sheepdog: code beautification'
> 005/7:[----] [--] 'block/curl: check error return of curl_global_init()'
> 006/7:[----] [--] 'block/curl: fix minor memory leaks'
> 007/7:[0002] [FC] 'block/curl: code cleanup to comply with coding style'
> 
> 
> Jeff Cody (7):
>   block/ssh: don't call libssh2_init() in block_init()
>   block/ssh: make compliant with coding guidelines
>   block/sheepdog: remove spurious NULL check
>   block/sheepdog: code beautification
>   block/curl: check error return of curl_global_init()
>   block/curl: fix minor memory leaks
>   block/curl: code cleanup to comply with coding style
> 
>  block/curl.c     | 124 +++++++++++++++++++++++------------------
>  block/sheepdog.c | 166 +++++++++++++++++++++++++++----------------------------
>  block/ssh.c      |  69 ++++++++++++++---------
>  3 files changed, 197 insertions(+), 162 deletions(-)
> 
> -- 
> 2.9.5
> 

Applied the non-controversial patches 3-6 to my block branch:

git://github.com/codyprime/qemu-kvm-jtc block

-Jeff