From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Thu, 15 May 2014 20:31:14 +0200 (CEST) Received: from b.ns.miles-group.at ([95.130.255.144]:9062 "EHLO radon.swed.at" rhost-flags-OK-OK-OK-FAIL) by eddie.linux-mips.org with ESMTP id S6822318AbaEOSbGc1nMX (ORCPT ); Thu, 15 May 2014 20:31:06 +0200 Received: (qmail 30739 invoked by uid 89); 15 May 2014 18:31:05 -0000 Received: by simscan 1.3.1 ppid: 30732, pid: 30735, t: 0.0930s scanners: attach: 1.3.1 clamav: 0.96.5/m: Received: from unknown (HELO ?192.168.0.11?) (richard@nod.at@213.47.235.169) by radon.swed.at with ESMTPA; 15 May 2014 18:31:05 -0000 Message-ID: <537507E8.1010600@nod.at> Date: Thu, 15 May 2014 20:31:04 +0200 From: Richard Weinberger User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.1.0 MIME-Version: 1.0 To: Leonid Yegoshin CC: linux-arch@vger.kernel.org, arnd@arndb.de, linux-kernel@vger.kernel.org, Ralf Baechle , John Crispin , Markos Chandras , linux-mips@linux-mips.org Subject: Re: [PATCH 16/27] mips: Use common bits from generic tlb.h References: <1400093999-18703-1-git-send-email-richard@nod.at> <1400093999-18703-17-git-send-email-richard@nod.at> <53750129.6060902@imgtec.com> In-Reply-To: <53750129.6060902@imgtec.com> X-Enigmail-Version: 1.6 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 40115 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: richard@nod.at Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips Am 15.05.2014 20:02, schrieb Leonid Yegoshin: > On 05/14/2014 11:59 AM, Richard Weinberger wrote: >> It is no longer needed to define them on our own. >> >> Cc: Ralf Baechle >> Cc: John Crispin >> Cc: Markos Chandras >> Cc: Leonid Yegoshin >> Cc: Richard Weinberger >> Cc: linux-mips@linux-mips.org >> Cc: linux-kernel@vger.kernel.org >> Signed-off-by: Richard Weinberger >> --- >> arch/mips/include/asm/tlb.h | 7 ------- >> 1 file changed, 7 deletions(-) >> >> diff --git a/arch/mips/include/asm/tlb.h b/arch/mips/include/asm/tlb.h >> index 4a23493..5ea43ca 100644 >> --- a/arch/mips/include/asm/tlb.h >> +++ b/arch/mips/include/asm/tlb.h >> @@ -10,13 +10,6 @@ >> if (!tlb->fullmm) \ >> flush_cache_range(vma, vma->vm_start, vma->vm_end); \ >> } while (0) >> -#define tlb_end_vma(tlb, vma) do { } while (0) >> -#define __tlb_remove_tlb_entry(tlb, ptep, address) do { } while (0) >> - >> -/* >> - * .. because we flush the whole mm when it fills up. >> - */ >> -#define tlb_flush(tlb) flush_tlb_mm((tlb)->mm) >> #define UNIQUE_ENTRYHI(idx) \ >> ((CKSEG0 + ((idx) << (PAGE_SHIFT + 1))) | \ > > I would like to know why these functions are eliminated (don't find any clue). > Is it just because there will be a generic one or the calls would be eliminated? There will be a generic one. See [PATCH 03/27] generic/tlb.h: Move common defines into generic tlb.h > And if there are generic - can I tune it later? Yes. You can always define your own and override the generic one. Thanks, //richard