Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 5 Dec 2019 18:47:29 +0000 (UTC)
From:      John Baldwin <jhb@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r355426 - in head/sys: dev/ahci dev/ata dev/mvs dev/siis net/altq powerpc/mpc85xx
Message-ID:  <201912051847.xB5IlTmK069792@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: jhb
Date: Thu Dec  5 18:47:29 2019
New Revision: 355426
URL: https://svnweb.freebsd.org/changeset/base/355426

Log:
  Use a void * argument to callout handlers instead of timeout_t casts.
  
  Reviewed by:	kib
  Differential Revision:	https://reviews.freebsd.org/D22684

Modified:
  head/sys/dev/ahci/ahci.c
  head/sys/dev/ata/ata-all.c
  head/sys/dev/ata/ata-all.h
  head/sys/dev/ata/ata-lowlevel.c
  head/sys/dev/mvs/mvs.c
  head/sys/dev/siis/siis.c
  head/sys/net/altq/altq_rmclass.c
  head/sys/powerpc/mpc85xx/fsl_sata.c

Modified: head/sys/dev/ahci/ahci.c
==============================================================================
--- head/sys/dev/ahci/ahci.c	Thu Dec  5 16:50:54 2019	(r355425)
+++ head/sys/dev/ahci/ahci.c	Thu Dec  5 18:47:29 2019	(r355426)
@@ -67,7 +67,7 @@ static void ahci_ch_intr_main(struct ahci_channel *ch,
 static void ahci_begin_transaction(struct ahci_channel *ch, union ccb *ccb);
 static void ahci_dmasetprd(void *arg, bus_dma_segment_t *segs, int nsegs, int error);
 static void ahci_execute_transaction(struct ahci_slot *slot);
-static void ahci_timeout(struct ahci_slot *slot);
+static void ahci_timeout(void *arg);
 static void ahci_end_transaction(struct ahci_slot *slot, enum ahci_err_type et);
 static int ahci_setup_fis(struct ahci_channel *ch, struct ahci_cmd_tab *ctp, union ccb *ccb, int tag);
 static void ahci_dmainit(device_t dev);
@@ -1793,7 +1793,7 @@ ahci_execute_transaction(struct ahci_slot *slot)
 	}
 	/* Start command execution timeout */
 	callout_reset_sbt(&slot->timeout, SBT_1MS * ccb->ccb_h.timeout / 2,
-	    0, (timeout_t*)ahci_timeout, slot, 0);
+	    0, ahci_timeout, slot, 0);
 	return;
 }
 
@@ -1830,14 +1830,15 @@ ahci_rearm_timeout(struct ahci_channel *ch)
 			continue;
 		callout_reset_sbt(&slot->timeout,
     	    	    SBT_1MS * slot->ccb->ccb_h.timeout / 2, 0,
-		    (timeout_t*)ahci_timeout, slot, 0);
+		    ahci_timeout, slot, 0);
 	}
 }
 
 /* Locked by callout mechanism. */
 static void
-ahci_timeout(struct ahci_slot *slot)
+ahci_timeout(void *arg)
 {
+	struct ahci_slot *slot = arg;
 	struct ahci_channel *ch = slot->ch;
 	device_t dev = ch->dev;
 	uint32_t sstatus;
@@ -1864,7 +1865,7 @@ ahci_timeout(struct ahci_slot *slot)
 
 		callout_reset_sbt(&slot->timeout,
 	    	    SBT_1MS * slot->ccb->ccb_h.timeout / 2, 0,
-		    (timeout_t*)ahci_timeout, slot, 0);
+		    ahci_timeout, slot, 0);
 		return;
 	}
 

Modified: head/sys/dev/ata/ata-all.c
==============================================================================
--- head/sys/dev/ata/ata-all.c	Thu Dec  5 16:50:54 2019	(r355425)
+++ head/sys/dev/ata/ata-all.c	Thu Dec  5 18:47:29 2019	(r355426)
@@ -702,10 +702,12 @@ ata_atapi(device_t dev, int target)
 }
 
 void
-ata_timeout(struct ata_request *request)
+ata_timeout(void *arg)
 {
+	struct ata_request *request;
 	struct ata_channel *ch;
 
+	request = arg;
 	ch = device_get_softc(request->parent);
 	//request->flags |= ATA_R_DEBUG;
 	ATA_DEBUG_RQ(request, "timeout");

Modified: head/sys/dev/ata/ata-all.h
==============================================================================
--- head/sys/dev/ata/ata-all.h	Thu Dec  5 16:50:54 2019	(r355425)
+++ head/sys/dev/ata/ata-all.h	Thu Dec  5 18:47:29 2019	(r355426)
@@ -477,7 +477,7 @@ const char *ata_mode2str(int mode);
 void ata_setmode(device_t dev);
 void ata_print_cable(device_t dev, u_int8_t *who);
 int ata_atapi(device_t dev, int target);
-void ata_timeout(struct ata_request *);
+void ata_timeout(void *);
 
 /* ata-lowlevel.c: */
 void ata_generic_hw(device_t dev);

Modified: head/sys/dev/ata/ata-lowlevel.c
==============================================================================
--- head/sys/dev/ata/ata-lowlevel.c	Thu Dec  5 16:50:54 2019	(r355425)
+++ head/sys/dev/ata/ata-lowlevel.c	Thu Dec  5 18:47:29 2019	(r355426)
@@ -237,7 +237,7 @@ begin_finished:
 
 begin_continue:
     callout_reset(&request->callout, request->timeout * hz,
-		  (timeout_t*)ata_timeout, request);
+		  ata_timeout, request);
     return ATA_OP_CONTINUES;
 }
 

Modified: head/sys/dev/mvs/mvs.c
==============================================================================
--- head/sys/dev/mvs/mvs.c	Thu Dec  5 16:50:54 2019	(r355425)
+++ head/sys/dev/mvs/mvs.c	Thu Dec  5 18:47:29 2019	(r355426)
@@ -82,7 +82,7 @@ static void mvs_legacy_intr(device_t dev, int poll);
 static void mvs_crbq_intr(device_t dev);
 static void mvs_begin_transaction(device_t dev, union ccb *ccb);
 static void mvs_legacy_execute_transaction(struct mvs_slot *slot);
-static void mvs_timeout(struct mvs_slot *slot);
+static void mvs_timeout(void *arg);
 static void mvs_dmasetprd(void *arg,
 	bus_dma_segment_t *segs, int nsegs, int error);
 static void mvs_requeue_frozen(device_t dev);
@@ -1418,7 +1418,7 @@ mvs_legacy_execute_transaction(struct mvs_slot *slot)
 	}
 	/* Start command execution timeout */
 	callout_reset_sbt(&slot->timeout, SBT_1MS * ccb->ccb_h.timeout, 0,
-	    (timeout_t*)mvs_timeout, slot, 0);
+	    mvs_timeout, slot, 0);
 }
 
 /* Must be called with channel locked. */
@@ -1532,7 +1532,7 @@ mvs_execute_transaction(struct mvs_slot *slot)
 	    ch->dma.workrq_bus + MVS_CRQB_OFFSET + (MVS_CRQB_SIZE * ch->out_idx));
 	/* Start command execution timeout */
 	callout_reset_sbt(&slot->timeout, SBT_1MS * ccb->ccb_h.timeout, 0,
-	    (timeout_t*)mvs_timeout, slot, 0);
+	    mvs_timeout, slot, 0);
 	return;
 }
 
@@ -1571,14 +1571,15 @@ mvs_rearm_timeout(device_t dev)
 			continue;
 		callout_reset_sbt(&slot->timeout,
 		    SBT_1MS * slot->ccb->ccb_h.timeout / 2, 0,
-		    (timeout_t*)mvs_timeout, slot, 0);
+		    mvs_timeout, slot, 0);
 	}
 }
 
 /* Locked by callout mechanism. */
 static void
-mvs_timeout(struct mvs_slot *slot)
+mvs_timeout(void *arg)
 {
+	struct mvs_slot *slot = arg;
 	device_t dev = slot->dev;
 	struct mvs_channel *ch = device_get_softc(dev);
 

Modified: head/sys/dev/siis/siis.c
==============================================================================
--- head/sys/dev/siis/siis.c	Thu Dec  5 16:50:54 2019	(r355425)
+++ head/sys/dev/siis/siis.c	Thu Dec  5 18:47:29 2019	(r355426)
@@ -72,7 +72,7 @@ static void siis_ch_led(void *priv, int onoff);
 static void siis_begin_transaction(device_t dev, union ccb *ccb);
 static void siis_dmasetprd(void *arg, bus_dma_segment_t *segs, int nsegs, int error);
 static void siis_execute_transaction(struct siis_slot *slot);
-static void siis_timeout(struct siis_slot *slot);
+static void siis_timeout(void *arg);
 static void siis_end_transaction(struct siis_slot *slot, enum siis_err_type et);
 static int siis_setup_fis(device_t dev, struct siis_cmd *ctp, union ccb *ccb, int tag);
 static void siis_dmainit(device_t dev);
@@ -1123,7 +1123,7 @@ siis_execute_transaction(struct siis_slot *slot)
 	ATA_OUTL(ch->r_mem, SIIS_P_CACTH(slot->slot), prb_bus >> 32);
 	/* Start command execution timeout */
 	callout_reset_sbt(&slot->timeout, SBT_1MS * ccb->ccb_h.timeout, 0,
-	    (timeout_t*)siis_timeout, slot, 0);
+	    siis_timeout, slot, 0);
 	return;
 }
 
@@ -1166,14 +1166,15 @@ siis_rearm_timeout(device_t dev)
 			continue;
 		callout_reset_sbt(&slot->timeout,
 		    SBT_1MS * slot->ccb->ccb_h.timeout, 0,
-		    (timeout_t*)siis_timeout, slot, 0);
+		    siis_timeout, slot, 0);
 	}
 }
 
 /* Locked by callout mechanism. */
 static void
-siis_timeout(struct siis_slot *slot)
+siis_timeout(void *arg)
 {
+	struct siis_slot *slot = arg;
 	device_t dev = slot->dev;
 	struct siis_channel *ch = device_get_softc(dev);
 	union ccb *ccb = slot->ccb;

Modified: head/sys/net/altq/altq_rmclass.c
==============================================================================
--- head/sys/net/altq/altq_rmclass.c	Thu Dec  5 16:50:54 2019	(r355425)
+++ head/sys/net/altq/altq_rmclass.c	Thu Dec  5 18:47:29 2019	(r355426)
@@ -87,7 +87,7 @@ static mbuf_t	*_rmc_pollq(rm_class_t *);
 static int	rmc_under_limit(struct rm_class *, struct timeval *);
 static void	rmc_tl_satisfied(struct rm_ifdat *, struct timeval *);
 static void	rmc_drop_action(struct rm_class *);
-static void	rmc_restart(struct rm_class *);
+static void	rmc_restart(void *);
 static void	rmc_root_overlimit(struct rm_class *, struct rm_class *);
 
 #define	BORROW_OFFTIME
@@ -1530,8 +1530,7 @@ rmc_delay_action(struct rm_class *cl, struct rm_class 
 			t = hzto(&cl->undertime_);
 		} else
 			t = 2;
-		CALLOUT_RESET(&cl->callout_, t,
-			      (timeout_t *)rmc_restart, (caddr_t)cl);
+		CALLOUT_RESET(&cl->callout_, t, rmc_restart, cl);
 	}
 }
 
@@ -1553,8 +1552,9 @@ rmc_delay_action(struct rm_class *cl, struct rm_class 
  */
 
 static void
-rmc_restart(struct rm_class *cl)
+rmc_restart(void *arg)
 {
+	struct rm_class *cl = arg;
 	struct rm_ifdat	*ifd = cl->ifdat_;
 	int		 s;
 

Modified: head/sys/powerpc/mpc85xx/fsl_sata.c
==============================================================================
--- head/sys/powerpc/mpc85xx/fsl_sata.c	Thu Dec  5 16:50:54 2019	(r355425)
+++ head/sys/powerpc/mpc85xx/fsl_sata.c	Thu Dec  5 18:47:29 2019	(r355426)
@@ -70,7 +70,7 @@ static void fsl_sata_intr_main(struct fsl_sata_channel
 static void fsl_sata_begin_transaction(struct fsl_sata_channel *ch, union ccb *ccb);
 static void fsl_sata_dmasetprd(void *arg, bus_dma_segment_t *segs, int nsegs, int error);
 static void fsl_sata_execute_transaction(struct fsl_sata_slot *slot);
-static void fsl_sata_timeout(struct fsl_sata_slot *slot);
+static void fsl_sata_timeout(void *arg);
 static void fsl_sata_end_transaction(struct fsl_sata_slot *slot, enum fsl_sata_err_type et);
 static int fsl_sata_setup_fis(struct fsl_sata_channel *ch, struct fsl_sata_cmd_tab *ctp, union ccb *ccb, int tag);
 static void fsl_sata_dmainit(device_t dev);
@@ -1107,7 +1107,7 @@ fsl_sata_execute_transaction(struct fsl_sata_slot *slo
 	}
 	/* Start command execution timeout */
 	callout_reset_sbt(&slot->timeout, SBT_1MS * ccb->ccb_h.timeout / 2,
-	    0, (timeout_t*)fsl_sata_timeout, slot, 0);
+	    0, fsl_sata_timeout, slot, 0);
 	return;
 }
 
@@ -1144,14 +1144,15 @@ fsl_sata_rearm_timeout(struct fsl_sata_channel *ch)
 			continue;
 		callout_reset_sbt(&slot->timeout,
  	    	    SBT_1MS * slot->ccb->ccb_h.timeout / 2, 0,
-		    (timeout_t*)fsl_sata_timeout, slot, 0);
+		    fsl_sata_timeout, slot, 0);
 	}
 }
 
 /* Locked by callout mechanism. */
 static void
-fsl_sata_timeout(struct fsl_sata_slot *slot)
+fsl_sata_timeout(void *arg)
 {
+	struct fsl_sata_slot *slot = arg;
 	struct fsl_sata_channel *ch = slot->ch;
 	device_t dev = ch->dev;
 	uint32_t sstatus;
@@ -1169,7 +1170,7 @@ fsl_sata_timeout(struct fsl_sata_slot *slot)
 
 		callout_reset_sbt(&slot->timeout,
 	    	    SBT_1MS * slot->ccb->ccb_h.timeout / 2, 0,
-		    (timeout_t*)fsl_sata_timeout, slot, 0);
+		    fsl_sata_timeout, slot, 0);
 		return;
 	}
 



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201912051847.xB5IlTmK069792>