<feed xmlns='http://www.w3.org/2005/Atom'>
<title>src-test/sys/dev/mrsas/mrsas_ioctl.c, branch main</title>
<subtitle>FreeBSD source tree</subtitle>
<id>https://cgit-dev.freebsd.org/src-test/atom?h=main</id>
<link rel='self' href='https://cgit-dev.freebsd.org/src-test/atom?h=main'/>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/'/>
<updated>2020-12-16T00:13:32Z</updated>
<entry>
<title>Use uintptr_t instead of unsigned long for pointers.</title>
<updated>2020-12-16T00:13:32Z</updated>
<author>
<name>John Baldwin</name>
<email>jhb@FreeBSD.org</email>
</author>
<published>2020-12-16T00:13:32Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=f83d3280f60d79f1fdaf6b0ff833d33b36ec0545'/>
<id>urn:sha1:f83d3280f60d79f1fdaf6b0ff833d33b36ec0545</id>
<content type='text'>
The sense_ptr thing is quite broken.  As near as I can tell, the
driver tries to copyout to a physical address rather than whatever
user address the sense buffer should be copied to.  It is not
immediately obvious what user address the sense buffer should be
copied to.

Reviewed by:	imp
Obtained from:	CheriBSD
Sponsored by:	DARPA
Differential Revision:	https://reviews.freebsd.org/D27578
</content>
</entry>
<entry>
<title>Following bugs fixed as part of this patch:</title>
<updated>2016-05-13T12:21:50Z</updated>
<author>
<name>Kashyap D Desai</name>
<email>kadesai@FreeBSD.org</email>
</author>
<published>2016-05-13T12:21:50Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=4bb0a4f0484299a5692fe7c2c6cead08f50b2b84'/>
<id>urn:sha1:4bb0a4f0484299a5692fe7c2c6cead08f50b2b84</id>
<content type='text'>
.Kernel panic while collecting kdump (reported by Doug A.)
.NULL pointer dereference at sertain places
.Removed dead codes

Submitted by:   Sumit Saxena &lt;sumit.saxena@broadcom.com&gt;
Reviewed by:    Kashyap Desai &lt;Kashyap.Desai@broadcom.com&gt;
MFC after:  3 days
Sponsored by:   AVAGO Technologies
</content>
</entry>
<entry>
<title>There was no ERROR handling for firmware command TIMEOUT. This patch</title>
<updated>2016-05-13T12:00:46Z</updated>
<author>
<name>Kashyap D Desai</name>
<email>kadesai@FreeBSD.org</email>
</author>
<published>2016-05-13T12:00:46Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=f0c7594bf3e5d70a8cdb3f58cc4849def5486f9b'/>
<id>urn:sha1:f0c7594bf3e5d70a8cdb3f58cc4849def5486f9b</id>
<content type='text'>
takes care of any firmware command timeout scenarios by initiating OCR.

Submitted by:   Sumit Saxena &lt;sumit.saxena@broadcom.com&gt;
Reviewed by:    Kashyap Desai &lt;Kashyap.Desai@broadcom.com&gt;
MFC after:  3 days
Sponsored by:   AVAGO Technologies
</content>
</entry>
<entry>
<title>Replace 0 with NULL for pointers in misc. device drivers.</title>
<updated>2016-04-12T17:23:03Z</updated>
<author>
<name>Pedro F. Giffuni</name>
<email>pfg@FreeBSD.org</email>
</author>
<published>2016-04-12T17:23:03Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=87d8fcc80e5c6bb3f11858d9866d3c83b1df896d'/>
<id>urn:sha1:87d8fcc80e5c6bb3f11858d9866d3c83b1df896d</id>
<content type='text'>
Found with devel/coccinelle.
</content>
</entry>
<entry>
<title>Corrected indentation on conflicted source files.</title>
<updated>2015-05-06T10:46:28Z</updated>
<author>
<name>Kashyap D Desai</name>
<email>kadesai@FreeBSD.org</email>
</author>
<published>2015-05-06T10:46:28Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=dbcc81dfdcc32cad95a1e34e1927e0771cadfe9d'/>
<id>urn:sha1:dbcc81dfdcc32cad95a1e34e1927e0771cadfe9d</id>
<content type='text'>
Reviewed by:	ambrisko
MFC after:	2 weeks
Sponsored by:	AVAGO Technologies
</content>
</entry>
<entry>
<title>1. All LSI namings are converted to AVAGO Tech.</title>
<updated>2015-05-06T10:45:13Z</updated>
<author>
<name>Kashyap D Desai</name>
<email>kadesai@FreeBSD.org</email>
</author>
<published>2015-05-06T10:45:13Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=ecea5be46436cb40dd739704a3fa446af15a16ee'/>
<id>urn:sha1:ecea5be46436cb40dd739704a3fa446af15a16ee</id>
<content type='text'>
2. Fix in AEN path(suggested by John Baldwin).
3. Fix IOCTL path w.r.t Sense key handling

Reviewed by:	ambrisko
MFC after:	2 weeks
Sponsored by:	AVAGO Technologies
</content>
</entry>
<entry>
<title>No logical code change in this pathc.</title>
<updated>2014-10-08T10:14:37Z</updated>
<author>
<name>Kashyap D Desai</name>
<email>kadesai@FreeBSD.org</email>
</author>
<published>2014-10-08T10:14:37Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=8e7273711914b9e78cae60a1c13f81e5e588c0b6'/>
<id>urn:sha1:8e7273711914b9e78cae60a1c13f81e5e588c0b6</id>
<content type='text'>
Only Style 9 changes for mrsas driver.

Reviewed by:	ambrisko
MFC after:	2 weeks
Sponsored by:	AVAGO Technologies
</content>
</entry>
<entry>
<title>In the passthru IOCTL path, the mfi command pool was freely accessible N times</title>
<updated>2014-10-08T09:37:47Z</updated>
<author>
<name>Kashyap D Desai</name>
<email>kadesai@FreeBSD.org</email>
</author>
<published>2014-10-08T09:37:47Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=839ee02531b5a3f7e52188011c355a6044748de3'/>
<id>urn:sha1:839ee02531b5a3f7e52188011c355a6044748de3</id>
<content type='text'>
where as there are limited number(32) of mfi commands in the pool.
The mfi command pool is now restricted to 27 simultaneous accesses by using
a counting semaphore while calling the passthru function.

In the mrsas_cam.c source file there was a same function name mrsas_poll(),
which was same as the mrsas_poll() implemented in the mrsas.c file for the
polling interface.
To clearly distinguish the functionality by usage we have renamed the former
as mrsas_cam_poll().

In the passthru function let's say it has got an mfi command from the pool
but it has failed in one of the DMA function call which will lead to leak
an mfi command because in the ERROR case it directly returns and not freeing up
the occupied mfi command.

Reviewed by:	ambrisko
MFC after:	2 weeks
Sponsored by:	AVAGO Technologies
</content>
</entry>
<entry>
<title>This is a feature provided to run 32-bit linux binaries on FreeBSD 64bit</title>
<updated>2014-10-08T09:19:35Z</updated>
<author>
<name>Kashyap D Desai</name>
<email>kadesai@FreeBSD.org</email>
</author>
<published>2014-10-08T09:19:35Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=536094dc79eb98d023bcfce564fc3cdc00787090'/>
<id>urn:sha1:536094dc79eb98d023bcfce564fc3cdc00787090</id>
<content type='text'>
machine, for which 32bit compatibilty code has been added.
As in linux there is only one device entry that is used to fire IOCTL commands,
a new device entry megaraid_sas_ioctl_node is added for solely this
purpose.

From one dev node i.e mrgaraid_sa_ioctl_node we have to find out the
controller instance in case of multicontroller, for which one management info
structure has been added.

Reviewed by:	ambrisko
MFC after:		2 weeks
Sponsored by:	AVAGO Technologies
</content>
</entry>
<entry>
<title>Add mrsas(4) driver from LSI official support of newer MegaRAID SAS</title>
<updated>2014-05-07T16:16:49Z</updated>
<author>
<name>Doug Ambrisko</name>
<email>ambrisko@FreeBSD.org</email>
</author>
<published>2014-05-07T16:16:49Z</published>
<link rel='alternate' type='text/html' href='https://cgit-dev.freebsd.org/src-test/commit/?id=665484d8f0c735d367ecdfc53aa49c19060703b1'/>
<id>urn:sha1:665484d8f0c735d367ecdfc53aa49c19060703b1</id>
<content type='text'>
cards.  LSI has been maintaining this driver outside of the FreeBSD
tree.  It overlaps support of ThunderBolt and Invader cards that mfi(4)
supports.  By default mfi(4) will attach to cards.  If the tunable:
	hw.mfi.mrsas_enable=1
is set then mfi(4) will not probe and attach to these newer cards and
allow mrsas(4) to attach.  So by default this driver will not effect
a FreeBSD system unless mfi(4) is removed from the kernel or the
tunable is enabled.

mrsas(4) attaches disks to the CAM layer so it depends on CAM and devices
show up as /dev/daX.  mfiutil(8) does not work with mrsas.  The FreeBSD
version of MegaCli and StorCli from LSI do work with mrsas.  It appears
that StorCli only works with mrsas.  MegaCli appears to work with mfi(4)
and mrsas(4).

It would be good to add mfiutil(4) support to mrsas, emulations modes,
kernel logging, device aliases to ease the transition between mfi(4)
and mrsas(4).

Style issues should be resolved by LSI when they get committers approved.
The plan is get this driver in FreeBSD 9.3 to improve HW support.

Thanks to LSI for developing, testing and working with FreeBSD to
make this driver co-exist in FreeBSD.  This improves the overall
support of MegaRAID SAS.

Submitted by:	Kashyap Desai &lt;Kashyap.Desai@lsi.com&gt;
Reviewed by:	scottl
MFC after:	3 days
Sponsored by:	LSI
</content>
</entry>
</feed>
