From mboxrd@z Thu Jan 1 00:00:00 1970 From: WANG Cong Subject: Re: [PATCH v2 1/3] coredump: use get_task_comm for %e filename format Date: Thu, 12 May 2011 10:49:10 +0000 (UTC) Message-ID: References: <1305181093-20871-1-git-send-email-jslaby@suse.cz> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Cc: linux-fsdevel@vger.kernel.org,linux-kernel@vger.kernel.org To: linux-mm@kvack.org Return-path: Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org On Thu, 12 May 2011 08:18:11 +0200, Jiri Slaby wrote: > We currently access current->comm directly. As we have > prctl(PR_SET_NAME), we need the access be protected by task_lock. This > is exactly what get_task_comm does, so use it. >=20 > I'm not 100% convinced prctl(PR_SET_NAME) may be called at the time of > core dump, but the locking won't hurt. Note that siglock is not held in > format_corename. John Stultz is working on some patches to convert get_task_common() to printk %ptc, you will not need to worry about the locking issue. Thanks. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter= .ca/ Don't email: email@kvack.org