Path: csiph.com!feeder.erje.net!2.eu.feeder.erje.net!news.etla.org!aioe.org!bofh.it!news.nic.it!robomod From: Ingo Molnar Newsgroups: linux.kernel Subject: Re: [PATCHv2] x86/vdso: add mremap hook to vm_special_mapping Date: Fri, 15 Apr 2016 11:20:04 +0200 Message-ID: References: Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=RUC7vq37dOlSNd3R+RkJEMrYw08LYgG7CcLFPO9wM6E=; b=pjcmeYsm5cY3fpZpCCPy3GnMmskFtpTXdAJQEF3aERfM2n4Riz4i9Ig9uhV7uUpxRK Miu9t1E4DC+UFQnF1CBxOu7+ckwhSExmcDU10sIJmhxIls8PO9Pl2YWVx9IgfeHCB+b7 g473pPp8pwXNWQXMQnOQ0xJjM0ThHSt+tEgg1I8SxgCkIXAGygAi2KZ+3sWihZS8q0ku 7P9ROBtVfdRFek27aKOAZJ/SwMQEyu1ZqGJ1ZfOMgexbZa+AqLtaIc1Axq64l64i218L v6LGXLAQUGICYDmhwm+A/kHTKsa78db5xe2bLOeIpojoBwfZyN9dEnxdrjvhR5yVwl/j kaFg== X-Google-Dkim-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:sender:date:from:to:cc:subject:message-id :references:mime-version:content-disposition:in-reply-to:user-agent; bh=RUC7vq37dOlSNd3R+RkJEMrYw08LYgG7CcLFPO9wM6E=; b=cdjEvthTfoW+zQ+31A4kvrzB3V37k0hKVpW+SmnXtPmfEKCh4PqHe3G9OG3ZEwjAcg Qlj6qpK6rqDmf1BbxHgav23dHwPe5sx/RvPoBhlaEVWSmsQVj6Xp8vpCMwfxFiQF5P5W 34DYrCuuwB2LUNEB58TNO4D2swdIEY6PGaIkMCadRPhzBb4CF29VCi4h3PHmwl9O0+K5 O4OQyxYKidZOS5vDaK8nTPCb0c+ks0g6N7ZWkK0HQ6f7J26Cfr55yAsIPSyPgqGkkYwZ R38FwTi+fZygomIo/wYq33RY3zVezWBOenTmJh8CWxpQS9MAAx9l2BcX5gfk8EH8WE5O sn2w== X-Gm-Message-State: AOPr4FXP89XxgPMA+/Af3Y8hv0fC6MN1L8gOIfSse1BOy4qZZY45o7u1H/VYzeke93fazQ== X-Received: by 10.194.202.133 with SMTP id ki5mr20596762wjc.27.1460711942243; Fri, 15 Apr 2016 02:19:02 -0700 (PDT) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.23 (2014-03-12) Sender: robomod@news.nic.it List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Approved: robomod@news.nic.it Lines: 17 Organization: linux.* mail to news gateway X-Original-Cc: Dmitry Safonov , "linux-kernel@vger.kernel.org" , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , X86 ML , Andrew Morton , "linux-mm@kvack.org" , Dmitry Safonov <0x7f454c46@gmail.com> X-Original-Date: Fri, 15 Apr 2016 11:18:59 +0200 X-Original-Message-ID: <20160415091859.GA10167@gmail.com> X-Original-References: <1460388169-13340-1-git-send-email-dsafonov@virtuozzo.com> <1460651571-10545-1-git-send-email-dsafonov@virtuozzo.com> X-Original-Sender: linux-kernel-owner@vger.kernel.org Xref: csiph.com linux.kernel:1379609 * Andy Lutomirski wrote: > > + if (regs->ip == (unsigned long)current->mm->context.vdso + > > + vdso_image_32.sym_int80_landing_pad > > +#ifdef CONFIG_IA32_EMULATION > > + && current_thread_info()->status & TS_COMPAT > > +#endif > > Instead of ifdef, use the (grossly misnamed) is_ia32_task() helper for > this, please. Please also let's do the rename. Thanks, Ingo