Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756326Ab1F0E3i (ORCPT ); Mon, 27 Jun 2011 00:29:38 -0400 Received: from mail-vx0-f174.google.com ([209.85.220.174]:43860 "EHLO mail-vx0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756134Ab1F0E3O convert rfc822-to-8bit (ORCPT ); Mon, 27 Jun 2011 00:29:14 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=googlemail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=Jxm8n3qKRPECe+S9sA1VnOusrGOz3ZMpnWf7Bb87EMdY2O6TXDpUqxoyA/kPzbiZT3 DlKDBRVUljuwZ+W8Sd+GEMQoDOKx9J0bImH/Aa+B6TY01U30youXB96gkWUx+hwqrS8D w9BhVuD89V1RFnxlQqNaqNEj2ncNZ9Qw8aSc0= MIME-Version: 1.0 In-Reply-To: References: <1307642718-22257-1-git-send-email-minipli@googlemail.com> <20110609155630.0f734351.akpm@linux-foundation.org> <1307711337-4042-1-git-send-email-minipli@googlemail.com> <20110610134821.GL24424@n2100.arm.linux.org.uk> Date: Mon, 27 Jun 2011 06:29:12 +0200 Message-ID: Subject: Re: [PATCH] arm, exec: remove redundant set_fs(USER_DS) From: Mathias Krause To: Russell King - ARM Linux Cc: Andrew Morton , Linus Torvalds , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1016 Lines: 23 On Fri, Jun 10, 2011 at 3:53 PM, Mathias Krause wrote: > On Fri, Jun 10, 2011 at 3:48 PM, Russell King - ARM Linux > wrote: >> On Fri, Jun 10, 2011 at 03:08:57PM +0200, Mathias Krause wrote: >>> The address limit is already set in flush_old_exec() so this >>> set_fs(USER_DS) is redundant. >> >> Please show where and how this is done. ?I've looked and can't see >> any equivalent call to set_fs() in flush_old_exec(). > > Before dac853a (exec: delay address limit change until point of no > return) it was done in search_binary_handler(), now it is done in > flush_old_exec(). Either way set_fs(USER_DS) was/gets called before > start_thread() so the call there is redundant. Russell, any new opinion on this? Mathias -- 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/