Project

General

Profile

Actions

Bug #10835

closed

SAMBA VFS module: free/total disk space return values

Added by Dennis Kramer about 9 years ago. Updated about 5 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
-
Target version:
% Done:

0%

Source:
other
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
Labels (FS):
Samba/CIFS
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

The return values for free/total disk space is not working correctly when you mount a share in Windows. My 340TB cluster had 7.3EB storage available in Windows which obviously is incorrect. I'm using a workaround by using a custom "dfree command =" script in the smb.conf, but VFS will override this and thus this script will not work (unless you remove the lines of codes for these specific disk operations in vfs_ceph.c).

I've compiled samba from samba-latest source (4.1.16) on Ubuntu 14.04LTS
My CEPH cluster is running version 0.87 (c51c8f9d80fa4e0168aa52685b8de40e42758578) on Ubuntu 14.04LTS


Files

patch (699 Bytes) patch Zheng Yan, 02/13/2015 09:52 AM
Actions #1

Updated by Zheng Yan about 9 years ago

remove which line of code of vfs_ceph.c can fix the issue?

Actions #2

Updated by Dennis Kramer about 9 years ago

It's not a fix, its a workaround to get the smb.conf parameter "dfree command" working. It seems like if the ".disk_free_fn = cephwrap_disk_free" (which doesn't give the correct return values) is defined in vfs_ceph.c it will override the parameter "dfree command" in smb.conf if set.

Actions #3

Updated by Zheng Yan about 9 years ago

  • File patch patch added
  • Status changed from New to 7

could you try the attached patch

Actions #4

Updated by Dennis Kramer about 9 years ago

Awesome! It works.
Thank you.

Actions #5

Updated by Zheng Yan about 9 years ago

  • Status changed from 7 to Resolved

upstreamed by commit 2501afe08b94a514d8e2f3eeb4a2c4edc9764979

Actions #6

Updated by Patrick Donnelly about 5 years ago

  • Category deleted (43)
  • Labels (FS) Samba/CIFS added
Actions

Also available in: Atom PDF