Opened 11 years ago

Closed 11 years ago

#21 closed defect (fixed)

/R option should be "on" by default...

Reported by: ecsnl Owned by: rousseau
Priority: major Milestone:
Component: driver Version:
Keywords: Cc:

Description

I'm sending this message to Ben Rietbroek who is working on some
OS2AHCI points.

One thing that keeps coming up in some cases is that on systems with an
Intel disc controller you need the /R option specified.
In some cases the system is able to boot with from the CD but it does
not recognize the hard disc.
Using the /R option fixes.

In the documentation this is written:
/r Reset ports during initialization (default = off

unless the [Intel] AHCI controller was found to

be

initialized by the BIOS in SATA mode)

My question what would the result be to do this by default (to always
initialize) the controller.
Or will this just "fix" some systems (no user action needed) and make
other systems hang ?

Change History (5)

comment:1 Changed 11 years ago by ecsnl

Hi Roderick, hi Ben,

I talked to Christian about this and he, too, has no objections:

Resetting the devices during startup will require slightly more time during the boot sequence, othewise there should be no negative effect. In theory, resetting a CD recorder may cause it to lose information about half-complete CDs (e.g. on an old Yamaha CDR100) but this is a moot point today as most recorders update the PMA after each track and nobody really expects a half-written CD to survive a reboot and still be usable.

Bye
Markus

comment:2 Changed 11 years ago by ecsnl

In other words Ben could you modify the behavior that the driver always behaves as if the /R is done by default.

And create a /!R switch so you can switch it of.

Thanks,

Roderick

comment:3 Changed 11 years ago by rousseau

Owner: changed from somebody to rousseau
Status: newaccepted

comment:4 Changed 11 years ago by rousseau

Hi all,

The change is made in r147.
Option /r is now on by default and can be turned off with /!r.
However, when an Intel controller is detected to be programmed in SATA mode by the BIOS,
then the ports will still be reset anyway, even when /!r was specified.

Thanks Markus and Christian for your feedback.
(And kudo's for the super readable code you guys wrote!)

Regards,

Ben.

Last edited 11 years ago by rousseau (previous) (diff)

comment:5 Changed 11 years ago by David Azarewicz

Resolution: fixed
Status: acceptedclosed
Note: See TracTickets for help on using tickets.