[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cifs-protocol
Subject:    Re: [cifs-protocol] [EXTERNAL] Re: [REG:120030321000983] SMB2: opening file for READ_CONTROL doesn't
From:       Jeff McCashland via cifs-protocol <cifs-protocol () lists ! samba ! org>
Date:       2020-03-04 17:08:39
Message-ID: BN8PR21MB121944DDBE7728A820442008A3E50 () BN8PR21MB1219 ! namprd21 ! prod ! outlook ! com
[Download RAW message or body]

Hi slow,

Thank you for the trace!

That's not my call and will be verified when the change request is worked on. \
However, commentary in the code from Win7 days suggests it is by design. 

Best regards,
Jeff McCashland | Senior Escalation Engineer | Microsoft Protocol Open Specifications \
                Team 
Phone: +1 (425) 703-8300 x38300 | Hours: 9am-5pm | Time zone: (UTC-08:00) Pacific \
Time (US and Canada) Local country phone number found here: \
http://support.microsoft.com/globalenglish | Extension 1138300 We value your \
feedback.   My manager is Jeremy Chapman (jeremyc), +1 (469) 775-2475

-----Original Message-----
From: Ralph Boehme <slow@samba.org> 
Sent: Wednesday, March 4, 2020 6:20 AM
To: Jeff McCashland <jeffm@microsoft.com>
Cc: cifs-protocol@lists.samba.org; support <support@mail.support.microsoft.com>
Subject: Re: [cifs-protocol] [EXTERNAL] Re: [REG:120030321000983] SMB2: opening file \
for READ_CONTROL doesn't trigger lease break

HI Jeff,

Am 3/4/20 um 10:53 AM schrieb Ralph Boehme via cifs-protocol:
> Am 3/3/20 um 10:23 PM schrieb Jeff McCashland:
> > However, note that legacy Oplocks do break on READ_CONTROL. If I understand our \
> > source code correctly, Leases never broke on READ_CONTROL, but we didn't catch \
> > the variance in the doc. 
> 
> I was afraid that this would be the case. :)
> 
> Just to be sure, I'll rewrite the testcase with oplocks instead of leases.

confirmed, trace attached (packets 123 ff). Are we sure this behaviour is by design \
and not a bug?

Thanks!
-slow

-- 
Ralph Boehme, Samba Team                \
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsamba.org%2F&amp;dat \
a=02%7C01%7Cjeffm%40microsoft.com%7C556db01e42714d7d529408d7c0472de0%7C72f988bf86f141a \
f91ab2d7cd011db47%7C1%7C0%7C637189284265492194&amp;sdata=0Emcz%2F7Wr%2FVLPhNLRced%2Bq744meKQiZ8mY80qrdnCg4%3D&amp;reserved=0
 Samba Developer, SerNet GmbH   \
https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsernet.de%2Fen%2Fsam \
ba%2F&amp;data=02%7C01%7Cjeffm%40microsoft.com%7C556db01e42714d7d529408d7c0472de0%7C72 \
f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C637189284265492194&amp;sdata=fmgYg6hvC9FBCtA0KRCAfdJytKFegwsIGKQ6627%2FnSM%3D&amp;reserved=0
 GPG-Fingerprint   FAE2C6088A24252051C559E4AA1E9B7126399E46
_______________________________________________
cifs-protocol mailing list
cifs-protocol@lists.samba.org
https://lists.samba.org/mailman/listinfo/cifs-protocol


[prev in list] [next in list] [prev in thread] [next in thread]