Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1163273Ab3DFJrz (ORCPT ); Sat, 6 Apr 2013 05:47:55 -0400 Received: from intranet.asianux.com ([58.214.24.6]:57600 "EHLO intranet.asianux.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1163260Ab3DFJry (ORCPT ); Sat, 6 Apr 2013 05:47:54 -0400 X-Spam-Score: -100.8 Message-ID: <515FEF2C.4080705@asianux.com> Date: Sat, 06 Apr 2013 17:47:24 +0800 From: Chen Gang User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 MIME-Version: 1.0 To: =?UTF-8?B?QXJ2ZSBIasO4bm5ldsOlZw==?= CC: Greg KH , Dan Carpenter , Al Viro , Mirsal Ennaime , LKML Subject: Re: [PATCH] Staging: Android: looping issue, need break when get value firstly. References: <515E85C5.7010706@asianux.com> <20130405220156.GC31680@kroah.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1374 Lines: 37 On 2013年04月06日 07:48, � wrote: > Theoretically this should not change the behavior. The purpose of this > code it to make sure only thread per process is part of a transaction > stack, so if it finds more than one transaction with a matching > process, they should all point to the same thread object. I think a > better change description is needed though. after reading code again, I think: in our condition, one proc can be related with multiple threads. can you provide more information for: why can we be sure "they should all point to the same thread object" ? result: if it is an issue (can not be sure all point to the same thread object) it seems need additional considerations for it. what my fix seems incorrect, either. (for its transaction processing, it seems not should obey stack rules). else (can be sure) my patch seems minor (only can increasing a little performance). it is not quite suitable to spend much time to test a minor patch !! so whether what will happen, my patch seems obsoleted !! :-( -- Chen Gang Asianux Corporation -- 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/