Core ReactOS
  1. Core ReactOS
  2. CORE-11894

Multiple restarts causes problems at startup

    Details

    • Type: Bug Bug
    • Status: Reopened Reopened
    • Priority: Critical Critical
    • Resolution: Unresolved
    • Fix Version/s: 0.4.4
    • Component/s: None
    • Labels:
      None

      Description

      After restarting ReactOS a few times, programs/DLLs start to not load during startup (giving Bad Image/failed to initialize errors). A debug log is attached.

      1. debug_erroronly.log
        115 kB
        Zero3K
      2. debug.log
        2.28 MB
        Zero3K
      3. debug.log
        377 kB
        Zero3K
      4. result.log
        33 kB
        Zero3K
      5. result.log
        33 kB
        Zero3K
      6. uniata-touch-physical.patch
        0.5 kB
        Thomas Faber

        Activity

        Hide
        Alter
        added a comment -

        MmGetPhysicalAddress is a hack. It is used to workaround ScsiPort DMA/PIO on the same channel issue. Scsiport assumes these modes are mutually exclusive.
        Unfortunately, IoGetDmaAdapter conflicts with ScsiMiniport model

        The problem can be completly solved when scsiport functionality is implemented inside uniata (for entire Windows family) or ReactOS scsiport is updated.

        Show
        Alter
        added a comment - MmGetPhysicalAddress is a hack. It is used to workaround ScsiPort DMA/PIO on the same channel issue. Scsiport assumes these modes are mutually exclusive. Unfortunately, IoGetDmaAdapter conflicts with ScsiMiniport model The problem can be completly solved when scsiport functionality is implemented inside uniata (for entire Windows family) or ReactOS scsiport is updated.
        Hide
        Zero3K
        added a comment -

        I can confirm that the crash is fixed by the patch that was recently attached.

        Show
        Zero3K
        added a comment - I can confirm that the crash is fixed by the patch that was recently attached.
        Hide
        Alter
        added a comment -

        Thanks, I will merge this update

        Show
        Alter
        added a comment - Thanks, I will merge this update
        Hide
        Alter
        added a comment -

        Patch is merged and published with 0.46e2

        Show
        Alter
        added a comment - Patch is merged and published with 0.46e2
        Hide
        Amine Khaldi
        added a comment -

        Alter would you suggest that we commit the fix attached here? or do a complete sync to 0.46e2?

        Show
        Amine Khaldi
        added a comment - Alter would you suggest that we commit the fix attached here? or do a complete sync to 0.46e2?

          People

          • Assignee:
            Bug Zilla
            Reporter:
            Zero3K
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated: