Доступ к структурному элементу из указателя структуры с помощью встроенной сборки Open Watcom

Question

У меня есть код, который записывает и считывает порты ввода-вывода для VGA. Я пытаюсь реализовать функциональность C-кода в встроенном ассемблере. Я использую Open Watcom 2.0 и компилирую для DOS 16 бит.

Чтобы написать цветную палитру на VGA, я придумал это. Это работает неправильно.

EDIT: Код для setPaletteColor не совсем точен. Я обновил, чтобы отразить фактический код.

void setPaletteColor (unsigned char index, rgbColor *p_color) 
{ 
    _asm 
    { 
     ; tell VGA card we are going to update a palette register 
     mov dx,PALETTE_MASK 
     mov al,0xff 
     out dx,al 

     ; tell VGA which register we will be updating 
     mov dx,PALETTE_REGISTER_WR 
     mov al,index 
     out dx,al 

     ; update the color in the register at index 
     mov dx,PALETTE_DATA 
     mov al,*p_color 
     out dx,al 
     mov al,*p_color // this is actually *(p_color+1) but this actually gets the next structure not the next data member, so I left it out of the code I typed for my question. 
     out dx,al 
     mov al,*p_color // same here, actually is *(p_color+2) 
     out dx,al 
    } 
} 

И для чтения у меня это есть. Это также не работает правильно.

void getPaletteColor (unsigned char index, rgbColor *p_color) 
{ 
    unsigned char *p_red = &p_color->red; 
    unsigned char *p_green = &p_color->green; 
    unsigned char *p_blue = &p_color->blue; 
    _asm 
    { 
     ; tell VGA card we are going to read a palette register 
     mov dx,PALETTE_MASK 
     mov al,0xff 
     out dx,al 

     ; tell VGA which register we will be reading 
     mov dx,PALETTE_REGISTER_RD 
     mov al,index 
     out dx,al 

     ; read the data into the color struct at 'p_color' 
     mov dx,PALETTE_DATA 
     in al,dx 
     mov *p_red,al 
     in al,dx 
     mov *p_green,al 
     in al,dx 
     mov *p_blue,al 
    } 
} 

Теперь вот чистые версии C, которые работают DO.

void setPaletteColor (unsigned char index, rgbColor *p_color) 
{ 
    outp(PALETTE_MASK,0xff); 
    outp(PALETTE_REGISTER_WR, index); 
    outp(PALETTE_DATA,p_color->red); 
    outp(PALETTE_DATA,p_color->green); 
    outp(PALETTE_DATA,p_color->blue); 
} 

И для чтения.

void getPaletteColor (unsigned char index, rgbColor *p_color) 
{ 
    outp(PALETTE_MASK,0xff); 
    outp(PALETTE_REGISTER_RD, index); 
    p_color->red = inp(PALETTE_DATA); 
    p_color->green = inp(PALETTE_DATA); 
    p_color->blue = inp(PALETTE_DATA); 
} 

ПРИМЕЧАНИЕ: Я не могу использовать '.' оператора или оператора '->' в встроенной сборке, потому что компилятор его не поддерживает.

Вот определение структуры rgbColor.

typedef struct rgbColorTag 
{ 
    unsigned char red; 
    unsigned char green; 
    unsigned char blue; 
} rgbColor; 

Answer 1

Хороший вопрос описал бы, как он не работает. Просто говорю, что это «не работает» заставило меня предположить, что это была синтаксическая ошибка, потому что я не знаю встроенный asm в стиле Watcom. Я просто предположил, что он похож на MSVC-стиль, и что использование оператора d-разметки C в asm является синтаксической ошибкой (например, в mov al,*p_color).

Видимо, это допустимый синтаксис для Open Watcom, но Вы загружаете один и тот же байт 3 раза. Может быть, попробуйте mov al, *(p_color+1) для второго байта? Хотя это может просто сделать C-указатель math и получить начало следующей структуры. Ознакомьтесь с руководством по компилятору для доступных вариантов синтаксиса.

---

Вы также можете просто загрузить указатель в регистр и использовать смещение от него самостоятельно (с режимами адресации, как mov al, [si+1]). Это зависит от того, как 3 элемента структуры будут выложены, чтобы не заполнить, но это должно быть безопасным предположением, я думаю. Вы всегда можете проверить выход asm компилятора, чтобы увидеть, как он описывает структуру.

Поскольку ваша структура выложена в правильном порядке, вы должны иметь возможность перебирать через нее 3 байта, используя OUTS. Или даже REP OUTS, поэтому вам не нужно писать цикл.

cld 
    mov si, p_color  ; get the function arg in a register 
    mov dx, PALETTE_DATA 
    mov cx, 3 
    rep outsb    ; OUT 3 times, to port DX, using data from DS:[SI] (and do SI++ post-increment) 

Аналогично, для чтения,

cld 
    mov di, p_color 
    mov dx, PALETTE_DATA 
    mov cx, 3 
    rep insb    ; IN 3 times, to port DX, using data from DS:[DI] (and do DI++)