History log of /src/sys/kern/vfs_quotactl.c |
Revision | | Date | Author | Comments |
1.40 |
| 28-Jun-2014 |
dholland | Revert the following changes:
src/sys/sys/quotactl.h 1.37 src/sys/compat/netbsd32/netbsd32.h 1.101 src/sys/compat/netbsd32/netbsd32_netbsd.c 1.188, 1.189 src/sys/kern/vfs_quotactl.c 1.39 src/sys/kern/vfs_syscalls.c 1.483 src/sys/ufs/lfs/ulfs_quota.c 1.11 src/sys/ufs/ufs/ufs_quota.c 1.116 src/lib/libquota/quota_kernel.c 1.5
and do them correctly.
If you're going to change the name of something, you need to change the name of *all* the things with the same name, not just a handful, and you should change it to something similar so it still matches the rest of the system rather than just picking an arbitrarily different name.
Hi, Joerg.
To wit, rename the quotactl "delete" operation to "del", because "delete" is a reserved word in C++ and for some reason Joerg wants to run internal interfaces used only by C code through his C++ compiler. Do not rename it to "remove" instead, because this doesn't match libquota or the rest of the usage throughout the system; and rename all the related identifiers, not just the ones that blew the mind of Joerg's C++ compiler.
Because this is not a user-facing API (the only userland consumer sys/quotactl.h is libquota) it is sort of ok to make arbitrary source-incompatible changes; however, by the same token it's completely unnecessary. If it *were* a user-facing API that someone might have a semi-rational reason to want to run a C++ compiler on, it would be incorrect to change it at this point.
|
1.39 |
| 12-Jun-2014 |
joerg | Don't t use a C++ keyword as field name.
|
1.38 |
| 01-Feb-2012 |
dholland | branches: 1.38.4; 1.38.6; 1.38.10; 1.38.24; Improve the names of some members of struct quotactl_args. These are effectively function parameter names, but since they need to be described with the same names in the man page the choices do matter. Some.
|
1.37 |
| 01-Feb-2012 |
dholland | Change the syscall API for quotas over to the new non-proplib one.
- struct vfs_quotactl_args -> struct quotactl_args - add sys/stdint.h to sys/quotactl.h for clean userland build - install sys/quotactl.h in /usr/include - update set lists for same - add new marshalling code in libquota - add new unmarshalling code in vfs_syscalls.c - discard proplib interpreter code in vfs_quotactl.c - add dispatching code for the 14 quotactl ops in vfs_quotactl.c - mark the proplib quotactl syscall obsolete - add a new syscall number for the new quotactl syscall - change the name of the syscall to __quotactl() - remove the decl of the old quotactl from quota/quotaprop.h - add a decl of the new quotactl to sys/quotactl.h - update the libc build - update ktruss - remove proplib marshalling code from libquota - update copy of syscall table in gdb ppc sources - hack rumphijack to accomodate new quotactl name (as I recall, pooka wanted such a name change to simplify something, but I don't really see what/how)
This change appears to require a kernel version bump for rumpish reasons.
|
1.36 |
| 29-Jan-2012 |
dholland | malloc -> kmem_alloc, by request of rmind
|
1.35 |
| 29-Jan-2012 |
dholland | Remove the extra op argument to VFS_QUOTACTL() - the op is now stored purely in the args structure.
This change requires a kernel version bump.
|
1.34 |
| 29-Jan-2012 |
dholland | Tidy up the VFS_QUOTACTL interface. Renumber the command codes in a logical order (as opposed to the previous order, which accumulated arbitrarily), remove the separate codes for argument encoding as there's now a 1-1 mapping between ops and argument substructures, and assert in VFS_QUOTACTL() itself that the op in the args structure matches the op passed directly.
This change requires a kernel version bump.
|
1.33 |
| 29-Jan-2012 |
dholland | Change QUOTACTL_GETVERSION to QUOTACTL_STAT. Add struct quotastat.
This change requires a kernel version bump.
|
1.32 |
| 29-Jan-2012 |
dholland | Move proplib bits for QUOTACTL_QUOTAOFF out of the ufs code.
This change requires a kernel version bump.
|
1.31 |
| 29-Jan-2012 |
dholland | Move the proplib bits for QUOTACTL_QUOTAON out of the ufs code.
This change requires a kernel version bump.
|
1.30 |
| 29-Jan-2012 |
dholland | Use QUOTACTL_CURSORSKIPIDTYPE and QUOTACTL_CURSORATEND in vfs_quotactl. Have it restart from the beginning if it receives EDEADLK, which requires QUOTACTL_CURSORREWIND.
|
1.29 |
| 29-Jan-2012 |
dholland | Add QUOTACTL_CURSORSKIPIDTYPE, QUOTACTL_CURSORATEND, QUOTACTL_CURSORREWIND.
This change requires a kernel version bump.
|
1.28 |
| 29-Jan-2012 |
dholland | Don't pass the idtype to QUOTACTL_GETALL. Instead, iterate both users and groups.
This change requires a kernel version bump.
|
1.27 |
| 29-Jan-2012 |
dholland | Call QUOTACTL_GETALL in a loop to get results 8 at a time. Make the QUOTACTL_GETALL interface less abusive.
Note: this change requires a kernel version bump.
|
1.26 |
| 29-Jan-2012 |
dholland | Rearrange result processing for QUOTACTL_GETALL.
|
1.25 |
| 29-Jan-2012 |
dholland | Stop treating the default values specially in QUOTACTL_GETALL.
Note: this change requires a kernel version bump.
|
1.24 |
| 29-Jan-2012 |
dholland | Teach quota2 QUOTACTL_GETALL to acecpt a limit on how much it sends back. Pass in a dummy limit for now.
Note: this change requires a kernel version bump.
|
1.23 |
| 29-Jan-2012 |
dholland | Hack QUOTACTL_GETALL to return results without using proplib.
(this interface is abusive and is going to be cleaned up in the immediate future)
Note: this change requires a kernel version bump.
|
1.22 |
| 29-Jan-2012 |
dholland | Pass the cursor to QUOTACTL_GETALL. Don't pass unused proplib items.
Note: this change requires a kernel version bump.
|
1.21 |
| 29-Jan-2012 |
dholland | Begin adding quota cursor/iteration interface to VFS_QUOTACTL.
Add struct quotakcursor. Add QUOTACTL_CURSOROPEN and QUOTACTL_CURSORCLOSE operations. Implement the plumbing for them. Add trivial implementations of them for quota2. (iteration is not supported on quota1 for the time being, just as getall isn't) Have the proplib interpreter open and close a cursor around doing QUOTACTL_GETALL.
Note: this change requires a kernel version bump.
|
1.20 |
| 29-Jan-2012 |
dholland | Package up the args of QUOTACTL_DELETE as a struct quotakey.
|
1.19 |
| 29-Jan-2012 |
dholland | QUOTACTL_CLEAR -> QUOTACTL_DELETE to match intended API and user API.
|
1.18 |
| 29-Jan-2012 |
dholland | Improve the quota2 QUOTACTL_CLEAR code to allow clearing blocks and files independently.
Note: this change requires a kernel version bump.
|
1.17 |
| 29-Jan-2012 |
dholland | The handling of QUOTACTL_CLEAR does not use the proplib data dictionary, so don't pass it.
Note: this change requires a kernel version bump.
|
1.16 |
| 29-Jan-2012 |
dholland | Move toplevel proplib iteration of QUOTACTL_CLEAR to fs-independent code.
Note: this change requires a kernel version bump.
|
1.15 |
| 29-Jan-2012 |
dholland | Rename QUOTACTL_SET to QUOTACTL_PUT, to match future intended API.
|
1.14 |
| 29-Jan-2012 |
dholland | Combine the miscellaneous QUOTACTL_SET args into a struct quotakey.
Note: this change requires a kernel version bump.
|
1.13 |
| 29-Jan-2012 |
dholland | Pass only one objtype and its quotaval to QUOTACTL_SET at one time.
(The backend code to handle this is a lot tidier than I expected given that the proplib code doesn't allow setting blocks and files independently; I was afraid there would turn out to be a reason for that...)
Note: this change requires a kernel version bump.
|
1.12 |
| 29-Jan-2012 |
dholland | Provide quota info to QUOTACTL_SET as two struct quotaval points as well as via proplib.
Note: this change requires a kernel version bump.
|
1.11 |
| 29-Jan-2012 |
dholland | Move the top level iteration for QUOTACTL_SET from ufs to vfs_quotactl.
Note: this change requires a kernel version bump.
|
1.10 |
| 29-Jan-2012 |
dholland | Use struct quotakey with QUOTACTL_GET. Tidy up accordingly.
Step 5 of 5 for QUOTACTL_GET.
Note: this change requires a kernel version bump.
|
1.9 |
| 29-Jan-2012 |
dholland | Per the FS-independent schema, get one quotaval at a time from the filesystem, instead of blocks and files together.
This results in fetching each FS-level quota entry twice and therefore doing slightly more work, but (1) quota access isn't a critical path and (2) after fetching the block values the file values will be hot in the cache, so it won't add much total time.
Also move more of the FS-independent defintions from <quota.h> to <sys/quota.h> so we can use them internally.
Step 4 of 5 for QUOTACTL_GET.
Note: this change requires a kernel version bump.
|
1.8 |
| 29-Jan-2012 |
dholland | Move what was second-layer proplib frobbing for QUOTACTL_GET to FS-independent code. (Step 3 of probably 5 for QUOTACTL_GET.)
Note: this change requires a kernel version bump.
|
1.7 |
| 29-Jan-2012 |
dholland | Move first-layer proplib frobbing for QUOTACTL_GET to FS-independent code. (step 1 of several)
Note: this change requires a kernel version bump.
|
1.6 |
| 29-Jan-2012 |
dholland | Move proplib frobbing for QUOTACTL_GETVERSION to FS-independent code.
Note: this change requires a kernel version bump.
|
1.5 |
| 29-Jan-2012 |
dholland | Introduce struct vfs_quotactl_args. Use it.
This change uglifies vfs_quotactl some in order to make room for moving operation-specific but FS-independent logic out of ufs_quota.c.
Note: this change requires a kernel version bump.
|
1.4 |
| 29-Jan-2012 |
dholland | Move the proplib-based quota command dispatching (that is, the code that knows the magic string names for the allowed actions) out of UFS-specific code and to fs-independent code.
This introduces QUOTACTL_* operation codes and changes the signature of VFS_QUOTACTL() again for compile safety.
Note: this change requires a kernel version bump.
|
1.3 |
| 29-Jan-2012 |
dholland | Move the code for iterating over the multiple RPC calls in a quota proplib XML packet to vfs_quotactl.c out of sys/ufs/ufs.
Add a dummy extra arg to VFS_QUOTACTL for compile safety.
Note: this change requires a kernel version bump.
|
1.2 |
| 29-Jan-2012 |
dholland | Add vfs_quotactl() in between the syscall and VFS_QUOTACTL. Call it from the COMPAT_50 code as well as the current sys_quotactl instead of going directly to VFS_QUOTACTL. Doesn't actually do anything yet.
|
1.1 |
| 29-Jan-2012 |
dholland | Add vfs_quotactl.c. This is where filesystem-independent quota handling will go.
|
1.38.24.1 |
| 10-Aug-2014 |
tls | Rebase.
|
1.38.10.1 |
| 20-Aug-2014 |
tls | Rebase to HEAD as of a few days ago.
|
1.38.6.2 |
| 17-Apr-2012 |
yamt | sync with head
|
1.38.6.1 |
| 01-Feb-2012 |
yamt | file vfs_quotactl.c was added on branch yamt-pagecache on 2012-04-17 00:08:31 +0000
|
1.38.4.2 |
| 18-Feb-2012 |
mrg | merge to -current.
|
1.38.4.1 |
| 01-Feb-2012 |
mrg | file vfs_quotactl.c was added on branch jmcneill-usbmp on 2012-02-18 07:35:34 +0000
|