From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oleg Nesterov Subject: Re: [PATCH v3 1/3] fs/binfmts: Add a name field to the binfmt struct Date: Sat, 3 Aug 2013 18:41:03 +0200 Message-ID: <20130803164103.GA32568@redhat.com> References: <1374766845-13565-1-git-send-email-zml@linux.vnet.ibm.com> <1375485703-4077-2-git-send-email-zml@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: akpm@linux-foundation.org, viro@zeniv.linux.org.uk, linux-kernel@vger.kernel.org, linux-fsdevel@vger.kernel.org, dan.carpenter@oracle.com, Zach Levis To: Zach Levis Return-path: Received: from mx1.redhat.com ([209.132.183.28]:49093 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751417Ab3HCQqc (ORCPT ); Sat, 3 Aug 2013 12:46:32 -0400 Content-Disposition: inline In-Reply-To: <1375485703-4077-2-git-send-email-zml@linux.vnet.ibm.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On 08/02, Zach Levis wrote: > > static struct linux_binfmt elf_format = { > + .name = "ELF", Once again, I think it would be better if elf_format and compat_elf_format have the different names. But with this patch they are both "ELF". Oleg.