Reviewed-by: Pavel Shilovsky <pshilov@xxxxxxxxxxxxx> -- Best regards, Pavel Shilovsky вт, 3 дек. 2019 г. в 13:49, Steve French <smfrench@xxxxxxxxx>: > > Updated patch with Pavel's recommended change included > > On Tue, Dec 3, 2019 at 1:19 PM Pavel Shilovsky <piastryyy@xxxxxxxxx> wrote: > > > > The inode locking is missed in the patch. See cifs_fattr_to_inode() > > that takes inode->i_lock to atomically update all the attributes. The > > similar thing is needed in +smb2_close_getattr(). > > > > -- > > Best regards, > > Pavel Shilovsky > > > > вт, 3 дек. 2019 г. в 10:05, Steve French <smfrench@xxxxxxxxx>: > > > > > > Fixed typo, and added the update of AllocationSize (and added reviewed > > > by). See attached. > > > > > > > -- > Thanks, > > Steve