Uploaded image for project: 'Core ReactOS'
  1. Core ReactOS
  2. CORE-20078

FreeLoader IDE driver failure: GCC string I/O inline asm error

    XMLWordPrintable

Details

    Description

      Starting with 0.4.16-dev-828-g787f81f it's not possible to boot ReactOS on platforms that use FreeLoader IDE driver (including OG Xbox, NEC PC-98, and others).

        0.4.16-dev-827-gddf55b3 0.4.16-dev-828-g787f81f
      OG Xbox
      NEC PC-98

      `__outwordstring` clobbers the buffer operand:

      ERR("1 %p\n", Request->Cdb);
      ATA_WRITE_BLOCK_16(DeviceUnit->Registers.Data, Request->Cdb, DeviceUnit->CdbSize);
      ERR("2 %p\n", Request->Cdb);

      err: 1 0000EF34
      err: 2 0000EF3A

      I tried to fix this, but I don't understand the GNU assembly fix.patch.

      MinGW implementation: https://github.com/mirror/mingw-w64/blob/master/mingw-w64-headers/include/psdk_inc/intrin-impl.h#L379

      Attachments

        1. 0.4.16-dev-827-gddf55b3-ok.png
          8 kB
          Stas'M
        2. 0.4.16-dev-827-gddf55b3-xbox.png
          24 kB
          Stas'M
        3. 0.4.16-dev-828-g787f81f-fail1.png
          8 kB
          Stas'M
        4. 0.4.16-dev-828-g787f81f-xbox.png
          5 kB
          Stas'M
        5. fix.patch
          1 kB
          disean

        Issue Links

          blocks

          Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. CORE-16216 Revive ReactOS port for original Xbox

          • Major - Major loss of function.
          • Open

          Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. CORE-17977 ReactOS NEC PC-98 port Epic to group tickets

          • Major - Major loss of function.
          • Open
          relates to

          New Feature - A new feature of the product, which has yet to be developed. CORE-17256 Implement IDE port driver

          • Major - Major loss of function.
          • Open

          Activity

            People

              Unassigned Unassigned
              walean disean
              Votes:
              2 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated: