diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2008-05-19 02:31:57 -0400 |
---|---|---|
committer | Niv Sardi <xaiki@debian.org> | 2008-07-28 02:58:07 -0400 |
commit | f0e2d93c29dc39ffd24cac180a19d48f700c0706 (patch) | |
tree | 5bfac66c6a5cda98373eea222834a37877a590f9 /fs | |
parent | 7c12f296500e1157872ef45b3f3bb06b4b73f1c1 (diff) |
[XFS] Remove unused arg from kmem_free()
kmem_free() function takes (ptr, size) arguments but doesn't actually use
second one.
This patch removes size argument from all callsites.
SGI-PV: 981498
SGI-Modid: xfs-linux-melb:xfs-kern:31050a
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Signed-off-by: David Chinner <dgc@sgi.com>
Signed-off-by: Lachlan McIlroy <lachlan@sgi.com>
Diffstat (limited to 'fs')
31 files changed, 122 insertions, 138 deletions
diff --git a/fs/xfs/linux-2.6/kmem.c b/fs/xfs/linux-2.6/kmem.c index 9b1bb17a0501..69233a52f0a6 100644 --- a/fs/xfs/linux-2.6/kmem.c +++ b/fs/xfs/linux-2.6/kmem.c | |||
@@ -90,7 +90,7 @@ kmem_zalloc_greedy(size_t *size, size_t minsize, size_t maxsize, | |||
90 | } | 90 | } |
91 | 91 | ||
92 | void | 92 | void |
93 | kmem_free(void *ptr, size_t size) | 93 | kmem_free(void *ptr) |
94 | { | 94 | { |
95 | if (!is_vmalloc_addr(ptr)) { | 95 | if (!is_vmalloc_addr(ptr)) { |
96 | kfree(ptr); | 96 | kfree(ptr); |
@@ -110,7 +110,7 @@ kmem_realloc(void *ptr, size_t newsize, size_t oldsize, | |||
110 | if (new) | 110 | if (new) |
111 | memcpy(new, ptr, | 111 | memcpy(new, ptr, |
112 | ((oldsize < newsize) ? oldsize : newsize)); | 112 | ((oldsize < newsize) ? oldsize : newsize)); |
113 | kmem_free(ptr, oldsize); | 113 | kmem_free(ptr); |
114 | } | 114 | } |
115 | return new; | 115 | return new; |
116 | } | 116 | } |
diff --git a/fs/xfs/linux-2.6/kmem.h b/fs/xfs/linux-2.6/kmem.h index a20683cf74dd..a3c96207e60e 100644 --- a/fs/xfs/linux-2.6/kmem.h +++ b/fs/xfs/linux-2.6/kmem.h | |||
@@ -58,7 +58,7 @@ extern void *kmem_alloc(size_t, unsigned int __nocast); | |||
58 | extern void *kmem_zalloc(size_t, unsigned int __nocast); | 58 | extern void *kmem_zalloc(size_t, unsigned int __nocast); |
59 | extern void *kmem_zalloc_greedy(size_t *, size_t, size_t, unsigned int __nocast); | 59 | extern void *kmem_zalloc_greedy(size_t *, size_t, size_t, unsigned int __nocast); |
60 | extern void *kmem_realloc(void *, size_t, size_t, unsigned int __nocast); | 60 | extern void *kmem_realloc(void *, size_t, size_t, unsigned int __nocast); |
61 | extern void kmem_free(void *, size_t); | 61 | extern void kmem_free(void *); |
62 | 62 | ||
63 | /* | 63 | /* |
64 | * Zone interfaces | 64 | * Zone interfaces |
diff --git a/fs/xfs/linux-2.6/xfs_buf.c b/fs/xfs/linux-2.6/xfs_buf.c index 98e0e86093b4..ed03c6d3c9c1 100644 --- a/fs/xfs/linux-2.6/xfs_buf.c +++ b/fs/xfs/linux-2.6/xfs_buf.c | |||
@@ -310,8 +310,7 @@ _xfs_buf_free_pages( | |||
310 | xfs_buf_t *bp) | 310 | xfs_buf_t *bp) |
311 | { | 311 | { |
312 | if (bp->b_pages != bp->b_page_array) { | 312 | if (bp->b_pages != bp->b_page_array) { |
313 | kmem_free(bp->b_pages, | 313 | kmem_free(bp->b_pages); |
314 | bp->b_page_count * sizeof(struct page *)); | ||
315 | } | 314 | } |
316 | } | 315 | } |
317 | 316 | ||
@@ -1398,7 +1397,7 @@ STATIC void | |||
1398 | xfs_free_bufhash( | 1397 | xfs_free_bufhash( |
1399 | xfs_buftarg_t *btp) | 1398 | xfs_buftarg_t *btp) |
1400 | { | 1399 | { |
1401 | kmem_free(btp->bt_hash, (1<<btp->bt_hashshift) * sizeof(xfs_bufhash_t)); | 1400 | kmem_free(btp->bt_hash); |
1402 | btp->bt_hash = NULL; | 1401 | btp->bt_hash = NULL; |
1403 | } | 1402 | } |
1404 | 1403 | ||
@@ -1444,7 +1443,7 @@ xfs_free_buftarg( | |||
1444 | xfs_unregister_buftarg(btp); | 1443 | xfs_unregister_buftarg(btp); |
1445 | kthread_stop(btp->bt_task); | 1444 | kthread_stop(btp->bt_task); |
1446 | 1445 | ||
1447 | kmem_free(btp, sizeof(*btp)); | 1446 | kmem_free(btp); |
1448 | } | 1447 | } |
1449 | 1448 | ||
1450 | STATIC int | 1449 | STATIC int |
@@ -1575,7 +1574,7 @@ xfs_alloc_buftarg( | |||
1575 | return btp; | 1574 | return btp; |
1576 | 1575 | ||
1577 | error: | 1576 | error: |
1578 | kmem_free(btp, sizeof(*btp)); | 1577 | kmem_free(btp); |
1579 | return NULL; | 1578 | return NULL; |
1580 | } | 1579 | } |
1581 | 1580 | ||
diff --git a/fs/xfs/linux-2.6/xfs_super.c b/fs/xfs/linux-2.6/xfs_super.c index 1b60e46f527f..5c7144bc3106 100644 --- a/fs/xfs/linux-2.6/xfs_super.c +++ b/fs/xfs/linux-2.6/xfs_super.c | |||
@@ -1074,7 +1074,7 @@ xfssyncd( | |||
1074 | list_del(&work->w_list); | 1074 | list_del(&work->w_list); |
1075 | if (work == &mp->m_sync_work) | 1075 | if (work == &mp->m_sync_work) |
1076 | continue; | 1076 | continue; |
1077 | kmem_free(work, sizeof(struct bhv_vfs_sync_work)); | 1077 | kmem_free(work); |
1078 | } | 1078 | } |
1079 | } | 1079 | } |
1080 | 1080 | ||
@@ -1222,7 +1222,7 @@ xfs_fs_remount( | |||
1222 | error = xfs_parseargs(mp, options, args, 1); | 1222 | error = xfs_parseargs(mp, options, args, 1); |
1223 | if (!error) | 1223 | if (!error) |
1224 | error = xfs_mntupdate(mp, flags, args); | 1224 | error = xfs_mntupdate(mp, flags, args); |
1225 | kmem_free(args, sizeof(*args)); | 1225 | kmem_free(args); |
1226 | return -error; | 1226 | return -error; |
1227 | } | 1227 | } |
1228 | 1228 | ||
@@ -1369,7 +1369,7 @@ xfs_fs_fill_super( | |||
1369 | 1369 | ||
1370 | xfs_itrace_exit(XFS_I(sb->s_root->d_inode)); | 1370 | xfs_itrace_exit(XFS_I(sb->s_root->d_inode)); |
1371 | 1371 | ||
1372 | kmem_free(args, sizeof(*args)); | 1372 | kmem_free(args); |
1373 | return 0; | 1373 | return 0; |
1374 | 1374 | ||
1375 | fail_vnrele: | 1375 | fail_vnrele: |
@@ -1384,7 +1384,7 @@ fail_unmount: | |||
1384 | xfs_unmount(mp, 0, NULL); | 1384 | xfs_unmount(mp, 0, NULL); |
1385 | 1385 | ||
1386 | fail_vfsop: | 1386 | fail_vfsop: |
1387 | kmem_free(args, sizeof(*args)); | 1387 | kmem_free(args); |
1388 | return -error; | 1388 | return -error; |
1389 | } | 1389 | } |
1390 | 1390 | ||
diff --git a/fs/xfs/quota/xfs_dquot_item.c b/fs/xfs/quota/xfs_dquot_item.c index 36e05ca78412..08d2fc89e6a1 100644 --- a/fs/xfs/quota/xfs_dquot_item.c +++ b/fs/xfs/quota/xfs_dquot_item.c | |||
@@ -576,8 +576,8 @@ xfs_qm_qoffend_logitem_committed( | |||
576 | * xfs_trans_delete_ail() drops the AIL lock. | 576 | * xfs_trans_delete_ail() drops the AIL lock. |
577 | */ | 577 | */ |
578 | xfs_trans_delete_ail(qfs->qql_item.li_mountp, (xfs_log_item_t *)qfs); | 578 | xfs_trans_delete_ail(qfs->qql_item.li_mountp, (xfs_log_item_t *)qfs); |
579 | kmem_free(qfs, sizeof(xfs_qoff_logitem_t)); | 579 | kmem_free(qfs); |
580 | kmem_free(qfe, sizeof(xfs_qoff_logitem_t)); | 580 | kmem_free(qfe); |
581 | return (xfs_lsn_t)-1; | 581 | return (xfs_lsn_t)-1; |
582 | } | 582 | } |
583 | 583 | ||
diff --git a/fs/xfs/quota/xfs_qm.c b/fs/xfs/quota/xfs_qm.c index d31cce1165c5..cde5c508f0e0 100644 --- a/fs/xfs/quota/xfs_qm.c +++ b/fs/xfs/quota/xfs_qm.c | |||
@@ -192,8 +192,8 @@ xfs_qm_destroy( | |||
192 | xfs_qm_list_destroy(&(xqm->qm_usr_dqhtable[i])); | 192 | xfs_qm_list_destroy(&(xqm->qm_usr_dqhtable[i])); |
193 | xfs_qm_list_destroy(&(xqm->qm_grp_dqhtable[i])); | 193 | xfs_qm_list_destroy(&(xqm->qm_grp_dqhtable[i])); |
194 | } | 194 | } |
195 | kmem_free(xqm->qm_usr_dqhtable, hsize * sizeof(xfs_dqhash_t)); | 195 | kmem_free(xqm->qm_usr_dqhtable); |
196 | kmem_free(xqm->qm_grp_dqhtable, hsize * sizeof(xfs_dqhash_t)); | 196 | kmem_free(xqm->qm_grp_dqhtable); |
197 | xqm->qm_usr_dqhtable = NULL; | 197 | xqm->qm_usr_dqhtable = NULL; |
198 | xqm->qm_grp_dqhtable = NULL; | 198 | xqm->qm_grp_dqhtable = NULL; |
199 | xqm->qm_dqhashmask = 0; | 199 | xqm->qm_dqhashmask = 0; |
@@ -201,7 +201,7 @@ xfs_qm_destroy( | |||
201 | #ifdef DEBUG | 201 | #ifdef DEBUG |
202 | mutex_destroy(&qcheck_lock); | 202 | mutex_destroy(&qcheck_lock); |
203 | #endif | 203 | #endif |
204 | kmem_free(xqm, sizeof(xfs_qm_t)); | 204 | kmem_free(xqm); |
205 | } | 205 | } |
206 | 206 | ||
207 | /* | 207 | /* |
@@ -1134,7 +1134,7 @@ xfs_qm_init_quotainfo( | |||
1134 | * and change the superblock accordingly. | 1134 | * and change the superblock accordingly. |
1135 | */ | 1135 | */ |
1136 | if ((error = xfs_qm_init_quotainos(mp))) { | 1136 | if ((error = xfs_qm_init_quotainos(mp))) { |
1137 | kmem_free(qinf, sizeof(xfs_quotainfo_t)); | 1137 | kmem_free(qinf); |
1138 | mp->m_quotainfo = NULL; | 1138 | mp->m_quotainfo = NULL; |
1139 | return error; | 1139 | return error; |
1140 | } | 1140 | } |
@@ -1248,7 +1248,7 @@ xfs_qm_destroy_quotainfo( | |||
1248 | qi->qi_gquotaip = NULL; | 1248 | qi->qi_gquotaip = NULL; |
1249 | } | 1249 | } |
1250 | mutex_destroy(&qi->qi_quotaofflock); | 1250 | mutex_destroy(&qi->qi_quotaofflock); |
1251 | kmem_free(qi, sizeof(xfs_quotainfo_t)); | 1251 | kmem_free(qi); |
1252 | mp->m_quotainfo = NULL; | 1252 | mp->m_quotainfo = NULL; |
1253 | } | 1253 | } |
1254 | 1254 | ||
@@ -1623,7 +1623,7 @@ xfs_qm_dqiterate( | |||
1623 | break; | 1623 | break; |
1624 | } while (nmaps > 0); | 1624 | } while (nmaps > 0); |
1625 | 1625 | ||
1626 | kmem_free(map, XFS_DQITER_MAP_SIZE * sizeof(*map)); | 1626 | kmem_free(map); |
1627 | 1627 | ||
1628 | return error; | 1628 | return error; |
1629 | } | 1629 | } |
diff --git a/fs/xfs/quota/xfs_qm_syscalls.c b/fs/xfs/quota/xfs_qm_syscalls.c index 768a3b27d2b6..413671523cb5 100644 --- a/fs/xfs/quota/xfs_qm_syscalls.c +++ b/fs/xfs/quota/xfs_qm_syscalls.c | |||
@@ -1449,14 +1449,14 @@ xfs_qm_internalqcheck( | |||
1449 | for (d = (xfs_dqtest_t *) h1->qh_next; d != NULL; ) { | 1449 | for (d = (xfs_dqtest_t *) h1->qh_next; d != NULL; ) { |
1450 | xfs_dqtest_cmp(d); | 1450 | xfs_dqtest_cmp(d); |
1451 | e = (xfs_dqtest_t *) d->HL_NEXT; | 1451 | e = (xfs_dqtest_t *) d->HL_NEXT; |
1452 | kmem_free(d, sizeof(xfs_dqtest_t)); | 1452 | kmem_free(d); |
1453 | d = e; | 1453 | d = e; |
1454 | } | 1454 | } |
1455 | h1 = &qmtest_gdqtab[i]; | 1455 | h1 = &qmtest_gdqtab[i]; |
1456 | for (d = (xfs_dqtest_t *) h1->qh_next; d != NULL; ) { | 1456 | for (d = (xfs_dqtest_t *) h1->qh_next; d != NULL; ) { |
1457 | xfs_dqtest_cmp(d); | 1457 | xfs_dqtest_cmp(d); |
1458 | e = (xfs_dqtest_t *) d->HL_NEXT; | 1458 | e = (xfs_dqtest_t *) d->HL_NEXT; |
1459 | kmem_free(d, sizeof(xfs_dqtest_t)); | 1459 | kmem_free(d); |
1460 | d = e; | 1460 | d = e; |
1461 | } | 1461 | } |
1462 | } | 1462 | } |
@@ -1467,8 +1467,8 @@ xfs_qm_internalqcheck( | |||
1467 | } else { | 1467 | } else { |
1468 | cmn_err(CE_DEBUG, "******** quotacheck successful! ********"); | 1468 | cmn_err(CE_DEBUG, "******** quotacheck successful! ********"); |
1469 | } | 1469 | } |
1470 | kmem_free(qmtest_udqtab, qmtest_hashmask * sizeof(xfs_dqhash_t)); | 1470 | kmem_free(qmtest_udqtab); |
1471 | kmem_free(qmtest_gdqtab, qmtest_hashmask * sizeof(xfs_dqhash_t)); | 1471 | kmem_free(qmtest_gdqtab); |
1472 | mutex_unlock(&qcheck_lock); | 1472 | mutex_unlock(&qcheck_lock); |
1473 | return (qmtest_nfails); | 1473 | return (qmtest_nfails); |
1474 | } | 1474 | } |
diff --git a/fs/xfs/support/ktrace.c b/fs/xfs/support/ktrace.c index 0b75d302508f..a34ef05489b1 100644 --- a/fs/xfs/support/ktrace.c +++ b/fs/xfs/support/ktrace.c | |||
@@ -89,7 +89,7 @@ ktrace_alloc(int nentries, unsigned int __nocast sleep) | |||
89 | if (sleep & KM_SLEEP) | 89 | if (sleep & KM_SLEEP) |
90 | panic("ktrace_alloc: NULL memory on KM_SLEEP request!"); | 90 | panic("ktrace_alloc: NULL memory on KM_SLEEP request!"); |
91 | 91 | ||
92 | kmem_free(ktp, sizeof(*ktp)); | 92 | kmem_free(ktp); |
93 | 93 | ||
94 | return NULL; | 94 | return NULL; |
95 | } | 95 | } |
@@ -126,7 +126,7 @@ ktrace_free(ktrace_t *ktp) | |||
126 | } else { | 126 | } else { |
127 | entries_size = (int)(ktp->kt_nentries * sizeof(ktrace_entry_t)); | 127 | entries_size = (int)(ktp->kt_nentries * sizeof(ktrace_entry_t)); |
128 | 128 | ||
129 | kmem_free(ktp->kt_entries, entries_size); | 129 | kmem_free(ktp->kt_entries); |
130 | } | 130 | } |
131 | 131 | ||
132 | kmem_zone_free(ktrace_hdr_zone, ktp); | 132 | kmem_zone_free(ktrace_hdr_zone, ktp); |
diff --git a/fs/xfs/xfs_attr_leaf.c b/fs/xfs/xfs_attr_leaf.c index 303d41e4217b..a85e9caf0156 100644 --- a/fs/xfs/xfs_attr_leaf.c +++ b/fs/xfs/xfs_attr_leaf.c | |||
@@ -555,7 +555,7 @@ xfs_attr_shortform_to_leaf(xfs_da_args_t *args) | |||
555 | out: | 555 | out: |
556 | if(bp) | 556 | if(bp) |
557 | xfs_da_buf_done(bp); | 557 | xfs_da_buf_done(bp); |
558 | kmem_free(tmpbuffer, size); | 558 | kmem_free(tmpbuffer); |
559 | return(error); | 559 | return(error); |
560 | } | 560 | } |
561 | 561 | ||
@@ -676,7 +676,7 @@ xfs_attr_shortform_list(xfs_attr_list_context_t *context) | |||
676 | XFS_ERRLEVEL_LOW, | 676 | XFS_ERRLEVEL_LOW, |
677 | context->dp->i_mount, sfe); | 677 | context->dp->i_mount, sfe); |
678 | xfs_attr_trace_l_c("sf corrupted", context); | 678 | xfs_attr_trace_l_c("sf corrupted", context); |
679 | kmem_free(sbuf, sbsize); | 679 | kmem_free(sbuf); |
680 | return XFS_ERROR(EFSCORRUPTED); | 680 | return XFS_ERROR(EFSCORRUPTED); |
681 | } | 681 | } |
682 | if (!xfs_attr_namesp_match_overrides(context->flags, sfe->flags)) { | 682 | if (!xfs_attr_namesp_match_overrides(context->flags, sfe->flags)) { |
@@ -717,7 +717,7 @@ xfs_attr_shortform_list(xfs_attr_list_context_t *context) | |||
717 | } | 717 | } |
718 | } | 718 | } |
719 | if (i == nsbuf) { | 719 | if (i == nsbuf) { |
720 | kmem_free(sbuf, sbsize); | 720 | kmem_free(sbuf); |
721 | xfs_attr_trace_l_c("blk end", context); | 721 | xfs_attr_trace_l_c("blk end", context); |
722 | return(0); | 722 | return(0); |
723 | } | 723 | } |
@@ -747,7 +747,7 @@ xfs_attr_shortform_list(xfs_attr_list_context_t *context) | |||
747 | cursor->offset++; | 747 | cursor->offset++; |
748 | } | 748 | } |
749 | 749 | ||
750 | kmem_free(sbuf, sbsize); | 750 | kmem_free(sbuf); |
751 | xfs_attr_trace_l_c("sf E-O-F", context); | 751 | xfs_attr_trace_l_c("sf E-O-F", context); |
752 | return(0); | 752 | return(0); |
753 | } | 753 | } |
@@ -873,7 +873,7 @@ xfs_attr_leaf_to_shortform(xfs_dabuf_t *bp, xfs_da_args_t *args, int forkoff) | |||
873 | error = 0; | 873 | error = 0; |
874 | 874 | ||
875 | out: | 875 | out: |
876 | kmem_free(tmpbuffer, XFS_LBSIZE(dp->i_mount)); | 876 | kmem_free(tmpbuffer); |
877 | return(error); | 877 | return(error); |
878 | } | 878 | } |
879 | 879 | ||
@@ -1271,7 +1271,7 @@ xfs_attr_leaf_compact(xfs_trans_t *trans, xfs_dabuf_t *bp) | |||
1271 | be16_to_cpu(hdr_s->count), mp); | 1271 | be16_to_cpu(hdr_s->count), mp); |
1272 | xfs_da_log_buf(trans, bp, 0, XFS_LBSIZE(mp) - 1); | 1272 | xfs_da_log_buf(trans, bp, 0, XFS_LBSIZE(mp) - 1); |
1273 | 1273 | ||
1274 | kmem_free(tmpbuffer, XFS_LBSIZE(mp)); | 1274 | kmem_free(tmpbuffer); |
1275 | } | 1275 | } |
1276 | 1276 | ||
1277 | /* | 1277 | /* |
@@ -1921,7 +1921,7 @@ xfs_attr_leaf_unbalance(xfs_da_state_t *state, xfs_da_state_blk_t *drop_blk, | |||
1921 | be16_to_cpu(drop_hdr->count), mp); | 1921 | be16_to_cpu(drop_hdr->count), mp); |
1922 | } | 1922 | } |
1923 | memcpy((char *)save_leaf, (char *)tmp_leaf, state->blocksize); | 1923 | memcpy((char *)save_leaf, (char *)tmp_leaf, state->blocksize); |
1924 | kmem_free(tmpbuffer, state->blocksize); | 1924 | kmem_free(tmpbuffer); |
1925 | } | 1925 | } |
1926 | 1926 | ||
1927 | xfs_da_log_buf(state->args->trans, save_blk->bp, 0, | 1927 | xfs_da_log_buf(state->args->trans, save_blk->bp, 0, |
@@ -2451,7 +2451,7 @@ xfs_attr_leaf_list_int(xfs_dabuf_t *bp, xfs_attr_list_context_t *context) | |||
2451 | (int)name_rmt->namelen, | 2451 | (int)name_rmt->namelen, |
2452 | valuelen, | 2452 | valuelen, |
2453 | (char*)args.value); | 2453 | (char*)args.value); |
2454 | kmem_free(args.value, valuelen); | 2454 | kmem_free(args.value); |
2455 | } | 2455 | } |
2456 | else { | 2456 | else { |
2457 | retval = context->put_listent(context, | 2457 | retval = context->put_listent(context, |
@@ -2954,7 +2954,7 @@ xfs_attr_leaf_inactive(xfs_trans_t **trans, xfs_inode_t *dp, xfs_dabuf_t *bp) | |||
2954 | error = tmp; /* save only the 1st errno */ | 2954 | error = tmp; /* save only the 1st errno */ |
2955 | } | 2955 | } |
2956 | 2956 | ||
2957 | kmem_free((xfs_caddr_t)list, size); | 2957 | kmem_free((xfs_caddr_t)list); |
2958 | return(error); | 2958 | return(error); |
2959 | } | 2959 | } |
2960 | 2960 | ||
diff --git a/fs/xfs/xfs_bmap.c b/fs/xfs/xfs_bmap.c index 53c259f5a5af..a612a90aae4a 100644 --- a/fs/xfs/xfs_bmap.c +++ b/fs/xfs/xfs_bmap.c | |||
@@ -5970,7 +5970,7 @@ unlock_and_return: | |||
5970 | xfs_iunlock_map_shared(ip, lock); | 5970 | xfs_iunlock_map_shared(ip, lock); |
5971 | xfs_iunlock(ip, XFS_IOLOCK_SHARED); | 5971 | xfs_iunlock(ip, XFS_IOLOCK_SHARED); |
5972 | 5972 | ||
5973 | kmem_free(map, subnex * sizeof(*map)); | 5973 | kmem_free(map); |
5974 | 5974 | ||
5975 | return error; | 5975 | return error; |
5976 | } | 5976 | } |
diff --git a/fs/xfs/xfs_buf_item.c b/fs/xfs/xfs_buf_item.c index 53a71c62025d..d86ca2c03a70 100644 --- a/fs/xfs/xfs_buf_item.c +++ b/fs/xfs/xfs_buf_item.c | |||
@@ -889,9 +889,9 @@ xfs_buf_item_relse( | |||
889 | } | 889 | } |
890 | 890 | ||
891 | #ifdef XFS_TRANS_DEBUG | 891 | #ifdef XFS_TRANS_DEBUG |
892 | kmem_free(bip->bli_orig, XFS_BUF_COUNT(bp)); | 892 | kmem_free(bip->bli_orig); |
893 | bip->bli_orig = NULL; | 893 | bip->bli_orig = NULL; |
894 | kmem_free(bip->bli_logged, XFS_BUF_COUNT(bp) / NBBY); | 894 | kmem_free(bip->bli_logged); |
895 | bip->bli_logged = NULL; | 895 | bip->bli_logged = NULL; |
896 | #endif /* XFS_TRANS_DEBUG */ | 896 | #endif /* XFS_TRANS_DEBUG */ |
897 | 897 | ||
@@ -1138,9 +1138,9 @@ xfs_buf_iodone( | |||
1138 | xfs_trans_delete_ail(mp, (xfs_log_item_t *)bip); | 1138 | xfs_trans_delete_ail(mp, (xfs_log_item_t *)bip); |
1139 | 1139 | ||
1140 | #ifdef XFS_TRANS_DEBUG | 1140 | #ifdef XFS_TRANS_DEBUG |
1141 | kmem_free(bip->bli_orig, XFS_BUF_COUNT(bp)); | 1141 | kmem_free(bip->bli_orig); |
1142 | bip->bli_orig = NULL; | 1142 | bip->bli_orig = NULL; |
1143 | kmem_free(bip->bli_logged, XFS_BUF_COUNT(bp) / NBBY); | 1143 | kmem_free(bip->bli_logged); |
1144 | bip->bli_logged = NULL; | 1144 | bip->bli_logged = NULL; |
1145 | #endif /* XFS_TRANS_DEBUG */ | 1145 | #endif /* XFS_TRANS_DEBUG */ |
1146 | 1146 | ||
diff --git a/fs/xfs/xfs_da_btree.c b/fs/xfs/xfs_da_btree.c index 021a8f7e563f..294780427abb 100644 --- a/fs/xfs/xfs_da_btree.c +++ b/fs/xfs/xfs_da_btree.c | |||
@@ -1598,7 +1598,7 @@ xfs_da_grow_inode(xfs_da_args_t *args, xfs_dablk_t *new_blkno) | |||
1598 | args->firstblock, args->total, | 1598 | args->firstblock, args->total, |
1599 | &mapp[mapi], &nmap, args->flist, | 1599 | &mapp[mapi], &nmap, args->flist, |
1600 | NULL))) { | 1600 | NULL))) { |
1601 | kmem_free(mapp, sizeof(*mapp) * count); | 1601 | kmem_free(mapp); |
1602 | return error; | 1602 | return error; |
1603 | } | 1603 | } |
1604 | if (nmap < 1) | 1604 | if (nmap < 1) |
@@ -1620,11 +1620,11 @@ xfs_da_grow_inode(xfs_da_args_t *args, xfs_dablk_t *new_blkno) | |||
1620 | mapp[mapi - 1].br_startoff + mapp[mapi - 1].br_blockcount != | 1620 | mapp[mapi - 1].br_startoff + mapp[mapi - 1].br_blockcount != |
1621 | bno + count) { | 1621 | bno + count) { |
1622 | if (mapp != &map) | 1622 | if (mapp != &map) |
1623 | kmem_free(mapp, sizeof(*mapp) * count); | 1623 | kmem_free(mapp); |
1624 | return XFS_ERROR(ENOSPC); | 1624 | return XFS_ERROR(ENOSPC); |
1625 | } | 1625 | } |
1626 | if (mapp != &map) | 1626 | if (mapp != &map) |
1627 | kmem_free(mapp, sizeof(*mapp) * count); | 1627 | kmem_free(mapp); |
1628 | *new_blkno = (xfs_dablk_t)bno; | 1628 | *new_blkno = (xfs_dablk_t)bno; |
1629 | return 0; | 1629 | return 0; |
1630 | } | 1630 | } |
@@ -2090,10 +2090,10 @@ xfs_da_do_buf( | |||
2090 | } | 2090 | } |
2091 | } | 2091 | } |
2092 | if (bplist) { | 2092 | if (bplist) { |
2093 | kmem_free(bplist, sizeof(*bplist) * nmap); | 2093 | kmem_free(bplist); |
2094 | } | 2094 | } |
2095 | if (mapp != &map) { | 2095 | if (mapp != &map) { |
2096 | kmem_free(mapp, sizeof(*mapp) * nfsb); | 2096 | kmem_free(mapp); |
2097 | } | 2097 | } |
2098 | if (bpp) | 2098 | if (bpp) |
2099 | *bpp = rbp; | 2099 | *bpp = rbp; |
@@ -2102,11 +2102,11 @@ exit1: | |||
2102 | if (bplist) { | 2102 | if (bplist) { |
2103 | for (i = 0; i < nbplist; i++) | 2103 | for (i = 0; i < nbplist; i++) |
2104 | xfs_trans_brelse(trans, bplist[i]); | 2104 | xfs_trans_brelse(trans, bplist[i]); |
2105 | kmem_free(bplist, sizeof(*bplist) * nmap); | 2105 | kmem_free(bplist); |
2106 | } | 2106 | } |
2107 | exit0: | 2107 | exit0: |
2108 | if (mapp != &map) | 2108 | if (mapp != &map) |
2109 | kmem_free(mapp, sizeof(*mapp) * nfsb); | 2109 | kmem_free(mapp); |
2110 | if (bpp) | 2110 | if (bpp) |
2111 | *bpp = NULL; | 2111 | *bpp = NULL; |
2112 | return error; | 2112 | return error; |
@@ -2315,7 +2315,7 @@ xfs_da_buf_done(xfs_dabuf_t *dabuf) | |||
2315 | if (dabuf->dirty) | 2315 | if (dabuf->dirty) |
2316 | xfs_da_buf_clean(dabuf); | 2316 | xfs_da_buf_clean(dabuf); |
2317 | if (dabuf->nbuf > 1) | 2317 | if (dabuf->nbuf > 1) |
2318 | kmem_free(dabuf->data, BBTOB(dabuf->bbcount)); | 2318 | kmem_free(dabuf->data); |
2319 | #ifdef XFS_DABUF_DEBUG | 2319 | #ifdef XFS_DABUF_DEBUG |
2320 | { | 2320 | { |
2321 | spin_lock(&xfs_dabuf_global_lock); | 2321 | spin_lock(&xfs_dabuf_global_lock); |
@@ -2332,7 +2332,7 @@ xfs_da_buf_done(xfs_dabuf_t *dabuf) | |||
2332 | if (dabuf->nbuf == 1) | 2332 | if (dabuf->nbuf == 1) |
2333 | kmem_zone_free(xfs_dabuf_zone, dabuf); | 2333 | kmem_zone_free(xfs_dabuf_zone, dabuf); |
2334 | else | 2334 | else |
2335 | kmem_free(dabuf, XFS_DA_BUF_SIZE(dabuf->nbuf)); | 2335 | kmem_free(dabuf); |
2336 | } | 2336 | } |
2337 | 2337 | ||
2338 | /* | 2338 | /* |
@@ -2403,7 +2403,7 @@ xfs_da_brelse(xfs_trans_t *tp, xfs_dabuf_t *dabuf) | |||
2403 | for (i = 0; i < nbuf; i++) | 2403 | for (i = 0; i < nbuf; i++) |
2404 | xfs_trans_brelse(tp, bplist[i]); | 2404 | xfs_trans_brelse(tp, bplist[i]); |
2405 | if (bplist != &bp) | 2405 | if (bplist != &bp) |
2406 | kmem_free(bplist, nbuf * sizeof(*bplist)); | 2406 | kmem_free(bplist); |
2407 | } | 2407 | } |
2408 | 2408 | ||
2409 | /* | 2409 | /* |
@@ -2429,7 +2429,7 @@ xfs_da_binval(xfs_trans_t *tp, xfs_dabuf_t *dabuf) | |||
2429 | for (i = 0; i < nbuf; i++) | 2429 | for (i = 0; i < nbuf; i++) |
2430 | xfs_trans_binval(tp, bplist[i]); | 2430 | xfs_trans_binval(tp, bplist[i]); |
2431 | if (bplist != &bp) | 2431 | if (bplist != &bp) |
2432 | kmem_free(bplist, nbuf * sizeof(*bplist)); | 2432 | kmem_free(bplist); |
2433 | } | 2433 | } |
2434 | 2434 | ||
2435 | /* | 2435 | /* |
diff --git a/fs/xfs/xfs_dfrag.c b/fs/xfs/xfs_dfrag.c index 5f3647cb9885..2211e885ef24 100644 --- a/fs/xfs/xfs_dfrag.c +++ b/fs/xfs/xfs_dfrag.c | |||
@@ -116,7 +116,7 @@ xfs_swapext( | |||
116 | out_put_file: | 116 | out_put_file: |
117 | fput(file); | 117 | fput(file); |
118 | out_free_sxp: | 118 | out_free_sxp: |
119 | kmem_free(sxp, sizeof(xfs_swapext_t)); | 119 | kmem_free(sxp); |
120 | out: | 120 | out: |
121 | return error; | 121 | return error; |
122 | } | 122 | } |
@@ -381,6 +381,6 @@ xfs_swap_extents( | |||
381 | xfs_iunlock(tip, lock_flags); | 381 | xfs_iunlock(tip, lock_flags); |
382 | } | 382 | } |
383 | if (tempifp != NULL) | 383 | if (tempifp != NULL) |
384 | kmem_free(tempifp, sizeof(xfs_ifork_t)); | 384 | kmem_free(tempifp); |
385 | return error; | 385 | return error; |
386 | } | 386 | } |
diff --git a/fs/xfs/xfs_dir2.c b/fs/xfs/xfs_dir2.c index 7cb26529766b..0284af1734bd 100644 --- a/fs/xfs/xfs_dir2.c +++ b/fs/xfs/xfs_dir2.c | |||
@@ -493,7 +493,7 @@ xfs_dir2_grow_inode( | |||
493 | args->firstblock, args->total, | 493 | args->firstblock, args->total, |
494 | &mapp[mapi], &nmap, args->flist, | 494 | &mapp[mapi], &nmap, args->flist, |
495 | NULL))) { | 495 | NULL))) { |
496 | kmem_free(mapp, sizeof(*mapp) * count); | 496 | kmem_free(mapp); |
497 | return error; | 497 | return error; |
498 | } | 498 | } |
499 | if (nmap < 1) | 499 | if (nmap < 1) |
@@ -525,14 +525,14 @@ xfs_dir2_grow_inode( | |||
525 | mapp[mapi - 1].br_startoff + mapp[mapi - 1].br_blockcount != | 525 | mapp[mapi - 1].br_startoff + mapp[mapi - 1].br_blockcount != |
526 | bno + count) { | 526 | bno + count) { |
527 | if (mapp != &map) | 527 | if (mapp != &map) |
528 | kmem_free(mapp, sizeof(*mapp) * count); | 528 | kmem_free(mapp); |
529 | return XFS_ERROR(ENOSPC); | 529 | return XFS_ERROR(ENOSPC); |
530 | } | 530 | } |
531 | /* | 531 | /* |
532 | * Done with the temporary mapping table. | 532 | * Done with the temporary mapping table. |
533 | */ | 533 | */ |
534 | if (mapp != &map) | 534 | if (mapp != &map) |
535 | kmem_free(mapp, sizeof(*mapp) * count); | 535 | kmem_free(mapp); |
536 | *dbp = xfs_dir2_da_to_db(mp, (xfs_dablk_t)bno); | 536 | *dbp = xfs_dir2_da_to_db(mp, (xfs_dablk_t)bno); |
537 | /* | 537 | /* |
538 | * Update file's size if this is the data space and it grew. | 538 | * Update file's size if this is the data space and it grew. |
diff --git a/fs/xfs/xfs_dir2_block.c b/fs/xfs/xfs_dir2_block.c index fb5a556725b3..e8a7aca5fe23 100644 --- a/fs/xfs/xfs_dir2_block.c +++ b/fs/xfs/xfs_dir2_block.c | |||
@@ -1071,7 +1071,7 @@ xfs_dir2_sf_to_block( | |||
1071 | */ | 1071 | */ |
1072 | error = xfs_dir2_grow_inode(args, XFS_DIR2_DATA_SPACE, &blkno); | 1072 | error = xfs_dir2_grow_inode(args, XFS_DIR2_DATA_SPACE, &blkno); |
1073 | if (error) { | 1073 | if (error) { |
1074 | kmem_free(buf, buf_len); | 1074 | kmem_free(buf); |
1075 | return error; | 1075 | return error; |
1076 | } | 1076 | } |
1077 | /* | 1077 | /* |
@@ -1079,7 +1079,7 @@ xfs_dir2_sf_to_block( | |||
1079 | */ | 1079 | */ |
1080 | error = xfs_dir2_data_init(args, blkno, &bp); | 1080 | error = xfs_dir2_data_init(args, blkno, &bp); |
1081 | if (error) { | 1081 | if (error) { |
1082 | kmem_free(buf, buf_len); | 1082 | kmem_free(buf); |
1083 | return error; | 1083 | return error; |
1084 | } | 1084 | } |
1085 | block = bp->data; | 1085 | block = bp->data; |
@@ -1198,7 +1198,7 @@ xfs_dir2_sf_to_block( | |||
1198 | sfep = xfs_dir2_sf_nextentry(sfp, sfep); | 1198 | sfep = xfs_dir2_sf_nextentry(sfp, sfep); |
1199 | } | 1199 | } |
1200 | /* Done with the temporary buffer */ | 1200 | /* Done with the temporary buffer */ |
1201 | kmem_free(buf, buf_len); | 1201 | kmem_free(buf); |
1202 | /* | 1202 | /* |
1203 | * Sort the leaf entries by hash value. | 1203 | * Sort the leaf entries by hash value. |
1204 | */ | 1204 | */ |
diff --git a/fs/xfs/xfs_dir2_leaf.c b/fs/xfs/xfs_dir2_leaf.c index bc52b803d79b..e33433408e4a 100644 --- a/fs/xfs/xfs_dir2_leaf.c +++ b/fs/xfs/xfs_dir2_leaf.c | |||
@@ -1110,7 +1110,7 @@ xfs_dir2_leaf_getdents( | |||
1110 | *offset = XFS_DIR2_MAX_DATAPTR; | 1110 | *offset = XFS_DIR2_MAX_DATAPTR; |
1111 | else | 1111 | else |
1112 | *offset = xfs_dir2_byte_to_dataptr(mp, curoff); | 1112 | *offset = xfs_dir2_byte_to_dataptr(mp, curoff); |
1113 | kmem_free(map, map_size * sizeof(*map)); | 1113 | kmem_free(map); |
1114 | if (bp) | 1114 | if (bp) |
1115 | xfs_da_brelse(NULL, bp); | 1115 | xfs_da_brelse(NULL, bp); |
1116 | return error; | 1116 | return error; |
diff --git a/fs/xfs/xfs_dir2_sf.c b/fs/xfs/xfs_dir2_sf.c index 919d275a1cef..ca33bc62edc2 100644 --- a/fs/xfs/xfs_dir2_sf.c +++ b/fs/xfs/xfs_dir2_sf.c | |||
@@ -255,7 +255,7 @@ xfs_dir2_block_to_sf( | |||
255 | xfs_dir2_sf_check(args); | 255 | xfs_dir2_sf_check(args); |
256 | out: | 256 | out: |
257 | xfs_trans_log_inode(args->trans, dp, logflags); | 257 | xfs_trans_log_inode(args->trans, dp, logflags); |
258 | kmem_free(block, mp->m_dirblksize); | 258 | kmem_free(block); |
259 | return error; | 259 | return error; |
260 | } | 260 | } |
261 | 261 | ||
@@ -512,7 +512,7 @@ xfs_dir2_sf_addname_hard( | |||
512 | sfep = xfs_dir2_sf_nextentry(sfp, sfep); | 512 | sfep = xfs_dir2_sf_nextentry(sfp, sfep); |
513 | memcpy(sfep, oldsfep, old_isize - nbytes); | 513 | memcpy(sfep, oldsfep, old_isize - nbytes); |
514 | } | 514 | } |
515 | kmem_free(buf, old_isize); | 515 | kmem_free(buf); |
516 | dp->i_d.di_size = new_isize; | 516 | dp->i_d.di_size = new_isize; |
517 | xfs_dir2_sf_check(args); | 517 | xfs_dir2_sf_check(args); |
518 | } | 518 | } |
@@ -1174,7 +1174,7 @@ xfs_dir2_sf_toino4( | |||
1174 | /* | 1174 | /* |
1175 | * Clean up the inode. | 1175 | * Clean up the inode. |
1176 | */ | 1176 | */ |
1177 | kmem_free(buf, oldsize); | 1177 | kmem_free(buf); |
1178 | dp->i_d.di_size = newsize; | 1178 | dp->i_d.di_size = newsize; |
1179 | xfs_trans_log_inode(args->trans, dp, XFS_ILOG_CORE | XFS_ILOG_DDATA); | 1179 | xfs_trans_log_inode(args->trans, dp, XFS_ILOG_CORE | XFS_ILOG_DDATA); |
1180 | } | 1180 | } |
@@ -1251,7 +1251,7 @@ xfs_dir2_sf_toino8( | |||
1251 | /* | 1251 | /* |
1252 | * Clean up the inode. | 1252 | * Clean up the inode. |
1253 | */ | 1253 | */ |
1254 | kmem_free(buf, oldsize); | 1254 | kmem_free(buf); |
1255 | dp->i_d.di_size = newsize; | 1255 | dp->i_d.di_size = newsize; |
1256 | xfs_trans_log_inode(args->trans, dp, XFS_ILOG_CORE | XFS_ILOG_DDATA); | 1256 | xfs_trans_log_inode(args->trans, dp, XFS_ILOG_CORE | XFS_ILOG_DDATA); |
1257 | } | 1257 | } |
diff --git a/fs/xfs/xfs_error.c b/fs/xfs/xfs_error.c index 05e5365d3c31..7380a00644c8 100644 --- a/fs/xfs/xfs_error.c +++ b/fs/xfs/xfs_error.c | |||
@@ -150,8 +150,7 @@ xfs_errortag_clearall(xfs_mount_t *mp, int loud) | |||
150 | xfs_etest[i]); | 150 | xfs_etest[i]); |
151 | xfs_etest[i] = 0; | 151 | xfs_etest[i] = 0; |
152 | xfs_etest_fsid[i] = 0LL; | 152 | xfs_etest_fsid[i] = 0LL; |
153 | kmem_free(xfs_etest_fsname[i], | 153 | kmem_free(xfs_etest_fsname[i]); |
154 | strlen(xfs_etest_fsname[i]) + 1); | ||
155 | xfs_etest_fsname[i] = NULL; | 154 | xfs_etest_fsname[i] = NULL; |
156 | } | 155 | } |
157 | } | 156 | } |
@@ -175,7 +174,7 @@ xfs_fs_vcmn_err(int level, xfs_mount_t *mp, char *fmt, va_list ap) | |||
175 | newfmt = kmem_alloc(len, KM_SLEEP); | 174 | newfmt = kmem_alloc(len, KM_SLEEP); |
176 | sprintf(newfmt, "Filesystem \"%s\": %s", mp->m_fsname, fmt); | 175 | sprintf(newfmt, "Filesystem \"%s\": %s", mp->m_fsname, fmt); |
177 | icmn_err(level, newfmt, ap); | 176 | icmn_err(level, newfmt, ap); |
178 | kmem_free(newfmt, len); | 177 | kmem_free(newfmt); |
179 | } else { | 178 | } else { |
180 | icmn_err(level, fmt, ap); | 179 | icmn_err(level, fmt, ap); |
181 | } | 180 | } |
diff --git a/fs/xfs/xfs_extfree_item.c b/fs/xfs/xfs_extfree_item.c index 132bd07b9bb8..8aa28f751b2a 100644 --- a/fs/xfs/xfs_extfree_item.c +++ b/fs/xfs/xfs_extfree_item.c | |||
@@ -41,8 +41,7 @@ xfs_efi_item_free(xfs_efi_log_item_t *efip) | |||
41 | int nexts = efip->efi_format.efi_nextents; | 41 | int nexts = efip->efi_format.efi_nextents; |
42 | 42 | ||
43 | if (nexts > XFS_EFI_MAX_FAST_EXTENTS) { | 43 | if (nexts > XFS_EFI_MAX_FAST_EXTENTS) { |
44 | kmem_free(efip, sizeof(xfs_efi_log_item_t) + | 44 | kmem_free(efip); |
45 | (nexts - 1) * sizeof(xfs_extent_t)); | ||
46 | } else { | 45 | } else { |
47 | kmem_zone_free(xfs_efi_zone, efip); | 46 | kmem_zone_free(xfs_efi_zone, efip); |
48 | } | 47 | } |
@@ -374,8 +373,7 @@ xfs_efd_item_free(xfs_efd_log_item_t *efdp) | |||
374 | int nexts = efdp->efd_format.efd_nextents; | 373 | int nexts = efdp->efd_format.efd_nextents; |
375 | 374 | ||
376 | if (nexts > XFS_EFD_MAX_FAST_EXTENTS) { | 375 | if (nexts > XFS_EFD_MAX_FAST_EXTENTS) { |
377 | kmem_free(efdp, sizeof(xfs_efd_log_item_t) + | 376 | kmem_free(efdp); |
378 | (nexts - 1) * sizeof(xfs_extent_t)); | ||
379 | } else { | 377 | } else { |
380 | kmem_zone_free(xfs_efd_zone, efdp); | 378 | kmem_zone_free(xfs_efd_zone, efdp); |
381 | } | 379 | } |
diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c index e569bf5d6cf0..4b21490334b1 100644 --- a/fs/xfs/xfs_inode.c +++ b/fs/xfs/xfs_inode.c | |||
@@ -2258,7 +2258,7 @@ xfs_ifree_cluster( | |||
2258 | xfs_trans_binval(tp, bp); | 2258 | xfs_trans_binval(tp, bp); |
2259 | } | 2259 | } |
2260 | 2260 | ||
2261 | kmem_free(ip_found, ninodes * sizeof(xfs_inode_t *)); | 2261 | kmem_free(ip_found); |
2262 | xfs_put_perag(mp, pag); | 2262 | xfs_put_perag(mp, pag); |
2263 | } | 2263 | } |
2264 | 2264 | ||
@@ -2470,7 +2470,7 @@ xfs_iroot_realloc( | |||
2470 | (int)new_size); | 2470 | (int)new_size); |
2471 | memcpy(np, op, new_max * (uint)sizeof(xfs_dfsbno_t)); | 2471 | memcpy(np, op, new_max * (uint)sizeof(xfs_dfsbno_t)); |
2472 | } | 2472 | } |
2473 | kmem_free(ifp->if_broot, ifp->if_broot_bytes); | 2473 | kmem_free(ifp->if_broot); |
2474 | ifp->if_broot = new_broot; | 2474 | ifp->if_broot = new_broot; |
2475 | ifp->if_broot_bytes = (int)new_size; | 2475 | ifp->if_broot_bytes = (int)new_size; |
2476 | ASSERT(ifp->if_broot_bytes <= | 2476 | ASSERT(ifp->if_broot_bytes <= |
@@ -2514,7 +2514,7 @@ xfs_idata_realloc( | |||
2514 | 2514 | ||
2515 | if (new_size == 0) { | 2515 | if (new_size == 0) { |
2516 | if (ifp->if_u1.if_data != ifp->if_u2.if_inline_data) { | 2516 | if (ifp->if_u1.if_data != ifp->if_u2.if_inline_data) { |
2517 | kmem_free(ifp->if_u1.if_data, ifp->if_real_bytes); | 2517 | kmem_free(ifp->if_u1.if_data); |
2518 | } | 2518 | } |
2519 | ifp->if_u1.if_data = NULL; | 2519 | ifp->if_u1.if_data = NULL; |
2520 | real_size = 0; | 2520 | real_size = 0; |
@@ -2529,7 +2529,7 @@ xfs_idata_realloc( | |||
2529 | ASSERT(ifp->if_real_bytes != 0); | 2529 | ASSERT(ifp->if_real_bytes != 0); |
2530 | memcpy(ifp->if_u2.if_inline_data, ifp->if_u1.if_data, | 2530 | memcpy(ifp->if_u2.if_inline_data, ifp->if_u1.if_data, |
2531 | new_size); | 2531 | new_size); |
2532 | kmem_free(ifp->if_u1.if_data, ifp->if_real_bytes); | 2532 | kmem_free(ifp->if_u1.if_data); |
2533 | ifp->if_u1.if_data = ifp->if_u2.if_inline_data; | 2533 | ifp->if_u1.if_data = ifp->if_u2.if_inline_data; |
2534 | } | 2534 | } |
2535 | real_size = 0; | 2535 | real_size = 0; |
@@ -2636,7 +2636,7 @@ xfs_idestroy_fork( | |||
2636 | 2636 | ||
2637 | ifp = XFS_IFORK_PTR(ip, whichfork); | 2637 | ifp = XFS_IFORK_PTR(ip, whichfork); |
2638 | if (ifp->if_broot != NULL) { | 2638 | if (ifp->if_broot != NULL) { |
2639 | kmem_free(ifp->if_broot, ifp->if_broot_bytes); | 2639 | kmem_free(ifp->if_broot); |
2640 | ifp->if_broot = NULL; | 2640 | ifp->if_broot = NULL; |
2641 | } | 2641 | } |
2642 | 2642 | ||
@@ -2650,7 +2650,7 @@ xfs_idestroy_fork( | |||
2650 | if ((ifp->if_u1.if_data != ifp->if_u2.if_inline_data) && | 2650 | if ((ifp->if_u1.if_data != ifp->if_u2.if_inline_data) && |
2651 | (ifp->if_u1.if_data != NULL)) { | 2651 | (ifp->if_u1.if_data != NULL)) { |
2652 | ASSERT(ifp->if_real_bytes != 0); | 2652 | ASSERT(ifp->if_real_bytes != 0); |
2653 | kmem_free(ifp->if_u1.if_data, ifp->if_real_bytes); | 2653 | kmem_free(ifp->if_u1.if_data); |
2654 | ifp->if_u1.if_data = NULL; | 2654 | ifp->if_u1.if_data = NULL; |
2655 | ifp->if_real_bytes = 0; | 2655 | ifp->if_real_bytes = 0; |
2656 | } | 2656 | } |
@@ -3058,7 +3058,7 @@ xfs_iflush_cluster( | |||
3058 | 3058 | ||
3059 | out_free: | 3059 | out_free: |
3060 | read_unlock(&pag->pag_ici_lock); | 3060 | read_unlock(&pag->pag_ici_lock); |
3061 | kmem_free(ilist, ilist_size); | 3061 | kmem_free(ilist); |
3062 | return 0; | 3062 | return 0; |
3063 | 3063 | ||
3064 | 3064 | ||
@@ -3102,7 +3102,7 @@ cluster_corrupt_out: | |||
3102 | * Unlocks the flush lock | 3102 | * Unlocks the flush lock |
3103 | */ | 3103 | */ |
3104 | xfs_iflush_abort(iq); | 3104 | xfs_iflush_abort(iq); |
3105 | kmem_free(ilist, ilist_size); | 3105 | kmem_free(ilist); |
3106 | return XFS_ERROR(EFSCORRUPTED); | 3106 | return XFS_ERROR(EFSCORRUPTED); |
3107 | } | 3107 | } |
3108 | 3108 | ||
@@ -3836,7 +3836,7 @@ xfs_iext_add_indirect_multi( | |||
3836 | erp = xfs_iext_irec_new(ifp, erp_idx); | 3836 | erp = xfs_iext_irec_new(ifp, erp_idx); |
3837 | } | 3837 | } |
3838 | memmove(&erp->er_extbuf[i], nex2_ep, byte_diff); | 3838 | memmove(&erp->er_extbuf[i], nex2_ep, byte_diff); |
3839 | kmem_free(nex2_ep, byte_diff); | 3839 | kmem_free(nex2_ep); |
3840 | erp->er_extcount += nex2; | 3840 | erp->er_extcount += nex2; |
3841 | xfs_iext_irec_update_extoffs(ifp, erp_idx + 1, nex2); | 3841 | xfs_iext_irec_update_extoffs(ifp, erp_idx + 1, nex2); |
3842 | } | 3842 | } |
@@ -4112,7 +4112,7 @@ xfs_iext_direct_to_inline( | |||
4112 | */ | 4112 | */ |
4113 | memcpy(ifp->if_u2.if_inline_ext, ifp->if_u1.if_extents, | 4113 | memcpy(ifp->if_u2.if_inline_ext, ifp->if_u1.if_extents, |
4114 | nextents * sizeof(xfs_bmbt_rec_t)); | 4114 | nextents * sizeof(xfs_bmbt_rec_t)); |
4115 | kmem_free(ifp->if_u1.if_extents, ifp->if_real_bytes); | 4115 | kmem_free(ifp->if_u1.if_extents); |
4116 | ifp->if_u1.if_extents = ifp->if_u2.if_inline_ext; | 4116 | ifp->if_u1.if_extents = ifp->if_u2.if_inline_ext; |
4117 | ifp->if_real_bytes = 0; | 4117 | ifp->if_real_bytes = 0; |
4118 | } | 4118 | } |
@@ -4186,7 +4186,7 @@ xfs_iext_indirect_to_direct( | |||
4186 | ASSERT(ifp->if_real_bytes == XFS_IEXT_BUFSZ); | 4186 | ASSERT(ifp->if_real_bytes == XFS_IEXT_BUFSZ); |
4187 | 4187 | ||
4188 | ep = ifp->if_u1.if_ext_irec->er_extbuf; | 4188 | ep = ifp->if_u1.if_ext_irec->er_extbuf; |
4189 | kmem_free(ifp->if_u1.if_ext_irec, sizeof(xfs_ext_irec_t)); | 4189 | kmem_free(ifp->if_u1.if_ext_irec); |
4190 | ifp->if_flags &= ~XFS_IFEXTIREC; | 4190 | ifp->if_flags &= ~XFS_IFEXTIREC; |
4191 | ifp->if_u1.if_extents = ep; | 4191 | ifp->if_u1.if_extents = ep; |
4192 | ifp->if_bytes = size; | 4192 | ifp->if_bytes = size; |
@@ -4212,7 +4212,7 @@ xfs_iext_destroy( | |||
4212 | } | 4212 | } |
4213 | ifp->if_flags &= ~XFS_IFEXTIREC; | 4213 | ifp->if_flags &= ~XFS_IFEXTIREC; |
4214 | } else if (ifp->if_real_bytes) { | 4214 | } else if (ifp->if_real_bytes) { |
4215 | kmem_free(ifp->if_u1.if_extents, ifp->if_real_bytes); | 4215 | kmem_free(ifp->if_u1.if_extents); |
4216 | } else if (ifp->if_bytes) { | 4216 | } else if (ifp->if_bytes) { |
4217 | memset(ifp->if_u2.if_inline_ext, 0, XFS_INLINE_EXTS * | 4217 | memset(ifp->if_u2.if_inline_ext, 0, XFS_INLINE_EXTS * |
4218 | sizeof(xfs_bmbt_rec_t)); | 4218 | sizeof(xfs_bmbt_rec_t)); |
@@ -4483,7 +4483,7 @@ xfs_iext_irec_remove( | |||
4483 | if (erp->er_extbuf) { | 4483 | if (erp->er_extbuf) { |
4484 | xfs_iext_irec_update_extoffs(ifp, erp_idx + 1, | 4484 | xfs_iext_irec_update_extoffs(ifp, erp_idx + 1, |
4485 | -erp->er_extcount); | 4485 | -erp->er_extcount); |
4486 | kmem_free(erp->er_extbuf, XFS_IEXT_BUFSZ); | 4486 | kmem_free(erp->er_extbuf); |
4487 | } | 4487 | } |
4488 | /* Compact extent records */ | 4488 | /* Compact extent records */ |
4489 | erp = ifp->if_u1.if_ext_irec; | 4489 | erp = ifp->if_u1.if_ext_irec; |
@@ -4501,8 +4501,7 @@ xfs_iext_irec_remove( | |||
4501 | xfs_iext_realloc_indirect(ifp, | 4501 | xfs_iext_realloc_indirect(ifp, |
4502 | nlists * sizeof(xfs_ext_irec_t)); | 4502 | nlists * sizeof(xfs_ext_irec_t)); |
4503 | } else { | 4503 | } else { |
4504 | kmem_free(ifp->if_u1.if_ext_irec, | 4504 | kmem_free(ifp->if_u1.if_ext_irec); |
4505 | sizeof(xfs_ext_irec_t)); | ||
4506 | } | 4505 | } |
4507 | ifp->if_real_bytes = nlists * XFS_IEXT_BUFSZ; | 4506 | ifp->if_real_bytes = nlists * XFS_IEXT_BUFSZ; |
4508 | } | 4507 | } |
@@ -4571,7 +4570,7 @@ xfs_iext_irec_compact_pages( | |||
4571 | * so er_extoffs don't get modified in | 4570 | * so er_extoffs don't get modified in |
4572 | * xfs_iext_irec_remove. | 4571 | * xfs_iext_irec_remove. |
4573 | */ | 4572 | */ |
4574 | kmem_free(erp_next->er_extbuf, XFS_IEXT_BUFSZ); | 4573 | kmem_free(erp_next->er_extbuf); |
4575 | erp_next->er_extbuf = NULL; | 4574 | erp_next->er_extbuf = NULL; |
4576 | xfs_iext_irec_remove(ifp, erp_idx + 1); | 4575 | xfs_iext_irec_remove(ifp, erp_idx + 1); |
4577 | nlists = ifp->if_real_bytes / XFS_IEXT_BUFSZ; | 4576 | nlists = ifp->if_real_bytes / XFS_IEXT_BUFSZ; |
@@ -4614,8 +4613,7 @@ xfs_iext_irec_compact_full( | |||
4614 | * so er_extoffs don't get modified in | 4613 | * so er_extoffs don't get modified in |
4615 | * xfs_iext_irec_remove. | 4614 | * xfs_iext_irec_remove. |
4616 | */ | 4615 | */ |
4617 | kmem_free(erp_next->er_extbuf, | 4616 | kmem_free(erp_next->er_extbuf); |
4618 | erp_next->er_extcount * sizeof(xfs_bmbt_rec_t)); | ||
4619 | erp_next->er_extbuf = NULL; | 4617 | erp_next->er_extbuf = NULL; |
4620 | xfs_iext_irec_remove(ifp, erp_idx + 1); | 4618 | xfs_iext_irec_remove(ifp, erp_idx + 1); |
4621 | erp = &ifp->if_u1.if_ext_irec[erp_idx]; | 4619 | erp = &ifp->if_u1.if_ext_irec[erp_idx]; |
diff --git a/fs/xfs/xfs_inode_item.c b/fs/xfs/xfs_inode_item.c index 167b33f15772..0eee08a32c26 100644 --- a/fs/xfs/xfs_inode_item.c +++ b/fs/xfs/xfs_inode_item.c | |||
@@ -686,7 +686,7 @@ xfs_inode_item_unlock( | |||
686 | ASSERT(ip->i_d.di_nextents > 0); | 686 | ASSERT(ip->i_d.di_nextents > 0); |
687 | ASSERT(iip->ili_format.ilf_fields & XFS_ILOG_DEXT); | 687 | ASSERT(iip->ili_format.ilf_fields & XFS_ILOG_DEXT); |
688 | ASSERT(ip->i_df.if_bytes > 0); | 688 | ASSERT(ip->i_df.if_bytes > 0); |
689 | kmem_free(iip->ili_extents_buf, ip->i_df.if_bytes); | 689 | kmem_free(iip->ili_extents_buf); |
690 | iip->ili_extents_buf = NULL; | 690 | iip->ili_extents_buf = NULL; |
691 | } | 691 | } |
692 | if (iip->ili_aextents_buf != NULL) { | 692 | if (iip->ili_aextents_buf != NULL) { |
@@ -694,7 +694,7 @@ xfs_inode_item_unlock( | |||
694 | ASSERT(ip->i_d.di_anextents > 0); | 694 | ASSERT(ip->i_d.di_anextents > 0); |
695 | ASSERT(iip->ili_format.ilf_fields & XFS_ILOG_AEXT); | 695 | ASSERT(iip->ili_format.ilf_fields & XFS_ILOG_AEXT); |
696 | ASSERT(ip->i_afp->if_bytes > 0); | 696 | ASSERT(ip->i_afp->if_bytes > 0); |
697 | kmem_free(iip->ili_aextents_buf, ip->i_afp->if_bytes); | 697 | kmem_free(iip->ili_aextents_buf); |
698 | iip->ili_aextents_buf = NULL; | 698 | iip->ili_aextents_buf = NULL; |
699 | } | 699 | } |
700 | 700 | ||
@@ -957,8 +957,7 @@ xfs_inode_item_destroy( | |||
957 | { | 957 | { |
958 | #ifdef XFS_TRANS_DEBUG | 958 | #ifdef XFS_TRANS_DEBUG |
959 | if (ip->i_itemp->ili_root_size != 0) { | 959 | if (ip->i_itemp->ili_root_size != 0) { |
960 | kmem_free(ip->i_itemp->ili_orig_root, | 960 | kmem_free(ip->i_itemp->ili_orig_root); |
961 | ip->i_itemp->ili_root_size); | ||
962 | } | 961 | } |
963 | #endif | 962 | #endif |
964 | kmem_zone_free(xfs_ili_zone, ip->i_itemp); | 963 | kmem_zone_free(xfs_ili_zone, ip->i_itemp); |
diff --git a/fs/xfs/xfs_itable.c b/fs/xfs/xfs_itable.c index 419de15aeb43..9a3ef9dcaeb9 100644 --- a/fs/xfs/xfs_itable.c +++ b/fs/xfs/xfs_itable.c | |||
@@ -257,7 +257,7 @@ xfs_bulkstat_one( | |||
257 | *ubused = error; | 257 | *ubused = error; |
258 | 258 | ||
259 | out_free: | 259 | out_free: |
260 | kmem_free(buf, sizeof(*buf)); | 260 | kmem_free(buf); |
261 | return error; | 261 | return error; |
262 | } | 262 | } |
263 | 263 | ||
@@ -708,7 +708,7 @@ xfs_bulkstat( | |||
708 | /* | 708 | /* |
709 | * Done, we're either out of filesystem or space to put the data. | 709 | * Done, we're either out of filesystem or space to put the data. |
710 | */ | 710 | */ |
711 | kmem_free(irbuf, irbsize); | 711 | kmem_free(irbuf); |
712 | *ubcountp = ubelem; | 712 | *ubcountp = ubelem; |
713 | /* | 713 | /* |
714 | * Found some inodes, return them now and return the error next time. | 714 | * Found some inodes, return them now and return the error next time. |
@@ -914,7 +914,7 @@ xfs_inumbers( | |||
914 | } | 914 | } |
915 | *lastino = XFS_AGINO_TO_INO(mp, agno, agino); | 915 | *lastino = XFS_AGINO_TO_INO(mp, agno, agino); |
916 | } | 916 | } |
917 | kmem_free(buffer, bcount * sizeof(*buffer)); | 917 | kmem_free(buffer); |
918 | if (cur) | 918 | if (cur) |
919 | xfs_btree_del_cursor(cur, (error ? XFS_BTREE_ERROR : | 919 | xfs_btree_del_cursor(cur, (error ? XFS_BTREE_ERROR : |
920 | XFS_BTREE_NOERROR)); | 920 | XFS_BTREE_NOERROR)); |
diff --git a/fs/xfs/xfs_log.c b/fs/xfs/xfs_log.c index ad3d26ddfe31..16a01abe2490 100644 --- a/fs/xfs/xfs_log.c +++ b/fs/xfs/xfs_log.c | |||
@@ -1570,7 +1570,7 @@ xlog_dealloc_log(xlog_t *log) | |||
1570 | } | 1570 | } |
1571 | #endif | 1571 | #endif |
1572 | next_iclog = iclog->ic_next; | 1572 | next_iclog = iclog->ic_next; |
1573 | kmem_free(iclog, sizeof(xlog_in_core_t)); | 1573 | kmem_free(iclog); |
1574 | iclog = next_iclog; | 1574 | iclog = next_iclog; |
1575 | } | 1575 | } |
1576 | freesema(&log->l_flushsema); | 1576 | freesema(&log->l_flushsema); |
@@ -1587,7 +1587,7 @@ xlog_dealloc_log(xlog_t *log) | |||
1587 | } | 1587 | } |
1588 | #endif | 1588 | #endif |
1589 | log->l_mp->m_log = NULL; | 1589 | log->l_mp->m_log = NULL; |
1590 | kmem_free(log, sizeof(xlog_t)); | 1590 | kmem_free(log); |
1591 | } /* xlog_dealloc_log */ | 1591 | } /* xlog_dealloc_log */ |
1592 | 1592 | ||
1593 | /* | 1593 | /* |
diff --git a/fs/xfs/xfs_log_recover.c b/fs/xfs/xfs_log_recover.c index e65ab4af0955..9eb722ec744e 100644 --- a/fs/xfs/xfs_log_recover.c +++ b/fs/xfs/xfs_log_recover.c | |||
@@ -1715,8 +1715,7 @@ xlog_check_buffer_cancelled( | |||
1715 | } else { | 1715 | } else { |
1716 | prevp->bc_next = bcp->bc_next; | 1716 | prevp->bc_next = bcp->bc_next; |
1717 | } | 1717 | } |
1718 | kmem_free(bcp, | 1718 | kmem_free(bcp); |
1719 | sizeof(xfs_buf_cancel_t)); | ||
1720 | } | 1719 | } |
1721 | } | 1720 | } |
1722 | return 1; | 1721 | return 1; |
@@ -2519,7 +2518,7 @@ write_inode_buffer: | |||
2519 | 2518 | ||
2520 | error: | 2519 | error: |
2521 | if (need_free) | 2520 | if (need_free) |
2522 | kmem_free(in_f, sizeof(*in_f)); | 2521 | kmem_free(in_f); |
2523 | return XFS_ERROR(error); | 2522 | return XFS_ERROR(error); |
2524 | } | 2523 | } |
2525 | 2524 | ||
@@ -2830,16 +2829,14 @@ xlog_recover_free_trans( | |||
2830 | item = item->ri_next; | 2829 | item = item->ri_next; |
2831 | /* Free the regions in the item. */ | 2830 | /* Free the regions in the item. */ |
2832 | for (i = 0; i < free_item->ri_cnt; i++) { | 2831 | for (i = 0; i < free_item->ri_cnt; i++) { |
2833 | kmem_free(free_item->ri_buf[i].i_addr, | 2832 | kmem_free(free_item->ri_buf[i].i_addr); |
2834 | free_item->ri_buf[i].i_len); | ||
2835 | } | 2833 | } |
2836 | /* Free the item itself */ | 2834 | /* Free the item itself */ |
2837 | kmem_free(free_item->ri_buf, | 2835 | kmem_free(free_item->ri_buf); |
2838 | (free_item->ri_total * sizeof(xfs_log_iovec_t))); | 2836 | kmem_free(free_item); |
2839 | kmem_free(free_item, sizeof(xlog_recover_item_t)); | ||
2840 | } while (first_item != item); | 2837 | } while (first_item != item); |
2841 | /* Free the transaction recover structure */ | 2838 | /* Free the transaction recover structure */ |
2842 | kmem_free(trans, sizeof(xlog_recover_t)); | 2839 | kmem_free(trans); |
2843 | } | 2840 | } |
2844 | 2841 | ||
2845 | STATIC int | 2842 | STATIC int |
@@ -3786,8 +3783,7 @@ xlog_do_log_recovery( | |||
3786 | error = xlog_do_recovery_pass(log, head_blk, tail_blk, | 3783 | error = xlog_do_recovery_pass(log, head_blk, tail_blk, |
3787 | XLOG_RECOVER_PASS1); | 3784 | XLOG_RECOVER_PASS1); |
3788 | if (error != 0) { | 3785 | if (error != 0) { |
3789 | kmem_free(log->l_buf_cancel_table, | 3786 | kmem_free(log->l_buf_cancel_table); |
3790 | XLOG_BC_TABLE_SIZE * sizeof(xfs_buf_cancel_t*)); | ||
3791 | log->l_buf_cancel_table = NULL; | 3787 | log->l_buf_cancel_table = NULL; |
3792 | return error; | 3788 | return error; |
3793 | } | 3789 | } |
@@ -3806,8 +3802,7 @@ xlog_do_log_recovery( | |||
3806 | } | 3802 | } |
3807 | #endif /* DEBUG */ | 3803 | #endif /* DEBUG */ |
3808 | 3804 | ||
3809 | kmem_free(log->l_buf_cancel_table, | 3805 | kmem_free(log->l_buf_cancel_table); |
3810 | XLOG_BC_TABLE_SIZE * sizeof(xfs_buf_cancel_t*)); | ||
3811 | log->l_buf_cancel_table = NULL; | 3806 | log->l_buf_cancel_table = NULL; |
3812 | 3807 | ||
3813 | return error; | 3808 | return error; |
diff --git a/fs/xfs/xfs_mount.c b/fs/xfs/xfs_mount.c index 361c7a755a07..c63f410ccfaa 100644 --- a/fs/xfs/xfs_mount.c +++ b/fs/xfs/xfs_mount.c | |||
@@ -161,11 +161,8 @@ xfs_mount_free( | |||
161 | 161 | ||
162 | for (agno = 0; agno < mp->m_maxagi; agno++) | 162 | for (agno = 0; agno < mp->m_maxagi; agno++) |
163 | if (mp->m_perag[agno].pagb_list) | 163 | if (mp->m_perag[agno].pagb_list) |
164 | kmem_free(mp->m_perag[agno].pagb_list, | 164 | kmem_free(mp->m_perag[agno].pagb_list); |
165 | sizeof(xfs_perag_busy_t) * | 165 | kmem_free(mp->m_perag); |
166 | XFS_PAGB_NUM_SLOTS); | ||
167 | kmem_free(mp->m_perag, | ||
168 | sizeof(xfs_perag_t) * mp->m_sb.sb_agcount); | ||
169 | } | 166 | } |
170 | 167 | ||
171 | spinlock_destroy(&mp->m_ail_lock); | 168 | spinlock_destroy(&mp->m_ail_lock); |
@@ -176,11 +173,11 @@ xfs_mount_free( | |||
176 | XFS_QM_DONE(mp); | 173 | XFS_QM_DONE(mp); |
177 | 174 | ||
178 | if (mp->m_fsname != NULL) | 175 | if (mp->m_fsname != NULL) |
179 | kmem_free(mp->m_fsname, mp->m_fsname_len); | 176 | kmem_free(mp->m_fsname); |
180 | if (mp->m_rtname != NULL) | 177 | if (mp->m_rtname != NULL) |
181 | kmem_free(mp->m_rtname, strlen(mp->m_rtname) + 1); | 178 | kmem_free(mp->m_rtname); |
182 | if (mp->m_logname != NULL) | 179 | if (mp->m_logname != NULL) |
183 | kmem_free(mp->m_logname, strlen(mp->m_logname) + 1); | 180 | kmem_free(mp->m_logname); |
184 | 181 | ||
185 | xfs_icsb_destroy_counters(mp); | 182 | xfs_icsb_destroy_counters(mp); |
186 | } | 183 | } |
@@ -1265,9 +1262,8 @@ xfs_mountfs( | |||
1265 | error2: | 1262 | error2: |
1266 | for (agno = 0; agno < sbp->sb_agcount; agno++) | 1263 | for (agno = 0; agno < sbp->sb_agcount; agno++) |
1267 | if (mp->m_perag[agno].pagb_list) | 1264 | if (mp->m_perag[agno].pagb_list) |
1268 | kmem_free(mp->m_perag[agno].pagb_list, | 1265 | kmem_free(mp->m_perag[agno].pagb_list); |
1269 | sizeof(xfs_perag_busy_t) * XFS_PAGB_NUM_SLOTS); | 1266 | kmem_free(mp->m_perag); |
1270 | kmem_free(mp->m_perag, sbp->sb_agcount * sizeof(xfs_perag_t)); | ||
1271 | mp->m_perag = NULL; | 1267 | mp->m_perag = NULL; |
1272 | /* FALLTHROUGH */ | 1268 | /* FALLTHROUGH */ |
1273 | error1: | 1269 | error1: |
diff --git a/fs/xfs/xfs_mru_cache.c b/fs/xfs/xfs_mru_cache.c index a0b2c0a2589a..26d14a1e0e14 100644 --- a/fs/xfs/xfs_mru_cache.c +++ b/fs/xfs/xfs_mru_cache.c | |||
@@ -382,9 +382,9 @@ xfs_mru_cache_create( | |||
382 | 382 | ||
383 | exit: | 383 | exit: |
384 | if (err && mru && mru->lists) | 384 | if (err && mru && mru->lists) |
385 | kmem_free(mru->lists, mru->grp_count * sizeof(*mru->lists)); | 385 | kmem_free(mru->lists); |
386 | if (err && mru) | 386 | if (err && mru) |
387 | kmem_free(mru, sizeof(*mru)); | 387 | kmem_free(mru); |
388 | 388 | ||
389 | return err; | 389 | return err; |
390 | } | 390 | } |
@@ -424,8 +424,8 @@ xfs_mru_cache_destroy( | |||
424 | 424 | ||
425 | xfs_mru_cache_flush(mru); | 425 | xfs_mru_cache_flush(mru); |
426 | 426 | ||
427 | kmem_free(mru->lists, mru->grp_count * sizeof(*mru->lists)); | 427 | kmem_free(mru->lists); |
428 | kmem_free(mru, sizeof(*mru)); | 428 | kmem_free(mru); |
429 | } | 429 | } |
430 | 430 | ||
431 | /* | 431 | /* |
diff --git a/fs/xfs/xfs_rtalloc.c b/fs/xfs/xfs_rtalloc.c index a0dc6e5bc5b9..bf87a5913504 100644 --- a/fs/xfs/xfs_rtalloc.c +++ b/fs/xfs/xfs_rtalloc.c | |||
@@ -2062,7 +2062,7 @@ xfs_growfs_rt( | |||
2062 | /* | 2062 | /* |
2063 | * Free the fake mp structure. | 2063 | * Free the fake mp structure. |
2064 | */ | 2064 | */ |
2065 | kmem_free(nmp, sizeof(*nmp)); | 2065 | kmem_free(nmp); |
2066 | 2066 | ||
2067 | return error; | 2067 | return error; |
2068 | } | 2068 | } |
diff --git a/fs/xfs/xfs_trans.c b/fs/xfs/xfs_trans.c index 140386434aa3..e4ebddd3c500 100644 --- a/fs/xfs/xfs_trans.c +++ b/fs/xfs/xfs_trans.c | |||
@@ -889,7 +889,7 @@ shut_us_down: | |||
889 | 889 | ||
890 | tp->t_commit_lsn = commit_lsn; | 890 | tp->t_commit_lsn = commit_lsn; |
891 | if (nvec > XFS_TRANS_LOGVEC_COUNT) { | 891 | if (nvec > XFS_TRANS_LOGVEC_COUNT) { |
892 | kmem_free(log_vector, nvec * sizeof(xfs_log_iovec_t)); | 892 | kmem_free(log_vector); |
893 | } | 893 | } |
894 | 894 | ||
895 | /* | 895 | /* |
@@ -1265,7 +1265,7 @@ xfs_trans_committed( | |||
1265 | ASSERT(!XFS_LIC_ARE_ALL_FREE(licp)); | 1265 | ASSERT(!XFS_LIC_ARE_ALL_FREE(licp)); |
1266 | xfs_trans_chunk_committed(licp, tp->t_lsn, abortflag); | 1266 | xfs_trans_chunk_committed(licp, tp->t_lsn, abortflag); |
1267 | next_licp = licp->lic_next; | 1267 | next_licp = licp->lic_next; |
1268 | kmem_free(licp, sizeof(xfs_log_item_chunk_t)); | 1268 | kmem_free(licp); |
1269 | licp = next_licp; | 1269 | licp = next_licp; |
1270 | } | 1270 | } |
1271 | 1271 | ||
diff --git a/fs/xfs/xfs_trans_inode.c b/fs/xfs/xfs_trans_inode.c index 4c70bf5e9985..2a1c0f071f91 100644 --- a/fs/xfs/xfs_trans_inode.c +++ b/fs/xfs/xfs_trans_inode.c | |||
@@ -291,7 +291,7 @@ xfs_trans_inode_broot_debug( | |||
291 | iip = ip->i_itemp; | 291 | iip = ip->i_itemp; |
292 | if (iip->ili_root_size != 0) { | 292 | if (iip->ili_root_size != 0) { |
293 | ASSERT(iip->ili_orig_root != NULL); | 293 | ASSERT(iip->ili_orig_root != NULL); |
294 | kmem_free(iip->ili_orig_root, iip->ili_root_size); | 294 | kmem_free(iip->ili_orig_root); |
295 | iip->ili_root_size = 0; | 295 | iip->ili_root_size = 0; |
296 | iip->ili_orig_root = NULL; | 296 | iip->ili_orig_root = NULL; |
297 | } | 297 | } |
diff --git a/fs/xfs/xfs_trans_item.c b/fs/xfs/xfs_trans_item.c index 66a09f0d894b..db5c83595526 100644 --- a/fs/xfs/xfs_trans_item.c +++ b/fs/xfs/xfs_trans_item.c | |||
@@ -161,7 +161,7 @@ xfs_trans_free_item(xfs_trans_t *tp, xfs_log_item_desc_t *lidp) | |||
161 | licpp = &((*licpp)->lic_next); | 161 | licpp = &((*licpp)->lic_next); |
162 | } | 162 | } |
163 | *licpp = licp->lic_next; | 163 | *licpp = licp->lic_next; |
164 | kmem_free(licp, sizeof(xfs_log_item_chunk_t)); | 164 | kmem_free(licp); |
165 | tp->t_items_free -= XFS_LIC_NUM_SLOTS; | 165 | tp->t_items_free -= XFS_LIC_NUM_SLOTS; |
166 | } | 166 | } |
167 | } | 167 | } |
@@ -314,7 +314,7 @@ xfs_trans_free_items( | |||
314 | ASSERT(!XFS_LIC_ARE_ALL_FREE(licp)); | 314 | ASSERT(!XFS_LIC_ARE_ALL_FREE(licp)); |
315 | (void) xfs_trans_unlock_chunk(licp, 1, abort, NULLCOMMITLSN); | 315 | (void) xfs_trans_unlock_chunk(licp, 1, abort, NULLCOMMITLSN); |
316 | next_licp = licp->lic_next; | 316 | next_licp = licp->lic_next; |
317 | kmem_free(licp, sizeof(xfs_log_item_chunk_t)); | 317 | kmem_free(licp); |
318 | licp = next_licp; | 318 | licp = next_licp; |
319 | } | 319 | } |
320 | 320 | ||
@@ -363,7 +363,7 @@ xfs_trans_unlock_items(xfs_trans_t *tp, xfs_lsn_t commit_lsn) | |||
363 | next_licp = licp->lic_next; | 363 | next_licp = licp->lic_next; |
364 | if (XFS_LIC_ARE_ALL_FREE(licp)) { | 364 | if (XFS_LIC_ARE_ALL_FREE(licp)) { |
365 | *licpp = next_licp; | 365 | *licpp = next_licp; |
366 | kmem_free(licp, sizeof(xfs_log_item_chunk_t)); | 366 | kmem_free(licp); |
367 | freed -= XFS_LIC_NUM_SLOTS; | 367 | freed -= XFS_LIC_NUM_SLOTS; |
368 | } else { | 368 | } else { |
369 | licpp = &(licp->lic_next); | 369 | licpp = &(licp->lic_next); |
@@ -530,7 +530,7 @@ xfs_trans_free_busy(xfs_trans_t *tp) | |||
530 | lbcp = tp->t_busy.lbc_next; | 530 | lbcp = tp->t_busy.lbc_next; |
531 | while (lbcp != NULL) { | 531 | while (lbcp != NULL) { |
532 | lbcq = lbcp->lbc_next; | 532 | lbcq = lbcp->lbc_next; |
533 | kmem_free(lbcp, sizeof(xfs_log_busy_chunk_t)); | 533 | kmem_free(lbcp); |
534 | lbcp = lbcq; | 534 | lbcp = lbcq; |
535 | } | 535 | } |
536 | 536 | ||
diff --git a/fs/xfs/xfs_vfsops.c b/fs/xfs/xfs_vfsops.c index bbc911720d81..a005cebf5041 100644 --- a/fs/xfs/xfs_vfsops.c +++ b/fs/xfs/xfs_vfsops.c | |||
@@ -639,7 +639,7 @@ out: | |||
639 | xfs_unmountfs(mp, credp); | 639 | xfs_unmountfs(mp, credp); |
640 | xfs_qmops_put(mp); | 640 | xfs_qmops_put(mp); |
641 | xfs_dmops_put(mp); | 641 | xfs_dmops_put(mp); |
642 | kmem_free(mp, sizeof(xfs_mount_t)); | 642 | kmem_free(mp); |
643 | } | 643 | } |
644 | 644 | ||
645 | return XFS_ERROR(error); | 645 | return XFS_ERROR(error); |
@@ -1055,7 +1055,7 @@ xfs_sync_inodes( | |||
1055 | 1055 | ||
1056 | if (XFS_FORCED_SHUTDOWN(mp) && !(flags & SYNC_CLOSE)) { | 1056 | if (XFS_FORCED_SHUTDOWN(mp) && !(flags & SYNC_CLOSE)) { |
1057 | XFS_MOUNT_IUNLOCK(mp); | 1057 | XFS_MOUNT_IUNLOCK(mp); |
1058 | kmem_free(ipointer, sizeof(xfs_iptr_t)); | 1058 | kmem_free(ipointer); |
1059 | return 0; | 1059 | return 0; |
1060 | } | 1060 | } |
1061 | 1061 | ||
@@ -1201,7 +1201,7 @@ xfs_sync_inodes( | |||
1201 | } | 1201 | } |
1202 | XFS_MOUNT_IUNLOCK(mp); | 1202 | XFS_MOUNT_IUNLOCK(mp); |
1203 | ASSERT(ipointer_in == B_FALSE); | 1203 | ASSERT(ipointer_in == B_FALSE); |
1204 | kmem_free(ipointer, sizeof(xfs_iptr_t)); | 1204 | kmem_free(ipointer); |
1205 | return XFS_ERROR(error); | 1205 | return XFS_ERROR(error); |
1206 | } | 1206 | } |
1207 | 1207 | ||
@@ -1231,7 +1231,7 @@ xfs_sync_inodes( | |||
1231 | 1231 | ||
1232 | ASSERT(ipointer_in == B_FALSE); | 1232 | ASSERT(ipointer_in == B_FALSE); |
1233 | 1233 | ||
1234 | kmem_free(ipointer, sizeof(xfs_iptr_t)); | 1234 | kmem_free(ipointer); |
1235 | return XFS_ERROR(last_error); | 1235 | return XFS_ERROR(last_error); |
1236 | } | 1236 | } |
1237 | 1237 | ||