lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 19 Mar 2007 17:24:43 -0700 (PDT) From: Davide Libenzi <davidel@...ilserver.org> To: Oleg Nesterov <oleg@...sign.ru> cc: Linux Kernel Mailing List <linux-kernel@...r.kernel.org>, Andrew Morton <akpm@...ux-foundation.org>, Linus Torvalds <torvalds@...ux-foundation.org>, "Eric W. Biederman" <ebiederm@...ssion.com> Subject: Re: [patch 2/13] signal/timer/event fds v7 - signalfd core ... On Tue, 20 Mar 2007, Oleg Nesterov wrote: > On 03/19, Davide Libenzi wrote: > > > > +static void signalfd_unlock(struct signalfd_ctx *ctx, > > + struct signalfd_lockctx *lk) > > +{ > > + unlock_task_sighand(lk->tsk, &lk->flags); > > +} > > Again, this is a matter of taste. But I can't understand why signalfd_unlock() > needs "signalfd_ctx *ctx" parameter. If we have "struct signalfd_lockctx *lk", > signalfd_lock() can setup lk->ctx if it is ever needed. With the new API, I agree. Removed. - Davide - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists