Difference between
modified post 09f by Anonymous on Wed 20th Aug 2025 08:58
original post 72f6de by Anonymous on Tue 19th Aug 2025 20:02
Download diff
Show
old version |
new version |
both versions
| 1 | 1 | diff --git a/daemon/daemon_debug.c b/daemon/daemon_debug.c | |
| 2 | 2 | index 20fd420..02a0aec 100644 | |
| 3 | 3 | --- a/daemon/daemon_debug.c | |
| 4 | 4 | +++ b/daemon/daemon_debug.c | |
| 5 | 5 | @@ -485,6 +485,7 @@ const char* opcode2string(nfs41_opcodes opcode) | |
| 6 | 6 | NFSOPCODE_TO_STRLITERAL(NFS41_SYSOP_FSCTL_QUERYALLOCATEDRANGES) | |
| 7 | 7 | NFSOPCODE_TO_STRLITERAL(NFS41_SYSOP_FSCTL_SET_ZERO_DATA) | |
| 8 | 8 | NFSOPCODE_TO_STRLITERAL(NFS41_SYSOP_FSCTL_DUPLICATE_DATA) | |
| 9 | 9 | + NFSOPCODE_TO_STRLITERAL(NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY) | |
| 10 | 10 | NFSOPCODE_TO_STRLITERAL(NFS41_SYSOP_INVALID_OPCODE1) | |
| 11 | 11 | default: break; | |
| 12 | 12 | } | |
| 13 | 13 | diff --git a/daemon/fsctl.c b/daemon/fsctl.c | |
| 14 | - | index e1c4f19..a6c5048 100644 | |
| 14 | + | index e1c4f19..fbbd8aa 100644 | |
| 15 | 15 | --- a/daemon/fsctl.c | |
| 16 | 16 | +++ b/daemon/fsctl.c | |
| 17 | 17 | @@ -27,9 +27,6 @@ | |
| 18 | 18 | #include "daemon_debug.h" | |
| 19 | 19 | #include "util.h" | |
| 20 | 20 | ||
| 21 | 21 | -/* Testing only: Use NFS COPY instead of NFS CLONE */ | |
| 22 | 22 | -// #define DUP_DATA_USE_NFSCOPY 1 | |
| 23 | 23 | - | |
| 24 | 24 | #define QARLVL 2 /* dprintf level for "query allocated ranges" logging */ | |
| 25 | 25 | #define SZDLVL 2 /* dprintf level for "set zero data" logging */ | |
| 26 | 26 | #define DDLVL 2 /* dprintf level for "duplicate data" logging */ | |
| 27 | 27 | @@ -470,7 +467,8 @@ out: | |
| 28 | 28 | } | |
| 29 | 29 | ||
| 30 | 30 | static | |
| 31 | 31 | -int duplicate_sparsefile(nfs41_open_state *src_state, | |
| 32 | 32 | +int duplicate_sparsefile(nfs41_opcodes opcode, | |
| 33 | 33 | + nfs41_open_state *src_state, | |
| 34 | 34 | nfs41_open_state *dst_state, | |
| 35 | 35 | uint64_t srcfileoffset, | |
| 36 | 36 | uint64_t destfileoffset, | |
| 37 | 37 | @@ -498,7 +496,8 @@ int duplicate_sparsefile(nfs41_open_state *src_state, | |
| 38 | 38 | (void)memset(info, 0, sizeof(*info)); | |
| 39 | 39 | ||
| 40 | 40 | DPRINTF(DDLVL, | |
| 41 | 41 | - ("--> duplicate_sparsefile(src_state->path.path='%s')\n", | |
| 42 | 42 | + ("--> duplicate_sparsefile(opcode='%s',src_state->path.path='%s')\n", | |
| 43 | 43 | + opcode2string(opcode), | |
| 44 | 44 | src_state->path.path)); | |
| 45 | 45 | ||
| 46 | 46 | nfs41_open_stateid_arg(src_state, &src_stateid); | |
| 47 | 47 | @@ -518,9 +517,11 @@ int duplicate_sparsefile(nfs41_open_state *src_state, | |
| 48 | 48 | if (status) { | |
| 49 | 49 | DPRINTF(0/*DDLVL*/, | |
| 50 | 50 | ("duplicate_sparsefile(" | |
| 51 | 51 | + "opcode='%s'," | |
| 52 | 52 | "src_state->path.path='%s' " | |
| 53 | 53 | "dst_state->path.path='%s'): " | |
| 54 | 54 | "DEALLOCATE failed with '%s'\n", | |
| 55 | 55 | + opcode2string(opcode), | |
| 56 | 56 | src_state->path.path, | |
| 57 | 57 | dst_state->path.path, | |
| 58 | 58 | nfs_error_string(status))); | |
| 59 | - | @@ -594,46 +595,45 @@ int duplicate_sparsefile(nfs41_open_state *src_state, | |
| 59 | + | @@ -594,46 +595,60 @@ int duplicate_sparsefile(nfs41_open_state *src_state, | |
| 60 | 60 | (int)data_seek_sr_eof, | |
| 61 | 61 | (int)hole_seek_sr_eof)); | |
| 62 | 62 | ||
| 63 | 63 | -#ifdef DUP_DATA_USE_NFSCOPY | |
| 64 | 64 | - nfs41_write_verf verf; | |
| 65 | 65 | - status = nfs42_copy(session, | |
| 66 | 66 | - src_file, | |
| 67 | 67 | - dst_file, | |
| 68 | 68 | - &src_stateid, | |
| 69 | 69 | - &dst_stateid, | |
| 70 | 70 | - data_seek_sr_offset, | |
| 71 | 71 | - destfileoffset + (data_seek_sr_offset-srcfileoffset), | |
| 72 | 72 | - data_size, | |
| 73 | 73 | - &verf, | |
| 74 | 74 | - info); | |
| 75 | 75 | - /* FIXME: What should we do with |verf| ? Should we COMMIT this ? */ | |
| 76 | 76 | -#else | |
| 77 | 77 | - status = nfs42_clone(session, | |
| 78 | 78 | - src_file, | |
| 79 | 79 | - dst_file, | |
| 80 | 80 | - &src_stateid, | |
| 81 | 81 | - &dst_stateid, | |
| 82 | 82 | - data_seek_sr_offset, | |
| 83 | 83 | - destfileoffset + (data_seek_sr_offset-srcfileoffset), | |
| 84 | 84 | - data_size, | |
| 85 | 85 | - info); | |
| 86 | 86 | -#endif /* DUP_DATA_USE_NFSCOPY */ | |
| 87 | 87 | - if (status) { | |
| 88 | 88 | - const char dup_op_name[] = | |
| 89 | 89 | -#ifdef DUP_DATA_USE_NFSCOPY | |
| 90 | 90 | - "COPY"; | |
| 91 | 91 | -#else | |
| 92 | 92 | - "CLONE"; | |
| 93 | 93 | -#endif /* DUP_DATA_USE_NFSCOPY */ | |
| 94 | 94 | + if (opcode == NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY) { | |
| 95 | - | + nfs41_write_verf verf; | |
| 95 | + | + uint64_t bytes_written; | |
| 96 | - | + status = nfs42_copy(session, | |
| 96 | + | + uint64_t bytestowrite = data_size; | |
| 97 | - | + src_file, | |
| 97 | + | + uint64_t writeoffset = 0ULL; | |
| 98 | - | + dst_file, | |
| 98 | + | + | |
| 99 | - | + &src_stateid, | |
| 99 | + | + do | |
| 100 | - | + &dst_stateid, | |
| 100 | + | + { | |
| 101 | - | + data_seek_sr_offset, | |
| 101 | + | + nfs41_write_verf verf; | |
| 102 | - | + destfileoffset + (data_seek_sr_offset-srcfileoffset), | |
| 102 | + | + bytes_written = 0ULL; | |
| 103 | - | + data_size, | |
| 103 | + | + | |
| 104 | - | + &verf, | |
| 104 | + | + status = nfs42_copy(session, | |
| 105 | - | + info); | |
| 105 | + | + src_file, | |
| 106 | - | + /* FIXME: What should we do with |verf| ? Should we COMMIT this ? */ | |
| 106 | + | + dst_file, | |
| 107 | - | + } | |
| 107 | + | + &src_stateid, | |
| 108 | - | + else if (opcode == NFS41_SYSOP_FSCTL_DUPLICATE_DATA) { | |
| 108 | + | + &dst_stateid, | |
| 109 | - | + status = nfs42_clone(session, | |
| 109 | + | + (data_seek_sr_offset + writeoffset), | |
| 110 | - | + src_file, | |
| 110 | + | + (destfileoffset + (data_seek_sr_offset-srcfileoffset) + writeoffset), | |
| 111 | - | + dst_file, | |
| 111 | + | + bytestowrite, | |
| 112 | - | + &src_stateid, | |
| 112 | + | + &bytes_written, | |
| 113 | - | + &dst_stateid, | |
| 113 | + | + &verf, | |
| 114 | - | + data_seek_sr_offset, | |
| 114 | + | + info); | |
| 115 | - | + destfileoffset + (data_seek_sr_offset-srcfileoffset), | |
| 115 | + | + if (status) | |
| 116 | - | + data_size, | |
| 116 | + | + break; | |
| 117 | - | + info); | |
| 117 | + | + | |
| 118 | + | + bytestowrite -= bytes_written; | |
| 118 | 122 | + } | |
| 119 | - | + else { | |
| 123 | + | + else if (opcode == NFS41_SYSOP_FSCTL_DUPLICATE_DATA) { | |
| 120 | - | + EASSERT(0); | |
| 124 | + | + status = nfs42_clone(session, | |
| 125 | + | + src_file, | |
| 121 | 133 | + } | |
| 134 | + | + else { | |
| 122 | 137 | ||
| 123 | 138 | + if (status) { | |
| 124 | 139 | DPRINTF(0/*DDLVL*/, | |
| 125 | 140 | ("duplicate_sparsefile(" | |
| 126 | 141 | + "opcode='%s'," | |
| 127 | 142 | "src_state->path.path='%s' " | |
| 128 | 143 | "dst_state->path.path='%s'): " | |
| 129 | 144 | - "'%s' failed with '%s'\n", | |
| 130 | 145 | + "failed with '%s'\n", | |
| 131 | 146 | + opcode2string(opcode), | |
| 132 | 147 | src_state->path.path, | |
| 133 | 148 | dst_state->path.path, | |
| 134 | 149 | - dup_op_name, | |
| 135 | 150 | nfs_error_string(status))); | |
| 136 | 151 | status = nfs_to_windows_error(status, ERROR_BAD_NET_RESP); | |
| 137 | 152 | goto out; | |
| 138 | - | @@ -688,25 +688,35 @@ int handle_duplicatedata(void *daemon_context, | |
| 153 | + | @@ -688,25 +703,35 @@ int handle_duplicatedata(void *daemon_context, | |
| 139 | 154 | dst_state->path.path, | |
| 140 | 155 | src_state->path.path)); | |
| 141 | 156 | ||
| 142 | 157 | - /* NFS SEEK supported ? */ | |
| 143 | 158 | - if (src_session->client->root->supports_nfs42_seek == false) { | |
| 144 | 159 | - status = ERROR_NOT_SUPPORTED; | |
| 145 | 160 | - goto out; | |
| 146 | 161 | + /* | |
| 147 | 162 | + * Check whether we support the required NFS operations... | |
| 148 | 163 | + */ | |
| 149 | 164 | + if (upcall->opcode == NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY) { | |
| 150 | 165 | + if ((src_session->client->root->supports_nfs42_seek == false) || | |
| 151 | 166 | + (src_session->client->root->supports_nfs42_copy == false) || | |
| 152 | 167 | + (src_session->client->root->supports_nfs42_deallocate == false)) { | |
| 153 | 168 | + status = ERROR_NOT_SUPPORTED; | |
| 154 | 169 | + goto out; | |
| 155 | 170 | + } | |
| 156 | 171 | } | |
| 157 | 172 | - /* NFS CLONE supported ? */ | |
| 158 | 173 | - if ( | |
| 159 | 174 | -#ifdef DUP_DATA_USE_NFSCOPY | |
| 160 | 175 | - src_session->client->root->supports_nfs42_copy == false | |
| 161 | 176 | -#else | |
| 162 | 177 | - src_session->client->root->supports_nfs42_clone == false | |
| 163 | 178 | -#endif /* DUP_DATA_USE_NFSCOPY */ | |
| 164 | 179 | - ) { | |
| 165 | 180 | - status = ERROR_NOT_SUPPORTED; | |
| 166 | 181 | - goto out; | |
| 167 | 182 | + else if (upcall->opcode == NFS41_SYSOP_FSCTL_DUPLICATE_DATA) { | |
| 168 | 183 | + if ((src_session->client->root->supports_nfs42_seek == false) || | |
| 169 | 184 | + (src_session->client->root->supports_nfs42_clone == false) || | |
| 170 | 185 | + (src_session->client->root->supports_nfs42_deallocate == false)) { | |
| 171 | 186 | + status = ERROR_NOT_SUPPORTED; | |
| 172 | 187 | + goto out; | |
| 173 | 188 | + } | |
| 174 | 189 | } | |
| 175 | 190 | - /* NFS DEALLOCATE supported ? */ | |
| 176 | 191 | - if (src_session->client->root->supports_nfs42_deallocate == false) { | |
| 177 | 192 | - status = ERROR_NOT_SUPPORTED; | |
| 178 | 193 | + else { | |
| 179 | 194 | + status = ERROR_INVALID_PARAMETER; | |
| 180 | 195 | + eprintf("duplicate_sparsefile(" | |
| 181 | 196 | + "opcode='%s'," | |
| 182 | 197 | + "src_state->path.path='%s' " | |
| 183 | 198 | + "dst_state->path.path='%s'): " | |
| 184 | 199 | + "Unknown opcode.\n", | |
| 185 | 200 | + opcode2string(upcall->opcode), | |
| 186 | 201 | + src_state->path.path, | |
| 187 | 202 | + dst_state->path.path); | |
| 188 | 203 | goto out; | |
| 189 | 204 | } | |
| 190 | 205 | ||
| 191 | - | @@ -837,7 +847,8 @@ int handle_duplicatedata(void *daemon_context, | |
| 206 | + | @@ -837,7 +862,8 @@ int handle_duplicatedata(void *daemon_context, | |
| 192 | 207 | ||
| 193 | 208 | (void)memset(&info, 0, sizeof(info)); | |
| 194 | 209 | ||
| 195 | 210 | - status = duplicate_sparsefile(src_state, | |
| 196 | 211 | + status = duplicate_sparsefile(upcall->opcode, | |
| 197 | 212 | + src_state, | |
| 198 | 213 | dst_state, | |
| 199 | 214 | args->srcfileoffset, | |
| 200 | 215 | args->destfileoffset, | |
| 201 | - | @@ -879,3 +890,10 @@ const nfs41_upcall_op nfs41_op_duplicatedata = { | |
| 216 | + | @@ -879,3 +905,10 @@ const nfs41_upcall_op nfs41_op_duplicatedata = { | |
| 202 | 217 | .marshall = marshall_duplicatedata, | |
| 203 | 218 | .arg_size = sizeof(duplicatedata_upcall_args) | |
| 204 | 219 | }; | |
| 205 | 220 | + | |
| 206 | 221 | +const nfs41_upcall_op nfs41_op_offload_datacopy = { | |
| 207 | 222 | + .parse = parse_duplicatedata, | |
| 208 | 223 | + .handle = handle_duplicatedata, | |
| 209 | 224 | + .marshall = marshall_duplicatedata, | |
| 210 | 225 | + .arg_size = sizeof(duplicatedata_upcall_args) | |
| 211 | 226 | +}; | |
| 212 | - | diff --git a/daemon/upcall.c b/daemon/upcall.c | |
| 227 | + | diff --git a/daemon/nfs41_ops.h b/daemon/nfs41_ops.h | |
| 213 | - | index 45e0729..53433c4 100644 | |
| 228 | + | index dd9dcb4..bd93fbf 100644 | |
| 214 | - | --- a/daemon/upcall.c | |
| 229 | + | --- a/daemon/nfs41_ops.h | |
| 215 | - | +++ b/daemon/upcall.c | |
| 230 | + | +++ b/daemon/nfs41_ops.h | |
| 216 | - | @@ -52,6 +52,7 @@ extern const nfs41_upcall_op nfs41_op_setacl; | |
| 231 | + | @@ -1299,6 +1299,7 @@ int nfs42_copy( | |
| 217 | - | extern const nfs41_upcall_op nfs41_op_queryallocatedranges; | |
| 232 | + | IN uint64_t src_offset, | |
| 218 | - | extern const nfs41_upcall_op nfs41_op_setzerodata; | |
| 233 | + | IN uint64_t dst_offset, | |
| 219 | - | extern const nfs41_upcall_op nfs41_op_duplicatedata; | |
| 234 | + | IN uint64_t length, | |
| 220 | - | +extern const nfs41_upcall_op nfs41_op_offload_datacopy; | |
| 235 | + | + OUT uint64_t *bytes_written, | |
| 236 | + | OUT nfs41_write_verf *writeverf, | |
| 221 | 238 | ||
| 222 | - | /* |_nfs41_opcodes| and |g_upcall_op_table| must be in sync! */ | |
| 239 | + | diff --git a/daemon/nfs42_ops.c b/daemon/nfs42_ops.c | |
| 223 | - | static const nfs41_upcall_op *g_upcall_op_table[] = { | |
| 240 | + | index 582a968..3494a3b 100644 | |
| 224 | - | @@ -78,6 +79,7 @@ static const nfs41_upcall_op *g_upcall_op_table[] = { | |
| 241 | + | --- a/daemon/nfs42_ops.c | |
| 225 | - | &nfs41_op_queryallocatedranges, | |
| 242 | + | +++ b/daemon/nfs42_ops.c | |
| 226 | - | &nfs41_op_setzerodata, | |
| 243 | + | @@ -122,6 +122,7 @@ int nfs42_copy( | |
| 227 | - | &nfs41_op_duplicatedata, | |
| 244 | + | IN uint64_t src_offset, | |
| 228 | - | + &nfs41_op_offload_datacopy, | |
| 245 | + | IN uint64_t dst_offset, | |
| 229 | - | NULL, | |
| 246 | + | IN uint64_t length, | |
| 230 | - | NULL | |
| 247 | + | + OUT uint64_t *bytes_written, | |
| 231 | - | }; | |
| 248 | + | OUT nfs41_write_verf *writeverf, | |
| 232 | - | diff --git a/include/nfs41_driver.h b/include/nfs41_driver.h | |
| 249 | + | OUT nfs41_file_info *cinfo) | |
| 233 | - | index bf956f4..7a4ddfc 100644 | |
| 250 | + | { | |
| 234 | - | --- a/include/nfs41_driver.h | |
| 251 | + | @@ -205,13 +206,7 @@ int nfs42_copy( | |
| 235 | - | +++ b/include/nfs41_driver.h | |
| 252 | + | ||
| 236 | - | @@ -88,6 +88,7 @@ typedef enum _nfs41_opcodes { | |
| 253 | + | nfs41_superblock_space_changed(dst_file->fh.superblock); | |
| 237 | - | NFS41_SYSOP_FSCTL_QUERYALLOCATEDRANGES, | |
| 254 | + | ||
| 238 | - | NFS41_SYSOP_FSCTL_SET_ZERO_DATA, | |
| 255 | + | - if (copy_res.u.resok4.response.count != length) { | |
| 239 | - | NFS41_SYSOP_FSCTL_DUPLICATE_DATA, | |
| 256 | + | - DPRINTF(0, | |
| 240 | - | + NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY, | |
| 257 | + | - ("nfs42_copy: " | |
| 241 | - | NFS41_SYSOP_SHUTDOWN, | |
| 258 | + | - "copy_res.u.resok4.response.count(=%lld) < length(=%lld)\n", | |
| 242 | - | NFS41_SYSOP_INVALID_OPCODE1 | |
| 259 | + | - (long long)copy_res.u.resok4.response.count, (long long)length)); | |
| 243 | - | } nfs41_opcodes; | |
| 260 | + | - status = ERROR_NET_WRITE_FAULT; | |
| 244 | - | diff --git a/sys/nfs41sys_debug.c b/sys/nfs41sys_debug.c | |
| 261 | + | - } | |
| 245 | - | index ee95c9a..ffebee3 100644 | |
| 262 | + | + *bytes_written = copy_res.u.resok4.response.count; | |
| 246 | - | --- a/sys/nfs41sys_debug.c | |
| 263 | + | ||
| 247 | - | +++ b/sys/nfs41sys_debug.c | |
| 264 | + | out: | |
| 248 | - | @@ -683,6 +683,8 @@ const char *opcode2string(int opcode) | |
| 265 | + | return status; | |
| 249 | - | case NFS41_SYSOP_FSCTL_QUERYALLOCATEDRANGES: return "NFS41_SYSOP_FSCTL_QUERYALLOCATEDRANGES"; | |
| 266 | + | diff --git a/daemon/upcall.c b/daemon/upcall.c | |
| 250 | - | case NFS41_SYSOP_FSCTL_SET_ZERO_DATA: return "NFS41_SYSOP_FSCTL_SET_ZERO_DATA"; | |
| 267 | + | index 45e0729..53433c4 100644 | |
| 251 | - | case NFS41_SYSOP_FSCTL_DUPLICATE_DATA: return "NFS41_SYSOP_FSCTL_DUPLICATE_DATA"; | |
| 268 | + | --- a/daemon/upcall.c | |
| 252 | - | + case NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY: | |
| 269 | + | +++ b/daemon/upcall.c | |
| 253 | - | + return "NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY"; | |
| 270 | + | @@ -52,6 +52,7 @@ extern const nfs41_upcall_op nfs41_op_setacl; | |
| 254 | - | default: return "UNKNOWN"; | |
| 271 | + | extern const nfs41_upcall_op nfs41_op_queryallocatedranges; | |
| 255 | - | } | |
| 272 | + | extern const nfs41_upcall_op nfs41_op_setzerodata; | |
| 256 | - | } | |
| 273 | + | extern const nfs41_upcall_op nfs41_op_duplicatedata; | |
| 257 | - | diff --git a/sys/nfs41sys_driver.c b/sys/nfs41sys_driver.c | |
| 274 | + | +extern const nfs41_upcall_op nfs41_op_offload_datacopy; | |
| 267 | 275 | ||
| 268 | - | LONGLONG xid = 0; | |
| 276 | + | /* |_nfs41_opcodes| and |g_upcall_op_table| must be in sync! */ | |
| 269 | - | LONG open_owner_id = 1; | |
| 277 | + | static const nfs41_upcall_op *g_upcall_op_table[] = { | |
| 270 | - | @@ -772,6 +774,7 @@ NTSTATUS nfs41_DeallocateForFobx( | |
| 278 | + | @@ -78,6 +79,7 @@ static const nfs41_upcall_op *g_upcall_op_table[] = { | |
| 271 | - | __notnull PNFS41_FOBX nfs41_fobx = NFS41GetFobxExtension(pFobx); | |
| 279 | + | &nfs41_op_queryallocatedranges, | |
| 272 | - | ||
| 280 | + | &nfs41_op_setzerodata, | |
| 273 | - | nfs41_invalidate_fobx_entry(pFobx); | |
| 281 | + | &nfs41_op_duplicatedata, | |
| 274 | - | + nfs41_remove_offloadcontext_for_fobx(pFobx); | |
| 282 | + | + &nfs41_op_offload_datacopy, | |
| 275 | - | ||
| 283 | + | NULL, | |
| 276 | - | if (nfs41_fobx->acl) { | |
| 284 | + | NULL | |
| 277 | - | RxFreePool(nfs41_fobx->acl); | |
| 285 | + | }; | |
| 278 | - | @@ -1396,9 +1399,11 @@ NTSTATUS DriverEntry( | |
| 286 | + | diff --git a/include/nfs41_driver.h b/include/nfs41_driver.h | |
| 279 | - | ExInitializeFastMutex(&downcallLock); | |
| 287 | + | index bf956f4..7a4ddfc 100644 | |
| 280 | - | ExInitializeFastMutex(&openOwnerLock); | |
| 288 | + | --- a/include/nfs41_driver.h | |
| 281 | - | ExInitializeFastMutex(&fcblistLock); | |
| 289 | + | +++ b/include/nfs41_driver.h | |
| 282 | - | + ExInitializeFastMutex(&offloadcontextLock); | |
| 290 | + | @@ -88,6 +88,7 @@ typedef enum _nfs41_opcodes { | |
| 283 | - | InitializeListHead(&upcall.head); | |
| 291 | + | NFS41_SYSOP_FSCTL_QUERYALLOCATEDRANGES, | |
| 284 | - | InitializeListHead(&downcall.head); | |
| 292 | + | NFS41_SYSOP_FSCTL_SET_ZERO_DATA, | |
| 285 | - | InitializeListHead(&openlist.head); | |
| 293 | + | NFS41_SYSOP_FSCTL_DUPLICATE_DATA, | |
| 286 | - | + InitializeListHead(&offloadcontext_list.head); | |
| 294 | + | + NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY, | |
| 287 | - | #ifdef USE_LOOKASIDELISTS_FOR_UPDOWNCALLENTRY_MEM | |
| 295 | + | NFS41_SYSOP_SHUTDOWN, | |
| 288 | - | /* | |
| 296 | + | NFS41_SYSOP_INVALID_OPCODE1 | |
| 289 | - | * The |Depth| parameter is unfortunately ignored in Win10, | |
| 297 | + | } nfs41_opcodes; | |
| 290 | - | diff --git a/sys/nfs41sys_driver.h b/sys/nfs41sys_driver.h | |
| 298 | + | diff --git a/sys/nfs41sys_debug.c b/sys/nfs41sys_debug.c | |
| 291 | - | index 597d207..3f8cc68 100644 | |
| 299 | + | index ee95c9a..ffebee3 100644 | |
| 292 | - | --- a/sys/nfs41sys_driver.h | |
| 300 | + | --- a/sys/nfs41sys_debug.c | |
| 293 | - | +++ b/sys/nfs41sys_driver.h | |
| 301 | + | +++ b/sys/nfs41sys_debug.c | |
| 294 | - | @@ -507,6 +507,12 @@ typedef struct _nfs41_fcb_list { | |
| 302 | + | @@ -683,6 +683,8 @@ const char *opcode2string(int opcode) | |
| 295 | - | } nfs41_fcb_list; | |
| 303 | + | case NFS41_SYSOP_FSCTL_QUERYALLOCATEDRANGES: return "NFS41_SYSOP_FSCTL_QUERYALLOCATEDRANGES"; | |
| 296 | - | nfs41_fcb_list openlist; | |
| 304 | + | case NFS41_SYSOP_FSCTL_SET_ZERO_DATA: return "NFS41_SYSOP_FSCTL_SET_ZERO_DATA"; | |
| 297 | - | ||
| 305 | + | case NFS41_SYSOP_FSCTL_DUPLICATE_DATA: return "NFS41_SYSOP_FSCTL_DUPLICATE_DATA"; | |
| 298 | - | +typedef struct _nfs41_offloadcontext_list { | |
| 306 | + | + case NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY: | |
| 299 | - | + LIST_ENTRY head; | |
| 307 | + | + return "NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY"; | |
| 300 | - | +} nfs41_offloadcontext_list; | |
| 308 | + | default: return "UNKNOWN"; | |
| 301 | - | +extern nfs41_offloadcontext_list offloadcontext_list; | |
| 309 | + | } | |
| 302 | - | +extern FAST_MUTEX offloadcontextLock; | |
| 310 | + | } | |
| 303 | - | + | |
| 311 | + | diff --git a/sys/nfs41sys_driver.c b/sys/nfs41sys_driver.c | |
| 304 | - | typedef enum _NULMRX_STORAGE_TYPE_CODES { | |
| 312 | + | index b079180..fd72305 100644 | |
| 305 | - | NTC_NFS41_DEVICE_EXTENSION = (NODE_TYPE_CODE)0xFC00, | |
| 313 | + | --- a/sys/nfs41sys_driver.c | |
| 306 | - | } NFS41_STORAGE_TYPE_CODES; | |
| 314 | + | +++ b/sys/nfs41sys_driver.c | |
| 307 | - | @@ -685,6 +691,8 @@ NTSTATUS marshal_nfs41_duplicatedata( | |
| 315 | + | @@ -131,6 +131,8 @@ PRDBSS_DEVICE_OBJECT nfs41_dev; | |
| 308 | - | NTSTATUS unmarshal_nfs41_duplicatedata( | |
| 316 | + | KEVENT upcallEvent; | |
| 309 | - | nfs41_updowncall_entry *cur, | |
| 317 | + | FAST_MUTEX upcallLock, downcallLock, fcblistLock; | |
| 310 | - | unsigned char **buf); | |
| 318 | + | FAST_MUTEX openOwnerLock; | |
| 311 | - | +void nfs41_remove_offloadcontext_for_fobx( | |
| 319 | + | +FAST_MUTEX offloadcontextLock; | |
| 312 | - | + IN PMRX_FOBX pFobx); | |
| 320 | + | +nfs41_offloadcontext_list offloadcontext_list; | |
| 313 | 321 | ||
| 314 | - | /* nfs41sys_ioctl.c */ | |
| 322 | + | LONGLONG xid = 0; | |
| 315 | - | NTSTATUS nfs41_IoCtl( | |
| 323 | + | LONG open_owner_id = 1; | |
| 316 | - | diff --git a/sys/nfs41sys_fsctl.c b/sys/nfs41sys_fsctl.c | |
| 324 | + | @@ -772,6 +774,7 @@ NTSTATUS nfs41_DeallocateForFobx( | |
| 317 | - | index c1a3f2b..5e2eb9f 100644 | |
| 325 | + | __notnull PNFS41_FOBX nfs41_fobx = NFS41GetFobxExtension(pFobx); | |
| 327 | 326 | ||
| 328 | - | #include "nfs41sys_buildconfig.h" | |
| 327 | + | nfs41_invalidate_fobx_entry(pFobx); | |
| 329 | - | @@ -863,6 +863,434 @@ NTSTATUS unmarshal_nfs41_duplicatedata( | |
| 328 | + | + nfs41_remove_offloadcontext_for_fobx(pFobx); | |
| 332 | 329 | ||
| 333 | - | +/* | |
| 330 | + | if (nfs41_fobx->acl) { | |
| 334 | - | + * |offloadcontext_entry| - context to store |FSCTL_OFFLOAD_READ| token | |
| 331 | + | RxFreePool(nfs41_fobx->acl); | |
| 335 | - | + * information | |
| 332 | + | @@ -1396,9 +1399,11 @@ NTSTATUS DriverEntry( | |
| 336 | - | + * | |
| 333 | + | ExInitializeFastMutex(&downcallLock); | |
| 337 | - | + * * Notes: | |
| 334 | + | ExInitializeFastMutex(&openOwnerLock); | |
| 338 | - | + * - These are stored in a global list, as |FSCTL_OFFLOAD_READ|+ | |
| 335 | + | ExInitializeFastMutex(&fcblistLock); | |
| 339 | - | + * |FSCTL_OFFLOAD_WRITE| is intended to work for intra-server and | |
| 336 | + | + ExInitializeFastMutex(&offloadcontextLock); | |
| 340 | - | + * inter-server copies, so |FSCTL_OFFLOAD_READ| might be done on one | |
| 337 | + | InitializeListHead(&upcall.head); | |
| 341 | - | + * filesystem but |FSCTL_OFFLOAD_WRITE| might be done on a different | |
| 338 | + | InitializeListHead(&downcall.head); | |
| 342 | - | + * one | |
| 339 | + | InitializeListHead(&openlist.head); | |
| 343 | - | + * | |
| 340 | + | + InitializeListHead(&offloadcontext_list.head); | |
| 344 | - | + * * FIXME: | |
| 341 | + | #ifdef USE_LOOKASIDELISTS_FOR_UPDOWNCALLENTRY_MEM | |
| 345 | - | + * - Is it legal if one user passes a token to another user, or | |
| 342 | + | /* | |
| 346 | - | + * should this be prevented ? | |
| 343 | + | * The |Depth| parameter is unfortunately ignored in Win10, | |
| 347 | - | + */ | |
| 344 | + | diff --git a/sys/nfs41sys_driver.h b/sys/nfs41sys_driver.h | |
| 348 | - | +typedef struct _offloadcontext_entry | |
| 345 | + | index 597d207..3f8cc68 100644 | |
| 349 | - | +{ | |
| 346 | + | --- a/sys/nfs41sys_driver.h | |
| 350 | - | + LIST_ENTRY next; | |
| 347 | + | +++ b/sys/nfs41sys_driver.h | |
| 351 | - | + /* | |
| 348 | + | @@ -507,6 +507,12 @@ typedef struct _nfs41_fcb_list { | |
| 352 | - | + * r/w lock - shared access for |FSCTL_OFFLOAD_WRITE|, so one token can | |
| 349 | + | } nfs41_fcb_list; | |
| 353 | - | + * be used for multiple parallel writes, exclusive access for file delete | |
| 350 | + | nfs41_fcb_list openlist; | |
| 354 | - | + * (i.e. wait until all shared access before deleting the context) | |
| 351 | + | ||
| 355 | - | + */ | |
| 352 | + | +typedef struct _nfs41_offloadcontext_list { | |
| 356 | - | + ERESOURCE resource; | |
| 353 | + | + LIST_ENTRY head; | |
| 357 | - | + STORAGE_OFFLOAD_TOKEN token; | |
| 354 | + | +} nfs41_offloadcontext_list; | |
| 358 | - | + PNFS41_FOBX src_fobx; | |
| 355 | + | +extern nfs41_offloadcontext_list offloadcontext_list; | |
| 359 | - | + ULONGLONG src_fileoffset; | |
| 356 | + | +extern FAST_MUTEX offloadcontextLock; | |
| 362 | 357 | + | |
| 363 | - | + | |
| 358 | + | typedef enum _NULMRX_STORAGE_TYPE_CODES { | |
| 359 | + | NTC_NFS41_DEVICE_EXTENSION = (NODE_TYPE_CODE)0xFC00, | |
| 364 | 365 | +void nfs41_remove_offloadcontext_for_fobx( | |
| 365 | - | + IN PMRX_FOBX pFobx) | |
| 366 | + | + IN PMRX_FOBX pFobx); | |
| 366 | - | +{ | |
| 367 | + | ||
| 367 | - | + PLIST_ENTRY pEntry; | |
| 368 | + | /* nfs41sys_ioctl.c */ | |
| 368 | - | + offloadcontext_entry *cur, *found = NULL; | |
| 369 | + | NTSTATUS nfs41_IoCtl( | |
| 369 | - | + __notnull PNFS41_FOBX nfs41_fobx = NFS41GetFobxExtension(pFobx); | |
| 370 | + | diff --git a/sys/nfs41sys_fsctl.c b/sys/nfs41sys_fsctl.c | |
| 370 | - | + | |
| 371 | + | index c1a3f2b..5e2eb9f 100644 | |
| 371 | - | + ExAcquireFastMutexUnsafe(&offloadcontextLock); | |
| 372 | + | --- a/sys/nfs41sys_fsctl.c | |
| 372 | - | + | |
| 373 | + | +++ b/sys/nfs41sys_fsctl.c | |
| 373 | - | + pEntry = offloadcontext_list.head.Flink; | |
| 374 | + | @@ -56,7 +56,7 @@ | |
| 374 | - | + while (!IsListEmpty(&offloadcontext_list.head)) { | |
| 375 | + | #include <rx.h> | |
| 375 | - | + cur = (offloadcontext_entry *)CONTAINING_RECORD(pEntry, | |
| 376 | + | #include <windef.h> | |
| 376 | - | + offloadcontext_entry, next); | |
| 377 | + | #include <winerror.h> | |
| 377 | - | + if (cur->src_fobx == nfs41_fobx) { | |
| 378 | + | - | |
| 378 | - | + found = cur; | |
| 379 | + | +#include <ntddstor.h> | |
| 379 | - | + break; | |
| 380 | + | #include <Ntstrsafe.h> | |
| 380 | - | + } | |
| 381 | + | ||
| 381 | - | + if (pEntry->Flink == &offloadcontext_list.head) { | |
| 382 | + | #include "nfs41sys_buildconfig.h" | |
| 382 | - | + break; | |
| 383 | + | @@ -863,6 +863,434 @@ NTSTATUS unmarshal_nfs41_duplicatedata( | |
| 383 | - | + } | |
| 384 | + | return status; | |
| 384 | - | + pEntry = pEntry->Flink; | |
| 385 | + | } | |
| 385 | - | + } | |
| 386 | + | ||
| 386 | - | + | |
| 387 | + | +/* | |
| 387 | - | + if (found) { | |
| 388 | + | + * |offloadcontext_entry| - context to store |FSCTL_OFFLOAD_READ| token | |
| 388 | - | + DbgP("nfs41_remove_offloadcontext(pFobx=0x%p): " | |
| 389 | + | + * information | |
| 389 | - | + "removing found=0x%p\n", | |
| 390 | + | + * | |
| 390 | - | + pFobx, | |
| 391 | + | + * * Notes: | |
| 391 | - | + found); | |
| 392 | + | + * - These are stored in a global list, as |FSCTL_OFFLOAD_READ|+ | |
| 392 | - | + | |
| 393 | + | + * |FSCTL_OFFLOAD_WRITE| is intended to work for intra-server and | |
| 393 | - | + /* Wait for any shared access in |nfs41_OffloadWrite()| to finish */ | |
| 394 | + | + * inter-server copies, so |FSCTL_OFFLOAD_READ| might be done on one | |
| 394 | - | + (void)ExAcquireResourceExclusiveLite(&found->resource, TRUE); | |
| 395 | + | + * filesystem but |FSCTL_OFFLOAD_WRITE| might be done on a different | |
| 395 | - | + ExReleaseResourceLite(&found->resource); | |
| 396 | + | + * one | |
| 396 | - | + | |
| 397 | + | + * | |
| 397 | - | + RemoveEntryList(&found->next); | |
| 398 | + | + * * FIXME: | |
| 398 | - | + | |
| 399 | + | + * - Is it legal if one user passes a token to another user, or | |
| 399 | - | + (void)ExDeleteResourceLite(&found->resource); | |
| 400 | + | + * should this be prevented ? | |
| 400 | - | + RxFreePool(found); | |
| 401 | + | + */ | |
| 401 | - | + } | |
| 402 | + | +typedef struct _offloadcontext_entry | |
| 402 | - | + else { | |
| 403 | + | +{ | |
| 403 | - | + DbgP("nfs41_remove_offloadcontext(pFobx=0x%p): Nothing found.\n", | |
| 404 | + | + LIST_ENTRY next; | |
| 404 | - | + pFobx); | |
| 405 | + | + /* | |
| 405 | - | + } | |
| 406 | + | + * r/w lock - shared access for |FSCTL_OFFLOAD_WRITE|, so one token can | |
| 407 | + | + * be used for multiple parallel writes, exclusive access for file delete | |
| 406 | 416 | + | |
| 409 | 417 | + | |
| 410 | - | +static | |
| 418 | + | +void nfs41_remove_offloadcontext_for_fobx( | |
| 411 | - | +offloadcontext_entry *nfs41_find_offloadcontext_acquireshared( | |
| 419 | + | + IN PMRX_FOBX pFobx) | |
| 413 | 420 | +{ | |
| 414 | 421 | + PLIST_ENTRY pEntry; | |
| 415 | 422 | + offloadcontext_entry *cur, *found = NULL; | |
| 423 | + | + __notnull PNFS41_FOBX nfs41_fobx = NFS41GetFobxExtension(pFobx); | |
| 416 | 424 | + | |
| 417 | 425 | + ExAcquireFastMutexUnsafe(&offloadcontextLock); | |
| 418 | 426 | + | |
| 419 | 427 | + pEntry = offloadcontext_list.head.Flink; | |
| 420 | 428 | + while (!IsListEmpty(&offloadcontext_list.head)) { | |
| 421 | 429 | + cur = (offloadcontext_entry *)CONTAINING_RECORD(pEntry, | |
| 422 | 430 | + offloadcontext_entry, next); | |
| 423 | - | + if (cur == unvalidated_oce) { | |
| 431 | + | + if (cur->src_fobx == nfs41_fobx) { | |
| 424 | 432 | + found = cur; | |
| 425 | 433 | + break; | |
| 426 | 434 | + } | |
| 427 | 435 | + if (pEntry->Flink == &offloadcontext_list.head) { | |
| 428 | 436 | + break; | |
| 429 | 437 | + } | |
| 430 | 438 | + pEntry = pEntry->Flink; | |
| 431 | 439 | + } | |
| 432 | 440 | + | |
| 433 | 441 | + if (found) { | |
| 434 | - | + DbgP("nfs41_find_offloadcontext_acquireshared(unvalidated_oce=0x%p): " | |
| 442 | + | + DbgP("nfs41_remove_offloadcontext(pFobx=0x%p): " | |
| 435 | - | + "found=0x%p\n", | |
| 443 | + | + "removing found=0x%p\n", | |
| 436 | - | + unvalidated_oce); | |
| 444 | + | + pFobx, | |
| 445 | + | + found); | |
| 437 | 446 | + | |
| 438 | - | + (void)ExAcquireSharedStarveExclusive(&found->resource, TRUE); | |
| 447 | + | + /* Wait for any shared access in |nfs41_OffloadWrite()| to finish */ | |
| 439 | - | + ExReleaseFastMutexUnsafe(&offloadcontextLock); | |
| 448 | + | + (void)ExAcquireResourceExclusiveLite(&found->resource, TRUE); | |
| 440 | - | + return found; | |
| 449 | + | + ExReleaseResourceLite(&found->resource); | |
| 450 | + | + | |
| 441 | 455 | + } | |
| 442 | 456 | + else { | |
| 443 | - | + DbgP("nfs41_find_offloadcontext_acquireshared(unvalidated_oce=0x%p): " | |
| 457 | + | + DbgP("nfs41_remove_offloadcontext(pFobx=0x%p): Nothing found.\n", | |
| 444 | - | + "Nothing found.\n", | |
| 458 | + | + pFobx); | |
| 448 | 459 | + } | |
| 450 | 460 | + | |
| 461 | + | + ExReleaseFastMutexUnsafe(&offloadcontextLock); | |
| 451 | 464 | +static | |
| 452 | - | +NTSTATUS nfs41_OffloadRead( | |
| 465 | + | +offloadcontext_entry *nfs41_find_offloadcontext_acquireshared( | |
| 453 | - | + IN OUT PRX_CONTEXT RxContext) | |
| 466 | + | + IN offloadcontext_entry *unvalidated_oce) | |
| 454 | 467 | +{ | |
| 455 | - | + NTSTATUS status = STATUS_INVALID_DEVICE_REQUEST; | |
| 468 | + | + PLIST_ENTRY pEntry; | |
| 456 | - | + __notnull XXCTL_LOWIO_COMPONENT *FsCtl = | |
| 469 | + | + offloadcontext_entry *cur, *found = NULL; | |
| 459 | 470 | + | |
| 460 | - | + DbgEn(); | |
| 471 | + | + ExAcquireFastMutexUnsafe(&offloadcontextLock); | |
| 461 | 472 | + | |
| 462 | - | + RxContext->IoStatusBlock.Information = 0; | |
| 473 | + | + pEntry = offloadcontext_list.head.Flink; | |
| 463 | - | + | |
| 474 | + | + while (!IsListEmpty(&offloadcontext_list.head)) { | |
| 464 | - | + if (FsCtl->pInputBuffer == NULL) { | |
| 475 | + | + cur = (offloadcontext_entry *)CONTAINING_RECORD(pEntry, | |
| 465 | - | + status = STATUS_INVALID_USER_BUFFER; | |
| 476 | + | + offloadcontext_entry, next); | |
| 466 | - | + goto out; | |
| 477 | + | + if (cur == unvalidated_oce) { | |
| 478 | + | + found = cur; | |
| 467 | 485 | + } | |
| 468 | 486 | + | |
| 469 | - | + if (FsCtl->pOutputBuffer == NULL) { | |
| 487 | + | + if (found) { | |
| 470 | - | + status = STATUS_INVALID_USER_BUFFER; | |
| 488 | + | + DbgP("nfs41_find_offloadcontext_acquireshared(unvalidated_oce=0x%p): " | |
| 471 | - | + goto out; | |
| 489 | + | + "found=0x%p\n", | |
| 490 | + | + unvalidated_oce); | |
| 473 | 491 | + | |
| 474 | - | + if (FsCtl->InputBufferLength < sizeof(FSCTL_OFFLOAD_READ_INPUT)) { | |
| 492 | + | + (void)ExAcquireSharedStarveExclusive(&found->resource, TRUE); | |
| 475 | - | + DbgP("nfs41_OffloadRead: " | |
| 493 | + | + ExReleaseFastMutexUnsafe(&offloadcontextLock); | |
| 476 | - | + "buffer too small for FSCTL_OFFLOAD_READ_INPUT\n"); | |
| 494 | + | + return found; | |
| 479 | 495 | + } | |
| 480 | - | + if (FsCtl->OutputBufferLength < sizeof(FSCTL_OFFLOAD_READ_OUTPUT)) { | |
| 496 | + | + else { | |
| 481 | - | + DbgP("nfs41_OffloadRead: " | |
| 497 | + | + DbgP("nfs41_find_offloadcontext_acquireshared(unvalidated_oce=0x%p): " | |
| 482 | - | + "buffer too small for FSCTL_OFFLOAD_READ_OUTPUT\n"); | |
| 498 | + | + "Nothing found.\n", | |
| 483 | - | + status = STATUS_BUFFER_TOO_SMALL; | |
| 499 | + | + unvalidated_oce); | |
| 484 | - | + goto out; | |
| 500 | + | + ExReleaseFastMutexUnsafe(&offloadcontextLock); | |
| 501 | + | + return NULL; | |
| 485 | 502 | + } | |
| 503 | + | +} | |
| 486 | 504 | + | |
| 487 | - | + PFSCTL_OFFLOAD_READ_INPUT ori = | |
| 505 | + | +static | |
| 488 | - | + (PFSCTL_OFFLOAD_READ_INPUT)FsCtl->pInputBuffer; | |
| 506 | + | +NTSTATUS nfs41_OffloadRead( | |
| 489 | - | + PFSCTL_OFFLOAD_READ_OUTPUT oro = | |
| 507 | + | + IN OUT PRX_CONTEXT RxContext) | |
| 490 | - | + (PFSCTL_OFFLOAD_READ_OUTPUT)FsCtl->pOutputBuffer; | |
| 508 | + | +{ | |
| 509 | + | + NTSTATUS status = STATUS_INVALID_DEVICE_REQUEST; | |
| 491 | 513 | + | |
| 492 | - | + DbgP("nfs41_OffloadRead: " | |
| 514 | + | + DbgEn(); | |
| 501 | 515 | + | |
| 502 | - | + offloadcontext_entry *oce = RxAllocatePoolWithTag(NonPagedPoolNx, | |
| 516 | + | + RxContext->IoStatusBlock.Information = 0; | |
| 508 | 517 | + | |
| 509 | - | + DbgP("nfs41_OffloadRead: oce=0x%p\n", oce); | |
| 518 | + | + if (FsCtl->pInputBuffer == NULL) { | |
| 519 | + | + status = STATUS_INVALID_USER_BUFFER; | |
| 510 | 522 | + | |
| 511 | - | + (void)ExInitializeResourceLite(&oce->resource); | |
| 523 | + | + if (FsCtl->pOutputBuffer == NULL) { | |
| 524 | + | + status = STATUS_INVALID_USER_BUFFER; | |
| 512 | 527 | + | |
| 513 | - | + (void)memset(&oce->token, 0, sizeof(oce->token)); | |
| 528 | + | + if (FsCtl->InputBufferLength < sizeof(FSCTL_OFFLOAD_READ_INPUT)) { | |
| 514 | - | + /* Add safeguard to |TokenType| */ | |
| 529 | + | + DbgP("nfs41_OffloadRead: " | |
| 515 | - | + oce->token.TokenType[0] = 'N'; | |
| 530 | + | + "buffer too small for FSCTL_OFFLOAD_READ_INPUT\n"); | |
| 516 | - | + oce->token.TokenType[1] = 'F'; | |
| 531 | + | + status = STATUS_BUFFER_TOO_SMALL; | |
| 517 | - | + oce->token.TokenType[2] = 'S'; | |
| 532 | + | + goto out; | |
| 518 | - | + oce->token.TokenType[3] = '4'; | |
| 533 | + | + } | |
| 519 | - | + /* FIXME: What about the endianness of |TokenIdLength| ? */ | |
| 534 | + | + if (FsCtl->OutputBufferLength < sizeof(FSCTL_OFFLOAD_READ_OUTPUT)) { | |
| 520 | - | + *((USHORT *)(&oce->token.TokenIdLength[0])) = | |
| 535 | + | + DbgP("nfs41_OffloadRead: " | |
| 521 | - | + STORAGE_OFFLOAD_TOKEN_ID_LENGTH; | |
| 536 | + | + "buffer too small for FSCTL_OFFLOAD_READ_OUTPUT\n"); | |
| 522 | - | + *((void **)(&oce->token.Token[0])) = oce; | |
| 537 | + | + status = STATUS_BUFFER_TOO_SMALL; | |
| 523 | - | + oce->src_fobx = nfs41_fobx; | |
| 538 | + | + goto out; | |
| 524 | - | + oce->src_fileoffset = ori->FileOffset; | |
| 539 | + | + } | |
| 526 | 540 | + | |
| 527 | - | + oro->Size = sizeof(FSCTL_OFFLOAD_READ_OUTPUT); | |
| 541 | + | + PFSCTL_OFFLOAD_READ_INPUT ori = | |
| 528 | - | + oro->Flags = 0; | |
| 542 | + | + (PFSCTL_OFFLOAD_READ_INPUT)FsCtl->pInputBuffer; | |
| 529 | - | + oro->TransferLength = ori->CopyLength; | |
| 543 | + | + PFSCTL_OFFLOAD_READ_OUTPUT oro = | |
| 530 | - | + (void)memcpy(&oro->Token[0], &oce->token, sizeof(oce->token)); | |
| 544 | + | + (PFSCTL_OFFLOAD_READ_OUTPUT)FsCtl->pOutputBuffer; | |
| 531 | 545 | + | |
| 532 | - | + nfs41_AddEntry(offloadcontextLock, offloadcontext_list, oce); | |
| 546 | + | + DbgP("nfs41_OffloadRead: " | |
| 547 | + | + "ori->(Size=%lu, Flags=0x%lx, TokenTimeToLive=%lu, Reserved=%lu, " | |
| 533 | 555 | + | |
| 534 | - | + RxContext->CurrentIrp->IoStatus.Status = status = STATUS_SUCCESS; | |
| 556 | + | + offloadcontext_entry *oce = RxAllocatePoolWithTag(NonPagedPoolNx, | |
| 535 | - | + RxContext->InformationToReturn = sizeof(FSCTL_OFFLOAD_READ_OUTPUT); | |
| 557 | + | + sizeof(offloadcontext_entry), NFS41_MM_POOLTAG); | |
| 558 | + | + if (oce == NULL) { | |
| 536 | 562 | + | |
| 537 | - | +out: | |
| 563 | + | + DbgP("nfs41_OffloadRead: oce=0x%p\n", oce); | |
| 541 | 564 | + | |
| 542 | - | +static | |
| 565 | + | + (void)ExInitializeResourceLite(&oce->resource); | |
| 550 | 566 | + | |
| 551 | - | + /* access checks */ | |
| 567 | + | + (void)memset(&oce->token, 0, sizeof(oce->token)); | |
| 552 | - | + if (VNetRootContext->read_only) { | |
| 568 | + | + /* Add safeguard to |TokenType| */ | |
| 553 | - | + status = STATUS_MEDIA_WRITE_PROTECTED; | |
| 569 | + | + oce->token.TokenType[0] = 'N'; | |
| 554 | - | + goto out; | |
| 570 | + | + oce->token.TokenType[1] = 'F'; | |
| 555 | - | + } | |
| 571 | + | + oce->token.TokenType[2] = 'S'; | |
| 556 | - | + if (!(SrvOpen->DesiredAccess & | |
| 572 | + | + oce->token.TokenType[3] = '4'; | |
| 557 | - | + (FILE_WRITE_DATA|FILE_WRITE_ATTRIBUTES))) { | |
| 573 | + | + /* FIXME: What about the endianness of |TokenIdLength| ? */ | |
| 558 | - | + status = STATUS_ACCESS_DENIED; | |
| 574 | + | + *((USHORT *)(&oce->token.TokenIdLength[0])) = | |
| 559 | - | + goto out; | |
| 575 | + | + STORAGE_OFFLOAD_TOKEN_ID_LENGTH; | |
| 560 | - | + } | |
| 576 | + | + *((void **)(&oce->token.Token[0])) = oce; | |
| 577 | + | + oce->src_fobx = nfs41_fobx; | |
| 561 | 580 | + | |
| 562 | - | +out: | |
| 581 | + | + oro->Size = sizeof(FSCTL_OFFLOAD_READ_OUTPUT); | |
| 563 | - | + return status; | |
| 582 | + | + oro->Flags = 0; | |
| 564 | - | +} | |
| 583 | + | + oro->TransferLength = ori->CopyLength; | |
| 584 | + | + (void)memcpy(&oro->Token[0], &oce->token, sizeof(oce->token)); | |
| 565 | 585 | + | |
| 566 | - | +static | |
| 586 | + | + nfs41_AddEntry(offloadcontextLock, offloadcontext_list, oce); | |
| 567 | - | +NTSTATUS nfs41_OffloadWrite( | |
| 587 | + | + | |
| 568 | - | + IN OUT PRX_CONTEXT RxContext) | |
| 588 | + | + RxContext->CurrentIrp->IoStatus.Status = status = STATUS_SUCCESS; | |
| 569 | - | +{ | |
| 589 | + | + RxContext->InformationToReturn = sizeof(FSCTL_OFFLOAD_READ_OUTPUT); | |
| 570 | - | + NTSTATUS status = STATUS_INVALID_DEVICE_REQUEST; | |
| 590 | + | + | |
| 571 | - | + nfs41_updowncall_entry *entry = NULL; | |
| 591 | + | +out: | |
| 592 | + | + DbgEx(); | |
| 572 | 601 | + __notnull PMRX_SRV_OPEN SrvOpen = RxContext->pRelevantSrvOpen; | |
| 573 | - | + __notnull PNFS41_V_NET_ROOT_EXTENSION pVNetRootContext = | |
| 602 | + | + __notnull PNFS41_V_NET_ROOT_EXTENSION VNetRootContext = | |
| 574 | 603 | + NFS41GetVNetRootExtension(SrvOpen->pVNetRoot); | |
| 581 | 604 | + | |
| 582 | - | + struct { | |
| 605 | + | + /* access checks */ | |
| 583 | - | + LONGLONG srcfileoffset; | |
| 606 | + | + if (VNetRootContext->read_only) { | |
| 584 | - | + LONGLONG destfileoffset; | |
| 607 | + | + status = STATUS_MEDIA_WRITE_PROTECTED; | |
| 585 | - | + LONGLONG bytecount; | |
| 608 | + | + goto out; | |
| 586 | - | + } dd; | |
| 609 | + | + } | |
| 610 | + | + if (!(SrvOpen->DesiredAccess & | |
| 587 | 615 | + | |
| 588 | - | + DbgEn(); | |
| 616 | + | +out: | |
| 617 | + | + return status; | |
| 589 | 619 | + | |
| 590 | - | + LONGLONG io_delay; | |
| 620 | + | +static | |
| 591 | - | + RxContext->IoStatusBlock.Information = 0; | |
| 621 | + | +NTSTATUS nfs41_OffloadWrite( | |
| 592 | - | + | |
| 622 | + | + IN OUT PRX_CONTEXT RxContext) | |
| 593 | - | + status = check_nfs41_offload_write_args(RxContext); | |
| 623 | + | +{ | |
| 594 | - | + if (status) | |
| 624 | + | + NTSTATUS status = STATUS_INVALID_DEVICE_REQUEST; | |
| 595 | - | + goto out; | |
| 625 | + | + nfs41_updowncall_entry *entry = NULL; | |
| 626 | + | + __notnull PMRX_SRV_OPEN SrvOpen = RxContext->pRelevantSrvOpen; | |
| 596 | 635 | + | |
| 597 | - | + if (FsCtl->pInputBuffer == NULL) { | |
| 636 | + | + struct { | |
| 598 | - | + status = STATUS_INVALID_USER_BUFFER; | |
| 637 | + | + LONGLONG srcfileoffset; | |
| 599 | - | + goto out; | |
| 638 | + | + LONGLONG destfileoffset; | |
| 600 | - | + } | |
| 639 | + | + LONGLONG bytecount; | |
| 601 | - | + if (FsCtl->pOutputBuffer == NULL) { | |
| 640 | + | + } dd; | |
| 602 | - | + status = STATUS_INVALID_USER_BUFFER; | |
| 641 | + | + | |
| 642 | + | + DbgEn(); | |
| 603 | 649 | + goto out; | |
| 650 | + | + | |
| 604 | 654 | + } | |
| 655 | + | + if (FsCtl->pOutputBuffer == NULL) { | |
| 605 | 659 | + if (FsCtl->InputBufferLength < sizeof(FSCTL_OFFLOAD_WRITE_INPUT)) { | |
| 606 | 660 | + DbgP("nfs41_OffloadWrite: " | |
| 607 | 661 | + "buffer too small for FSCTL_OFFLOAD_WRITE_INPUT\n"); | |
| 608 | 662 | + status = STATUS_BUFFER_TOO_SMALL; | |
| 609 | 663 | + goto out; | |
| 610 | 664 | + } | |
| 611 | 665 | + if (FsCtl->OutputBufferLength < sizeof(FSCTL_OFFLOAD_WRITE_OUTPUT)) { | |
| 612 | 666 | + DbgP("nfs41_OffloadWrite: " | |
| 613 | 667 | + "buffer too small for FSCTL_OFFLOAD_WRITE_OUTPUT\n"); | |
| 614 | 668 | + status = STATUS_BUFFER_TOO_SMALL; | |
| 615 | 669 | + goto out; | |
| 616 | 670 | + } | |
| 617 | 671 | + | |
| 618 | 672 | + PFSCTL_OFFLOAD_WRITE_INPUT owi = | |
| 619 | 673 | + (PFSCTL_OFFLOAD_WRITE_INPUT)FsCtl->pInputBuffer; | |
| 620 | 674 | + PFSCTL_OFFLOAD_WRITE_OUTPUT owo = | |
| 621 | 675 | + (PFSCTL_OFFLOAD_WRITE_OUTPUT)FsCtl->pOutputBuffer; | |
| 622 | 676 | + | |
| 623 | 677 | + offloadcontext_entry *unvalidated_src_oce; | |
| 624 | 678 | + | |
| 625 | 679 | + /* | |
| 626 | 680 | + * Peel |offloadcontext_entry| pointer from token... | |
| 627 | 681 | + */ | |
| 628 | 682 | + unvalidated_src_oce = | |
| 629 | 683 | + *((void **)(&(((STORAGE_OFFLOAD_TOKEN *)(&owi->Token[0]))->Token[0]))); | |
| 630 | 684 | + DbgP("nfs41_OffloadWrite: " | |
| 631 | 685 | + "unvalidated_src_oce=0x%p\n", unvalidated_src_oce); | |
| 632 | 686 | + | |
| 633 | 687 | + /* | |
| 634 | 688 | + * ... and validate it (and take a shared lock if validation was | |
| 635 | 689 | + * successful, so nobody can delete the context while we use it)! | |
| 636 | 690 | + */ | |
| 637 | 691 | + src_oce = nfs41_find_offloadcontext_acquireshared(unvalidated_src_oce); | |
| 638 | 692 | + if (src_oce == NULL) { | |
| 639 | 693 | + DbgP("nfs41_OffloadWrite: " | |
| 640 | 694 | + "nfs41_find_offloadcontext_acquireshared() failed\n"); | |
| 641 | 695 | + status = STATUS_INVALID_PARAMETER; | |
| 642 | 696 | + goto out; | |
| 643 | 697 | + } | |
| 644 | 698 | + | |
| 645 | 699 | + DbgP("nfs41_OffloadWrite: src_oce=0x%p\n", src_oce); | |
| 646 | 700 | + | |
| 647 | 701 | + /* Check safeguard... */ | |
| 648 | 702 | + if ((src_oce->token.TokenType[0] != 'N') || | |
| 649 | 703 | + (src_oce->token.TokenType[1] != 'F') || | |
| 650 | 704 | + (src_oce->token.TokenType[2] != 'S') || | |
| 651 | 705 | + (src_oce->token.TokenType[3] != '4')) { | |
| 652 | 706 | + DbgP("nfs41_OffloadWrite: " | |
| 653 | 707 | + "token in src_oce=0x%p not a 'NFS4' token\n", | |
| 654 | 708 | + src_oce); | |
| 655 | 709 | + status = STATUS_INVALID_PARAMETER; | |
| 656 | 710 | + goto out; | |
| 657 | 711 | + } | |
| 658 | 712 | + | |
| 659 | 713 | + /* | |
| 660 | 714 | + * FIXME: We should validate the length passed as | |
| 661 | 715 | + * |FSCTL_OFFLOAD_READ_INPUT.CopyLength| here better, because it is | |
| 662 | 716 | + * also used as some kind of access control to different parts of a | |
| 663 | 717 | + * file | |
| 664 | 718 | + */ | |
| 665 | 719 | + dd.srcfileoffset = src_oce->src_fileoffset + owi->TransferOffset; | |
| 666 | 720 | + dd.destfileoffset = owi->FileOffset; | |
| 667 | 721 | + dd.bytecount = owi->CopyLength; | |
| 668 | 722 | + | |
| 669 | 723 | + DbgP("nfs41_OffloadWrite: " | |
| 670 | 724 | + "dd=(srcfileoffset=%lld," | |
| 671 | 725 | + "destfileoffset=%lld," | |
| 672 | 726 | + "bytecount=%lld)\n", | |
| 673 | 727 | + (long long)dd.srcfileoffset, | |
| 674 | 728 | + (long long)dd.destfileoffset, | |
| 675 | 729 | + (long long)dd.bytecount); | |
| 676 | 730 | + | |
| 677 | 731 | + if (dd.bytecount == 0LL) { | |
| 678 | 732 | + status = STATUS_SUCCESS; | |
| 679 | 733 | + goto out; | |
| 680 | 734 | + } | |
| 681 | 735 | + | |
| 682 | 736 | + PNFS41_FOBX nfs41_src_fobx = src_oce->src_fobx; | |
| 683 | 737 | + if (!nfs41_src_fobx) { | |
| 684 | 738 | + DbgP("nfs41_OffloadWrite: No nfs41_src_fobx\n"); | |
| 685 | 739 | + status = STATUS_INVALID_PARAMETER; | |
| 686 | 740 | + goto out; | |
| 687 | 741 | + } | |
| 688 | 742 | + | |
| 689 | 743 | + /* | |
| 690 | 744 | + * Disable caching because NFSv4.2 COPY is basically a | |
| 691 | 745 | + * "write" operation. AFAIK we should flush the cache and wait | |
| 692 | 746 | + * for the kernel lazy writer (which |RxChangeBufferingState()| | |
| 693 | 747 | + * AFAIK does) before doing the COPY, to avoid that we | |
| 694 | 748 | + * have outstanding writes in the kernel cache at the same | |
| 695 | 749 | + * location where the COPY should do it's work | |
| 696 | 750 | + */ | |
| 697 | 751 | + ULONG flag = DISABLE_CACHING; | |
| 698 | 752 | + DbgP("nfs41_OffloadWrite: disableing caching for file '%wZ'\n", | |
| 699 | 753 | + SrvOpen->pAlreadyPrefixedName); | |
| 700 | 754 | + RxChangeBufferingState((PSRV_OPEN)SrvOpen, ULongToPtr(flag), 1); | |
| 701 | 755 | + | |
| 702 | 756 | + status = nfs41_UpcallCreate(NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY, | |
| 703 | 757 | + &nfs41_fobx->sec_ctx, | |
| 704 | 758 | + pVNetRootContext->session, | |
| 705 | 759 | + nfs41_fobx->nfs41_open_state, | |
| 706 | 760 | + pNetRootContext->nfs41d_version, | |
| 707 | 761 | + SrvOpen->pAlreadyPrefixedName, | |
| 708 | 762 | + &entry); | |
| 709 | 763 | + | |
| 710 | 764 | + if (status) | |
| 711 | 765 | + goto out; | |
| 712 | 766 | + | |
| 713 | 767 | + entry->u.DuplicateData.src_state = nfs41_src_fobx->nfs41_open_state; | |
| 714 | 768 | + entry->u.DuplicateData.srcfileoffset = dd.srcfileoffset; | |
| 715 | 769 | + entry->u.DuplicateData.destfileoffset = dd.destfileoffset; | |
| 716 | 770 | + entry->u.DuplicateData.bytecount = dd.bytecount; | |
| 717 | 771 | + | |
| 718 | 772 | + /* Add extra timeout depending on file size */ | |
| 719 | 773 | + io_delay = pVNetRootContext->timeout + | |
| 720 | 774 | + EXTRA_TIMEOUT_PER_BYTE(entry->u.DuplicateData.bytecount); | |
| 721 | 775 | + | |
| 722 | 776 | + status = nfs41_UpcallWaitForReply(entry, io_delay); | |
| 723 | 777 | + if (status) { | |
| 724 | 778 | + /* Timeout - |nfs41_downcall()| will free |entry|+contents */ | |
| 725 | 779 | + entry = NULL; | |
| 726 | 780 | + goto out; | |
| 727 | 781 | + } | |
| 728 | 782 | + | |
| 729 | 783 | + if (!entry->status) { | |
| 730 | 784 | + DbgP("nfs41_OffloadWrite: SUCCESS\n"); | |
| 731 | 785 | + | |
| 732 | 786 | + owo->Size = sizeof(FSCTL_OFFLOAD_READ_OUTPUT); | |
| 733 | 787 | + owo->Flags = 0; | |
| 734 | 788 | + owo->LengthWritten = dd.bytecount; | |
| 735 | 789 | + | |
| 736 | 790 | + RxContext->CurrentIrp->IoStatus.Status = status = STATUS_SUCCESS; | |
| 737 | 791 | + RxContext->InformationToReturn = sizeof(FSCTL_OFFLOAD_READ_OUTPUT); | |
| 738 | 792 | + } | |
| 739 | 793 | + else { | |
| 740 | 794 | + DbgP("nfs41_OffloadWrite: " | |
| 741 | 795 | + "FAILURE, entry->status=0x%lx\n", entry->status); | |
| 742 | 796 | + status = map_setfile_error(entry->status); | |
| 743 | 797 | + RxContext->CurrentIrp->IoStatus.Status = status; | |
| 744 | 798 | + RxContext->IoStatusBlock.Information = 0; | |
| 745 | 799 | + } | |
| 746 | 800 | + | |
| 747 | 801 | +out: | |
| 748 | 802 | + if (src_oce) { | |
| 749 | 803 | + /* Release resource we obtained in shared mode */ | |
| 750 | 804 | + ExReleaseResourceLite(&src_oce->resource); | |
| 751 | 805 | + } | |
| 752 | 806 | + | |
| 753 | 807 | + if (entry) { | |
| 754 | 808 | + nfs41_UpcallDestroy(entry); | |
| 755 | 809 | + } | |
| 756 | 810 | + | |
| 757 | 811 | + DbgEx(); | |
| 758 | 812 | + return status; | |
| 759 | 813 | +} | |
| 760 | 814 | + | |
| 761 | 815 | NTSTATUS nfs41_FsCtl( | |
| 762 | 816 | IN OUT PRX_CONTEXT RxContext) | |
| 763 | 817 | { | |
| 764 | 818 | @@ -895,6 +1323,12 @@ NTSTATUS nfs41_FsCtl( | |
| 765 | 819 | case FSCTL_DUPLICATE_EXTENTS_TO_FILE: | |
| 766 | 820 | status = nfs41_DuplicateData(RxContext); | |
| 767 | 821 | break; | |
| 768 | 822 | + case FSCTL_OFFLOAD_READ: | |
| 769 | 823 | + status = nfs41_OffloadRead(RxContext); | |
| 770 | 824 | + break; | |
| 771 | 825 | + case FSCTL_OFFLOAD_WRITE: | |
| 772 | 826 | + status = nfs41_OffloadWrite(RxContext); | |
| 773 | 827 | + break; | |
| 774 | 828 | default: | |
| 775 | 829 | break; | |
| 776 | 830 | } | |
| 777 | 831 | diff --git a/sys/nfs41sys_updowncall.c b/sys/nfs41sys_updowncall.c | |
| 778 | 832 | index cf5be35..fc523a4 100644 | |
| 779 | 833 | --- a/sys/nfs41sys_updowncall.c | |
| 780 | 834 | +++ b/sys/nfs41sys_updowncall.c | |
| 781 | 835 | @@ -321,6 +321,7 @@ NTSTATUS handle_upcall( | |
| 782 | 836 | pbOut, cbOut, len); | |
| 783 | 837 | break; | |
| 784 | 838 | case NFS41_SYSOP_FSCTL_DUPLICATE_DATA: | |
| 785 | 839 | + case NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY: | |
| 786 | 840 | status = marshal_nfs41_duplicatedata(entry, | |
| 787 | 841 | pbOut, cbOut, len); | |
| 788 | 842 | break; | |
| 789 | 843 | @@ -785,6 +786,7 @@ NTSTATUS nfs41_downcall( | |
| 790 | 844 | unmarshal_nfs41_setzerodata(cur, &buf); | |
| 791 | 845 | break; | |
| 792 | 846 | case NFS41_SYSOP_FSCTL_DUPLICATE_DATA: | |
| 793 | 847 | + case NFS41_SYSOP_FSCTL_OFFLOAD_DATACOPY: | |
| 794 | 848 | unmarshal_nfs41_duplicatedata(cur, &buf); | |
| 795 | 849 | break; | |
| 796 | 850 | } | |
| 797 | 851 |
Submit a correction or amendment below (click here to make a fresh posting)
After submitting an amendment, you'll be able to view the differences between the old and new posts easily.
nrubsig.kpaste.net RSS