diff for duplicates of <1305290370.2466.14.camel@twins> diff --git a/a/1.txt b/N1/1.txt index 9029d14..0350cd9 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,34 +1,42 @@ On Fri, 2011-05-13 at 14:26 +0200, Ingo Molnar wrote: > * Peter Zijlstra <peterz@infradead.org> wrote: -> +>=20 > > On Fri, 2011-05-13 at 14:10 +0200, Ingo Molnar wrote: -> > > err = event_vfs_getname(result); -> > -> > I really think we should not do this. Events like we have them should be -> > inactive, totally passive entities, only observe but not affect execution +> > > err =3D event_vfs_getname(result); +> >=20 +> > I really think we should not do this. Events like we have them should b= +e=20 +> > inactive, totally passive entities, only observe but not affect executi= +on=20 > > (other than the bare minimal time delay introduced by observance). -> -> Well, this patchset already demonstrates that we can use a single event +>=20 +> Well, this patchset already demonstrates that we can use a single event= +=20 > callback for a rather useful purpose. Can and should are two distinct things. -> Either it makes sense to do, in which case we should share facilities as much -> as possible, or it makes no sense, in which case we should not merge it at all. +> Either it makes sense to do, in which case we should share facilities as = +much=20 +> as possible, or it makes no sense, in which case we should not merge it a= +t all. And I'm arguing we should _not_. Observing is radically different from Affecting, at the very least the two things should have different permission schemes. We should not confuse these two matters. -> > If you want another entity that is more active, please invent a new name for -> > it and create a new subsystem for them, now you could have these active -> > entities also have an (automatic) passive event side, but that's some detail. -> +> > If you want another entity that is more active, please invent a new nam= +e for=20 +> > it and create a new subsystem for them, now you could have these active= +=20 +> > entities also have an (automatic) passive event side, but that's some d= +etail. +>=20 > Why should we have two callbacks next to each other: -> +>=20 > event_vfs_getname(result); -> result = check_event_vfs_getname(result); -> +> result =3D check_event_vfs_getname(result); +>=20 > if one could do it all? Did you actually read the bit where I said that check_event_* (although diff --git a/a/content_digest b/N1/content_digest index 4fd8bba..5750c99 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -11,80 +11,87 @@ "Subject\0Re: [PATCH 3/5] v2 seccomp_filters: Enable ftrace-based system call filtering\0" "Date\0Fri, 13 May 2011 14:39:30 +0200\0" "To\0Ingo Molnar <mingo@elte.hu>\0" - "Cc\0James Morris <jmorris@namei.org>" - Will Drewry <wad@chromium.org> - linux-kernel@vger.kernel.org - Steven Rostedt <rostedt@goodmis.org> + "Cc\0linux-mips@linux-mips.org" + linux-sh@vger.kernel.org Frederic Weisbecker <fweisbec@gmail.com> - Eric Paris <eparis@redhat.com> - kees.cook@canonical.com - agl@chromium.org - Serge E. Hallyn <serge@hallyn.com> - Ingo Molnar <mingo@redhat.com> - Andrew Morton <akpm@linux-foundation.org> - Tejun Heo <tj@kernel.org> - Michal Marek <mmarek@suse.cz> + Heiko Carstens <heiko.carstens@de.ibm.com> Oleg Nesterov <oleg@redhat.com> - Roland McGrath <roland@redhat.com> - Jiri Slaby <jslaby@suse.cz> David Howells <dhowells@redhat.com> - Russell King <linux@arm.linux.org.uk> - Michal Simek <monstr@monstr.eu> - Ralf Baechle <ralf@linux-mips.org> - Benjamin Herrenschmidt <benh@kernel.crashing.org> Paul Mackerras <paulus@samba.org> - Martin Schwidefsky <schwidefsky@de.ibm.com> - Heiko Carstens <heiko.carstens@de.ibm.com> - linux390@de.ibm.com - Paul Mundt <lethal@linux-sh.org> - David S. Miller <davem@davemloft.net> - Thomas Gleixner <tglx@linutronix.de> + Eric Paris <eparis@redhat.com> H. Peter Anvin <hpa@zytor.com> + sparclinux@vger.kernel.org + Jiri Slaby <jslaby@suse.cz> + linux-s390@vger.kernel.org + Russell King <linux@arm.linux.org.uk> x86@kernel.org + James Morris <jmorris@namei.org> + Linus Torvalds <torvalds@linux-foundation.org> + Ingo Molnar <mingo@redhat.com> linux-arm-kernel@lists.infradead.org + kees.cook@canonical.com + Serge E. Hallyn <serge@hallyn.com> microblaze-uclinux@itee.uq.edu.au - linux-mips@linux-mips.org + Steven Rostedt <rostedt@goodmis.org> + Martin Schwidefsky <schwidefsky@de.ibm.com> + Thomas Gleixner <tglx@linutronix.de> + Roland McGrath <roland@redhat.com> + Michal Marek <mmarek@suse.cz> + Michal Simek <monstr@monstr.eu> + Will Drewry <wad@chromium.org> linuxppc-dev@lists.ozlabs.org - linux-s390@vger.kernel.org - linux-sh@vger.kernel.org - sparclinux@vger.kernel.org - " Linus Torvalds <torvalds@linux-foundation.org>\0" + linux-kernel@vger.kernel.org + Ralf Baechle <ralf@linux-mips.org> + Paul Mundt <lethal@linux-sh.org> + Tejun Heo <tj@kernel.org> + linux390@de.ibm.com + Andrew Morton <akpm@linux-foundation.org> + agl@chromium.org + " David S. Miller <davem@davemloft.net>\0" "\00:1\0" "b\0" "On Fri, 2011-05-13 at 14:26 +0200, Ingo Molnar wrote:\n" "> * Peter Zijlstra <peterz@infradead.org> wrote:\n" - "> \n" + ">=20\n" "> > On Fri, 2011-05-13 at 14:10 +0200, Ingo Molnar wrote:\n" - "> > > err = event_vfs_getname(result);\n" - "> > \n" - "> > I really think we should not do this. Events like we have them should be \n" - "> > inactive, totally passive entities, only observe but not affect execution \n" + "> > > err =3D event_vfs_getname(result);\n" + "> >=20\n" + "> > I really think we should not do this. Events like we have them should b=\n" + "e=20\n" + "> > inactive, totally passive entities, only observe but not affect executi=\n" + "on=20\n" "> > (other than the bare minimal time delay introduced by observance).\n" - "> \n" - "> Well, this patchset already demonstrates that we can use a single event \n" + ">=20\n" + "> Well, this patchset already demonstrates that we can use a single event=\n" + "=20\n" "> callback for a rather useful purpose.\n" "\n" "Can and should are two distinct things.\n" "\n" - "> Either it makes sense to do, in which case we should share facilities as much \n" - "> as possible, or it makes no sense, in which case we should not merge it at all.\n" + "> Either it makes sense to do, in which case we should share facilities as =\n" + "much=20\n" + "> as possible, or it makes no sense, in which case we should not merge it a=\n" + "t all.\n" "\n" "And I'm arguing we should _not_. Observing is radically different from\n" "Affecting, at the very least the two things should have different\n" "permission schemes. We should not confuse these two matters.\n" "\n" - "> > If you want another entity that is more active, please invent a new name for \n" - "> > it and create a new subsystem for them, now you could have these active \n" - "> > entities also have an (automatic) passive event side, but that's some detail.\n" - "> \n" + "> > If you want another entity that is more active, please invent a new nam=\n" + "e for=20\n" + "> > it and create a new subsystem for them, now you could have these active=\n" + "=20\n" + "> > entities also have an (automatic) passive event side, but that's some d=\n" + "etail.\n" + ">=20\n" "> Why should we have two callbacks next to each other:\n" - "> \n" + ">=20\n" "> \tevent_vfs_getname(result);\n" - "> \tresult = check_event_vfs_getname(result);\n" - "> \n" + "> \tresult =3D check_event_vfs_getname(result);\n" + ">=20\n" "> if one could do it all?\n" "\n" "Did you actually read the bit where I said that check_event_* (although\n" I still think that name sucks) could imply a matching event_*? -c9e0640dc4eb0d0160efe2682f6b671905de2611462dc2cfd2d974a9c37fb13e +1cca12c65fe473e9fa6cadeba796d3b58004a7f75f66fcf834cbdff963f96643
diff --git a/a/content_digest b/N2/content_digest index 4fd8bba..059ea00 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -7,48 +7,10 @@ "ref\020110513121034.GG21022@elte.hu\0" "ref\01305289146.2466.8.camel@twins\0" "ref\020110513122646.GA3924@elte.hu\0" - "From\0Peter Zijlstra <peterz@infradead.org>\0" - "Subject\0Re: [PATCH 3/5] v2 seccomp_filters: Enable ftrace-based system call filtering\0" + "From\0peterz@infradead.org (Peter Zijlstra)\0" + "Subject\0[PATCH 3/5] v2 seccomp_filters: Enable ftrace-based system call filtering\0" "Date\0Fri, 13 May 2011 14:39:30 +0200\0" - "To\0Ingo Molnar <mingo@elte.hu>\0" - "Cc\0James Morris <jmorris@namei.org>" - Will Drewry <wad@chromium.org> - linux-kernel@vger.kernel.org - Steven Rostedt <rostedt@goodmis.org> - Frederic Weisbecker <fweisbec@gmail.com> - Eric Paris <eparis@redhat.com> - kees.cook@canonical.com - agl@chromium.org - Serge E. Hallyn <serge@hallyn.com> - Ingo Molnar <mingo@redhat.com> - Andrew Morton <akpm@linux-foundation.org> - Tejun Heo <tj@kernel.org> - Michal Marek <mmarek@suse.cz> - Oleg Nesterov <oleg@redhat.com> - Roland McGrath <roland@redhat.com> - Jiri Slaby <jslaby@suse.cz> - David Howells <dhowells@redhat.com> - Russell King <linux@arm.linux.org.uk> - Michal Simek <monstr@monstr.eu> - Ralf Baechle <ralf@linux-mips.org> - Benjamin Herrenschmidt <benh@kernel.crashing.org> - Paul Mackerras <paulus@samba.org> - Martin Schwidefsky <schwidefsky@de.ibm.com> - Heiko Carstens <heiko.carstens@de.ibm.com> - linux390@de.ibm.com - Paul Mundt <lethal@linux-sh.org> - David S. Miller <davem@davemloft.net> - Thomas Gleixner <tglx@linutronix.de> - H. Peter Anvin <hpa@zytor.com> - x86@kernel.org - linux-arm-kernel@lists.infradead.org - microblaze-uclinux@itee.uq.edu.au - linux-mips@linux-mips.org - linuxppc-dev@lists.ozlabs.org - linux-s390@vger.kernel.org - linux-sh@vger.kernel.org - sparclinux@vger.kernel.org - " Linus Torvalds <torvalds@linux-foundation.org>\0" + "To\0linux-arm-kernel@lists.infradead.org\0" "\00:1\0" "b\0" "On Fri, 2011-05-13 at 14:26 +0200, Ingo Molnar wrote:\n" @@ -87,4 +49,4 @@ "Did you actually read the bit where I said that check_event_* (although\n" I still think that name sucks) could imply a matching event_*? -c9e0640dc4eb0d0160efe2682f6b671905de2611462dc2cfd2d974a9c37fb13e +3498aadabf86130cd0ca2be4f4f3e047268d5797c1408aa3832513b9b052c039
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.