Re: PC-9800 patch for 2.5.47-ac4: not merged yet (15/15) SMP

[Andrey Panin]

--4bRzO86E/ozDv8r1
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On =D0=9F=D1=82=D0=BD, =D0=9D=D0=BE=D1=8F 15, 2002 at 10:08:41 +0900, Osamu=
 Tomita wrote:
> This is for SMP support.
=20
>  void __init find_smp_config (void)
>  {
> +#ifndef CONFIG_PC9800
>  	unsigned int address;
> +#endif
> =20
>  	/*
>  	 * FIXME: Linux assumes you have 640K of base ram..
> @@ -762,6 +793,7 @@
>  		smp_scan_config(639*0x400,0x400) ||
>  			smp_scan_config(0xF0000,0x10000))
>  		return;
> +#ifndef CONFIG_PC9800	/* PC-9800 has no EBDA area? */
>  	/*
>  	 * If it is an SMP machine we should know now, unless the
>  	 * configuration is in an EISA/MCA bus machine with an
> @@ -784,6 +816,7 @@
>  	smp_scan_config(address, 0x400);
>  	if (smp_found_config)
>  		printk(KERN_WARNING "WARNING: MP table in the EBDA can be UNSAFE, cont=
act [e-mail] if you experience SMP problems!\n");
> +#endif
>  }

Can you redo this fragment this way ?

#ifndef CONFIG_PC9800	/* PC-9800 has no EBDA area? */
	{
		unsigned int address =3D *(unsigned short *)phys_to_virt(0x40E);
		address <<=3D 4;
		smp_scan_config(address, 0x400);
		if (smp_found_config)
			printk(KERN_WARNING "WARNING: MP table in the EBDA can be UNSAFE, contac=
t [e-mail] if you experience SMP problems!\n");
	}
#endif

IMHO this is better way (one #ifndef less)

--=20
Andrey Panin            | Embedded systems software developer
[e-mail]        | PGP key: wwwkeys.eu.pgp.net
--4bRzO86E/ozDv8r1
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.1 (GNU/Linux)
Comment: For info see http://www.gnupg.org

iD4DBQE91PakBm4rlNOo3YgRAhYfAJUTxJBb7LzspoNt9nEuu8TSvrlIAJ4lPCB9
LbsYguqDan/2GuxKQhtmVA==
=BOsk
-----END PGP SIGNATURE-----

--4bRzO86E/ozDv8r1--