diff options
author | Eric Sandeen <sandeen@sandeen.net> | 2008-05-20 01:11:17 -0400 |
---|---|---|
committer | Niv Sardi <xaiki@debian.org> | 2008-07-28 02:58:35 -0400 |
commit | 68f34d5107dbace3d14a1c2f060fc8941894879c (patch) | |
tree | 79567f2b41853e805bf391e072d22085f3476f44 | |
parent | 120226c11a6277d3e761393f0995c55218fabebb (diff) |
[XFS]
de-duplicate calls to xfs_attr_trace_enter
Every call to xfs_attr_trace_enter() shares the exact same 16 args in the
middle... just send in the context pointer and let the next level down
split it into the ktrace.
Compile tested only.
SGI-PV: 976035
SGI-Modid: xfs-linux-melb:xfs-kern:31200a
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
Signed-off-by: Niv Sardi <xaiki@sgi.com>
Signed-off-by: Josef 'Jeff' Sipek <jeffpc@josefsipek.net>
Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
-rw-r--r-- | fs/xfs/xfs_attr.c | 106 | ||||
-rw-r--r-- | fs/xfs/xfs_attr_sf.h | 10 |
2 files changed, 28 insertions, 88 deletions
diff --git a/fs/xfs/xfs_attr.c b/fs/xfs/xfs_attr.c index 86d8619f279c..5e5dbe62b194 100644 --- a/fs/xfs/xfs_attr.c +++ b/fs/xfs/xfs_attr.c | |||
@@ -2300,23 +2300,7 @@ xfs_attr_rmtval_remove(xfs_da_args_t *args) | |||
2300 | void | 2300 | void |
2301 | xfs_attr_trace_l_c(char *where, struct xfs_attr_list_context *context) | 2301 | xfs_attr_trace_l_c(char *where, struct xfs_attr_list_context *context) |
2302 | { | 2302 | { |
2303 | xfs_attr_trace_enter(XFS_ATTR_KTRACE_L_C, where, | 2303 | xfs_attr_trace_enter(XFS_ATTR_KTRACE_L_C, where, context, |
2304 | (__psunsigned_t)context->dp, | ||
2305 | (__psunsigned_t)context->cursor->hashval, | ||
2306 | (__psunsigned_t)context->cursor->blkno, | ||
2307 | (__psunsigned_t)context->cursor->offset, | ||
2308 | (__psunsigned_t)context->alist, | ||
2309 | (__psunsigned_t)context->bufsize, | ||
2310 | (__psunsigned_t)context->count, | ||
2311 | (__psunsigned_t)context->firstu, | ||
2312 | (__psunsigned_t) | ||
2313 | ((context->count > 0) && | ||
2314 | !(context->flags & (ATTR_KERNAMELS|ATTR_KERNOVAL))) | ||
2315 | ? (ATTR_ENTRY(context->alist, | ||
2316 | context->count-1)->a_valuelen) | ||
2317 | : 0, | ||
2318 | (__psunsigned_t)context->dupcnt, | ||
2319 | (__psunsigned_t)context->flags, | ||
2320 | (__psunsigned_t)NULL, | 2304 | (__psunsigned_t)NULL, |
2321 | (__psunsigned_t)NULL, | 2305 | (__psunsigned_t)NULL, |
2322 | (__psunsigned_t)NULL); | 2306 | (__psunsigned_t)NULL); |
@@ -2329,23 +2313,7 @@ void | |||
2329 | xfs_attr_trace_l_cn(char *where, struct xfs_attr_list_context *context, | 2313 | xfs_attr_trace_l_cn(char *where, struct xfs_attr_list_context *context, |
2330 | struct xfs_da_intnode *node) | 2314 | struct xfs_da_intnode *node) |
2331 | { | 2315 | { |
2332 | xfs_attr_trace_enter(XFS_ATTR_KTRACE_L_CN, where, | 2316 | xfs_attr_trace_enter(XFS_ATTR_KTRACE_L_CN, where, context, |
2333 | (__psunsigned_t)context->dp, | ||
2334 | (__psunsigned_t)context->cursor->hashval, | ||
2335 | (__psunsigned_t)context->cursor->blkno, | ||
2336 | (__psunsigned_t)context->cursor->offset, | ||
2337 | (__psunsigned_t)context->alist, | ||
2338 | (__psunsigned_t)context->bufsize, | ||
2339 | (__psunsigned_t)context->count, | ||
2340 | (__psunsigned_t)context->firstu, | ||
2341 | (__psunsigned_t) | ||
2342 | ((context->count > 0) && | ||
2343 | !(context->flags & (ATTR_KERNAMELS|ATTR_KERNOVAL))) | ||
2344 | ? (ATTR_ENTRY(context->alist, | ||
2345 | context->count-1)->a_valuelen) | ||
2346 | : 0, | ||
2347 | (__psunsigned_t)context->dupcnt, | ||
2348 | (__psunsigned_t)context->flags, | ||
2349 | (__psunsigned_t)be16_to_cpu(node->hdr.count), | 2317 | (__psunsigned_t)be16_to_cpu(node->hdr.count), |
2350 | (__psunsigned_t)be32_to_cpu(node->btree[0].hashval), | 2318 | (__psunsigned_t)be32_to_cpu(node->btree[0].hashval), |
2351 | (__psunsigned_t)be32_to_cpu(node->btree[ | 2319 | (__psunsigned_t)be32_to_cpu(node->btree[ |
@@ -2359,23 +2327,7 @@ void | |||
2359 | xfs_attr_trace_l_cb(char *where, struct xfs_attr_list_context *context, | 2327 | xfs_attr_trace_l_cb(char *where, struct xfs_attr_list_context *context, |
2360 | struct xfs_da_node_entry *btree) | 2328 | struct xfs_da_node_entry *btree) |
2361 | { | 2329 | { |
2362 | xfs_attr_trace_enter(XFS_ATTR_KTRACE_L_CB, where, | 2330 | xfs_attr_trace_enter(XFS_ATTR_KTRACE_L_CB, where, context, |
2363 | (__psunsigned_t)context->dp, | ||
2364 | (__psunsigned_t)context->cursor->hashval, | ||
2365 | (__psunsigned_t)context->cursor->blkno, | ||
2366 | (__psunsigned_t)context->cursor->offset, | ||
2367 | (__psunsigned_t)context->alist, | ||
2368 | (__psunsigned_t)context->bufsize, | ||
2369 | (__psunsigned_t)context->count, | ||
2370 | (__psunsigned_t)context->firstu, | ||
2371 | (__psunsigned_t) | ||
2372 | ((context->count > 0) && | ||
2373 | !(context->flags & (ATTR_KERNAMELS|ATTR_KERNOVAL))) | ||
2374 | ? (ATTR_ENTRY(context->alist, | ||
2375 | context->count-1)->a_valuelen) | ||
2376 | : 0, | ||
2377 | (__psunsigned_t)context->dupcnt, | ||
2378 | (__psunsigned_t)context->flags, | ||
2379 | (__psunsigned_t)be32_to_cpu(btree->hashval), | 2331 | (__psunsigned_t)be32_to_cpu(btree->hashval), |
2380 | (__psunsigned_t)be32_to_cpu(btree->before), | 2332 | (__psunsigned_t)be32_to_cpu(btree->before), |
2381 | (__psunsigned_t)NULL); | 2333 | (__psunsigned_t)NULL); |
@@ -2388,23 +2340,7 @@ void | |||
2388 | xfs_attr_trace_l_cl(char *where, struct xfs_attr_list_context *context, | 2340 | xfs_attr_trace_l_cl(char *where, struct xfs_attr_list_context *context, |
2389 | struct xfs_attr_leafblock *leaf) | 2341 | struct xfs_attr_leafblock *leaf) |
2390 | { | 2342 | { |
2391 | xfs_attr_trace_enter(XFS_ATTR_KTRACE_L_CL, where, | 2343 | xfs_attr_trace_enter(XFS_ATTR_KTRACE_L_CL, where, context, |
2392 | (__psunsigned_t)context->dp, | ||
2393 | (__psunsigned_t)context->cursor->hashval, | ||
2394 | (__psunsigned_t)context->cursor->blkno, | ||
2395 | (__psunsigned_t)context->cursor->offset, | ||
2396 | (__psunsigned_t)context->alist, | ||
2397 | (__psunsigned_t)context->bufsize, | ||
2398 | (__psunsigned_t)context->count, | ||
2399 | (__psunsigned_t)context->firstu, | ||
2400 | (__psunsigned_t) | ||
2401 | ((context->count > 0) && | ||
2402 | !(context->flags & (ATTR_KERNAMELS|ATTR_KERNOVAL))) | ||
2403 | ? (ATTR_ENTRY(context->alist, | ||
2404 | context->count-1)->a_valuelen) | ||
2405 | : 0, | ||
2406 | (__psunsigned_t)context->dupcnt, | ||
2407 | (__psunsigned_t)context->flags, | ||
2408 | (__psunsigned_t)be16_to_cpu(leaf->hdr.count), | 2344 | (__psunsigned_t)be16_to_cpu(leaf->hdr.count), |
2409 | (__psunsigned_t)be32_to_cpu(leaf->entries[0].hashval), | 2345 | (__psunsigned_t)be32_to_cpu(leaf->entries[0].hashval), |
2410 | (__psunsigned_t)be32_to_cpu(leaf->entries[ | 2346 | (__psunsigned_t)be32_to_cpu(leaf->entries[ |
@@ -2417,22 +2353,30 @@ xfs_attr_trace_l_cl(char *where, struct xfs_attr_list_context *context, | |||
2417 | */ | 2353 | */ |
2418 | void | 2354 | void |
2419 | xfs_attr_trace_enter(int type, char *where, | 2355 | xfs_attr_trace_enter(int type, char *where, |
2420 | __psunsigned_t a2, __psunsigned_t a3, | 2356 | struct xfs_attr_list_context *context, |
2421 | __psunsigned_t a4, __psunsigned_t a5, | 2357 | __psunsigned_t a13, __psunsigned_t a14, |
2422 | __psunsigned_t a6, __psunsigned_t a7, | 2358 | __psunsigned_t a15) |
2423 | __psunsigned_t a8, __psunsigned_t a9, | ||
2424 | __psunsigned_t a10, __psunsigned_t a11, | ||
2425 | __psunsigned_t a12, __psunsigned_t a13, | ||
2426 | __psunsigned_t a14, __psunsigned_t a15) | ||
2427 | { | 2359 | { |
2428 | ASSERT(xfs_attr_trace_buf); | 2360 | ASSERT(xfs_attr_trace_buf); |
2429 | ktrace_enter(xfs_attr_trace_buf, (void *)((__psunsigned_t)type), | 2361 | ktrace_enter(xfs_attr_trace_buf, (void *)((__psunsigned_t)type), |
2430 | (void *)where, | 2362 | (void *)((__psunsigned_t)where), |
2431 | (void *)a2, (void *)a3, (void *)a4, | 2363 | (void *)((__psunsigned_t)context->dp), |
2432 | (void *)a5, (void *)a6, (void *)a7, | 2364 | (void *)((__psunsigned_t)context->cursor->hashval), |
2433 | (void *)a8, (void *)a9, (void *)a10, | 2365 | (void *)((__psunsigned_t)context->cursor->blkno), |
2434 | (void *)a11, (void *)a12, (void *)a13, | 2366 | (void *)((__psunsigned_t)context->cursor->offset), |
2435 | (void *)a14, (void *)a15); | 2367 | (void *)((__psunsigned_t)context->alist), |
2368 | (void *)((__psunsigned_t)context->bufsize), | ||
2369 | (void *)((__psunsigned_t)context->count), | ||
2370 | (void *)((__psunsigned_t)context->firstu), | ||
2371 | (void *)((__psunsigned_t) | ||
2372 | (((context->count > 0) && | ||
2373 | !(context->flags & (ATTR_KERNAMELS|ATTR_KERNOVAL))) | ||
2374 | ? (ATTR_ENTRY(context->alist, | ||
2375 | context->count-1)->a_valuelen) | ||
2376 | : 0)), | ||
2377 | (void *)((__psunsigned_t)context->dupcnt), | ||
2378 | (void *)((__psunsigned_t)context->flags), | ||
2379 | (void *)a13, (void *)a14, (void *)a15); | ||
2436 | } | 2380 | } |
2437 | #endif /* XFS_ATTR_TRACE */ | 2381 | #endif /* XFS_ATTR_TRACE */ |
2438 | 2382 | ||
diff --git a/fs/xfs/xfs_attr_sf.h b/fs/xfs/xfs_attr_sf.h index f67f917803b1..ea22839caed2 100644 --- a/fs/xfs/xfs_attr_sf.h +++ b/fs/xfs/xfs_attr_sf.h | |||
@@ -97,13 +97,9 @@ void xfs_attr_trace_l_cb(char *where, struct xfs_attr_list_context *context, | |||
97 | void xfs_attr_trace_l_cl(char *where, struct xfs_attr_list_context *context, | 97 | void xfs_attr_trace_l_cl(char *where, struct xfs_attr_list_context *context, |
98 | struct xfs_attr_leafblock *leaf); | 98 | struct xfs_attr_leafblock *leaf); |
99 | void xfs_attr_trace_enter(int type, char *where, | 99 | void xfs_attr_trace_enter(int type, char *where, |
100 | __psunsigned_t a2, __psunsigned_t a3, | 100 | struct xfs_attr_list_context *context, |
101 | __psunsigned_t a4, __psunsigned_t a5, | 101 | __psunsigned_t a13, __psunsigned_t a14, |
102 | __psunsigned_t a6, __psunsigned_t a7, | 102 | __psunsigned_t a15); |
103 | __psunsigned_t a8, __psunsigned_t a9, | ||
104 | __psunsigned_t a10, __psunsigned_t a11, | ||
105 | __psunsigned_t a12, __psunsigned_t a13, | ||
106 | __psunsigned_t a14, __psunsigned_t a15); | ||
107 | #else | 103 | #else |
108 | #define xfs_attr_trace_l_c(w,c) | 104 | #define xfs_attr_trace_l_c(w,c) |
109 | #define xfs_attr_trace_l_cn(w,c,n) | 105 | #define xfs_attr_trace_l_cn(w,c,n) |