Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751997Ab2JWDLz (ORCPT ); Mon, 22 Oct 2012 23:11:55 -0400 Received: from mail-pb0-f46.google.com ([209.85.160.46]:36574 "EHLO mail-pb0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750822Ab2JWDLy (ORCPT ); Mon, 22 Oct 2012 23:11:54 -0400 Date: Mon, 22 Oct 2012 20:12:22 -0700 From: Greg KH To: Arve Hj?nnev?g Cc: xiaobing.tu@intel.com, ccross@android.com, davej@redhat.com, akpm@linux-foundation.org, mingo@elte.hu, rusty@rustcorp.com.au, a.p.zijlstra@chello.nl, linux-kernel@vger.kernel.org, rostedt@goodmis.org, di.zhang@intel.com, xindong.ma@intel.com, alex.zuo@intel.com Subject: Re: [PATCH 1/2] Staging: android: binder: Fix memory leak on thread/process exit Message-ID: <20121023031222.GA22095@kroah.com> References: <20121015235836.GA3214@kroah.com> <1350347562-32643-1-git-send-email-arve@android.com> <20121016071136.GB5172@kroah.com> <20121022200047.GA18175@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3589 Lines: 74 On Mon, Oct 22, 2012 at 05:58:08PM -0700, Arve Hj?nnev?g wrote: > On Mon, Oct 22, 2012 at 1:00 PM, Greg KH wrote: > > On Tue, Oct 16, 2012 at 03:45:20PM -0700, Arve Hj?nnev?g wrote: > >> On Tue, Oct 16, 2012 at 12:11 AM, Greg KH wrote: > >> > On Mon, Oct 15, 2012 at 05:32:41PM -0700, Arve Hj?nnev?g wrote: > >> >> If a thread or process exited while a reply, one-way transaction or > >> >> death notification was pending, the struct holding the pending work > >> >> was leaked. > >> >> > >> >> Change-Id: I2eaafaba1c0ecda3ec0872d449dc16d0721c21e7 > >> > > >> > What is this field? Please don't include this in kernel patches, it > >> > forces me to edit the patch by hand :( > >> > > >> > >> It is a tag generated a git hook to uniquely identify a change through > >> multiple revisions of that change. If the tag does not already exist > >> the hook adds it. I removed it from the reposted patches as you > >> requested, but have you considered leaving it in? It is a useful > >> search token both for finding different revisions of a patch and for > >> finding which branch a change has been cherry-picked into. > > > > Sorry, I was being facetious, I knew what it really was, but the point > > was, why would I? > > > > So you can use it. If, for instance, you want to see which branches > you cherry-picked this change into you can run "git rev-list --all > --grep I2eaafaba1c0ecda3ec0872d449dc16d0721c21e7 | xargs -rl git > branch -a --contains". If it was more widely used, it would also be > useful for finding mailing list discussion of multiple versions of > patches (even when the title of the patch changes). Yes, if you want a unique identifier for a patch in order to try to track it through trees, you can do something like that (hint, see Alan Stern's patches, he has his own identifier for that.) But don't use the gerrit id for this, it's not useful and it is confusing. We have discussed this in the past, see the lkml archives for details. > > As has been stated before, you can include information like this in your > > patch, but you MUST reference it properly so that others can be able to > > figure out what you mean. A random Change-Id value means nothing given > > that lots of different groups use gerrit. You need a url, or a pointer > > to which gerrit instance this refers to before you can include it. > > > > There is no single url. The tag was generated by a local git hook > before it was uploaded anywhere. While gerrit looks for this tag and > use it when it is there, the same change can be uploaded to different > gerrit instances. While it would be useful to also include urls to the > gerrit instance(s) that a patch was discussed on, the Change-Id is > useful by itself. > > > Same goes for bugzilla entries, if you look in the kernel changelog, we > > reference lots of different ones, but we use urls to determine which one > > we are talking about (suse, red hat, kernel.org, etc.) > > > > Those IDs appear to be local to a specific server, so the id without > the url would not be very useful. Ok, then I don't recommend it be used at all, as we have a unique identifier, the git commit id in Linus's tree. That id does follow the patch around if it goes into any stable kernel tree, so let's stick with it. thanks, greg k-h -- 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/