linux-kernel - linux-next: manual merge of the arm-mpidr tree with the arm tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 19 Jun 2013 10:44:09 +1000
From:	Stephen Rothwell <sfr@...b.auug.org.au>
To:	Lorenzo Pieralisi <lorenzo.pieralisi@....com>
Cc:	linux-next@...r.kernel.org, linux-kernel@...r.kernel.org,
	Will Deacon <will.deacon@....com>,
	Jonathan Austin <jonathan.austin@....com>,
	Russell King <rmk@....linux.org.uk>
Subject: linux-next: manual merge of the arm-mpidr tree with the arm tree

Hi Lorenzo,

Today's linux-next merge of the arm-mpidr tree got a conflict in
arch/arm/kernel/suspend.c between commit aa1aadc3305c ("ARM: suspend: fix
CPU suspend code for !CONFIG_MMU configurations") from the arm tree and
commit 3fed6a1e3bf0 ("ARM: kernel: implement stack pointer save array
through MPIDR hashing") from the arm-mpidr tree.

I fixed it up (see below) and can carry the fix as necessary (no action
is required).

P.S. that arm tree commit above has no Signed-off-by from its
committer :-(
-- 
Cheers,
Stephen Rothwell                    sfr@...b.auug.org.au

diff --cc arch/arm/kernel/suspend.c
index 38a5067,17d02f6..0000000
--- a/arch/arm/kernel/suspend.c
+++ b/arch/arm/kernel/suspend.c
@@@ -82,3 -49,48 +85,20 @@@ void __cpu_suspend_save(u32 *ptr, u32 p
  	outer_clean_range(virt_to_phys(save_ptr),
  			  virt_to_phys(save_ptr) + sizeof(*save_ptr));
  }
+ 
 -/*
 - * Hide the first two arguments to __cpu_suspend - these are an implementation
 - * detail which platform code shouldn't have to know about.
 - */
 -int cpu_suspend(unsigned long arg, int (*fn)(unsigned long))
 -{
 -	struct mm_struct *mm = current->active_mm;
 -	int ret;
 -
 -	if (!idmap_pgd)
 -		return -EINVAL;
 -
 -	/*
 -	 * Provide a temporary page table with an identity mapping for
 -	 * the MMU-enable code, required for resuming.  On successful
 -	 * resume (indicated by a zero return code), we need to switch
 -	 * back to the correct page tables.
 -	 */
 -	ret = __cpu_suspend(arg, fn);
 -	if (ret == 0) {
 -		cpu_switch_mm(mm->pgd, mm);
 -		local_flush_bp_all();
 -		local_flush_tlb_all();
 -	}
 -
 -	return ret;
 -}
 -
+ extern struct sleep_save_sp sleep_save_sp;
+ 
+ static int cpu_suspend_alloc_sp(void)
+ {
+ 	void *ctx_ptr;
+ 	/* ctx_ptr is an array of physical addresses */
+ 	ctx_ptr = kcalloc(mpidr_hash_size(), sizeof(u32), GFP_KERNEL);
+ 
+ 	if (WARN_ON(!ctx_ptr))
+ 		return -ENOMEM;
+ 	sleep_save_sp.save_ptr_stash = ctx_ptr;
+ 	sleep_save_sp.save_ptr_stash_phys = virt_to_phys(ctx_ptr);
+ 	sync_cache_w(&sleep_save_sp);
+ 	return 0;
+ }
+ early_initcall(cpu_suspend_alloc_sp);

Content of type "application/pgp-signature" skipped