Re: BUG: sym53c8xx_2 and highmem_io

Gérard Roudier (groudier@free.fr)
Tue, 17 Sep 2002 22:48:56 +0200 (CEST)


On 16 Sep 2002, Todd Inglett wrote:

> I've been having an interesting experience getting sym53c8xx_2 working
> on ppc64 now that highio is in place. Of course ppc64 doesn't need
> highio, and it does set blk_nohighio = 1 in setup_arch(). So the
> sym53c8xx driver works.
>
> However, sym53c8xx_2 fails because after calling scsi_register() in its
> attach it blindly slams highmem_io on (in sym_glue.c). Is this
> correct? It seems to me that it should just leave it alone since
> scsi_register already handled that.

The may-be offending line is not from me. It has been added by a kernel
maintainer at the time the corresponding feature ;-) was added to the
Linux kernel.

Note that this let me think that it may well have been correct at that
time, at least.;)

The greatest guru for all this kinds of alchemy that address complex
beyond expectation architectures is David S. Miller IMO. You may submit
him your suggestion or just sent it to the linux Kernel list.

> I might be misunderstanding something here. Is there anything else a
> 64-bit arch must do for highio? I found we also weren't setting max_pfn
> which seemed bad...though maybe irrelevant.

highio very probably addresses antic 32 bit archs that want to address
more than 4 GB of physical memory, thus Intel PAE band-aid to IA32.

> The trivial patch to fix it is attached, but I haven't tested it on a
> system that supports highio.

Nor I can, since such systems are too expensive for me and, on the other
hand, I don't need nor want to use such horrible hardware.

Regards,
Gérard.

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