<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux.git/drivers/block/cciss.c, branch v2.6.38</title>
<subtitle>Linux kernel source tree</subtitle>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/'/>
<entry>
<title>cciss: make cciss_revalidate not loop through CISS_MAX_LUNS volumes unnecessarily.</title>
<updated>2011-01-19T15:25:02+00:00</updated>
<author>
<name>Stephen M. Cameron</name>
<email>StephenM.Cameron</email>
</author>
<published>2011-01-19T15:25:02+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=68264e9d6781f7163e92c517769bb470fa43f6cd'/>
<id>68264e9d6781f7163e92c517769bb470fa43f6cd</id>
<content type='text'>
Signed-off-by: Stephen M. Cameron &lt;scameron@beardog.cce.hp.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Signed-off-by: Stephen M. Cameron &lt;scameron@beardog.cce.hp.com&gt;
Signed-off-by: Andrew Morton &lt;akpm@linux-foundation.org&gt;
Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge branch 'for-2.6.38/drivers' of git://git.kernel.dk/linux-2.6-block</title>
<updated>2011-01-13T18:50:24+00:00</updated>
<author>
<name>Linus Torvalds</name>
<email>torvalds@linux-foundation.org</email>
</author>
<published>2011-01-13T18:50:24+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=7b0cb1bdac9e941141e3790785955ed933d476f3'/>
<id>7b0cb1bdac9e941141e3790785955ed933d476f3</id>
<content type='text'>
* 'for-2.6.38/drivers' of git://git.kernel.dk/linux-2.6-block:
  cciss: reinstate proper FIFO order of command queue list
  floppy: replace NO_GEOM macro with a function
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
* 'for-2.6.38/drivers' of git://git.kernel.dk/linux-2.6-block:
  cciss: reinstate proper FIFO order of command queue list
  floppy: replace NO_GEOM macro with a function
</pre>
</div>
</content>
</entry>
<entry>
<title>cciss: reinstate proper FIFO order of command queue list</title>
<updated>2011-01-10T20:50:33+00:00</updated>
<author>
<name>Jens Axboe</name>
<email>jaxboe@fusionio.com</email>
</author>
<published>2011-01-10T20:50:33+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=e6e1ee936d61d697735d17517678a626b7701ce4'/>
<id>e6e1ee936d61d697735d17517678a626b7701ce4</id>
<content type='text'>
Commit 8a3173de inadvertently changed the ordering when
switching to hlists. Change to regular list heads so we
can use tail list adds, this improves performance.

Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Commit 8a3173de inadvertently changed the ordering when
switching to hlists. Change to regular list heads so we
can use tail list adds, this improves performance.

Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix build error in drivers/block/cciss.c</title>
<updated>2010-12-21T05:21:49+00:00</updated>
<author>
<name>Linus Torvalds</name>
<email>torvalds@linux-foundation.org</email>
</author>
<published>2010-12-21T05:21:49+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=453434cf3fdcd3954bb52460e37d4945a0913d3e'/>
<id>453434cf3fdcd3954bb52460e37d4945a0913d3e</id>
<content type='text'>
.. caused by a missing semi-colon, introduced in commit 0fc13c8995cd
("cciss: fix cciss_revalidate panic").

Reported-by: Stephen Rothwell &lt;sfr@canb.auug.org.au&gt;
Reported-by: Thiago Farina &lt;tfransosi@gmail.com&gt;
Cc: Jens Axboe &lt;jaxboe@fusionio.com&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
.. caused by a missing semi-colon, introduced in commit 0fc13c8995cd
("cciss: fix cciss_revalidate panic").

Reported-by: Stephen Rothwell &lt;sfr@canb.auug.org.au&gt;
Reported-by: Thiago Farina &lt;tfransosi@gmail.com&gt;
Cc: Jens Axboe &lt;jaxboe@fusionio.com&gt;
Signed-off-by: Linus Torvalds &lt;torvalds@linux-foundation.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>cciss: fix cciss_revalidate panic</title>
<updated>2010-12-17T08:01:37+00:00</updated>
<author>
<name>Stephen M. Cameron</name>
<email>scameron@beardog.cce.hp.com</email>
</author>
<published>2010-12-17T08:01:37+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=0fc13c8995cd96f4123de400c71c223d80400ed9'/>
<id>0fc13c8995cd96f4123de400c71c223d80400ed9</id>
<content type='text'>
If you delete a logical drive, and then run BLKRRPART (e.g. via fdisk)
on a logical drive which is "after" the deleted logical drive in the h-&gt;drv[]
array, then cciss_revalidate panics because it will access the null pointer
h-&gt;drv[x] when x hits the deleted drive.

Signed-off-by: Stephen M. Cameron &lt;scameron@beardog.cce.hp.com&gt;
Cc: stable@kernel.org
Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
If you delete a logical drive, and then run BLKRRPART (e.g. via fdisk)
on a logical drive which is "after" the deleted logical drive in the h-&gt;drv[]
array, then cciss_revalidate panics because it will access the null pointer
h-&gt;drv[x] when x hits the deleted drive.

Signed-off-by: Stephen M. Cameron &lt;scameron@beardog.cce.hp.com&gt;
Cc: stable@kernel.org
Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>cciss: fix build for PROC_FS disabled</title>
<updated>2010-11-17T10:56:13+00:00</updated>
<author>
<name>Jens Axboe</name>
<email>jaxboe@fusionio.com</email>
</author>
<published>2010-11-17T10:56:13+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=bbe425cd9ae83eacd0c9f09df2bf56dc911a54cd'/>
<id>bbe425cd9ae83eacd0c9f09df2bf56dc911a54cd</id>
<content type='text'>
The recent patch to fix the removal of a non-existing proc
directory introduced this build problem for !CONFIG_PROC_FS:

drivers/block/cciss.c:4929: error: 'proc_cciss' undeclared (first use in this function)

Fix it by moving proc_cciss outside of the CONFIG_PROC_FS scope.

Reported-by: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The recent patch to fix the removal of a non-existing proc
directory introduced this build problem for !CONFIG_PROC_FS:

drivers/block/cciss.c:4929: error: 'proc_cciss' undeclared (first use in this function)

Fix it by moving proc_cciss outside of the CONFIG_PROC_FS scope.

Reported-by: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Merge branch 'for-2.6.37/drivers' into for-linus</title>
<updated>2010-11-10T13:51:27+00:00</updated>
<author>
<name>Jens Axboe</name>
<email>jaxboe@fusionio.com</email>
</author>
<published>2010-11-10T13:51:27+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=00e375e7e962f938f6b3c93e4cd097a5e26cc788'/>
<id>00e375e7e962f938f6b3c93e4cd097a5e26cc788</id>
<content type='text'>
Conflicts:
	drivers/block/cciss.c

Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Conflicts:
	drivers/block/cciss.c

Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>cciss: fix proc warning on attempt to remove non-existant directory</title>
<updated>2010-11-10T13:40:52+00:00</updated>
<author>
<name>Jens Axboe</name>
<email>jaxboe@fusionio.com</email>
</author>
<published>2010-11-08T13:29:13+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=90fdb0b98a62d78a0650b9fd3ddc58a48f71d740'/>
<id>90fdb0b98a62d78a0650b9fd3ddc58a48f71d740</id>
<content type='text'>
Randy reports that he gets the following stack trace when
removing the cciss module:

[  109.164277] Pid: 3463, comm: rmmod Not tainted 2.6.37-rc1 #7
[  109.164280] Call Trace:
[  109.164292]  [&lt;ffffffff8107eb8d&gt;] warn_slowpath_common+0xc6/0xf3
[  109.164299]  [&lt;ffffffff8107ecaa&gt;] warn_slowpath_fmt+0x5b/0x6b
[  109.164307]  [&lt;ffffffff8155175b&gt;] ? _raw_spin_unlock+0x40/0x4b
[  109.164313]  [&lt;ffffffff8123dd1e&gt;] remove_proc_entry+0x156/0x35e
[  109.164320]  [&lt;ffffffff812cd91b&gt;] ? do_raw_spin_unlock+0xff/0x10f
[  109.164327]  [&lt;ffffffff8113823d&gt;] ? trace_hardirqs_on+0x10/0x4a
[  109.164333]  [&lt;ffffffff8155162d&gt;] ? _raw_spin_unlock_irq+0x4c/0x7b
[  109.164339]  [&lt;ffffffff8154d4d1&gt;] ? wait_for_common+0x145/0x15e
[  109.164345]  [&lt;ffffffff81075337&gt;] ? default_wake_function+0x0/0x22
[  109.164357]  [&lt;ffffffffa0615a8f&gt;] cciss_cleanup+0xa9/0xc7 [cciss]
[  109.164365]  [&lt;ffffffff810d3cb0&gt;] sys_delete_module+0x2d6/0x368
[  109.164371]  [&lt;ffffffff8155036b&gt;] ? lockdep_sys_exit_thunk+0x35/0x67
[  109.164377]  [&lt;ffffffff810fdfaf&gt;] ? audit_syscall_entry+0x172/0x1a5
[  109.164383]  [&lt;ffffffff815502f5&gt;] ? trace_hardirqs_on_thunk+0x3a/0x3f
[  109.164389]  [&lt;ffffffff8100ea72&gt;] system_call_fastpath+0x16/0x1b
[  109.164394] ---[ end trace 88e8568246ed0b1d ]---

which will happen if you don't actually have an HP CISS adapter,
since it'll do an uncondional removal of a proc directory it
never attempted to create in that case.

Reported-by: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
Tested-by: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Randy reports that he gets the following stack trace when
removing the cciss module:

[  109.164277] Pid: 3463, comm: rmmod Not tainted 2.6.37-rc1 #7
[  109.164280] Call Trace:
[  109.164292]  [&lt;ffffffff8107eb8d&gt;] warn_slowpath_common+0xc6/0xf3
[  109.164299]  [&lt;ffffffff8107ecaa&gt;] warn_slowpath_fmt+0x5b/0x6b
[  109.164307]  [&lt;ffffffff8155175b&gt;] ? _raw_spin_unlock+0x40/0x4b
[  109.164313]  [&lt;ffffffff8123dd1e&gt;] remove_proc_entry+0x156/0x35e
[  109.164320]  [&lt;ffffffff812cd91b&gt;] ? do_raw_spin_unlock+0xff/0x10f
[  109.164327]  [&lt;ffffffff8113823d&gt;] ? trace_hardirqs_on+0x10/0x4a
[  109.164333]  [&lt;ffffffff8155162d&gt;] ? _raw_spin_unlock_irq+0x4c/0x7b
[  109.164339]  [&lt;ffffffff8154d4d1&gt;] ? wait_for_common+0x145/0x15e
[  109.164345]  [&lt;ffffffff81075337&gt;] ? default_wake_function+0x0/0x22
[  109.164357]  [&lt;ffffffffa0615a8f&gt;] cciss_cleanup+0xa9/0xc7 [cciss]
[  109.164365]  [&lt;ffffffff810d3cb0&gt;] sys_delete_module+0x2d6/0x368
[  109.164371]  [&lt;ffffffff8155036b&gt;] ? lockdep_sys_exit_thunk+0x35/0x67
[  109.164377]  [&lt;ffffffff810fdfaf&gt;] ? audit_syscall_entry+0x172/0x1a5
[  109.164383]  [&lt;ffffffff815502f5&gt;] ? trace_hardirqs_on_thunk+0x3a/0x3f
[  109.164389]  [&lt;ffffffff8100ea72&gt;] system_call_fastpath+0x16/0x1b
[  109.164394] ---[ end trace 88e8568246ed0b1d ]---

which will happen if you don't actually have an HP CISS adapter,
since it'll do an uncondional removal of a proc directory it
never attempted to create in that case.

Reported-by: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
Tested-by: Randy Dunlap &lt;randy.dunlap@oracle.com&gt;
Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>cciss: remove overlapping PCI IDs</title>
<updated>2010-10-28T12:33:27+00:00</updated>
<author>
<name>Mike Miller</name>
<email>mike.miller@hp.com</email>
</author>
<published>2010-10-28T12:33:27+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=6fa977520871b1fc0be64247abad02b678c13fda'/>
<id>6fa977520871b1fc0be64247abad02b678c13fda</id>
<content type='text'>
This patch removes the controller overlap between cciss and hpsa. It was
decided that no overlap should exist. All new controllers will use the hpsa
SCSI based driver.

Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch removes the controller overlap between cciss and hpsa. It was
decided that no overlap should exist. All new controllers will use the hpsa
SCSI based driver.

Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>block: cciss: fix information leak to userland</title>
<updated>2010-10-28T12:31:55+00:00</updated>
<author>
<name>Vasiliy Kulikov</name>
<email>segooon@gmail.com</email>
</author>
<published>2010-10-28T12:31:55+00:00</published>
<link rel='alternate' type='text/html' href='https://git.tavy.me/linux.git/commit/?id=7ab5118d7c2be650bc936894f159dc1c597badae'/>
<id>7ab5118d7c2be650bc936894f159dc1c597badae</id>
<content type='text'>
Structure IOCTL_Command_struct is copied to userland with
some padding fields at the end of the struct unitialized.
It leads to leaking of contents of kernel stack memory.

Signed-off-by: Vasiliy Kulikov &lt;segooon@gmail.com&gt;
Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Structure IOCTL_Command_struct is copied to userland with
some padding fields at the end of the struct unitialized.
It leads to leaking of contents of kernel stack memory.

Signed-off-by: Vasiliy Kulikov &lt;segooon@gmail.com&gt;
Signed-off-by: Jens Axboe &lt;jaxboe@fusionio.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
