Date: Mon, 2 Jun 2014 10:46:11 -0600 From: Warner Losh <imp@bsdimp.com> To: freebsd-scsi@freebsd.org Subject: Review needed Message-ID: <D530E825-A507-41D6-9ABE-A2A7F8F95394@bsdimp.com>
next in thread | raw e-mail | index | archive | help
--Apple-Mail=_73DD6227-4E02-4592-98BE-AF91A6D19F95 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset=us-ascii Greetings, The code that combines adjacent ranges for BIO_DELETEs to optimize trims to the device assumes the list is sorted. Don't apply the optimization of not sorting the queue when we have SSDs to the delete_queue, since it causes more discard traffic to the drive. While one could argue that the higher levels should coalesce the trims, that's not done today, so some optimization at this level is needed. Diffs and a (hopefully) cool review tool: https://phabric.freebsd.org/D142 Comments? Warner --Apple-Mail=_73DD6227-4E02-4592-98BE-AF91A6D19F95 Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename=signature.asc Content-Type: application/pgp-signature; name=signature.asc Content-Description: Message signed with OpenPGP using GPGMail -----BEGIN PGP SIGNATURE----- Comment: GPGTools - https://gpgtools.org iQIcBAEBCgAGBQJTjKpTAAoJEGwc0Sh9sBEA8ggQAJSOn9MsyR+di1XKAIUnK62x jnaSvkBD3Fq1Zd3aBeHgtarVE06eSLeufun/hgmrzbMHOENT+DQPckoXlw7rte7K eYrKusn5HPrxVEJLPVgtHv88960cso/qnttgxCuKtj+vCxg9X6+Cieb8rRYz3Cos M6L6c0wyy+4/hoxODvGU7hEdXrzWJiGCbb2Xg0Pzm/C2ExxP98dliXOhbNHbKNel iZdSPHNgw83NSBtMxrp/tmNBf2SnS3JiqsigJ7A6XB1ajs/K/Wh5l0ATPx+JKsG1 Y0O2m7ylcNOPD22vZq4LC6RjIpGXsDzjeAkQTWXuZzfHPQTVsvpclRViCsUokCnJ cnS31dtv8dqPSLBPfd1crklBofIOs7wlk2HCLxZcVKWiQzWldP9JAyWL8BApzYiU /f8hxielJZmIQbLjleWmv1YtbirfJNfCtIzcZQiiMidACqoXdplWMJOKS7Rvjek/ r9zxd8Bi/e7ORD3uAthVJc5oiO4+N/p4SbARxLiFy756UF2yOswFzamOExKD+d7M 8QuXxWQVc1wI2i2E6Xoa9/eUUMW9IEETU0KAEJRD6m/eNJHIH0ANTPncRtJQ9vBb hCvjcTdXJblr8fxrKIzo5rr9t6+rDUA10SXMqBXqEh6tujphKm229RWXwgeMI9fL q17MTB2gsHj63bbHN/eh =odDi -----END PGP SIGNATURE----- --Apple-Mail=_73DD6227-4E02-4592-98BE-AF91A6D19F95--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?D530E825-A507-41D6-9ABE-A2A7F8F95394>