[v3] block: deprecate the sheepdog driver

[PATCH v3 0/2] block: deprecate the sheepdog driver

Daniel P. Berrangé posted 2 patches 3 years, 6 months ago

Diff against v2
Download series mbox

Test docker-quick@centos7 failed

Test docker-mingw@fedora failed

Test checkpatch failed

Test FreeBSD failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20201002113243.2347710-1-berrange@redhat.com

MAINTAINERS                |  1 -
block/sheepdog.c           | 14 ++++++++++++++
configure                  |  5 +++--
docs/system/deprecated.rst |  9 +++++++++
4 files changed, 26 insertions(+), 3 deletions(-)

Expand all Fold all

[PATCH v3 0/2] block: deprecate the sheepdog driver

Posted by Daniel P. Berrangé 3 years, 6 months ago

2 years back I proposed dropping the sheepdog mailing list from the
MAINTAINERS file, but somehow the patch never got picked up:

  https://lists.gnu.org/archive/html/qemu-block/2018-03/msg01048.html

So here I am with the same patch again.

This time I go further and deprecate the sheepdog driver entirely.
See the rationale in the second patch commit message.

Changes in v3:

 - A few minor text changes
 - Don't initialize static variable to false

Daniel P. Berrangé (2):
  block: drop moderated sheepdog mailing list from MAINTAINERS file
  block: deprecate the sheepdog block driver

 MAINTAINERS                |  1 -
 block/sheepdog.c           | 14 ++++++++++++++
 configure                  |  5 +++--
 docs/system/deprecated.rst |  9 +++++++++
 4 files changed, 26 insertions(+), 3 deletions(-)

-- 
2.26.2

Re: [PATCH v3 0/2] block: deprecate the sheepdog driver

Posted by Neal Gompa 3 years, 6 months ago

On Fri, Oct 2, 2020 at 7:34 AM Daniel P. Berrangé <berrange@redhat.com> wrote:
>
> 2 years back I proposed dropping the sheepdog mailing list from the
> MAINTAINERS file, but somehow the patch never got picked up:
>
>   https://lists.gnu.org/archive/html/qemu-block/2018-03/msg01048.html
>
> So here I am with the same patch again.
>
> This time I go further and deprecate the sheepdog driver entirely.
> See the rationale in the second patch commit message.
>
> Changes in v3:
>
>  - A few minor text changes
>  - Don't initialize static variable to false
>
> Daniel P. Berrangé (2):
>   block: drop moderated sheepdog mailing list from MAINTAINERS file
>   block: deprecate the sheepdog block driver
>
>  MAINTAINERS                |  1 -
>  block/sheepdog.c           | 14 ++++++++++++++
>  configure                  |  5 +++--
>  docs/system/deprecated.rst |  9 +++++++++
>  4 files changed, 26 insertions(+), 3 deletions(-)
>
> --
> 2.26.2
>

Series looks good to me.

Reviewed-by: Neal Gompa <ngompa13@gmail.com>


--
真実はいつも一つ！/ Always, there's only one truth!

Re: [PATCH v3 0/2] block: deprecate the sheepdog driver

Posted by Kevin Wolf 3 years, 6 months ago

Am 02.10.2020 um 13:32 hat Daniel P. BerrangÃ© geschrieben:
> 2 years back I proposed dropping the sheepdog mailing list from the
> MAINTAINERS file, but somehow the patch never got picked up:
> 
>   https://lists.gnu.org/archive/html/qemu-block/2018-03/msg01048.html
> 
> So here I am with the same patch again.
> 
> This time I go further and deprecate the sheepdog driver entirely.
> See the rationale in the second patch commit message.

Thanks, applied to the block branch.

Kevin