[PATCH] rdmacg: fix a typo in rdmacg documentation

2018-10-23 Thread Rami Rosen
This patch fixes a typo in RDMA cgroup documentation. Signed-off-by: Rami Rosen --- Documentation/cgroup-v1/rdma.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/cgroup-v1/rdma.txt b/Documentation/cgroup-v1/rdma.txt index af618171e0eb..9bdb7fd03f83 100644

[PATCH] rdmacg: fix a typo in rdmacg documentation

2018-10-23 Thread Rami Rosen
This patch fixes a typo in RDMA cgroup documentation. Signed-off-by: Rami Rosen --- Documentation/cgroup-v1/rdma.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/cgroup-v1/rdma.txt b/Documentation/cgroup-v1/rdma.txt index af618171e0eb..9bdb7fd03f83 100644

[PATCH] devres: fix a typo

2018-10-21 Thread Rami Rosen
Signed-off-by: Rami Rosen This patch fixes a typo in lib/devres.c. --- lib/devres.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/devres.c b/lib/devres.c index faccf1a..f64cc21 100644 --- a/lib/devres.c +++ b/lib/devres.c @@ -290,7 +290,7 @@ static void

[PATCH] devres: fix a typo

2018-10-21 Thread Rami Rosen
Signed-off-by: Rami Rosen This patch fixes a typo in lib/devres.c. --- lib/devres.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/devres.c b/lib/devres.c index faccf1a..f64cc21 100644 --- a/lib/devres.c +++ b/lib/devres.c @@ -290,7 +290,7 @@ static void

[PATCH net-next] i40e: fix a typo in i40e_pf documentation.

2017-08-18 Thread Rami Rosen
This patch fixes a typo in i40e_pf object documentation; num_req_vfs refers to the number of VFs requested for the PF. Signed-off-by: Rami Rosen <rami.ro...@intel.com> --- drivers/net/ethernet/intel/i40e/i40e.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drive

[PATCH net-next] i40e: fix a typo in i40e_pf documentation.

2017-08-18 Thread Rami Rosen
This patch fixes a typo in i40e_pf object documentation; num_req_vfs refers to the number of VFs requested for the PF. Signed-off-by: Rami Rosen --- drivers/net/ethernet/intel/i40e/i40e.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/ethernet/intel/i40e/i40e.h

[PATCH] tools: remove unused variable in accounting/getdelays.c.

2017-05-18 Thread Rami Rosen
This patch removes the definition and the single assignement of the "done" variable in accounting/getdelays.c, as it is not used anywhere. Signed-off-by: Rami Rosen <rami.ro...@intel.com> --- tools/accounting/getdelays.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/

[PATCH] tools: remove unused variable in accounting/getdelays.c.

2017-05-18 Thread Rami Rosen
This patch removes the definition and the single assignement of the "done" variable in accounting/getdelays.c, as it is not used anywhere. Signed-off-by: Rami Rosen --- tools/accounting/getdelays.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/tools/accounting/getdelays

Re: [PATCH 0/9] net: thunderx: Adds XDP support

2017-05-03 Thread Rami Rosen
Thanks, Sunil. >with network stack: 0.32 Mpps >with XDP (XDP_TX): 3 Mpps >and XDP_DROP: 3.8 Mpps Interesting; May I ask - which packet size did you use ? Regards, Rami Rosen

Re: [PATCH 0/9] net: thunderx: Adds XDP support

2017-05-03 Thread Rami Rosen
Thanks, Sunil. >with network stack: 0.32 Mpps >with XDP (XDP_TX): 3 Mpps >and XDP_DROP: 3.8 Mpps Interesting; May I ask - which packet size did you use ? Regards, Rami Rosen

Re: [PATCH v3 08/23] Documentation: PCI: Add binding documentation for pci-test endpoint function

2017-03-09 Thread Rami Rosen
id : don't care > +interrupt_pin : Should be 1 - INTA, 2 - INTB, 3 - INTC, 4 -INTD > +msi_interrupts : Should be 1 to 32 depending on the number of msi Regards, Rami Rosen

Re: [PATCH v3 08/23] Documentation: PCI: Add binding documentation for pci-test endpoint function

2017-03-09 Thread Rami Rosen
id : don't care > +interrupt_pin : Should be 1 - INTA, 2 - INTB, 3 - INTC, 4 -INTD > +msi_interrupts : Should be 1 to 32 depending on the number of msi Regards, Rami Rosen

Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss gigabit ethernet driver

2016-11-26 Thread Rami Rosen
v_err(>pdev->dev, > + "invalid rcv-sequencer firmware size %zu\n", > fw->size); > + err = -EINVAL; Again, in the release label, always 0 is returned: > + goto release; > + } > + > > +release: > + release_firmware(fw); > + > + return 0; > +} > + Regards, Rami Rosen

Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss gigabit ethernet driver

2016-11-26 Thread Rami Rosen
v_err(>pdev->dev, > + "invalid rcv-sequencer firmware size %zu\n", > fw->size); > + err = -EINVAL; Again, in the release label, always 0 is returned: > + goto release; > + } > + > > +release: > + release_firmware(fw); > + > + return 0; > +} > + Regards, Rami Rosen

Re: [PATCH v4 01/10] ethernet: add sun8i-emac driver

2016-10-23 Thread Rami Rosen
> + u64 tx_used_desc; > + u64 napi_schedule; > + u64 napi_underflow; > +}; > + Trivial: typo, should be: can transfer > +/* The datasheet said that each descriptor can transfers up to 4096bytes > + * But latter, a register documentation reduce that value to 2048 Regards, Rami Rosen

Re: [PATCH v4 01/10] ethernet: add sun8i-emac driver

2016-10-23 Thread Rami Rosen
> + u64 tx_used_desc; > + u64 napi_schedule; > + u64 napi_underflow; > +}; > + Trivial: typo, should be: can transfer > +/* The datasheet said that each descriptor can transfers up to 4096bytes > + * But latter, a register documentation reduce that value to 2048 Regards, Rami Rosen

Re: [PATCHv11 3/3] rdmacg: Added documentation for rdmacg

2016-08-24 Thread Rami Rosen
of RDMA resources. "of of" should be only a single "of" > + mlx4_1 uctx=1 ah=0 pd=1 cq=4 qp=4 mr=100 srq=0 flow=10 > + ocrdma1 uctx=2 pd=2 ah=2 mr=20 mw=max cq=1 srq=1 qp=10 flow=10 Seems to be inconsistency here: in the first line you have qp=4 *before* srq=0, but in the seco

Re: [PATCHv11 3/3] rdmacg: Added documentation for rdmacg

2016-08-24 Thread Rami Rosen
of RDMA resources. "of of" should be only a single "of" > + mlx4_1 uctx=1 ah=0 pd=1 cq=4 qp=4 mr=100 srq=0 flow=10 > + ocrdma1 uctx=2 pd=2 ah=2 mr=20 mw=max cq=1 srq=1 qp=10 flow=10 Seems to be inconsistency here: in the first line you have qp=4 *before* srq=0, but in the seco

[PATCH] Documentation: ACPI: add copy_dsdt to ACPI format options.

2015-09-26 Thread Rami Rosen
commit aa2110cb1a7510f9b834adfb39b05d4843a35d35 ("ACPI: add boot option acpi=copy_dsdt to fix corrupt DSDT") added copy_dsdt as an ACPI boot option, but did not add it to ACPI format options in Documentation/kernel-parameters.txt. This patch fixes it. Signed-off-by:

[PATCH] Documentation: ACPI: add copy_dsdt to ACPI format options.

2015-09-26 Thread Rami Rosen
commit aa2110cb1a7510f9b834adfb39b05d4843a35d35 ("ACPI: add boot option acpi=copy_dsdt to fix corrupt DSDT") added copy_dsdt as an ACPI boot option, but did not add it to ACPI format options in Documentation/kernel-parameters.txt. This patch fixes it. Signed-off-by: Rami Ro

[PATCH] PCI: remove a broken link in quirks.c.

2015-07-01 Thread Rami Rosen
This patch removes a broken link in drivers/pci/quirks.c. Signed-off-by: Rami Rosen --- drivers/pci/quirks.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index c6dc1df..f78bcc0 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c

[PATCH] PCI: remove a broken link in quirks.c.

2015-07-01 Thread Rami Rosen
This patch removes a broken link in drivers/pci/quirks.c. Signed-off-by: Rami Rosen ramir...@gmail.com --- drivers/pci/quirks.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index c6dc1df..f78bcc0 100644 --- a/drivers/pci/quirks.c +++ b/drivers

Re: [PATCH v6] NET: Add ezchip ethernet driver

2015-06-22 Thread Rami Rosen
%d)\n", priv->irq); + return 0; + +out_netif_api: + netif_napi_del(>napi); +out_netdev: + if (err) + free_netdev(ndev); + + return err; +} Regards, Rami Rosen -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH v6] NET: Add ezchip ethernet driver

2015-06-22 Thread Rami Rosen
); +out_netdev: + if (err) + free_netdev(ndev); + + return err; +} Regards, Rami Rosen -- To unsubscribe from this list: send the line unsubscribe linux-kernel in Please read the FAQ at http://www.tux.org/lkml/

[PATCH v2] Fix typos in Documentation/edac.txt.

2015-06-19 Thread Rami Rosen
This patch fixes various typos in Documentation/edac.txt. Signed-off-by: Rami Rosen --- Documentation/edac.txt | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/Documentation/edac.txt b/Documentation/edac.txt index 73fff13..4df786e 100644 --- a/Documentation

[PATCH v2] Fix typos in Documentation/edac.txt.

2015-06-19 Thread Rami Rosen
This patch fixes various typos in Documentation/edac.txt. Signed-off-by: Rami Rosen ramir...@gmail.com --- Documentation/edac.txt | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/Documentation/edac.txt b/Documentation/edac.txt index 73fff13..4df786e 100644

[PATCH] Fix a typo in Documentation/edac.txt.

2015-06-17 Thread Rami Rosen
This patch Fixes a typo in Documentation/edac.txt. Signed-off-by: Rami Rosen --- Documentation/edac.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/edac.txt b/Documentation/edac.txt index 73fff13..9679433 100644 --- a/Documentation/edac.txt +++ b

[PATCH] Fix a typo in Documentation/edac.txt.

2015-06-17 Thread Rami Rosen
This patch Fixes a typo in Documentation/edac.txt. Signed-off-by: Rami Rosen ramir...@gmail.com --- Documentation/edac.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/edac.txt b/Documentation/edac.txt index 73fff13..9679433 100644 --- a/Documentation

Re: question about netif_rx

2013-08-14 Thread Rami Rosen
Hi, BTW, this is not the only NAPI issue here. When looking into cleanup of resources in this driver, a call to netif_napi_del() is missing (though there is a call to napi_disable(), which is not enough for proper cleanup). Best, Rami Rosen http://ramirose.wix.com/ramirosen On Tue, Aug 13, 2013

Re: question about netif_rx

2013-08-14 Thread Rami Rosen
Hi, BTW, this is not the only NAPI issue here. When looking into cleanup of resources in this driver, a call to netif_napi_del() is missing (though there is a call to napi_disable(), which is not enough for proper cleanup). Best, Rami Rosen http://ramirose.wix.com/ramirosen On Tue, Aug 13, 2013

Re: Missing iproute2 support for 802.1ad

2013-06-21 Thread Rami Rosen
uld look at the rest of the patch of Patrick McHardy ("add 802.1ad support"), it does not appear in the latest git tree of iproute2. Please look for example in ip/iplink_vlan.c b/ip/iplink_vlan.c in the explain() method and compare to the above mentioned patch. Regards, Rami Rosen htt

Re: Missing iproute2 support for 802.1ad

2013-06-21 Thread Rami Rosen
should look at the rest of the patch of Patrick McHardy (add 802.1ad support), it does not appear in the latest git tree of iproute2. Please look for example in ip/iplink_vlan.c b/ip/iplink_vlan.c in the explain() method and compare to the above mentioned patch. Regards, Rami Rosen http

[PATCH] sysfs: kill sysfs_sb declaration in fs/sysfs/inode.c.

2013-04-29 Thread Rami Rosen
This patch removes sysfs_sb declaration from fs/sysfs/inode.c (due to 0f4288ec6fcc1a47d1fa0241ec1c6dacd5a09e96, "Kill unused sysfs_sb variable"). Signed-off-by: Rami Rosen --- fs/sysfs/inode.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/fs/sysfs/inode.c b/fs/sysfs/ino

[PATCH] sysfs: kill sysfs_sb declaration in fs/sysfs/inode.c.

2013-04-29 Thread Rami Rosen
This patch removes sysfs_sb declaration from fs/sysfs/inode.c (due to 0f4288ec6fcc1a47d1fa0241ec1c6dacd5a09e96, Kill unused sysfs_sb variable). Signed-off-by: Rami Rosen ramir...@gmail.com --- fs/sysfs/inode.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/fs/sysfs/inode.c b/fs/sysfs

[PATCH] devcg: remove parent_cgroup.

2013-04-16 Thread Rami Rosen
In devcgroup_css_alloc(), there is no longer need for parent_cgroup. bd2953ebbb("devcg: propagate local changes down the hierarchy") made the variable parent_cgroup redundant. This patch removes parent_cgroup from devcgroup_css_alloc(). Signed-off-by: Rami Rosen --- security/device_cg

[PATCH] devcg: remove parent_cgroup.

2013-04-16 Thread Rami Rosen
In devcgroup_css_alloc(), there is no longer need for parent_cgroup. bd2953ebbb(devcg: propagate local changes down the hierarchy) made the variable parent_cgroup redundant. This patch removes parent_cgroup from devcgroup_css_alloc(). Signed-off-by: Rami Rosen ramir...@gmail.com --- security

Re: linux-next: Tree for Jan 4 (net/bridge)

2013-01-04 Thread Rami Rosen
/msg221168.html probably this patch, which was applied to net, was not pulled yet. I believe that since the commit above was not to net-next, this problem is not observed in your tree. Rami Rosen http://ramirose.wix.com/ramirosen On Fri, Jan 4, 2013 at 9:27 PM, Stephen Hemminger wrote: > O

Re: linux-next: Tree for Jan 4 (net/bridge)

2013-01-04 Thread Rami Rosen
.html probably this patch, which was applied to net, was not pulled yet. I believe that since the commit above was not to net-next, this problem is not observed in your tree. Rami Rosen http://ramirose.wix.com/ramirosen On Fri, Jan 4, 2013 at 9:27 PM, Stephen Hemminger shemmin...@vyatta.com wrote

Re: How do I get mac80211 debugfs?

2012-10-02 Thread Rami Rosen
[=y])│ -> Wireless (WIRELESS [=y]) Regards, Rami Rosen On Tue, Oct 2, 2012 at 1:06 AM, Kevin Cruz wrote: > Hey Johannes, > None of this should be too much of an effort for me. Thankfully I > don't find terminals to be daunting. >

Re: How do I get mac80211 debugfs?

2012-10-02 Thread Rami Rosen
])│ - Wireless (WIRELESS [=y]) Regards, Rami Rosen On Tue, Oct 2, 2012 at 1:06 AM, Kevin Cruz skate_...@yahoo.com wrote: Hey Johannes, None of this should be too much of an effort for me. Thankfully I don't find terminals to be daunting. Yeah ... we saw

[PATCH] [IPV6] Minor cleanup: remove unused definitions in net/ip6_fib.h

2008-02-07 Thread Rami Rosen
Hi, This patch removes some unused definitions and one method typedef declaration (f_pnode) in include/net/ip6_fib.h, as they are not used in the kernel. Regards, Rami Rosen Signed-off-by: Rami Rosen <[EMAIL PROTECTED]> diff --git a/include/net/ip6_fib.h b/include/net/ip6_fib.h index d

[PATCH] [IPV6] Minor clenup: remove two unused definitions in net/ip6_route.h

2008-02-07 Thread Rami Rosen
Hi, Remove IP6_RT_PRIO_FW and IP6_RT_FLOW_MASK definitions in include/net/ip6_route.h, as they are not used in the kernel. Regards, Rami Rosen Signed-off-by: Rami Rosen <[EMAIL PROTECTED]> diff --git a/include/net/ip6_route.h b/include/net/ip6_route.h index faac0ee..f99e4f0 100644 --- a/i

[PATCH] [IPV6] Minor cleanup: remove unused definitions in net/ip6_fib.h

2008-02-07 Thread Rami Rosen
Hi, This patch removes some unused definitions and one method typedef declaration (f_pnode) in include/net/ip6_fib.h, as they are not used in the kernel. Regards, Rami Rosen Signed-off-by: Rami Rosen [EMAIL PROTECTED] diff --git a/include/net/ip6_fib.h b/include/net/ip6_fib.h index d8d85b1

[PATCH] [IPV6] Minor clenup: remove two unused definitions in net/ip6_route.h

2008-02-07 Thread Rami Rosen
Hi, Remove IP6_RT_PRIO_FW and IP6_RT_FLOW_MASK definitions in include/net/ip6_route.h, as they are not used in the kernel. Regards, Rami Rosen Signed-off-by: Rami Rosen [EMAIL PROTECTED] diff --git a/include/net/ip6_route.h b/include/net/ip6_route.h index faac0ee..f99e4f0 100644 --- a/include

[PATCH] [PPPOL2TP] Label unused warning when CONFIG_PROC_FS is not set.

2008-02-05 Thread Rami Rosen
appropriate #ifdef. Regards, Rami Rosen Signed-off-by: Rami Rosen <[EMAIL PROTECTED]> diff --git a/drivers/net/pppol2tp.c b/drivers/net/pppol2tp.c index 1b51bb6..5aa0a80 100644 --- a/drivers/net/pppol2tp.c +++ b/drivers/net/pppol2tp.c @@ -2468,9 +2468,10 @@ static int __init pppol2tp_ini

[PATCH] [PPPOL2TP] Label unused warning when CONFIG_PROC_FS is not set.

2008-02-05 Thread Rami Rosen
appropriate #ifdef. Regards, Rami Rosen Signed-off-by: Rami Rosen [EMAIL PROTECTED] diff --git a/drivers/net/pppol2tp.c b/drivers/net/pppol2tp.c index 1b51bb6..5aa0a80 100644 --- a/drivers/net/pppol2tp.c +++ b/drivers/net/pppol2tp.c @@ -2468,9 +2468,10 @@ static int __init pppol2tp_init(void

Re: 2.6.24-mm1 - Build failure at net/sched/cls_flow.c:598

2008-02-04 Thread Rami Rosen
Hello, I had sent a patch recently (which is currently pending) which solves this problem. see: http://www.spinics.net/lists/netdev/msg54455.html Regards, Rami Rosen On Feb 5, 2008 1:25 AM, Andrew Morton <[EMAIL PROTECTED]> wrote: > On Mon, 04 Feb 2008 23:32:49 +0100 > Ti

Re: 2.6.24-mm1 - Build failure at net/sched/cls_flow.c:598

2008-02-04 Thread Rami Rosen
Hello, I had sent a patch recently (which is currently pending) which solves this problem. see: http://www.spinics.net/lists/netdev/msg54455.html Regards, Rami Rosen On Feb 5, 2008 1:25 AM, Andrew Morton [EMAIL PROTECTED] wrote: On Mon, 04 Feb 2008 23:32:49 +0100 Tilman Schmidt [EMAIL

[PATCH] [NET_SCHED] Add #ifdef CONFIG_NET_EMATCH in net/sched/cls_flow.c (latest git broken build)

2008-02-03 Thread Rami Rosen
. When CONFIG_NET_EMATCH is defined, the tcf_ematch_tree structure indeed holds a struct tcf_ematch_tree_hdr (hdr) as flow_dump() expects. This patch adds #ifdef CONFIG_NET_EMATCH in flow_dump to avoid this. The patch was build against latest 2.6 git net tree. Regards, Rami Rosen Signed-off

[PATCH] [NET_SCHED] Add #ifdef CONFIG_NET_EMATCH in net/sched/cls_flow.c (latest git broken build)

2008-02-03 Thread Rami Rosen
. When CONFIG_NET_EMATCH is defined, the tcf_ematch_tree structure indeed holds a struct tcf_ematch_tree_hdr (hdr) as flow_dump() expects. This patch adds #ifdef CONFIG_NET_EMATCH in flow_dump to avoid this. The patch was build against latest 2.6 git net tree. Regards, Rami Rosen Signed-off

[RFC][PATCH] Removal of FASTROUTE definition include/linux/if_packet.h

2007-08-02 Thread Rami Rosen
Hi, It seems that PACKET_FASTROUTE definition should be removed due to that fastroute is no longer supported. Regards, Rami Rosen -- Signed-off-by: Rami Rosen <[EMAIL PROTECTED]> --- linux-2.6.23-rc1-clean/include/linux/if_packet.h2007-05-03 12:07:59.000

[RFC][PATCH] Removal of FASTROUTE definition include/linux/if_packet.h

2007-08-02 Thread Rami Rosen
Hi, It seems that PACKET_FASTROUTE definition should be removed due to that fastroute is no longer supported. Regards, Rami Rosen -- Signed-off-by: Rami Rosen [EMAIL PROTECTED] --- linux-2.6.23-rc1-clean/include/linux/if_packet.h2007-05-03 12:07:59.0