LogicalVsPhysicalTrackCleanup: Difference between revisions

From vice-emu
Jump to navigation Jump to search
Line 46: Line 46:
== Things to test ==
== Things to test ==
* create G64/G71
* create G64/G71
* create D64/D71/D82/D81/D20/D40
* create D64/D71/D82/D81/D1M/D2M/D4M

Revision as of 20:47, 15 October 2021

In some part of the code Logical Track numbers are used, when it should be physical tracks instead. (This mostly applies to 1571 and 8250 images and drives - where the DOS would use logical track numbers > 35 to access the second disk side). this makes it kindof hard an annoying to add support for images that use the upper tracks (35-40) in a generic way.

This page is used for notes and keeping track of some things while i am trying to get this worked out and under control Gpz (talk) 22:09, 14 October 2021 (CEST)

Functions

 src/diskimage/diskimage.c: disk_image_speed_map(unsigned int format, unsigned int track) - input: logical track
  returns: speedzone for given track
  calls: -
  called by:
    disk_image_sector_per_track(unsigned int format, unsigned int track) - input not checked for 1571/d71, so either physical or logical
      returns: number of sectors for a given track
    disk_image_raw_track_size(unsigned int format, unsigned int track) - input: logical track
      returns: number of GCR bytes on a track
    disk_image_gap_size(unsigned int format, unsigned int track) - input: logical track
      returns: gap size for given track
    src/diskimage/fsimage-create.c:fsimage_create_gcr(disk_image_t *image)
    src/diskimage/fsimage-p64.c:fsimage_p64_read_half_track(const disk_image_t *image, unsigned int half_track, disk_track_t *raw)
    src/diskimage/fsimage-gcr.c:fsimage_gcr_write_half_track(disk_image_t *image, unsigned int half_track, const disk_track_t *raw)
 src/diskimage/fsimage-check.c: int fsimage_check_sector(const disk_image_t *image, unsigned int track, unsigned int sector)
  returns: offset in sectors in a image for track/sector or -1 on error
  calls:
    disk_image_sector_per_track(unsigned int format, unsigned int track) - input not checked for 1571/d71, so either physical or logical
      returns: number of sectors for a given track
  called by:
    src/diskimage/diskimage.c: disk_image_check_sector(const disk_image_t *image, unsigned int track, unsigned int sector)
      returns: offset in sectors in a image for track/sector or -1 on error - ONLY IF (image->device == DISK_IMAGE_DEVICE_FS)
 src/diskimage/fsimage-create.c: fsimage_create_gcr(disk_image_t *image)
   calls: 
     disk_image_speed_map(unsigned int format, unsigned int track) - input: logical track
     disk_image_gap_size(unsigned int format, unsigned int track) - input: logical track
     disk_image_header_gap_size(unsigned int format, unsigned int track) - input: track (not used)
     disk_image_sync_size(unsigned int format, unsigned int track) - input: track (not used)
     disk_image_raw_track_size(unsigned int format, unsigned int track) - input: logical track
     disk_image_sector_per_track(unsigned int format, unsigned int track) - input not checked for 1571/d71, so either physical or logical

Things to test

  • create G64/G71
  • create D64/D71/D82/D81/D1M/D2M/D4M