Hi linux
http://nextavailablespace.com/supper.php?ring=yc1gu3uw6zqv38ka
S
Hi linux
http://nextavailablespace.com/supper.php?ring=yc1gu3uw6zqv38ka
S
Move error handling to service thread, and use mtip_set_timeout()
to set timeouts for HDIO_DRIVE_TASK and HDIO_DRIVE_CMD IOCTL commands.
Signed-off-by: Selvan Mani
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c | 71 +
drivers/block
Move error handling to service thread, and use mtip_set_timeout()
to set timeouts for HDIO_DRIVE_TASK and HDIO_DRIVE_CMD IOCTL commands.
Signed-off-by: Selvan Mani sm...@micron.com
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c | 71
Add open/close handlers to 'block_device_operations' instance.
Signed-off-by: Selvan Mani
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers
Fix incorrectly setting MTIP_DDF_SEC_LOCK_BIT
Signed-off-by: Selvan Mani
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c |5 +
1 files changed, 1 insertions(+), 4 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers/block/mtip32xx/mtip32xx.c
Fix incorrectly setting MTIP_DDF_SEC_LOCK_BIT
Signed-off-by: Selvan Mani sm...@micron.com
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c |5 +
1 files changed, 1 insertions(+), 4 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
Add open/close handlers to 'block_device_operations' instance.
Signed-off-by: Selvan Mani sm...@micron.com
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/drivers
is passed on to block drivers.
With respect to the block driver, the device is removed and cleaned up the
data structures. This check will stop I/O to a non-existent device at the
block layer.
Signed-off-by: Asai Thambi S P
---
block/blk-core.c |2 +-
1 files changed, 1 insertions(+), 1
and then remove dfs_parent.
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers/block/mtip32xx/mtip32xx.c
index 51628eb..27641bc 100644
--- a/drivers/block/mtip32xx
Increased timeout for STANDBY IMMEDIATE command to 2 minutes.
Signed-off-by: Selvan Mani
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c | 66 +++-
1 files changed, 35 insertions(+), 31 deletions(-)
diff --git a/drivers/block/mtip32xx
Increased timeout for STANDBY IMMEDIATE command to 2 minutes.
Signed-off-by: Selvan Mani sm...@micron.com
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c | 66 +++-
1 files changed, 35 insertions(+), 31 deletions
and then remove dfs_parent.
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers/block/mtip32xx/mtip32xx.c
index 51628eb..27641bc 100644
is passed on to block drivers.
With respect to the block driver, the device is removed and cleaned up the
data structures. This check will stop I/O to a non-existent device at the
block layer.
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
block/blk-core.c |2 +-
1 files changed, 1
Fix incorrectly releasing a reference to pci_dev in pci_remove()
Signed-off-by: Rajesh Kumar S
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c |2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers/block
Fix incorrectly releasing a reference to pci_dev in pci_remove()
Signed-off-by: Rajesh Kumar S rsamban...@micron.com
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c |2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/drivers
with a PCIe upstream
device with device id 0x5aXX
Signed-off-by: Asai Thambi S P
Signed-off-by: Sam Bradshaw
---
drivers/block/mtip32xx/mtip32xx.c | 53 +
1 files changed, 53 insertions(+), 0 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers
with a PCIe upstream
device with device id 0x5aXX
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
Signed-off-by: Sam Bradshaw sbrads...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c | 53 +
1 files changed, 53 insertions(+), 0 deletions(-)
diff --git
On 2/26/2014 11:29 AM, Christoph Hellwig wrote:
> Hi all,
>
> with blk-mq stabilizing in mainline and Jens using mtip32xx as tje major
> example drivers in the past is there any progress on getting the
> conversion finished and merged?
We are working on few changes in mtip32xx. After that, we
On 2/26/2014 11:29 AM, Christoph Hellwig wrote:
Hi all,
with blk-mq stabilizing in mainline and Jens using mtip32xx as tje major
example drivers in the past is there any progress on getting the
conversion finished and merged?
We are working on few changes in mtip32xx. After that, we will
Hi Jens,
Please include this for 3.14. It is a trivial change, hope it should be ok.
--
Regards,
Asai
After several experiments, deduced the the optimal number of unaligned
writes to be 2. Changing the value accordingly.
Signed-off-by: Asai Thambi S P
Signed-off-by: Sam Bradshaw
Hi Jens,
Please include this for 3.14. It is a trivial change, hope it should be ok.
--
Regards,
Asai
After several experiments, deduced the the optimal number of unaligned
writes to be 2. Changing the value accordingly.
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
Signed-off
On 07/12/2013 3:59 PM, Asai Thambi S P wrote:
On 5/23/2013 2:23 PM, David Milburn wrote:
Dynamically allocate buf to prevent warnings:
drivers/block/mtip32xx/mtip32xx.c: In function ‘mtip_hw_read_device_status’:
drivers/block/mtip32xx/mtip32xx.c:2823: warning: the frame size of 1056 bytes
On 07/12/2013 3:59 PM, Asai Thambi S P wrote:
On 5/23/2013 2:23 PM, David Milburn wrote:
Dynamically allocate buf to prevent warnings:
drivers/block/mtip32xx/mtip32xx.c: In function ‘mtip_hw_read_device_status’:
drivers/block/mtip32xx/mtip32xx.c:2823: warning: the frame size of 1056 bytes
On 09/08/2013 5:28 PM, Guenter Roeck wrote:
Hi all,
powerpc allmodconfig build on the latest upstream kernel results in:
ERROR: ".cpu_to_chip_id" [drivers/block/mtip32xx/mtip32xx.ko] undefined!
This is due to commit 15863ff3b (powerpc: Make chip-id information
available to userspace).
Not
On 09/08/2013 5:28 PM, Guenter Roeck wrote:
Hi all,
powerpc allmodconfig build on the latest upstream kernel results in:
ERROR: .cpu_to_chip_id [drivers/block/mtip32xx/mtip32xx.ko] undefined!
This is due to commit 15863ff3b (powerpc: Make chip-id information
available to userspace).
Not
2xx.c:2917: warning: the frame size of 1056 bytes
> is larger than 1024 bytes
>
> Signed-off-by: David Milburn
Acked-by: Asai Thambi S P
> ---
> drivers/block/mtip32xx/mtip32xx.c | 47
> +
> 1 files changed, 37 insertions(+), 10 de
of 1056 bytes
is larger than 1024 bytes
Signed-off-by: David Milburn dmilb...@redhat.com
Acked-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c | 47
+
1 files changed, 37 insertions(+), 10 deletions(-)
diff --git
On 5/9/2013 11:48 PM, Asai Thambi S P wrote:
>
> This patch add support for SRSI(Surprise Removal Surprise Insertion).
Jens,
Forgot to include. This patch is for 3.10
>
> Approach:
> -
> Surprise Removal:
> -
> On surprise removal of the device
On 5/9/2013 11:48 PM, Asai Thambi S P wrote:
This patch add support for SRSI(Surprise Removal Surprise Insertion).
Jens,
Forgot to include. This patch is for 3.10
Approach:
-
Surprise Removal:
-
On surprise removal of the device, gendisk, request queue, device
:
---
No change, this scenario follows the normal pci probe() function flow.
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c | 453 ++---
drivers/block/mtip32xx/mtip32xx.h | 18 +-
2 files changed, 289 insertions(+), 182 deletions(-)
diff --git
:
---
No change, this scenario follows the normal pci probe() function flow.
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c | 453 ++---
drivers/block/mtip32xx/mtip32xx.h | 18 +-
2 files changed, 289 insertions(+), 182
Workaround for handling unaligned writes: limit number of outstanding
unaligned writes
Signed-off-by: Sam Bradshaw
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c | 59 +---
drivers/block/mtip32xx/mtip32xx.h | 11 +++
2 files changed
Workaround for handling unaligned writes: limit number of outstanding
unaligned writes
Signed-off-by: Sam Bradshaw sbrads...@micron.com
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c | 59 +---
drivers/block/mtip32xx
On 4/14/2013 1:55 PM, Jens Axboe wrote:
> On Fri, Apr 12 2013, Asai Thambi S P wrote:
>>
>> Temporarily disabling TRIM support until TRIM related issues
>> are addressed in the firmware.
>
> How serious is this? We do have released kernels out there with the
> driv
On 4/14/2013 1:55 PM, Jens Axboe wrote:
On Fri, Apr 12 2013, Asai Thambi S P wrote:
Temporarily disabling TRIM support until TRIM related issues
are addressed in the firmware.
How serious is this? We do have released kernels out there with the
driver, you might want to consider a stable
Temporarily disabling TRIM support until TRIM related issues
are addressed in the firmware.
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers/block/mtip32xx
ue.
Reported-by: Dan Carpenter
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c | 18 --
1 files changed, 8 insertions(+), 10 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers/block/mtip32xx/mtip32xx.c
index 32c6780..d11ea92 100644
--- a/d
dan.carpen...@oracle.com
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c | 18 --
1 files changed, 8 insertions(+), 10 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers/block/mtip32xx/mtip32xx.c
index 32c6780
Temporarily disabling TRIM support until TRIM related issues
are addressed in the firmware.
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b
This patch adds a new debugfs entry 'device_status' in
/sys/kernel/debug/rssd. The value of this entry shows
devices online and those in the process of removing.
Signed-off-by: Sam Bradshaw
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c | 148
Fix to return 0 from pci probe in case of rebuild. If not, pci consider
probe has failed, and crash during rmmod.
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b
To recover from command timeouts, reset the device. In addition
to that improved timeout handling of PIO commands.
Signed-off-by: Sam Bradshaw
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c | 177 ++---
1 files changed, 87 insertions
Hi Jens,
This patchset includes the following. It was generated against your
for-3.9/drivers
* improved recovery for command timeout
* fix for a crash during rmmod
* add new debugfs entry 'device_status'
Asai Thambi S P (3):
mtip32xx: recovery from command timeout
mtip32xx: return 0 from
Hi Jens,
This patchset includes the following. It was generated against your
for-3.9/drivers
* improved recovery for command timeout
* fix for a crash during rmmod
* add new debugfs entry 'device_status'
Asai Thambi S P (3):
mtip32xx: recovery from command timeout
mtip32xx: return 0 from
To recover from command timeouts, reset the device. In addition
to that improved timeout handling of PIO commands.
Signed-off-by: Sam Bradshaw sbrads...@micron.com
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c | 177
Fix to return 0 from pci probe in case of rebuild. If not, pci consider
probe has failed, and crash during rmmod.
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c |2 ++
1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/drivers/block
This patch adds a new debugfs entry 'device_status' in
/sys/kernel/debug/rssd. The value of this entry shows
devices online and those in the process of removing.
Signed-off-by: Sam Bradshaw sbrads...@micron.com
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx
in
* mtip_command_cleanup()
* mtip_timeout_function()
* mtip_handle_tfe()
Signed-off-by: Kent Overstreet
Signed-off-by: Andrew Morton
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c | 86
in
* mtip_command_cleanup()
* mtip_timeout_function()
* mtip_handle_tfe()
Signed-off-by: Kent Overstreet koverstr...@google.com
Signed-off-by: Andrew Morton a...@linux-foundation.org
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block
On 1/14/2013 9:28 AM, Stephen Rothwell wrote:
> Hi Andrew,
>
> Today's linux-next merge of the akpm tree got a conflict in
> drivers/block/mtip32xx/mtip32xx.c between commit 16c906e51c6f ("mtip32xx:
> Add workqueue and NUMA support") from the tree and commit "mtip32xx:
> convert to batch
On 1/14/2013 9:28 AM, Stephen Rothwell wrote:
Hi Andrew,
Today's linux-next merge of the akpm tree got a conflict in
drivers/block/mtip32xx/mtip32xx.c between commit 16c906e51c6f (mtip32xx:
Add workqueue and NUMA support) from the tree and commit mtip32xx:
convert to batch completion from
When rebuild is in progress, disk->queue is yet to be created. Surprise
removing the device will call remove()-> del_gendisk(). del_gendisk()
expect disk->queue be not NULL. Fix is to call put_disk() when disk_queue
is NULL.
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/m
If an I/O command times out when a PIO command is active,
MTIP_PF_EH_ACTIVE_BIT is not cleared. This results in I/O
hang in the driver. Fix is to clear this bit.
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c |9 +
1 files changed, 5 insertions(+), 4
TRIM support added through vendor unique command.
Signed-off-by: Sam Bradshaw < sbrads...@micron.com>
Signed-off-by: Asai Thambi S P
---
drivers/block/mtip32xx/mtip32xx.c | 96 +
drivers/block/mtip32xx/mtip32xx.h | 18 +++
2 files change
This patch series contain changes for trim support and couple of fixes
Asai Thambi S P (3):
mtip32xx: add trim support
mtip32xx: fix for driver hang after a command timeout
mtip32xx: fix for crash when the device surprise removed during
rebuild
drivers/block/mtip32xx/mtip32xx.c
This patch series contain changes for trim support and couple of fixes
Asai Thambi S P (3):
mtip32xx: add trim support
mtip32xx: fix for driver hang after a command timeout
mtip32xx: fix for crash when the device surprise removed during
rebuild
drivers/block/mtip32xx/mtip32xx.c
TRIM support added through vendor unique command.
Signed-off-by: Sam Bradshaw sbrads...@micron.com
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c | 96 +
drivers/block/mtip32xx/mtip32xx.h | 18 +++
2 files
If an I/O command times out when a PIO command is active,
MTIP_PF_EH_ACTIVE_BIT is not cleared. This results in I/O
hang in the driver. Fix is to clear this bit.
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c |9 +
1 files changed, 5
When rebuild is in progress, disk-queue is yet to be created. Surprise
removing the device will call remove()- del_gendisk(). del_gendisk()
expect disk-queue be not NULL. Fix is to call put_disk() when disk_queue
is NULL.
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
---
drivers/block
Removed the dead code in mtip_hw_read_registers() and mtip_hw_read_flags().
Reported-by: Coverity
Signed-off-by: Asai Thambi S P
Signed-off-by: Selvan Mani
---
drivers/block/mtip32xx/mtip32xx.c |6 --
1 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/drivers/block/mtip32xx
Changed printk to be compliant with latest style changes
Signed-off-by: Asai Thambi S P
Signed-off-by: Selvan Mani
---
drivers/block/mtip32xx/mtip32xx.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers/block/mtip32xx
Proper reporting of write protect status on big-endian
Signed-off-by: Asai Thambi S P
Signed-off-by: Selvan Mani
---
drivers/block/mtip32xx/mtip32xx.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers/block/mtip32xx
Increased timeout for standby command to work with larger capacity drives
Signed-off-by: Asai Thambi S P
Signed-off-by: Selvan Mani
---
drivers/block/mtip32xx/mtip32xx.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/block/mtip32xx/mtip32xx.c
b/drivers/block
Return error for NCQ commands when the drive is in security locked state
Signed-off-by: Asai Thambi S P
Signed-off-by: Selvan Mani
---
drivers/block/mtip32xx/mtip32xx.c | 10 +-
drivers/block/mtip32xx/mtip32xx.h |2 ++
2 files changed, 11 insertions(+), 1 deletions(-)
diff --git
Added supported device IDs in pci table
Signed-off-by: Asai Thambi S P
Signed-off-by: Selvan Mani
---
drivers/block/mtip32xx/mtip32xx.c |8 +++-
drivers/block/mtip32xx/mtip32xx.h |8 +++-
2 files changed, 14 insertions(+), 2 deletions(-)
diff --git a/drivers/block/mtip32xx
This patchset adds support for new devices and few minor changes.
Asai Thambi S P (6):
mtip32xx: Add support for new devices
mtip32xx: Handle NCQ commands during the security locked state
mtip32xx: Increase timeout for standby command
mtip32xx: Proper reporting of write protect status
This patchset adds support for new devices and few minor changes.
Asai Thambi S P (6):
mtip32xx: Add support for new devices
mtip32xx: Handle NCQ commands during the security locked state
mtip32xx: Increase timeout for standby command
mtip32xx: Proper reporting of write protect status
Added supported device IDs in pci table
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
Signed-off-by: Selvan Mani sm...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c |8 +++-
drivers/block/mtip32xx/mtip32xx.h |8 +++-
2 files changed, 14 insertions(+), 2 deletions
Return error for NCQ commands when the drive is in security locked state
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
Signed-off-by: Selvan Mani sm...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c | 10 +-
drivers/block/mtip32xx/mtip32xx.h |2 ++
2 files changed, 11
Increased timeout for standby command to work with larger capacity drives
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
Signed-off-by: Selvan Mani sm...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers
Proper reporting of write protect status on big-endian
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
Signed-off-by: Selvan Mani sm...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/block/mtip32xx
Changed printk to be compliant with latest style changes
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
Signed-off-by: Selvan Mani sm...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/block/mtip32xx
Removed the dead code in mtip_hw_read_registers() and mtip_hw_read_flags().
Reported-by: Coverity
Signed-off-by: Asai Thambi S P asamymuth...@micron.com
Signed-off-by: Selvan Mani sm...@micron.com
---
drivers/block/mtip32xx/mtip32xx.c |6 --
1 files changed, 0 insertions(+), 6 deletions
On Wed, Jul 25, 2007 at 12:21:49PM +0530, Srinivasa Ds wrote:
>
> When I was testing kprobes on x86_64 and I come across the below error
> message
> on latest 2.6.23-rc1 kernel.
> ==
> Trying to vfree() bad address (c20002233199)
> WARNING: at
On Wed, Jul 25, 2007 at 12:21:49PM +0530, Srinivasa Ds wrote:
When I was testing kprobes on x86_64 and I come across the below error
message
on latest 2.6.23-rc1 kernel.
==
Trying to vfree() bad address (c20002233199)
WARNING: at
allows to configure and mark the kernel text and
kernel data as read-only separately.
Signed-off-by: Prasanna S P. <[EMAIL PROTECTED]>
arch/i386/Kconfig.debug |8
arch/i386/mm/init.c | 22 --
2 files changed, 24 insertions(+), 6 deletions(-)
diff -p
to configure and mark the kernel text and
kernel data as read-only separately.
Signed-off-by: Prasanna S P. [EMAIL PROTECTED]
arch/i386/Kconfig.debug |8
arch/i386/mm/init.c | 22 --
2 files changed, 24 insertions(+), 6 deletions(-)
diff -puN arch/i386
Hi
I plan to hire a famous, professional Linux software
development team or company to port the whole Linux
Kernel and device drivers to a new type of CPU and
hardware, with fixed budget and a time limit.
What is the name of the professional, experienced
software company or team familiar with
Hi
I plan to hire a famous, professional Linux software
development team or company to port the whole Linux
Kernel and device drivers to a new type of CPU and
hardware, with fixed budget and a time limit.
What is the name of the professional, experienced
software company or team familiar with
On Wed, Jun 20, 2007 at 10:23:21PM -0700, Arjan van de Ven wrote:
> On Wed, 2007-06-20 at 16:50 -0700, Linus Torvalds wrote:
> >
> > On Wed, 20 Jun 2007, Arjan van de Ven wrote:
> > >
> > > the real fix would be something like this instead:
> >
> > If people can test this, and confirm it works,
On Wed, Jun 20, 2007 at 10:23:21PM -0700, Arjan van de Ven wrote:
On Wed, 2007-06-20 at 16:50 -0700, Linus Torvalds wrote:
On Wed, 20 Jun 2007, Arjan van de Ven wrote:
the real fix would be something like this instead:
If people can test this, and confirm it works, please send a
On Tue, Jun 19, 2007 at 06:44:30AM -0700, Arjan van de Ven wrote:
> On Tue, 2007-06-19 at 09:30 -0400, Mathieu Desnoyers wrote:
> > * Arjan van de Ven ([EMAIL PROTECTED]) wrote:
> > > On Tue, 2007-06-19 at 15:38 +0530, S. P. Prasanna wrote:
> > > > This patch is a
This patch is a quick fix for x86_64 to enable kprobes only if DEBUG_RODATA is
not configured. Since DEBUG_RODATA markes the kernel text pages as read-only.
Signed-off-by: Prasanna S P. <[EMAIL PROTECTED]>
arch/x86_64/Kconfig |6 +++---
1 files changed, 3 insertions(+), 3 del
KPROBES dependent on !DEBUG_RODATA. That would be a one liner.
>
Please find the quick fix as per your suggestion below.
Thanks
Prasanna
This patch is a quick fix to enable kprobes only if DEBUG_RODATA is
not configured. Since DEBUG_RODATA markes the kernel text pages as read-only.
Signed-off-b
the kernel text pages as read-only.
Signed-off-by: Prasanna S P. [EMAIL PROTECTED]
arch/i386/Kconfig |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff -puN arch/i386/Kconfig~kprobes-quick-fix-mark-ro-data-i386
arch/i386/Kconfig
--- linux-2.6.22-rc5/arch/i386/Kconfig~kprobes-quick
This patch is a quick fix for x86_64 to enable kprobes only if DEBUG_RODATA is
not configured. Since DEBUG_RODATA markes the kernel text pages as read-only.
Signed-off-by: Prasanna S P. [EMAIL PROTECTED]
arch/x86_64/Kconfig |6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff
On Tue, Jun 19, 2007 at 06:44:30AM -0700, Arjan van de Ven wrote:
On Tue, 2007-06-19 at 09:30 -0400, Mathieu Desnoyers wrote:
* Arjan van de Ven ([EMAIL PROTECTED]) wrote:
On Tue, 2007-06-19 at 15:38 +0530, S. P. Prasanna wrote:
This patch is a quick fix for x86_64 to enable kprobes only
On Mon, Jun 18, 2007 at 01:43:04AM +0200, Adrian Bunk wrote:
> Allow gcc to perform show_registers() type checking also with
> CONFIG_KPROBES=n.
>
Adrian,
Please implement it as Andrew suggested, when this was patch was posted
eariler. Refer this discussion thread below.
On Mon, Jun 18, 2007 at 01:43:04AM +0200, Adrian Bunk wrote:
Allow gcc to perform show_registers() type checking also with
CONFIG_KPROBES=n.
Adrian,
Please implement it as Andrew suggested, when this was patch was posted
eariler. Refer this discussion thread below.
This patch fixes the problem of page protection introduced by
CONFIG_DEBUG_RODATA for x86_64 architecture. As per Andi
Kleen's suggestion, the kernel text pages are marked writeable
only for a short duration to insert or remove the breakpoints.
Signed-off-by: Prasanna S P<[EMAIL PROTECTED]&g
gt;
> Ian
Ian,
Please find the fix as suggested by Andi Kleen
for the above stated problem.
Thanks
Prasanna
This patch fixes the problem of page protection introduced by
CONFIG_DEBUG_RODATA. CONFIG_DEBUG_RODATA marks the text pages as
read-only, hence kprobes is unable to insert breakpoints in the
marks the text pages as
read-only, hence kprobes is unable to insert breakpoints in the
kernel text. This patch overrides the page protection when adding
or removing a probe for the i386 architecture.
Signed-off-by: Prasanna S P[EMAIL PROTECTED]
Ack-ed-by: Jim Keniston [EMAIL PROTECTED]
arch/i386
This patch fixes the problem of page protection introduced by
CONFIG_DEBUG_RODATA for x86_64 architecture. As per Andi
Kleen's suggestion, the kernel text pages are marked writeable
only for a short duration to insert or remove the breakpoints.
Signed-off-by: Prasanna S P[EMAIL PROTECTED]
Ack-ed
On Wed, Feb 21, 2007 at 03:35:13PM -0500, Alan Stern wrote:
> Going back to something you mentioned earlier...
>
[...]
> On Fri, 9 Feb 2007, Roland McGrath wrote:
> There are two things I am uncertain about: vm86 mode and kprobes. I don't
> know anything about how either of them works. Judging
On Wed, Feb 21, 2007 at 03:35:13PM -0500, Alan Stern wrote:
Going back to something you mentioned earlier...
[...]
On Fri, 9 Feb 2007, Roland McGrath wrote:
There are two things I am uncertain about: vm86 mode and kprobes. I don't
know anything about how either of them works. Judging from
On Tue, Jan 16, 2007 at 04:27:40PM -0500, Mathieu Desnoyers wrote:
> Hi,
>
> I have looked at kprobes code and have some questions for you. I would really
> like to use it to patch dynamically my marker immediate value by doing code
> patching. Using an int3 seems like the right way to handle
On Tue, Jan 16, 2007 at 04:27:40PM -0500, Mathieu Desnoyers wrote:
Hi,
I have looked at kprobes code and have some questions for you. I would really
like to use it to patch dynamically my marker immediate value by doing code
patching. Using an int3 seems like the right way to handle this wrt
On Sun, Jan 07, 2007 at 01:00:08PM -0500, Alan Stern wrote:
> Has the kwatch patch (hardware watchpooint debugging for x86) been updated
> to the current kernel? Is it available anywhere?
Hi Alan,
On Sun, Jan 07,
On Sun, Jan 07, 2007 at 01:00:08PM -0500, Alan Stern wrote:
Has the kwatch patch (hardware watchpooint debugging for x86) been updated
to the current kernel? Is it available anywhere?
Hi Alan,
On Sun, Jan 07, 2007
100 matches
Mail list logo