Re: [PATCH-2.5.47-ac6] More IDE updates (BIOS, simplex, etc)

[Andre Hedrick]

I love it!  I finally got the inital model correct and with the aid and
trust of Alan to bring me back in the loop, everyone is doing great stuff.

Is there an interest in closing out the taskfile io model to complete the
transformation?  If so the following has to be added, and effects PIO
only.

Given the new BIO has the ablitity to force and match BIO atomic segments
with Disk Data Block (atomic segment) it make it much nicer!

The interrupt race or lost interrupts happens for the following reason and
only this reason:

interrupt_servicing:
check_status:
map_hi_bios:
output_ata_data_register:
	competiton (here begins the race!)
		interrupt: (this can happen almost instantly)
unmap_hi_bios:
update_partial_completion_of_original_larger:
	return_if_last_data_block_was_transfered_request_complete:
arm_isr:
	jump_to_interrupt:

The completion of the local atomic segement of data can cause the device
to pop an interrupt instant, and there is no handler armed to catch it.

interrupt_servicing:
check_status:
arm_isr:
	boost_wait_time_worst_case_to_prevent_early_timer_popping:
map_hi_bios:
output_ata_data_register:
	competiton (here begins the race!)
		interrupt: (this can happen almost instantly)
			who_cares_but_needs_a_sem_to_block:
unmap_hi_bios:
update_partial_completion_of_original_larger:
	if_last_data_block_dissarm_handler:
		return_if_last_data_block_was_transfered_request_complete:

Comments?

Andre Hedrick
LAD Storage Consulting Group

On Mon, 25 Nov 2002, Torben Mathiasen wrote:

> Hi Alan,
> 
> Please apply the attached patch. Its an update on my previos patch with a
> rewrite of the simplex code. The patch now does the following:
> 
> 	# Make simplex detection on a per drive basis and dynamic for use 
> 	  with hotplug.
> 	# Make sure we also enforce simplex rules when using hdparm.
> 	# Make sure we also enforce simplex rules when using BIOS timings.
> 	# Moves BIOS timings IDE detection into chipset drivers (by providing a
> 	  library function).
> 	# Provide the above library function that sets default values for tune
> 	  parameters. It also handles the special case where user has requested
> 	  to use BIOS IDE timings. Caller is assumed to support DMA
> 	  on/off probe using the dma_status register unless a dma_check funtion
> 	  is provided (note, the tekram driver is somewhat different from all
> 	  the others, and haven't been updated yet).
> 	# Makes BIOS IDE timings work for both IDE0 and IDE1 at the same time.
> 
> I'm not sure whether you already applied my previous patch, so let me know if
> you want this on top of that one.
> 
> Cheers,
> Torben
>