On Wednesday 23 April 2014, Geert Uytterhoeven wrote: > On Wed, Apr 23, 2014 at 4:47 AM, Ley Foon Tan <lftan@xxxxxxxxxx> wrote: > > On Tue, Apr 22, 2014 at 8:33 PM, Arnd Bergmann <arnd@xxxxxxxx> wrote: > >> On Friday 18 April 2014, Ley Foon Tan wrote: > >>> +static const struct file_operations misalign_fops = { > >>> + .open = proc_misaligned_open, > >>> + .read = seq_read, > >>> + .llseek = seq_lseek, > >>> + .release = single_release, > >>> + .write = proc_misaligned_write, > >>> +}; > >>> +#endif /* CONFIG_PROC_FS */ > >> > >> We really shouldn't be doing new architecture specific procfs files > >> any more. I suggest you drop this one for now, and add back the > >> functionality using perf or ftrace at a later point. > > > > Okay, will remove this. > > MIPS and powerpc handle this through debugfs, cfr. > arch/mips/kernel/unaligned.c and arch/powerpc/kernel/traps.c. Ah, right, that would work too. Being able to use perf from user space sounds more flexible though. It's certainly something that can be decided later. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-arch" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html