Messages in this thread Patch in this message |  | | Date | Thu, 15 Feb 2007 14:02:32 -0800 | From | Andrew Morton <> | Subject | Re: 2.6.20-mm1 |
| |
On Thu, 15 Feb 2007 15:37:20 +0100 Michal Piotrowski <michal.k.k.piotrowski@gmail.com> wrote:
> Andrew Morton napisa__(a): > > Temporarily at > > > > http://userweb.kernel.org/~akpm/2.6.20-mm1/ > > > > Will appear later at > > > > ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.20/2.6.20-mm1/ > > > > > > BUG: sleeping function called from invalid context at /mnt/md0/devel/linux-mm/mm/slab.c:3043 > in_atomic():1, irqs_disabled():0 > 1 lock held by artsd/3819: > #0: (&new->lock){--..}, at: [<c01d5b7c>] ipc_lock+0x35/0x4f > [<c0105312>] show_trace_log_lvl+0x1a/0x2f > [<c0105a25>] show_trace+0x12/0x14 > [<c0105ae7>] dump_stack+0x16/0x18 > [<c011db4a>] __might_sleep+0xc9/0xcf > [<c017c37a>] kmem_cache_zalloc+0x28/0xe5 > [<c01d8c7d>] do_shmat+0x111/0x372 > [<c0109151>] sys_ipc+0x148/0x1b5 > [<c010432c>] syscall_call+0x7/0xb
That's shm-make-sysv-ipc-shared-memory-use-stacked-files.patch, brought to us by Eric-who-hasnt-read-Documentation/SubmitChecklist.
Like this, I guess:
diff -puN ipc/shm.c~shm-make-sysv-ipc-shared-memory-use-stacked-files-fix ipc/shm.c --- a/ipc/shm.c~shm-make-sysv-ipc-shared-memory-use-stacked-files-fix +++ a/ipc/shm.c @@ -818,7 +818,7 @@ long do_shmat(int shmid, char __user *sh int acc_mode; void *user_addr; struct ipc_namespace *ns; - struct shm_file_data *sfd; + struct shm_file_data *sfd = NULL; mode_t f_mode; if (shmid < 0) { @@ -856,6 +856,8 @@ long do_shmat(int shmid, char __user *sh acc_mode |= S_IXUGO; } + sfd = kzalloc(sizeof(*sfd), GFP_KERNEL); + /* * We cannot rely on the fs check since SYSV IPC does have an * additional creator id... @@ -879,13 +881,12 @@ long do_shmat(int shmid, char __user *sh goto out_unlock; err = -ENOMEM; - sfd = kzalloc(sizeof(*sfd), GFP_KERNEL); if (!sfd) goto out_unlock; file = get_empty_filp(); if (!file) - goto out_free; + goto out_unlock; file->f_op = &shm_file_operations; file->private_data = sfd; @@ -939,9 +940,8 @@ invalid: if (IS_ERR(user_addr)) err = PTR_ERR(user_addr); out: - return err; -out_free: kfree(sfd); + return err; out_unlock: shm_unlock(shp); goto out; _ - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |