Re: amd64 bus_space_*_8() functions

[Andrew Doran <ad%netbsd.org@localhost>]

On Wed, Apr 23, 2008 at 04:46:09PM -0700, Dennis Ferguson wrote:

> On 23 Apr 2008, at 16:14 , Andrew Doran wrote:
> >
> >I think it is simply because there has not been a need for these  
> >functions
> >yet. I think that an 8-byte operation to I/O space should call  
> >panic().
> >Can you see how to implement them (arch/amd64/amd64/busfunc.S) or  
> >shall I
> >do that?
> 
> I'd  be very grateful if you would do it.  I've been making progress
> with
> 
> #define bus_space_read_8(t, h, o) \
>         (*(volatile uint64_t *)((h) + (o)))
> 
> #define bus_space_write_8(t, h, o, v) \
>         (*(volatile uint64_t *)((h) + (o)) = (v))
> 
> shoved in a .h file, and while I found the file they needed to be
> added to my Intel assembler skills are too obsolete to do a proper
> job without some relearning.  If you would do it, it would save me
> that last bit of trouble.

Ok, I added them but I haven't tested them.

Checking in busfunc.S;
/cvsroot/src/sys/arch/amd64/amd64/busfunc.S,v  <--  busfunc.S
new revision: 1.5; previous revision: 1.4

Thanks,
Andrew