linux-kernel - Re: [patch v2 0/3] OProfile support for System z's hardware sampling

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

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20110215172906.GG5874@erda.amd.com>
Date:	Tue, 15 Feb 2011 18:29:06 +0100
From:	Robert Richter <robert.richter@....com>
To:	Heinz Graalfs <graalfs@...ux.vnet.ibm.com>
CC:	Martin Schwidefsky <schwidefsky@...ibm.com>,
	Heiko Carstens <heiko.carstens@...ibm.com>,
	"mingo@...e.hu" <mingo@...e.hu>,
	"oprofile-list@...ts.sf.net" <oprofile-list@...ts.sf.net>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-s390@...r.kernel.org" <linux-s390@...r.kernel.org>,
	"borntraeger@...ibm.com" <borntraeger@...ibm.com>
Subject: Re: [patch v2 0/3] OProfile support for System z's hardware
 sampling

On 15.02.11 11:59:29, Heinz Graalfs wrote:

> > * Merge init.c and hwsampler_file.c, two files are bloated here and
> >   hwsampler_file.c is a bad and too long naming.
> > 
> 
> OK, I've merged all hwsampler_files.c contents into init.c
> 
> > * Rework functions in cpu_buffer.c (log_sample,
> >   __oprofile_add_ext_sample, oprofile_add_ext_hw_sample, etc.). All
> >   the (static) functions can be merged to a single functino by
> >   implementing a struct that holds all current function arguments.
> >   Something like:
> > 
> > static void __oprofile_add_ext_sample(struct *foobar fb);
> > void oprofile_add_sample(struct pt_regs * const regs, unsigned long event)
> > {
> > 	struct foobar fb = { .regs = regs, .event = event };
> > 	__oprofile_add_ext_sample(&fb);
> > }
> 
> OK, I've done this
> 
> > 
> >   The naming and description of oprofile_add_ext_hw_sample() is also
> >   not the best. As interface in include/linux/oprofile.h we could then
> >   merge oprofile_add_ext_sample() and oprofile_add_ext_hw_sample() to
> >   a single function.
> > 
> > It would be nice if you could implement this.
> 
> sure, I will do this, however I'm not sure what you exactly mean.
> Could you specify the interface in oprofile.h what you basically have in
> mind?

I mean to replace oprofile_add_ext_sample() and
oprofile_add_ext_hw_sample() by a new one. The interface would be in
the form of:

struct foobar {
       ...
}

static void __oprofile_add_ext_sample(struct *foobar fb);

Hope this makes sense. The advantage would be that we don't need to
extend the functions argument list anymore, we simply extend the
struct.

Please send me delta patches to oprofile/s390.

-Robert

-- 
Advanced Micro Devices, Inc.
Operating System Research Center

--
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/