Re: [PATCH -queue v0 1/6] [loongson] add basic loongson-2f support

From: Ralf Baechle
Date: Wed Nov 04 2009 - 15:14:18 EST


On Wed, Nov 04, 2009 at 11:23:46PM +0800, Wu Zhangjin wrote:

> > We have other systems where 32-bit kernel support is just remarkably ugly.
> > We've dropped 32-bit support for the SGI IP32 aka O2 - nobody seems to even
> > have really noticed that. The Sibyte systems would be good candidates to do
> > the same as accesses to outside the 32-bit address space are needed very
> > frequently.
> >
>
> So, we really remove the 32bit support?
>
> 1312 config CPU_LOONGSON2
> 1313 bool
> 1314 select CPU_SUPPORTS_32BIT_KERNEL --> remove it?
> 1315 select CPU_SUPPORTS_64BIT_KERNEL
> 1316 select CPU_SUPPORTS_HIGHMEM
>
> If you all agree, I will send a new patch to remove the above line and
> resend the corresponding patches without 32bit support, and removed the
> relative CONFIG_64BIT lines in the patches too.

If you need highmem with 32-bit (and with Loongson systems I assume that
virtually all systems will have enough RAM to require that) then you're
almost certainly better off going 64-bit. Highmem takes a performance toll
which for some workloads can be very significant. And while highmem won't
go away any time soon it's nothing kernel performance is being tuned for,
so it's only going to get worse into the future so I'd not waste time on
highmem unless I have to.

Ralf
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/