Subject: Re: [Patch v5 0/3] Introduce a driver to support host accelerated
access to Microsoft Azure Blob for Azure VM

On Sat, Aug 07, 2021 at 06:29:06PM +0000, Long Li wrote:

quoted

quoted

I still think this "model" is totally broken and wrong overall.
Again, you are creating a custom "block" layer with a character
device, forcing all userspace programs to use a custom library (where is it

at?) just to get their data.

quoted

The Azure Blob library (with source code) is available in the following

languages:

quoted

Java:
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
ub.com%2FAzure%2Fazure-sdk-for-

java%2Ftree%2Fmain%2Fsdk%2Fstorage%2Faz

quoted

ure-storage-

blob&data=04%7C01%7Clongli%40microsoft.com%7C778083147

quoted

8ed49b16e6308d95a2b7ae8%7C72f988bf86f141af91ab2d7cd011db47%7C1
%7C0%7C6

quoted

37639965101378114%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwM
DAiLCJQIjoi

quoted

V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=wcNhsEoH
LV0VBc

quoted

uDf0CVXl7W0Ug9Cj7Q92%2Bw6qizroU%3D&reserved=0
JavaScript:
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
ub.com%2FAzure%2Fazure-sdk-for-

js%2Ftree%2Fmain%2Fsdk%2Fstorage%2Fstor

quoted

age-

blob&data=04%7C01%7Clongli%40microsoft.com%7C7780831478ed49b
16

quoted

e6308d95a2b7ae8%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C
637639965

quoted

101378114%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjo
iV2luMzIi

quoted

LCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=I%2FfhdPX3Unz6S3
eBPcpl

quoted

%2Bh55nKoV0u%2FO0%2BYgjLy4grQ%3D&reserved=0
Python:
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
ub.com%2FAzure%2Fazure-sdk-for-

python%2Ftree%2Fmain%2Fsdk%2Fstorage%2F

quoted

azure-storage-

blob&data=04%7C01%7Clongli%40microsoft.com%7C7780831

quoted

478ed49b16e6308d95a2b7ae8%7C72f988bf86f141af91ab2d7cd011db47%7
C1%7C0%7

quoted

C637639965101378114%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAw
MDAiLCJQIj

quoted

oiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=aAwsi%2
BPVsN

quoted

tsDMJ7rKnRDigNc41fIao031lde247Nc0%3D&reserved=0
Go:
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
ub.com%2FAzure%2Fazure-storage-blob-

go&data=04%7C01%7Clongli%40mic

quoted

rosoft.com%7C7780831478ed49b16e6308d95a2b7ae8%7C72f988bf86f141a
f91ab2d

quoted

7cd011db47%7C1%7C0%7C637639965101378114%7CUnknown%7CTWFpbG
Zsb3d8eyJWIj

quoted

oiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C10
00&am

quoted

p;sdata=43JhbGsYQxA%2FoivNd7C3z7DSYO%2FPONCoaW2v7TN6xEU%3D&a
mp;reserve

quoted

d=0
.NET:
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
ub.com%2FAzure%2Fazure-sdk-for-

net%2Ftree%2Fmain%2Fsdk%2Fstorage%2FAzu

quoted

re.Storage.Blobs&data=04%7C01%7Clongli%40microsoft.com%7C77808
3147

quoted

8ed49b16e6308d95a2b7ae8%7C72f988bf86f141af91ab2d7cd011db47%7C1
%7C0%7C6

quoted

37639965101378114%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwM
DAiLCJQIjoi

quoted

V2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=6ClMeURlt
cBv1q

quoted

7l7PGGrxXVJbVDt9uMBlwoIVh7Wpw%3D&reserved=0
PHP:
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
ub.com%2FAzure%2Fazure-storage-php%2Ftree%2Fmaster%2Fazure-

storage-blo

quoted

b&data=04%7C01%7Clongli%40microsoft.com%7C7780831478ed49b16
e6308d9

quoted

5a2b7ae8%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C6376399
651013781

quoted

14%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIi
LCJBTiI

quoted

6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=DuZO539vd76c%2Byaqjn
hetp%2B3T

quoted

i0b74601ZkNe39SNK4%3D&reserved=0
Ruby:
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
ub.com%2Fazure%2Fazure-storage-

ruby%2Ftree%2Fmaster%2Fblob&data=04

quoted

%7C01%7Clongli%40microsoft.com%7C7780831478ed49b16e6308d95a2b

7ae8%7C72

quoted

f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637639965101378114%7
CUnknown%

quoted

7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwi
LCJX

quoted

VCI6Mn0%3D%7C1000&sdata=6Zviu1IuRQE2do9bDCae2iJv0W2KOJu90t
XSR6kDAR

quoted

4%3D&reserved=0
C++:

C++https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fg

quoted

C++ithub.com%2FAzure%2Fazure-sdk-for-

cpp%2Ftree%2Fmain%2Fsdk%2Fstorage

quoted

C++%23azure-storage-client-library-for-

c&data=04%7C01%7Clongli%40m

quoted

C++icrosoft.com%7C7780831478ed49b16e6308d95a2b7ae8%7C72f988bf86
f141af9

quoted

C++1ab2d7cd011db47%7C1%7C0%7C637639965101388074%7CUnknown%
7CTWFpbGZsb3

quoted

C++d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn
0%3

quoted

C++D%7C1000&sdata=HH6jrqREWQ%2BkoRR%2Fsb02wRXnuLU5il4Erzm
rBvUZu5w%

quoted

C++3D&reserved=0

And why wasn't this linked to in the changelog here?

In looking at the C code above, where is the interaction with this Linux driver?
I can't seem to find it...

Those are existing Blob client libraries. The new code using this driver is being
tested and has not been released to github.

I'm sorry I misunderstood your request. I'm asking the team to share the new
code for review. I will send the code location for review soon.

Thanks,
Long