Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753145Ab2FHEdt (ORCPT ); Fri, 8 Jun 2012 00:33:49 -0400 Received: from e3.ny.us.ibm.com ([32.97.182.143]:59941 "EHLO e3.ny.us.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751546Ab2FHEds (ORCPT ); Fri, 8 Jun 2012 00:33:48 -0400 Date: Fri, 8 Jun 2012 10:03:19 +0530 From: Ananth N Mavinakayanahalli To: Srikar Dronamraju Cc: Ingo Molnar , Peter Zijlstra , linuxppc-dev@lists.ozlabs.org, lkml , Paul Mackerras , benh@kernel.crashing.org, Anton Blanchard , Ingo Molnar , oleg@redhat.com Subject: Re: [PATCH 1/2] uprobes: Pass probed vaddr to arch_uprobe_analyze_insn() Message-ID: <20120608043319.GA13409@in.ibm.com> Reply-To: ananth@in.ibm.com References: <20120606091950.GB6745@in.ibm.com> <1338974632.2749.87.camel@twins> <20120606093744.GB29580@in.ibm.com> <20120606094014.GD9495@gmail.com> <20120606114423.GA32094@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20120606114423.GA32094@linux.vnet.ibm.com> User-Agent: Mutt/1.5.17 (2007-11-01) X-Content-Scanned: Fidelis XPS MAILER x-cbid: 12060804-8974-0000-0000-000009DE4CE2 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1332 Lines: 34 On Wed, Jun 06, 2012 at 05:14:23PM +0530, Srikar Dronamraju wrote: > * Ingo Molnar [2012-06-06 11:40:15]: > > > > > * Ananth N Mavinakayanahalli wrote: > > > > > On Wed, Jun 06, 2012 at 11:23:52AM +0200, Peter Zijlstra wrote: > > > > On Wed, 2012-06-06 at 14:49 +0530, Ananth N Mavinakayanahalli wrote: > > > > > +int arch_uprobe_analyze_insn(struct arch_uprobe *auprobe, struct mm_struct *mm, loff_t vaddr) > > > > > > > > Don't we traditionally use unsigned long to pass vaddrs? > > > > > > Right. But the vaddr we pass here is vma_info->vaddr which is loff_t. > > > I guess I should've made that clear in the patch description. > > > > Why not fix struct vma_info's vaddr type? > > > > Calculating and comparing vaddr results either uses variables of type loff_t. > To avoid typecasting and avoid overflow at each of these places, we used > loff_t. > > Ananth, install_breakpoint() already has a variable of type addr of type > unsigned long. Why dont you use addr instead of vaddr. Ok, makes sense. I'll change it in v2. Ananth -- 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/