kernel/linux.git/drivers/ntb/test, branch v6.18.21

Merge tag 'ntb-6.15' of https://github.com/jonmason/ntb

2025-04-04T21:23:07+00:00

Pull ntb fixes from Jon Mason: "Bug fixes for NTB Switchtec driver mw negative shift, Intel NTB link status db, ntb_perf double unmap (in error case), and MSI 64bit arithmetic. Also, add new AMD NTB PCI IDs, update AMD NTB maintainer, and pull in patch to reduce the stack usage in IDT driver" * tag 'ntb-6.15' of https://github.com/jonmason/ntb: ntb_hw_amd: Add NTB PCI ID for new gen CPU ntb: reduce stack usage in idt_scan_mws ntb: use 64-bit arithmetic for the MSI doorbell mask MAINTAINERS: Update AMD NTB maintainers ntb_perf: Delete duplicate dmaengine_unmap_put() call in perf_copy_chunk() ntb: intel: Fix using link status DB's ntb_hw_switchtec: Fix shift-out-of-bounds in switchtec_ntb_mw_set_trans

ntb_perf: Delete duplicate dmaengine_unmap_put() call in perf_copy_chunk()

2025-03-18T14:33:39+00:00

The function call “dmaengine_unmap_put(unmap)” was used in an if branch. The same call was immediately triggered by a subsequent goto statement. Thus avoid such a call repetition. This issue was detected by using the Coccinelle software. Fixes: 5648e56d03fa ("NTB: ntb_perf: Add full multi-port NTB API support") Cc: stable@vger.kernel.org Signed-off-by: Markus Elfring Signed-off-by: Jon Mason

ntb: ntb_pingpong: Switch to use hrtimer_setup()

2025-02-18T10:19:04+00:00

hrtimer_setup() takes the callback function pointer as argument and initializes the timer completely. Replace hrtimer_init() and the open coded initialization of hrtimer::function with the new setup mechanism. Patch was created by using Coccinelle. Signed-off-by: Nam Cao Signed-off-by: Thomas Gleixner Acked-by: Zack Rusin Link: https://lore.kernel.org/all/2b179e16ab830611fdcb09a33a5a531c9800679b.1738746904.git.namcao@linutronix.de

ntb_perf: Fix printk format

2024-09-20T14:50:38+00:00

The correct printk format is %pa or %pap, but not %pa[p]. Fixes: 99a06056124d ("NTB: ntb_perf: Fix address err in perf_copy_chunk") Signed-off-by: Max Hawking Signed-off-by: Jon Mason

NTB: ntb_tool: Switch to memdup_user_nul() helper

2023-08-18T13:50:06+00:00

Use memdup_user_nul() helper instead of open-coding to simplify the code. Signed-off-by: Ruan Jinjie Reviewed-by: Dave Jiang Reviewed-by: Dan Carpenter Reviewed-by: Serge Semin Signed-off-by: Jon Mason

dtivers: ntb: fix parameter check in perf_setup_dbgfs()

2023-08-18T13:50:06+00:00

Make IS_ERR() judge the debugfs_create_dir() function return in perf_setup_dbgfs(). Signed-off-by: Minjie Du Reviewed-by: Serge Semin Signed-off-by: Jon Mason

ntb: Remove error checking for debugfs_create_dir()

2023-08-18T13:50:06+00:00

It is expected that most callers should _ignore_ the errors return by debugfs_create_dir() in tool_setup_dbgfs() Signed-off-by: Wang Ming Reviewed-by: Dave Jiang Reviewed-by: Dan Carpenter Reviewed-by: Serge Semin Signed-off-by: Jon Mason

NTB: ntb_tool: Add check for devm_kcalloc

2023-07-08T14:51:56+00:00

As the devm_kcalloc may return NULL pointer, it should be better to add check for the return value, as same as the others. Fixes: 7f46c8b3a552 ("NTB: ntb_tool: Add full multi-port NTB API support") Signed-off-by: Jiasheng Jiang Reviewed-by: Serge Semin Reviewed-by: Dave Jiang Signed-off-by: Jon Mason

NTB: ntb_tool: uninitialized heap data in tool_fn_write()

2022-08-09T15:54:41+00:00

The call to: ret = simple_write_to_buffer(buf, size, offp, ubuf, size); will return success if it is able to write even one byte to "buf". The value of "*offp" controls which byte. This could result in reading uninitialized data when we do the sscanf() on the next line. This code is not really desigined to handle partial writes where *offp is non-zero and the "buf" is preserved and re-used between writes. Just ban partial writes and replace the simple_write_to_buffer() with copy_from_user(). Fixes: 578b881ba9c4 ("NTB: Add tool test client") Signed-off-by: Dan Carpenter Signed-off-by: Jon Mason

ntb: ntb_pingpong: remove redundant initialization of variables msg_data and spad_data

2021-09-05T22:04:46+00:00

The variables msg_data and spad_data are being initialized with values that are never read, they are being updated later on. The initializations are redundant and can be removed. Addresses-Coverity: ("Unused value") Signed-off-by: Colin Ian King Acked-by: Serge Semin Signed-off-by: Jon Mason