ExternalVendorCode/genode
1
0
Fork 0
mirror of https://github.com/genodelabs/genode.git synced 2025-05-03 01:02:56 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

ahci: SYNC with empty NCQ queue only

Browse Source 
Fixes #3746
This commit is contained in:
Sebastian Sumpf 2020-05-04 14:24:39 +02:00 committed by Norman Feske
parent 04aeaa25e5
commit ea78d85d35
1 changed files with 18 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
repos/os/src/drivers/ahci/ata_protocol.h
View File

@ -124,6 +124,8 @@ class Ata::Protocol : public Ahci::Protocol, Noncopyable
{ {
private: private:
bool _syncing { false };
struct Request : Block::Request struct Request : Block::Request
{ {
bool valid() const { return operation.valid(); } bool valid() const { return operation.valid(); }
@ -162,6 +164,9 @@ class Ata::Protocol : public Ahci::Protocol, Noncopyable
block_number_t end = block_number + request.operation.count - 1; block_number_t end = block_number + request.operation.count - 1;
auto overlap_check = [&] (Request const &req) { auto overlap_check = [&] (Request const &req) {
if (req.operation.type == Block::Operation::Type::SYNC)
return false;
block_number_t pending_start = req.operation.block_number; block_number_t pending_start = req.operation.block_number;
block_number_t pending_end = pending_start + req.operation.count - 1; block_number_t pending_end = pending_start + req.operation.count - 1;
@ -266,6 +271,8 @@ class Ata::Protocol : public Ahci::Protocol, Noncopyable
_slot_states = port.read<Port::Ci>() | port.read<Port::Sact>(); _slot_states = port.read<Port::Ci>() | port.read<Port::Sact>();
port.stop(); port.stop();
_syncing = false;
} }
Block::Session::Info info() const override Block::Session::Info info() const override
@ -280,10 +287,18 @@ class Ata::Protocol : public Ahci::Protocol, Noncopyable
Response submit(Port &port, Block::Request const request) override Response submit(Port &port, Block::Request const request) override
{ {
if (_writeable == false && request.operation.type == Block::Operation::Type::WRITE) Block::Operation const op = request.operation;
bool const sync = (op.type == Block::Operation::Type::SYNC);
bool const write = (op.type == Block::Operation::Type::WRITE);
if ((sync && _slot_states) || _syncing)
return Response::RETRY;
if (_writeable == false && write)
return Response::REJECTED; return Response::REJECTED;
if (Block::Operation::has_payload(request.operation.type)) { if (Block::Operation::has_payload(op.type)) {
if (port.sanity_check(request) == false || port.dma_base == 0) if (port.sanity_check(request) == false || port.dma_base == 0)
return Response::REJECTED; return Response::REJECTED;
@ -298,7 +313,6 @@ class Ata::Protocol : public Ahci::Protocol, Noncopyable
*r = request; *r = request;
Block::Operation op = request.operation;
size_t slot = _slots.index(*r); size_t slot = _slots.index(*r);
_slot_states |= 1u << slot; _slot_states |= 1u << slot;
@ -308,13 +322,9 @@ class Ata::Protocol : public Ahci::Protocol, Noncopyable
op.count * _block_size()); op.count * _block_size());
/* setup ATA command */ /* setup ATA command */
/* write bit for command header + read/write ATA requests */
bool const write = (op.type == Block::Operation::Type::WRITE);
bool const sync = (op.type == Block::Operation::Type::SYNC);
if (sync) { if (sync) {
table.fis.flush_cache_ext(); table.fis.flush_cache_ext();
_syncing = true;
} else if (_ncq_support(port)) { } else if (_ncq_support(port)) {
table.fis.fpdma(write == false, op.block_number, op.count, slot); table.fis.fpdma(write == false, op.block_number, op.count, slot);
/* ensure that 'Cmd::St' is 1 before writing 'Sact' */ /* ensure that 'Cmd::St' is 1 before writing 'Sact' */