From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from terminus.zytor.com ([2001:1868:205::10] helo=mail.zytor.com) by merlin.infradead.org with esmtps (Exim 4.80.1 #2 (Red Hat Linux)) id 1Ug8eK-0005TH-R7 for kexec@lists.infradead.org; Sat, 25 May 2013 07:12:21 +0000 In-Reply-To: <51A02E50.5020507@gmail.com> References: <51A02E50.5020507@gmail.com> MIME-Version: 1.0 Subject: Re: [PATCH 0/7] Remove unused /dev/oldmem interface From: "H. Peter Anvin" Date: Sat, 25 May 2013 00:11:33 -0700 Message-ID: List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "kexec" Errors-To: kexec-bounces+dwmw2=twosheds.infradead.org@lists.infradead.org To: Zhang Yanfei , "Eric W. Biederman" , Vivek Goyal , Andrew Morton , Simon Horman Cc: "kexec@lists.infradead.org" , "linux-kernel@vger.kernel.org" Awesome! Zhang Yanfei wrote: >/dev/oldmem provides the interface for us to access the "old memory" in >the dump-capture kernel. Unfortunately, no one actually uses this >interface. > >And this interface could actually cause some real problems if used on >ia64 >where the cached/uncached accesses are mixed. See the discussion from >the link: https://lkml.org/lkml/2013/4/12/386. > >So Eric suggested that we should remove /dev/oldmem as an unused piece >of >code. > >Besides, we used a global variable saved_max_pfn to let the capture >kernel >know the amount of memory that the previous kernel used. And for almost >all >architectures (except x86. In x86, saved_max_pfn is used by >detect_calgary()), >the only user of this variable is the read_oldmem interface of >/dev/oldmem, so >also remove the setting for saved_max_pfn in those architectures. > >Zhang Yanfei (7): > /dev/oldmem: Remove this interface > Documentation/devices.txt: Remove /dev/oldmem description > Documentation/kdump/kdump.txt: Remove /dev/oldmem description > mips: Remove savemaxmem parameter setup > powerpc: Remove savemaxmem parameter setup > ia64: Remove setting for saved_max_pfn > s390: Remove setting for saved_max_pfn > > Documentation/devices.txt | 2 - > Documentation/kdump/kdump.txt | 31 +++++-------------------- > arch/ia64/kernel/efi.c | 5 ---- > arch/mips/kernel/crash_dump.c | 10 -------- > arch/powerpc/kernel/crash_dump.c | 10 -------- > arch/s390/kernel/setup.c | 4 --- >drivers/char/mem.c | 47 >-------------------------------------- > 7 files changed, 6 insertions(+), 103 deletions(-) -- Sent from my mobile phone. Please excuse brevity and lack of formatting. _______________________________________________ kexec mailing list kexec@lists.infradead.org http://lists.infradead.org/mailman/listinfo/kexec