Commit ee364416 authored by John Snow's avatar John Snow
Browse files

ahci: add get_cmd_header helper 



cur_cmd is an internal bookmark that points to the
current AHCI Command Header being processed by the
AHCI state machine. With NCQ needing to occasionally
rely on some of the same AHCI helpers, we cannot use
cur_cmd and will need to grab explicit pointers instead.

In an attempt to begin relying on the cur_cmd pointer
less, add a helper to let us specifically get the pointer
to the command header of particular interest.

Signed-off-by: default avatarJohn Snow <jsnow@redhat.com>
Reviewed-by: default avatarStefan Hajnoczi <stefanha@redhat.com>
Message-id: 1435767578-32743-12-git-send-email-jsnow@redhat.com
parent c82bd3c8

hw/ide/ahci.c

+12 −3
Original line number Diff line number Diff line
@@ -1116,11 +1116,20 @@ static void process_ncq_command(AHCIState *s, int port, uint8_t *cmd_fis, 
    execute_ncq_command(ncq_tfs);

}



static AHCICmdHdr *get_cmd_header(AHCIState *s, uint8_t port, uint8_t slot)

{

    if (port >= s->ports || slot >= AHCI_MAX_CMDS) {

        return NULL;

    }



    return s->dev[port].lst ? &((AHCICmdHdr *)s->dev[port].lst)[slot] : NULL;

}



static void handle_reg_h2d_fis(AHCIState *s, int port,

                               uint8_t slot, uint8_t *cmd_fis)

{

    IDEState *ide_state = &s->dev[port].port.ifs[0];

    AHCICmdHdr *cmd = s->dev[port].cur_cmd;

    AHCICmdHdr *cmd = get_cmd_header(s, port, slot);

    uint16_t opts = le16_to_cpu(cmd->opts);



    if (cmd_fis[1] & 0x0F) {
@@ -1219,7 +1228,7 @@ static int handle_cmd(AHCIState *s, int port, uint8_t slot) 
        DPRINTF(port, "error: lst not given but cmd handled");

        return -1;

    }

    cmd = &((AHCICmdHdr *)s->dev[port].lst)[slot];

    cmd = get_cmd_header(s, port, slot);

    /* remember current slot handle for later */

    s->dev[port].cur_cmd = cmd;
@@ -1572,7 +1581,7 @@ static int ahci_state_post_load(void *opaque, int version_id) 
            if (ad->busy_slot < 0 || ad->busy_slot >= AHCI_MAX_CMDS) {

                return -1;

            }

            ad->cur_cmd = &((AHCICmdHdr *)ad->lst)[ad->busy_slot];

            ad->cur_cmd = get_cmd_header(s, i, ad->busy_slot);

        }

    }