qemu-commits
[Top][All Lists]
Advanced
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] d5895f: iscsi: Split URL into individual opti

From: GitHub
Subject: [Qemu-commits] [qemu/qemu] d5895f: iscsi: Split URL into individual options
Date: Tue, 21 Feb 2017 10:30:11 -0800 
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: d5895fcb1de8f6de3cc63a413d7052b98091e191
      
https://github.com/qemu/qemu/commit/d5895fcb1de8f6de3cc63a413d7052b98091e191
  Author: Kevin Wolf <address@hidden>
  Date:   2017-02-21 (Tue, 21 Feb 2017)

  Changed paths:
    M block/iscsi.c

  Log Message:
  -----------
  iscsi: Split URL into individual options

This introduces a .bdrv_parse_filename handler for iscsi which parses an
URL if given and translates it to individual options.

Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>
Signed-off-by: Jeff Cody <address@hidden>


  Commit: 4317142020921c6dcdcfda7a349a91088d969f14
      
https://github.com/qemu/qemu/commit/4317142020921c6dcdcfda7a349a91088d969f14
  Author: Kevin Wolf <address@hidden>
  Date:   2017-02-21 (Tue, 21 Feb 2017)

  Changed paths:
    M block/iscsi.c

  Log Message:
  -----------
  iscsi: Handle -iscsi user/password in bdrv_parse_filename()

This splits the logic in the old parse_chap() function into a part that
parses the -iscsi options into the new driver-specific options, and
another part that actually applies those options (called apply_chap()
now).

Note that this means that username and password specified with -iscsi
only take effect when a URL is provided. This is intentional, -iscsi is
a legacy interface only supported for compatibility, new users should
use the proper driver-specific options.

Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>
Signed-off-by: Jeff Cody <address@hidden>


  Commit: d4e799292c537789e5c9fee3a3e65ead37c5a302
      
https://github.com/qemu/qemu/commit/d4e799292c537789e5c9fee3a3e65ead37c5a302
  Author: Kevin Wolf <address@hidden>
  Date:   2017-02-21 (Tue, 21 Feb 2017)

  Changed paths:
    M block/iscsi.c

  Log Message:
  -----------
  iscsi: Add initiator-name option

This was previously only available with -iscsi. Again, after this patch,
the -iscsi option only takes effect if an URL is given. New users are
supposed to use the new driver-specific option.

Reviewed-by: Daniel P. Berrange <address@hidden>
Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>
Signed-off-by: Jeff Cody <address@hidden>


  Commit: 81aa2a0fb575df682c8ab4661287b24aba74d1f8
      
https://github.com/qemu/qemu/commit/81aa2a0fb575df682c8ab4661287b24aba74d1f8
  Author: Kevin Wolf <address@hidden>
  Date:   2017-02-21 (Tue, 21 Feb 2017)

  Changed paths:
    M block/iscsi.c

  Log Message:
  -----------
  iscsi: Add header-digest option

This was previously only available with -iscsi. Again, after this patch,
the -iscsi option only takes effect if an URL is given. New users are
supposed to use the new driver-specific option.

Reviewed-by: Daniel P. Berrange <address@hidden>
Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>
Signed-off-by: Jeff Cody <address@hidden>


  Commit: 1d56010482be1a29d02fbbf2cee46633298a5e81
      
https://github.com/qemu/qemu/commit/1d56010482be1a29d02fbbf2cee46633298a5e81
  Author: Kevin Wolf <address@hidden>
  Date:   2017-02-21 (Tue, 21 Feb 2017)

  Changed paths:
    M block/iscsi.c

  Log Message:
  -----------
  iscsi: Add timeout option

This was previously only available with -iscsi. Again, after this patch,
the -iscsi option only takes effect if an URL is given. New users are
supposed to use the new driver-specific option.

All -iscsi options have a corresponding driver-specific option for the
iscsi block driver now.

Reviewed-by: Daniel P. Berrange <address@hidden>
Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>
Signed-off-by: Jeff Cody <address@hidden>


  Commit: 31eb1202d391f7e35307c7aaddb03224efcba331
      
https://github.com/qemu/qemu/commit/31eb1202d391f7e35307c7aaddb03224efcba331
  Author: Kevin Wolf <address@hidden>
  Date:   2017-02-21 (Tue, 21 Feb 2017)

  Changed paths:
    M block/iscsi.c
    M qapi/block-core.json

  Log Message:
  -----------
  iscsi: Add blockdev-add support

This adds blockdev-add support for iscsi devices.

Reviewed-by: Daniel P. Berrange <address@hidden>
Reviewed-by: Fam Zheng <address@hidden>
Signed-off-by: Kevin Wolf <address@hidden>
Signed-off-by: Jeff Cody <address@hidden>


  Commit: b1660997125099db2c0cd7f54f6664fbd93bad00
      
https://github.com/qemu/qemu/commit/b1660997125099db2c0cd7f54f6664fbd93bad00
  Author: Jeff Cody <address@hidden>
  Date:   2017-02-21 (Tue, 21 Feb 2017)

  Changed paths:
    M qapi/block-core.json

  Log Message:
  -----------
  QAPI: Fix blockdev-add example documentation

Signed-off-by: Jeff Cody <address@hidden>


  Commit: 90ab48eb07d5d98b2fb81ced3c27bba828472569
      
https://github.com/qemu/qemu/commit/90ab48eb07d5d98b2fb81ced3c27bba828472569
  Author: Anton Nefedov <address@hidden>
  Date:   2017-02-21 (Tue, 21 Feb 2017)

  Changed paths:
    M block/mirror.c

  Log Message:
  -----------
  mirror: do not increase offset during initial zero_or_discard phase

If explicit zeroing out before mirroring is required for the target image,
it moves the block job offset counter to EOF, then offset and len counters
count the image size twice. There is no harm but stats are confusing,
specifically the progress of the operation is always reported as 99% by
management tools.

The patch skips offset increase for the first "technical" pass over the
image. This should not cause any further harm.

Signed-off-by: Anton Nefedov <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
Reviewed-by: Eric Blake <address@hidden>
Reviewed-by: Stefan Hajnoczi <address@hidden>
Message-id: address@hidden
CC: Jeff Cody <address@hidden>
CC: Kevin Wolf <address@hidden>
CC: Max Reitz <address@hidden>
CC: Eric Blake <address@hidden>
Signed-off-by: Jeff Cody <address@hidden>


  Commit: 6135c5e12606b8413708384e3e7d43f6010c5941
      
https://github.com/qemu/qemu/commit/6135c5e12606b8413708384e3e7d43f6010c5941
  Author: Thomas Huth <address@hidden>
  Date:   2017-02-21 (Tue, 21 Feb 2017)

  Changed paths:
    M qemu-options.hx

  Log Message:
  -----------
  qemu-options: Fix broken sheepdog URL

The sheepdog URL is broken twice: First it uses a duplicated
http:// prefix, second the website seems to have moved to
https://sheepdog.github.io/sheepdog/ instead.

Signed-off-by: Thomas Huth <address@hidden>
Signed-off-by: Jeff Cody <address@hidden>


  Commit: 796b288f7be875045670f963ce99991b3c8e96ac
      
https://github.com/qemu/qemu/commit/796b288f7be875045670f963ce99991b3c8e96ac
  Author: Peter Maydell <address@hidden>
  Date:   2017-02-21 (Tue, 21 Feb 2017)

  Changed paths:
    M block/iscsi.c
    M block/mirror.c
    M qapi/block-core.json
    M qemu-options.hx

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/cody/tags/block-pull-request' into 
staging

# gpg: Signature made Tue 21 Feb 2017 15:40:05 GMT
# gpg:                using RSA key 0xBDBE7B27C0DE3057
# gpg: Good signature from "Jeffrey Cody <address@hidden>"
# gpg:                 aka "Jeffrey Cody <address@hidden>"
# gpg:                 aka "Jeffrey Cody <address@hidden>"
# Primary key fingerprint: 9957 4B4D 3474 90E7 9D98  D624 BDBE 7B27 C0DE 3057

* remotes/cody/tags/block-pull-request:
  qemu-options: Fix broken sheepdog URL
  mirror: do not increase offset during initial zero_or_discard phase
  QAPI: Fix blockdev-add example documentation
  iscsi: Add blockdev-add support
  iscsi: Add timeout option
  iscsi: Add header-digest option
  iscsi: Add initiator-name option
  iscsi: Handle -iscsi user/password in bdrv_parse_filename()
  iscsi: Split URL into individual options

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/a1cf5fac2b92...796b288f7be8
reply via email to
[Prev in Thread] Current Thread [Next in Thread]