Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Mon, 28 Jun 2021 17:39:01 +0000
From:   Wei Liu <wei.liu@kernel.org>
To:     "Enrico Weigelt, metux IT consult" <lkml@metux.net>
Cc:     longli@linuxonhyperv.com, linux-kernel@vger.kernel.org,
        linux-hyperv@vger.kernel.org, Long Li <longli@microsoft.com>,
        Jonathan Corbet <corbet@lwn.net>,
        "K. Y. Srinivasan" <kys@microsoft.com>,
        Haiyang Zhang <haiyangz@microsoft.com>,
        Stephen Hemminger <sthemmin@microsoft.com>,
        Wei Liu <wei.liu@kernel.org>, Dexuan Cui <decui@microsoft.com>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Bjorn Andersson <bjorn.andersson@linaro.org>,
        Hans de Goede <hdegoede@redhat.com>,
        Dan Williams <dan.j.williams@intel.com>,
        Maximilian Luz <luzmaximilian@gmail.com>,
        Mike Rapoport <rppt@kernel.org>,
        Ben Widawsky <ben.widawsky@intel.com>,
        Jiri Slaby <jirislaby@kernel.org>,
        Andra Paraschiv <andraprs@amazon.com>,
        Siddharth Gupta <sidgup@codeaurora.org>,
        Hannes Reinecke <hare@suse.de>, linux-doc@vger.kernel.org
Subject: Re: [Patch v2 2/3] Drivers: hv: add Azure Blob driver
Message-ID: <20210628173901.mwo4ydfdmf3zd2zt@liuwe-devbox-debian-v2>
References: <1624689020-9589-1-git-send-email-longli@linuxonhyperv.com>
 <1624689020-9589-3-git-send-email-longli@linuxonhyperv.com>
 <ff05bb0d-bcad-8445-d410-7756e7502352@metux.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <ff05bb0d-bcad-8445-d410-7756e7502352@metux.net>
Precedence: bulk

On Mon, Jun 28, 2021 at 07:12:40PM +0200, Enrico Weigelt, metux IT consult wrote:
> On 26.06.21 08:30, longli@linuxonhyperv.com wrote:
> 
> Hi,
> 
> > From: Long Li <longli@microsoft.com>
> > 
> > Azure Blob storage provides scalable and durable data storage for Azure.
> > (https://azure.microsoft.com/en-us/services/storage/blobs/)
> 
> It's a bit hard to quickly find out the exact semantics of that (w/o fully
> reading the library code ...).  :(
> 
> > This driver adds support for accelerated access to Azure Blob storage. As an
> > alternative to REST APIs, it provides a fast data path that uses host native
> > network stack and secure direct data link for storage server access.
> 
> I don't think that yet another misc device with some very specific API
> is a good thing. We should have a more generic userland interface.
> 
> Maybe a file system ?

This can already be done if I'm not mistaken:

https://docs.microsoft.com/en-us/azure/storage/blobs/storage-how-to-mount-container-linux
https://docs.microsoft.com/en-us/azure/storage/blobs/network-file-system-protocol-support-how-to

> 
> OTOH, I've been considering adding other kind of blob stores (eg. venti
> and similar things) and inventing an own subsys for that, so we'd have
> some more universal interface. Unfortunately always been busy with other
> things.
> 
> Maybe we should go that route instead of introducing yet another device
> specific uapi.

This work of course does not preclude anyone from going that route.

Wei.

> 
> 
> --mtx
> 
> -- 
> ---
> Hinweis: unverschl?sselte E-Mails k?nnen leicht abgeh?rt und manipuliert
> werden ! F?r eine vertrauliche Kommunikation senden Sie bitte ihren
> GPG/PGP-Schl?ssel zu.
> ---
> Enrico Weigelt, metux IT consult
> Free software and Linux embedded engineering
> info@metux.net -- +49-151-27565287