The patch titled coda: replace upc_alloc/upc_free with kmalloc/kfree has been added to the -mm tree. Its filename is coda-replace-upc_alloc-upc_free-with-kmalloc-kfree.patch *** Remember to use Documentation/SubmitChecklist when testing your code *** See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find out what to do about this ------------------------------------------------------ Subject: coda: replace upc_alloc/upc_free with kmalloc/kfree From: Jan Harkes <jaharkes@xxxxxxxxxx> Signed-off-by: Jan Harkes <jaharkes@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- fs/coda/psdev.c | 6 ++---- fs/coda/upcall.c | 11 ++++------- 2 files changed, 6 insertions(+), 11 deletions(-) diff -puN fs/coda/psdev.c~coda-replace-upc_alloc-upc_free-with-kmalloc-kfree fs/coda/psdev.c --- a/fs/coda/psdev.c~coda-replace-upc_alloc-upc_free-with-kmalloc-kfree +++ a/fs/coda/psdev.c @@ -49,8 +49,6 @@ #include "coda_int.h" -#define upc_free(r) kfree(r) - /* statistics */ int coda_hard; /* allows signals during upcalls */ unsigned long coda_timeout = 30; /* .. secs, then signals will dequeue */ @@ -264,7 +262,7 @@ static ssize_t coda_psdev_read(struct fi } CODA_FREE(req->uc_data, sizeof(struct coda_in_hdr)); - upc_free(req); + kfree(req); out: unlock_kernel(); return (count ? count : retval); @@ -320,7 +318,7 @@ static int coda_psdev_release(struct ino /* Async requests need to be freed here */ if (req->uc_flags & REQ_ASYNC) { CODA_FREE(req->uc_data, sizeof(struct coda_in_hdr)); - upc_free(req); + kfree(req); continue; } req->uc_flags |= REQ_ABORT; diff -puN fs/coda/upcall.c~coda-replace-upc_alloc-upc_free-with-kmalloc-kfree fs/coda/upcall.c --- a/fs/coda/upcall.c~coda-replace-upc_alloc-upc_free-with-kmalloc-kfree +++ a/fs/coda/upcall.c @@ -37,9 +37,6 @@ #include <linux/coda_cache.h> #include <linux/coda_proc.h> -#define upc_alloc() kmalloc(sizeof(struct upc_req), GFP_KERNEL) -#define upc_free(r) kfree(r) - static int coda_upcall(struct coda_sb_info *mntinfo, int inSize, int *outSize, union inputArgs *buffer); @@ -745,7 +742,7 @@ static int coda_upcall(struct coda_sb_in } /* Format the request message. */ - req = upc_alloc(); + req = kmalloc(sizeof(struct upc_req), GFP_KERNEL); if (!req) return -ENOMEM; @@ -802,12 +799,12 @@ static int coda_upcall(struct coda_sb_in } error = -ENOMEM; - sig_req = upc_alloc(); + sig_req = kmalloc(sizeof(struct upc_req), GFP_KERNEL); if (!sig_req) goto exit; CODA_ALLOC((sig_req->uc_data), char *, sizeof(struct coda_in_hdr)); if (!sig_req->uc_data) { - upc_free(sig_req); + kfree(sig_req); goto exit; } @@ -827,7 +824,7 @@ static int coda_upcall(struct coda_sb_in wake_up_interruptible(&vcommp->vc_waitq); exit: - upc_free(req); + kfree(req); return error; } _ Patches currently in -mm which might be from jaharkes@xxxxxxxxxx are coda-do-not-grab-an-uninitialized-fd-when-the-open-upcall-returns-an-error.patch coda-correctly-invalidate-cached-access-rights.patch coda-fix-nlink-updates-for-directories.patch coda-allow-removal-of-busy-directories.patch coda-coda-doesnt-track-atime.patch coda-use-ilookup5.patch coda-cleanup-dev-cfs-open-and-close-handling.patch coda-cleanup-for-upcall-handling-path.patch coda-block-signals-during-upcall-processing.patch coda-avoid-lockdep-warning-in-coda_readdir.patch coda-replace-upc_alloc-upc_free-with-kmalloc-kfree.patch coda-ignore-returned-values-when-upcalls-return-errors.patch coda-cleanup-coda_lookup-use-dsplice_alias.patch coda-cleanup-downcall-handler.patch coda-remove-struct-coda_sb_info.patch coda-remove-statistics-counters-from-proc-fs-coda.patch coda-update-module-information.patch - To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html