Re: [PATCHv2 net-next] dropwatch: Support monitoring of dropped frames

2020-08-05 Thread Neil Horman
On Tue, Aug 04, 2020 at 04:14:14PM -0700, David Miller wrote: > From: izabela.bakoll...@gmail.com > Date: Tue, 4 Aug 2020 18:09:08 +0200 > > > @@ -1315,6 +1334,53 @@ static int net_dm_cmd_trace(struct sk_buff *skb, > > return -EOPNOTSUPP; > > } > > > > +static int net_dm_interface_start(st

Re: [Linux-kernel-mentees] [PATCHv2 net-next] dropwatch: Support monitoring of dropped frames

2020-08-04 Thread Neil Horman
On Tue, Aug 04, 2020 at 02:28:28PM -0700, Cong Wang wrote: > On Tue, Aug 4, 2020 at 9:14 AM wrote: > > > > From: Izabela Bakollari > > > > Dropwatch is a utility that monitors dropped frames by having userspace > > record them over the dropwatch protocol over a file. This augument > > allows live

Re: [PATCH] sctp: Fix sk_buff leak when receiving a datagram

2020-06-13 Thread Neil Horman
On Sat, Jun 13, 2020 at 08:39:25PM +0800, Xiyu Yang wrote: > In sctp_skb_recv_datagram(), the function fetch a sk_buff object from > the receiving queue to "skb" by calling skb_peek() or __skb_dequeue() > and return its reference to the caller. > > However, when calling __skb_dequeue() successfull

Re: [PATCH] sctp: Replace zero-length array with flexible-array

2020-05-07 Thread Neil Horman
__be16 hmac_id; > - __u8 hmac[0]; > + __u8 hmac[]; > }; > > struct sctp_auth_chunk { > @@ -733,7 +733,7 @@ struct sctp_infox { > > struct sctp_reconf_chunk { > struct sctp_chunkhdr chunk_hdr; > - __u8 params[0]; > + __u8 params[]; > }; > > struct sctp_strreset_outreq { > @@ -741,13 +741,13 @@ struct sctp_strreset_outreq { > __be32 request_seq; > __be32 response_seq; > __be32 send_reset_at_tsn; > - __be16 list_of_streams[0]; > + __be16 list_of_streams[]; > }; > > struct sctp_strreset_inreq { > struct sctp_paramhdr param_hdr; > __be32 request_seq; > - __be16 list_of_streams[0]; > + __be16 list_of_streams[]; > }; > > struct sctp_strreset_tsnreq { > > Acked-by: Neil Horman

Re: [PATCH 07/15] drop_monitor: work around gcc-10 stringop-overflow warning

2020-05-01 Thread Neil Horman
gt; *location) >*/ > __nla_reserve_nohdr(dskb, sizeof(struct net_dm_drop_point)); > nla->nla_len += NLA_ALIGN(sizeof(struct net_dm_drop_point)); > - memcpy(msg->points[msg->entries].pc, &location, sizeof(void *)); > - msg->points[msg->entries].count = 1; > + memcpy(point->pc, &location, sizeof(void *)); > + point->count = 1; > msg->entries++; > > if (!timer_pending(&data->send_timer)) { Acked-by: Neil Horman

Re: [PATCH 1/4] net: sctp: Rename fallthrough label to unhandled

2019-10-11 Thread Neil Horman
fy_param(struct net > *net, > retval = SCTP_IERROR_ABORT; > } > break; > -fallthrough: > +unhandled: > default: > pr_debug("%s: unrecognized param:%d for chunk:%d\n", >__func__, ntohs(param.p->type), cid); > -- > 2.15.0 > > I'm still not a fan of the pseudo keyword fallthrough, but I don't have a problem in renaming the label, so Acked-by: Neil Horman

Re: [PATCH] lib/generic-radix-tree.c: add kmemleak annotations

2019-10-04 Thread Neil Horman
_genradix *radix, > size_t offset, > } > > if (new_node) > - free_page((unsigned long) new_node); > + genradix_free_node(new_node); > > return &n->data[offset]; > } > @@ -191,7 +211,7 @@ static void genradix_free_recurse(struct genradix_node > *n, unsigned level) > genradix_free_recurse(n->children[i], level - > 1); > } > > - free_page((unsigned long) n); > + genradix_free_node(n); > } > > int __genradix_prealloc(struct __genradix *radix, size_t size, > -- > 2.23.0 > > Acked-by: Neil Horman

Re: [PATCH net 0/2] fix memory leak for sctp_do_bind

2019-09-10 Thread Neil Horman
ctp_get_port_local > sctp: destroy bucket if failed to bind addr > > net/sctp/socket.c | 13 +++-- > 1 file changed, 7 insertions(+), 6 deletions(-) > > -- > 2.20.1 > > Series Acked-by: Neil Horman

[tip: x86/apic] x86/apic/vector: Warn when vector space exhaustion breaks affinity

2019-08-28 Thread tip-bot2 for Neil Horman
The following commit has been merged into the x86/apic branch of tip: Commit-ID: 743dac494d61d991967ebcfab92e4f80dc7583b3 Gitweb: https://git.kernel.org/tip/743dac494d61d991967ebcfab92e4f80dc7583b3 Author:Neil Horman AuthorDate:Thu, 22 Aug 2019 10:34:21 -04:00 Committer

[PATCH V2] x86: Add irq spillover warning

2019-08-22 Thread Neil Horman
y t...@lintronix.de) Signed-off-by: Neil Horman Reported-by: dju...@redhat.com CC: Thomas Gleixner CC: Ingo Molnar CC: Borislav Petkov CC: "H. Peter Anvin" CC: x...@kernel.org --- arch/x86/kernel/apic/vector.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/x8

Re: [PATCH] net: sctp: Rename fallthrough label to unhandled

2019-08-04 Thread Neil Horman
On Fri, Aug 02, 2019 at 04:19:32PM -0700, David Miller wrote: > From: Joe Perches > Date: Fri, 02 Aug 2019 10:47:34 -0700 > > > On Wed, 2019-07-31 at 08:16 -0400, Neil Horman wrote: > >> On Wed, Jul 31, 2019 at 04:32:43AM -0700, Joe Perches wrote: > >> > On

Re: [PATCH] net: sctp: Rename fallthrough label to unhandled

2019-08-02 Thread Neil Horman
fy_param(struct net > *net, > retval = SCTP_IERROR_ABORT; > } > break; > -fallthrough: > +unhandled: > default: > pr_debug("%s: unrecognized param:%d for chunk:%d\n", > __func__, ntohs(param.p->type), cid); > -- > 2.15.0 > > Yeah, it seems reasonable to me, though I'm still not comfortable with defining fallthrough as a macrotized keyword, but thats a debate for another thread Acked-by: Neil Horman

Re: [RFC PATCH] compiler_attributes.h: Add 'fallthrough' pseudo keyword for switch/case use

2019-08-02 Thread Neil Horman
On Thu, Aug 01, 2019 at 10:26:29PM +0200, Miguel Ojeda wrote: > On Thu, Aug 1, 2019 at 10:10 PM wrote: > > > > I'm not disagreeing... I think using a macro makes sense. > > It is either a macro or waiting for 5+ years (while we keep using the > comment style) :-) > > In case it helps to make one

Re: [PATCH] net: sctp: Rename fallthrough label to unhandled

2019-08-01 Thread Neil Horman
On Thu, Aug 01, 2019 at 10:42:31AM -0700, Joe Perches wrote: > On Thu, 2019-08-01 at 06:50 -0400, Neil Horman wrote: > > On Wed, Jul 31, 2019 at 03:23:46PM -0700, Joe Perches wrote: > [] > > You can say that if you want, but you made the point that your think the > > macro

Re: [PATCH] net: sctp: Rename fallthrough label to unhandled

2019-08-01 Thread Neil Horman
On Wed, Jul 31, 2019 at 03:23:46PM -0700, Joe Perches wrote: > On Wed, 2019-07-31 at 16:58 -0400, Neil Horman wrote: > > On Wed, Jul 31, 2019 at 09:35:31AM -0700, Joe Perches wrote: > > > On Wed, 2019-07-31 at 08:16 -0400, Neil Horman wrote: > > > > On Wed, Jul 31,

Re: [PATCH] net: sctp: Rename fallthrough label to unhandled

2019-07-31 Thread Neil Horman
On Wed, Jul 31, 2019 at 09:35:31AM -0700, Joe Perches wrote: > On Wed, 2019-07-31 at 08:16 -0400, Neil Horman wrote: > > On Wed, Jul 31, 2019 at 04:32:43AM -0700, Joe Perches wrote: > > > On Wed, 2019-07-31 at 07:19 -0400, Neil Horman wrote: > > > > On Tue, Jul 30,

Re: [PATCH] net: sctp: Rename fallthrough label to unhandled

2019-07-31 Thread Neil Horman
On Wed, Jul 31, 2019 at 04:32:43AM -0700, Joe Perches wrote: > On Wed, 2019-07-31 at 07:19 -0400, Neil Horman wrote: > > On Tue, Jul 30, 2019 at 10:04:37PM -0700, Joe Perches wrote: > > > fallthrough may become a pseudo reserved keyword so this only use of > > > fallt

Re: [PATCH] net: sctp: Rename fallthrough label to unhandled

2019-07-31 Thread Neil Horman
On Tue, Jul 30, 2019 at 10:04:37PM -0700, Joe Perches wrote: > fallthrough may become a pseudo reserved keyword so this only use of > fallthrough is better renamed to allow it. > > Signed-off-by: Joe Perches Are you referring to the __attribute__((fallthrough)) statement that gcc supports? If so

Re: [PATCH] x86: Add irq spillover warning

2019-07-16 Thread Neil Horman
On Tue, Jul 16, 2019 at 09:05:30PM +0200, Thomas Gleixner wrote: > Neil, > > On Tue, 16 Jul 2019, Neil Horman wrote: > > On Tue, Jul 16, 2019 at 05:57:31PM +0200, Thomas Gleixner wrote: > > > On Tue, 16 Jul 2019, Neil Horman wrote: > > > > If a cpu has more th

Re: [PATCH] x86: Add irq spillover warning

2019-07-16 Thread Neil Horman
On Tue, Jul 16, 2019 at 05:57:31PM +0200, Thomas Gleixner wrote: > Neil, > > On Tue, 16 Jul 2019, Neil Horman wrote: > > > On Intel hardware, cpus are limited in the number of irqs they can > > have affined to them (currently 240), based on section 10.5.2 of: > >

[PATCH] x86: Add irq spillover warning

2019-07-16 Thread Neil Horman
-off-by: Neil Horman Reported-by: dju...@redhat.com CC: Thomas Gleixner CC: Ingo Molnar CC: Borislav Petkov CC: "H. Peter Anvin" CC: x...@kernel.org --- arch/x86/kernel/irq.c | 8 1 file changed, 8 insertions(+) diff --git a/arch/x86/kernel/irq.c b/arch/x86/kernel/i

Re: [PATCH] net: sctp: fix warning "NULL check before some freeing functions is not needed"

2019-07-16 Thread Neil Horman
-2680,8 +2677,7 @@ static int sctp_process_param(struct sctp_association > *asoc, > if (!ep->auth_enable) > goto fall_through; > > - if (asoc->peer.peer_chunks) > - kfree(asoc->peer.peer_chunks); > + kfree(asoc->peer.peer_chunks); > asoc->peer.peer_chunks = kmemdup(param.p, > ntohs(param.p->length), gfp); > if (!asoc->peer.peer_chunks) > -- > 2.7.4 > > Acked-by: Neil Horman

Re: [PATCH 19/87] i2c: busses: Remove call to memset after dmam_alloc_coherent

2019-06-28 Thread Neil Horman
+++ b/drivers/i2c/busses/i2c-ismt.c > @@ -781,8 +781,6 @@ static int ismt_dev_init(struct ismt_priv *priv) > if (!priv->hw) > return -ENOMEM; > > - memset(priv->hw, 0, (ISMT_DESC_ENTRIES * sizeof(struct ismt_desc))); > - > priv->head = 0; > init_completion(&priv->cmp); > > -- > 2.11.0 > > Acked-by: Neil Horman

Re: [PATCH] sctp: Add rcu lock to protect dst entry in sctp_transport_route

2019-06-13 Thread Neil Horman
On Thu, Jun 13, 2019 at 10:37:51AM +0800, Su Yanjun wrote: > > 在 2019/6/12 21:13, Neil Horman 写道: > > On Tue, Jun 11, 2019 at 10:33:17AM +0800, Su Yanjun wrote: > > > 在 2019/6/10 19:12, Neil Horman 写道: > > > > On Mon, Jun 10, 2019 at 11:20:00AM +0800, Su Yanju

Re: [PATCH] sctp: Add rcu lock to protect dst entry in sctp_transport_route

2019-06-12 Thread Neil Horman
On Tue, Jun 11, 2019 at 10:33:17AM +0800, Su Yanjun wrote: > > 在 2019/6/10 19:12, Neil Horman 写道: > > On Mon, Jun 10, 2019 at 11:20:00AM +0800, Su Yanjun wrote: > > > syzbot found a crash in rt_cache_valid. Problem is that when more > > > threads release dst in s

Re: [PATCH] sctp: Add rcu lock to protect dst entry in sctp_transport_route

2019-06-10 Thread Neil Horman
On Mon, Jun 10, 2019 at 11:20:00AM +0800, Su Yanjun wrote: > syzbot found a crash in rt_cache_valid. Problem is that when more > threads release dst in sctp_transport_route, the route cache can > be freed. > > As follows, > p1: > sctp_transport_route > dst_release > get_dst > > p2: > sctp_tra

Re: [PATCH v3] coredump: Split pipe command whitespace before expanding template

2019-06-06 Thread Neil Horman
kernel_siginfo_t *siginfo) > retval = call_usermodehelper_exec(sub_info, > UMH_WAIT_EXEC); > > - argv_free(helper_argv); > + kfree(helper_argv); > if (retval) { > printk(KERN_INFO "Core dump to |%s pipe failed\n", > cn.corename); > @@ -766,6 +799,7 @@ void do_coredump(const kernel_siginfo_t *siginfo) > if (ispipe) > atomic_dec(&core_dump_count); > fail_unlock: > + kfree(argv); > kfree(cn.corename); > coredump_finish(mm, core_dumped); > revert_creds(old_cred); > -- > 2.20.1 > > Acked-by: Neil Horman

Re: [PATCH net-next] net: Drop unlikely before IS_ERR(_OR_NULL)

2019-06-05 Thread Neil Horman
On Wed, Jun 05, 2019 at 10:24:26PM +0800, Kefeng Wang wrote: > IS_ERR(_OR_NULL) already contain an 'unlikely' compiler flag, > so no need to do that again from its callers. Drop it. > > Cc: "David S. Miller" > Cc: Alexey Kuznetsov > Cc: Hideaki YOSHIFUJI

Re: [PATCH] net: sctp: fix memory leak in sctp_send_reset_streams

2019-06-02 Thread Neil Horman
nstr_list slab released, if any outq is found > to be non empty. > > Freeing the slab in question before bailing out fixes it. > > Fixes: d570a59c5b5f ("sctp: only allow the out stream reset when the stream > outq is empty") > Reported-by: syzbot > Reported-by:

Re: memory leak in sctp_process_init

2019-05-31 Thread Neil Horman
On Fri, May 31, 2019 at 09:42:42AM -0300, Marcelo Ricardo Leitner wrote: > On Thu, May 30, 2019 at 03:56:34PM -0400, Neil Horman wrote: > > On Thu, May 30, 2019 at 12:17:05PM -0300, Marcelo Ricardo Leitner wrote: > ... > > > --- a/net/sctp/sm_sideeffect.c > > >

Re: memory leak in sctp_process_init

2019-05-30 Thread Neil Horman
On Thu, May 30, 2019 at 12:17:05PM -0300, Marcelo Ricardo Leitner wrote: > On Thu, May 30, 2019 at 10:20:11AM -0400, Neil Horman wrote: > > On Wed, May 29, 2019 at 08:37:57PM -0300, Marcelo Ricardo Leitner wrote: > > > On Wed, May 29, 2019 at 03:07:09PM -0400, Neil Horman wrote:

Re: memory leak in sctp_process_init

2019-05-30 Thread Neil Horman
On Wed, May 29, 2019 at 08:37:57PM -0300, Marcelo Ricardo Leitner wrote: > On Wed, May 29, 2019 at 03:07:09PM -0400, Neil Horman wrote: > > --- a/net/sctp/sm_make_chunk.c > > +++ b/net/sctp/sm_make_chunk.c > > @@ -2419,9 +2419,12 @@ int sctp_process_init(struct sctp_associati

Re: memory leak in sctp_process_init

2019-05-30 Thread Neil Horman
On Wed, May 29, 2019 at 08:37:57PM -0300, Marcelo Ricardo Leitner wrote: > On Wed, May 29, 2019 at 03:07:09PM -0400, Neil Horman wrote: > > --- a/net/sctp/sm_make_chunk.c > > +++ b/net/sctp/sm_make_chunk.c > > @@ -2419,9 +2419,12 @@ int sctp_process_init(struct sctp_associati

Re: [PATCH] Fix xoring of arch_get_random_long into crng->state array

2019-05-30 Thread Neil Horman
On Wed, May 29, 2019 at 11:12:01PM -0400, Theodore Ts'o wrote: > On Tue, Apr 02, 2019 at 06:00:25PM -0400, Neil Horman wrote: > > When _crng_extract is called, any arch that has a registered > > arch_get_random_long method, attempts to mix an unsigned long value into > >

Re: memory leak in sctp_process_init

2019-05-29 Thread Neil Horman
On Tue, May 28, 2019 at 07:15:50AM -0400, Neil Horman wrote: > On Mon, May 27, 2019 at 10:36:00PM -0300, Marcelo Ricardo Leitner wrote: > > On Mon, May 27, 2019 at 05:48:06PM -0700, syzbot wrote: > > > Hello, > > > > > > syzbot found the following crash on: &

Re: [PATCH net-next] sctp: deduplicate identical skb_checksum_ops

2019-05-29 Thread Neil Horman
2c_csum_ops = { > - .update = sctp_csum_update, > - .combine = sctp_csum_combine, > -}; > - > int __init sctp_offload_init(void) > { > int ret; > @@ -120,7 +115,7 @@ int __init sctp_offload_init(void) > if (ret) > goto ipv4; > > - crc32c_csum_stub = &crc32c_csum_ops; > + crc32c_csum_stub = &sctp_csum_ops; > return ret; > > ipv4: > -- > 2.21.0 > > Acked-by: Neil Horman

Re: [PATCH] Fix xoring of arch_get_random_long into crng->state array

2019-05-29 Thread Neil Horman
On Wed, May 29, 2019 at 03:57:07PM +, David Laight wrote: > From: Neil Horman [mailto:nhor...@tuxdriver.com] > > Sent: 29 May 2019 16:52 > > On Wed, May 29, 2019 at 01:51:24PM +, David Laight wrote: > > > From: Neil Horman > > > > Sent: 29 May 2019 14:4

Re: [PATCH] Fix xoring of arch_get_random_long into crng->state array

2019-05-29 Thread Neil Horman
On Wed, May 29, 2019 at 01:51:24PM +, David Laight wrote: > From: Neil Horman > > Sent: 29 May 2019 14:42 > > On Tue, Apr 02, 2019 at 06:00:25PM -0400, Neil Horman wrote: > > > When _crng_extract is called, any arch that has a registered > > > arch_get_rando

Re: [PATCH] Fix xoring of arch_get_random_long into crng->state array

2019-05-29 Thread Neil Horman
On Tue, Apr 02, 2019 at 06:00:25PM -0400, Neil Horman wrote: > When _crng_extract is called, any arch that has a registered > arch_get_random_long method, attempts to mix an unsigned long value into > the crng->state buffer, it only mixes in 32 of the 64 bits available, > because t

Re: memory leak in sctp_process_init

2019-05-28 Thread Neil Horman
On Mon, May 27, 2019 at 10:36:00PM -0300, Marcelo Ricardo Leitner wrote: > On Mon, May 27, 2019 at 05:48:06PM -0700, syzbot wrote: > > Hello, > > > > syzbot found the following crash on: > > > > HEAD commit:9c7db500 Merge tag 'selinux-pr-20190521' of git://git.kern.. > > git tree: upstr

Re: [PATCH net-next 10/12] net: hns3: Add handling of MAC tunnel interruption

2019-04-19 Thread Neil Horman
On Fri, Apr 19, 2019 at 11:05:45AM +0800, Huazhong Tan wrote: > From: Weihang Li > > MAC tnl interruptions are different from other type of RAS and MSI-X > errors, because some bits, such as OVF/LR/RF will occur during link up > and down. > > The drivers should clear status of all MAC tnl interr

Re: [PATCH ghak90 V6 05/10] audit: add contid support for signalling the audit daemon

2019-04-09 Thread Neil Horman
ow for just this message. A better approach would be to do this now, and in a subsequent patch, create an AUDIT version 2 netlink protocol that converts all the messages we send to that format for consistency. Such a change would be large and warrant its own patch set and review. I'm go

[PATCH] Fix xoring of arch_get_random_long into crng->state array

2019-04-02 Thread Neil Horman
be filled (owing to the fact that it expects indexes 14 and 15 to be filled). Bring the expected behavior into alignment by casting index 14 to an unsignled long pointer, and xoring that in instead. Tested successfully by myself Signed-off-by: Neil Horman Reported-by: Steve Grubb CC: "Theod

Re: [PATCH v19,RESEND 01/27] x86/cpufeatures: Add Intel-defined SGX feature bit

2019-03-20 Thread Neil Horman
On Wed, Mar 20, 2019 at 06:20:53PM +0200, Jarkko Sakkinen wrote: > From: Kai Huang > > X86_FEATURE_SGX reflects whether or not the CPU supports Intel's > Software Guard eXtensions (SGX). > > Signed-off-by: Kai Huang > Co-developed-by: Jarkko Sakkinen > Signed-off-by: Jarkko Sakkinen > Reviewe

Re: [PATCH ghak90 V5 08/10] audit: add containerid filtering

2019-03-18 Thread Neil Horman
a/kernel/auditsc.c b/kernel/auditsc.c > index aa5d13b4fbbb..2d74238e9638 100644 > --- a/kernel/auditsc.c > +++ b/kernel/auditsc.c > @@ -616,6 +616,9 @@ static int audit_filter_rules(struct task_struct *tsk, > case AUDIT_LOGINUID_SET: > result = audit_comparator(audit_loginuid_set(tsk), > f->op, f->val); > break; > + case AUDIT_CONTID: > + result = audit_comparator64(audit_get_contid(tsk), > f->op, f->val64); > + break; > case AUDIT_SUBJ_USER: > case AUDIT_SUBJ_ROLE: > case AUDIT_SUBJ_TYPE: > -- > 1.8.3.1 > > Acked-by: Neil Horman

Re: general protection fault in sctp_sched_rr_dequeue

2019-03-06 Thread Neil Horman
On Wed, Mar 06, 2019 at 06:43:48PM +0800, Xin Long wrote: > On Wed, Mar 6, 2019 at 9:42 AM syzbot > wrote: > > > > Hello, > > > > syzbot found the following crash on: > > > > HEAD commit:63bdf4284c38 Merge branch 'linus' of git://git.kernel.org/.. > > git tree: upstream > > console outpu

Re: [PATCH net] sctp: get sctphdr by offset in sctp_compute_cksum

2019-02-27 Thread Neil Horman
On Wed, Feb 27, 2019 at 08:53:26PM +0800, Xin Long wrote: > On Tue, Feb 26, 2019 at 8:29 PM Neil Horman wrote: > > > > On Tue, Feb 26, 2019 at 12:15:54AM +0800, Xin Long wrote: > > > On Mon, Feb 25, 2019 at 10:08 PM Neil Horman > > > wrote: > > > &g

Re: [PATCH net] sctp: get sctphdr by offset in sctp_compute_cksum

2019-02-26 Thread Neil Horman
On Tue, Feb 26, 2019 at 12:15:54AM +0800, Xin Long wrote: > On Mon, Feb 25, 2019 at 10:08 PM Neil Horman wrote: > > > > On Mon, Feb 25, 2019 at 09:20:44PM +0800, Xin Long wrote: > > > On Mon, Feb 25, 2019 at 8:47 PM Neil Horman wrote: > > > > > > > &g

Re: [PATCH net] sctp: get sctphdr by offset in sctp_compute_cksum

2019-02-25 Thread Neil Horman
On Mon, Feb 25, 2019 at 09:20:44PM +0800, Xin Long wrote: > On Mon, Feb 25, 2019 at 8:47 PM Neil Horman wrote: > > > > On Mon, Feb 25, 2019 at 07:25:37PM +0800, Xin Long wrote: > > > sctp_hdr(skb) only works when skb->transport_header is set properly. >

Re: [PATCH net] sctp: get sctphdr by offset in sctp_compute_cksum

2019-02-25 Thread Neil Horman
On Mon, Feb 25, 2019 at 07:25:37PM +0800, Xin Long wrote: > sctp_hdr(skb) only works when skb->transport_header is set properly. > > But in the path of nf_conntrack_in: sctp_packet() -> sctp_error() > > skb->transport_header is not guaranteed to be right value for sctp. > It will cause to fail to

Re: [PATCH 08/15] perf script python: add Python3 support to net_dropmonitor.py

2019-02-25 Thread Neil Horman
ld be unchanged. > > The use of 'from __future__' implies the minimum supported Python2 version > is now v2.6 > > Signed-off-by: Tony Jones > Signed-off-by: Seeteena Thoufeek > Cc: Neil Horman > --- > tools/perf/scripts/python/net_dropmonitor.py | 10 ++

Re: [PATCH net] sctp: set stream ext to NULL after freeing it in sctp_stream_outq_migrate

2019-02-12 Thread Neil Horman
*stream, > } > } > > - for (i = outcnt; i < stream->outcnt; i++) > + for (i = outcnt; i < stream->outcnt; i++) { > kfree(SCTP_SO(stream, i)->ext); > + SCTP_SO(stream, i)->ext = NULL; > + } > } > > static int sctp_stream_alloc_out(struct sctp_stream *stream, __u16 outcnt, > -- > 2.1.0 > > Acked-by: Neil Horman

Re: [PATCH net] sctp: call gso_reset_checksum when computing checksum in sctp_gso_segment

2019-02-12 Thread Neil Horman
ad.c > +++ b/net/sctp/offload.c > @@ -36,6 +36,7 @@ static __le32 sctp_gso_make_checksum(struct sk_buff *skb) > { > skb->ip_summed = CHECKSUM_NONE; > skb->csum_not_inet = 0; > + gso_reset_checksum(skb, ~0); > return sctp_compute_cksum(skb, skb_transport_offset(skb)); > } > > -- > 2.1.0 > > > Acked-by: Neil Horman

Re: [PATCH net] sctp: make sctp_setsockopt_events() less strict about the option length

2019-02-11 Thread Neil Horman
On Sun, Feb 10, 2019 at 10:46:16AM -0200, Marcelo Ricardo Leitner wrote: > On Sat, Feb 09, 2019 at 03:12:17PM -0800, David Miller wrote: > > From: Marcelo Ricardo Leitner > > Date: Wed, 6 Feb 2019 18:37:54 -0200 > > > > > On Wed, Feb 06, 2019 at 12:14:30PM -0800, Julien Gomes wrote: > > >> Make s

Re: [PATCH net] sctp: make sctp_setsockopt_events() less strict about the option length

2019-02-08 Thread Neil Horman
On Fri, Feb 08, 2019 at 09:53:03AM +, David Laight wrote: > From: 'Marcelo Ricardo Leitner' > > Sent: 07 February 2019 17:47 > ... > > > > Maybe what we want(ed) here then is explicit versioning, to have the 3 > > > > definitions available. Then the application is able to use, say struct > > >

Re: [PATCH net] sctp: make sctp_setsockopt_events() less strict about the option length

2019-02-07 Thread Neil Horman
On Wed, Feb 06, 2019 at 01:48:44PM -0800, Julien Gomes wrote: > > > On 2/6/19 1:39 PM, Neil Horman wrote: > > On Wed, Feb 06, 2019 at 01:26:55PM -0800, Julien Gomes wrote: > >> > >> > >> On 2/6/19 1:07 PM, Marcelo Ricardo Leitner wrote: > >>

Re: [PATCH net] sctp: make sctp_setsockopt_events() less strict about the option length

2019-02-07 Thread Neil Horman
On Wed, Feb 06, 2019 at 01:48:42PM -0800, Julien Gomes wrote: > > > On 2/6/19 1:23 PM, Neil Horman wrote: > > On Wed, Feb 06, 2019 at 07:07:23PM -0200, Marcelo Ricardo Leitner wrote: > >> On Wed, Feb 06, 2019 at 12:48:38PM -0800, Julien Gomes wrote: > >>> >

Re: [PATCH net] sctp: make sctp_setsockopt_events() less strict about the option length

2019-02-06 Thread Neil Horman
On Wed, Feb 06, 2019 at 01:26:55PM -0800, Julien Gomes wrote: > > > On 2/6/19 1:07 PM, Marcelo Ricardo Leitner wrote: > > On Wed, Feb 06, 2019 at 12:48:38PM -0800, Julien Gomes wrote: > >> > >> > >> On 2/6/19 12:37 PM, Marcelo Ricardo Leitner wrote: > >>> On Wed, Feb 06, 2019 at 12:14:30PM -0800,

Re: [PATCH net] sctp: make sctp_setsockopt_events() less strict about the option length

2019-02-06 Thread Neil Horman
On Wed, Feb 06, 2019 at 07:07:23PM -0200, Marcelo Ricardo Leitner wrote: > On Wed, Feb 06, 2019 at 12:48:38PM -0800, Julien Gomes wrote: > > > > > > On 2/6/19 12:37 PM, Marcelo Ricardo Leitner wrote: > > > On Wed, Feb 06, 2019 at 12:14:30PM -0800, Julien Gomes wrote: > > >> Make sctp_setsockopt_e

Re: [PATCH net] sctp: make sctp_setsockopt_events() less strict about the option length

2019-02-06 Thread Neil Horman
On Wed, Feb 06, 2019 at 12:48:38PM -0800, Julien Gomes wrote: > > > On 2/6/19 12:37 PM, Marcelo Ricardo Leitner wrote: > > On Wed, Feb 06, 2019 at 12:14:30PM -0800, Julien Gomes wrote: > >> Make sctp_setsockopt_events() able to accept sctp_event_subscribe > >> structures longer than the current d

Re: [PATCH net] sctp: make sctp_setsockopt_events() less strict about the option length

2019-02-06 Thread Neil Horman
On Wed, Feb 06, 2019 at 12:14:30PM -0800, Julien Gomes wrote: > Make sctp_setsockopt_events() able to accept sctp_event_subscribe > structures longer than the current definitions. > > This should prevent unjustified setsockopt() failures due to struct > sctp_event_subscribe extensions (as in 4.11

Re: [PATCHv3 net] sctp: check and update stream->out_curr when allocating stream_out

2019-02-03 Thread Neil Horman
m->out, stream->out_curr, > + stream->outcnt); > + > + BUG_ON(index == stream->outcnt); > + stream->out_curr = flex_array_get(out, index); > + } > fa_free(stream->out); > } > > -- > 2.1.0 > > Acked-by: Neil Horman

Re: [PATCH 6/7] sctp: Convert to genradix

2019-01-18 Thread Neil Horman
On Fri, Jan 18, 2019 at 11:35:34AM -0200, Marcelo Ricardo Leitner wrote: > On Fri, Jan 18, 2019 at 08:10:22AM -0500, Neil Horman wrote: > > On Tue, Jan 15, 2019 at 12:29:26PM -0200, Marcelo Ricardo Leitner wrote: > > > On Mon, Dec 17, 2018 at 04:00:21PM -0500, Kent Overstreet wro

Re: [PATCH 6/7] sctp: Convert to genradix

2019-01-18 Thread Neil Horman
On Tue, Jan 15, 2019 at 12:29:26PM -0200, Marcelo Ricardo Leitner wrote: > On Mon, Dec 17, 2018 at 04:00:21PM -0500, Kent Overstreet wrote: > > On Mon, Dec 17, 2018 at 12:50:01PM -0800, Andrew Morton wrote: > > > On Mon, 17 Dec 2018 08:19:28 -0500 Kent Overstreet > > > wrote: > > > > > > > @@ -5

Re: [PATCH net] sctp: allocate sctp_sockaddr_entry with kzalloc

2019-01-14 Thread Neil Horman
addr->a.v4.sin_family = AF_INET; > - addr->a.v4.sin_port = 0; > addr->a.v4.sin_addr.s_addr = ifa->ifa_local; > addr->valid = 1; > spin_lock_bh(&net->sctp.local_addr_lock); > -- > 2.1.0 > > Acked-by: Neil Horman

Re: [PATCH 6/6] Drop flex_arrays

2018-12-18 Thread Neil Horman
On Mon, Dec 17, 2018 at 07:50:11AM -0500, Kent Overstreet wrote: > On Thu, Dec 13, 2018 at 01:09:17PM -0500, Neil Horman wrote: > > On Thu, Dec 13, 2018 at 08:45:33AM -0800, Matthew Wilcox wrote: > > > On Thu, Dec 13, 2018 at 10:51:49AM -0500, Neil Horman wrote: > > > &

Re: [PATCH 1/1] net-next/hinic:optmize rx refill buffer mechanism

2018-12-17 Thread Neil Horman
On Sun, Dec 16, 2018 at 10:32:34PM +, Xue Chaojing wrote: > In rx_alloc_pkts(), there is no need to schedule a different tasklet for > refill and it will cause some extra overhead. this patch remove it. > > Suggested-by: Neil Horman > Signed-off-by: Xue Chaojing >

Re: [PATCH 6/6] Drop flex_arrays

2018-12-13 Thread Neil Horman
On Thu, Dec 13, 2018 at 08:45:33AM -0800, Matthew Wilcox wrote: > On Thu, Dec 13, 2018 at 10:51:49AM -0500, Neil Horman wrote: > > On Thu, Dec 13, 2018 at 06:41:11AM -0800, Matthew Wilcox wrote: > > > On Thu, Dec 13, 2018 at 09:30:47PM +0900, Xin Long wrote: > > > >

Re: [PATCH 6/6] Drop flex_arrays

2018-12-13 Thread Neil Horman
On Thu, Dec 13, 2018 at 06:41:11AM -0800, Matthew Wilcox wrote: > On Thu, Dec 13, 2018 at 09:30:47PM +0900, Xin Long wrote: > > On Sat, Sep 8, 2018 at 1:57 AM Kent Overstreet > > wrote: > > > > > > All existing users have been converted to generic radix trees > > NAK, SCTP is still using flex_arra

Re: [PATCH 1/1] net-next/hinic:optmize rx refill buffer mechanism

2018-12-13 Thread Neil Horman
On Wed, Dec 12, 2018 at 05:40:23PM +, Xue Chaojing wrote: > There is no need to schedule a different tasklet for refill, > This patch remove it. > > Suggested-by: Neil Horman > Signed-off-by: Xue Chaojing > --- > drivers/net/ethernet/huawei/

Re: [PATCHv2 net 0/3] net: add support for flex_array_resize in flex_array

2018-12-12 Thread Neil Horman
On Tue, Dec 11, 2018 at 10:50:00PM -0800, David Miller wrote: > From: Xin Long > Date: Fri, 7 Dec 2018 14:30:32 +0800 > > > Without the support for the total_nr_elements's growing or shrinking > > dynamically, flex_array is not that 'flexible'. Like when users want > > to change the size, they h

Re: [PATCH net] sctp: initialize sin6_flowinfo for ipv6 addrs in sctp_inet6addr_event

2018-12-10 Thread Neil Horman
gned long ev, > if (addr) { > addr->a.v6.sin6_family = AF_INET6; > addr->a.v6.sin6_port = 0; > + addr->a.v6.sin6_flowinfo = 0; > addr->a.v6.sin6_addr = ifa->addr; > addr->a.v6.sin6_scope_id = ifa->idev->dev->ifindex; > addr->valid = 1; > -- > 2.1.0 > > Acked-by: Neil Horman

Re: [PATCH] vmw_pvrdma: Release netdev when vmxnet3 module is removed

2018-07-01 Thread Neil Horman
On Sat, Jun 30, 2018 at 10:15:07PM +0300, Dan Carpenter wrote: > Hi Neil, > > I love your patch! Perhaps something to improve: > > url: > https://github.com/0day-ci/linux/commits/Neil-Horman/vmw_pvrdma-Release-netdev-when-vmxnet3-module-is-removed/20180628-232414 &g

[PATCH v2] vmw_pvrdma: Release netdev when vmxnet3 module is removed

2018-06-29 Thread Neil Horman
accordingly. This solution has been tested by myself and the reporter with successful results. This fix also allows the pvrdma driver to find its underlying ethernet device in the event that vmxnet3 is loaded after pvrdma, which it was not able to do before. Signed-off-by: Neil Horman Reported-by: ruq

Re: [PATCH] vmw_pvrdma: Release netdev when vmxnet3 module is removed

2018-06-29 Thread Neil Horman
On Thu, Jun 28, 2018 at 09:15:46PM +, Adit Ranadive wrote: > On 6/28/18, 1:37 PM, "Jason Gunthorpe" wrote: > > On Thu, Jun 28, 2018 at 03:45:26PM -0400, Neil Horman wrote: > > > On Thu, Jun 28, 2018 at 12:59:46PM -0600, Jason Gunthorpe wrote: > > > >

Re: [PATCH] vmw_pvrdma: Release netdev when vmxnet3 module is removed

2018-06-29 Thread Neil Horman
On Thu, Jun 28, 2018 at 02:37:09PM -0600, Jason Gunthorpe wrote: > On Thu, Jun 28, 2018 at 03:45:26PM -0400, Neil Horman wrote: > > On Thu, Jun 28, 2018 at 12:59:46PM -0600, Jason Gunthorpe wrote: > > > On Thu, Jun 28, 2018 at 09:59:38AM -0400, Neil Horman wrote: > > >

Re: [PATCH] vmw_pvrdma: Release netdev when vmxnet3 module is removed

2018-06-28 Thread Neil Horman
On Thu, Jun 28, 2018 at 12:59:46PM -0600, Jason Gunthorpe wrote: > On Thu, Jun 28, 2018 at 09:59:38AM -0400, Neil Horman wrote: > > On repeated module load/unload cycles, its possible for the pvrmda > > driver to encounter this crash: > > > > ..

[PATCH] vmw_pvrdma: Release netdev when vmxnet3 module is removed

2018-06-28 Thread Neil Horman
accordingly. This solution has been tested by myself and the reporter with successful results. This fix also allows the pvrdma driver to find its underlying ethernet device in the event that vmxnet3 is loaded after pvrdma, which it was not able to do before. Signed-off-by: Neil Horman Reported-by: ruq

Re: [PATCH v11 09/13] x86, sgx: basic routines for enclave page cache

2018-06-25 Thread Neil Horman
On Mon, Jun 25, 2018 at 12:21:22PM +0300, Jarkko Sakkinen wrote: > On Wed, 2018-06-20 at 06:21 -0700, Sean Christopherson wrote: > > On Fri, 2018-06-08 at 19:09 +0200, Jarkko Sakkinen wrote: > > > SGX has a set of data structures to maintain information about the > > > enclaves > > > and their sec

Re: [intel-sgx-kernel-dev] [PATCH v11 13/13] intel_sgx: in-kernel launch enclave

2018-06-21 Thread Neil Horman
On Thu, Jun 21, 2018 at 08:32:25AM -0400, Nathaniel McCallum wrote: > On Wed, Jun 20, 2018 at 5:02 PM Sean Christopherson > wrote: > > > > On Wed, Jun 20, 2018 at 11:39:00AM -0700, Jethro Beekman wrote: > > > On 2018-06-20 11:16, Jethro Beekman wrote: > > > > > This last bit is also repeated in di

Re: [PATCH v11 09/13] x86, sgx: basic routines for enclave page cache

2018-06-19 Thread Neil Horman
On Tue, Jun 19, 2018 at 05:57:53PM +0300, Jarkko Sakkinen wrote: > On Fri, Jun 08, 2018 at 11:24:12AM -0700, Dave Hansen wrote: > > On 06/08/2018 10:09 AM, Jarkko Sakkinen wrote: > > > SGX has a set of data structures to maintain information about the > > > enclaves > > > and their security proper

Re: [intel-sgx-kernel-dev] [PATCH v11 13/13] intel_sgx: in-kernel launch enclave

2018-06-19 Thread Neil Horman
On Mon, Jun 18, 2018 at 02:58:59PM -0700, Andy Lutomirski wrote: > On Tue, Jun 12, 2018 at 10:45 AM Neil Horman wrote: > > > > On Mon, Jun 11, 2018 at 09:55:29PM -0700, Andy Lutomirski wrote: > > > On Mon, Jun 11, 2018 at 4:52 AM Neil Horman wrote: > > > > &

Re: [intel-sgx-kernel-dev] [PATCH v11 13/13] intel_sgx: in-kernel launch enclave

2018-06-12 Thread Neil Horman
On Mon, Jun 11, 2018 at 09:55:29PM -0700, Andy Lutomirski wrote: > On Mon, Jun 11, 2018 at 4:52 AM Neil Horman wrote: > > > > On Sun, Jun 10, 2018 at 10:17:13PM -0700, Andy Lutomirski wrote: > > > > On Jun 9, 2018, at 10:39 PM, Andy Lutomirski wrote: > > > &g

Re: [intel-sgx-kernel-dev] [PATCH v11 13/13] intel_sgx: in-kernel launch enclave

2018-06-11 Thread Neil Horman
On Sun, Jun 10, 2018 at 10:17:13PM -0700, Andy Lutomirski wrote: > > On Jun 9, 2018, at 10:39 PM, Andy Lutomirski wrote: > > > > On Fri, Jun 8, 2018 at 10:32 AM Jarkko Sakkinen > > wrote: > >> > >> The Launch Enclave (LE) generates cryptographic launch tokens for user > >> enclaves. A launch toke

Re: [PATCH v11 07/13] x86, sgx: detect Intel SGX

2018-06-11 Thread Neil Horman
On Fri, Jun 08, 2018 at 07:09:42PM +0200, Jarkko Sakkinen wrote: > From: Sean Christopherson > > Intel(R) SGX is a set of CPU instructions that can be used by applications > to set aside private regions of code and data. The code outside the enclave > is disallowed to access the memory inside the

Re: INFO: rcu detected stall in kfree_skbmem

2018-05-14 Thread Neil Horman
On Fri, May 11, 2018 at 12:00:38PM +0200, Dmitry Vyukov wrote: > On Mon, Apr 30, 2018 at 8:09 PM, syzbot > wrote: > > Hello, > > > > syzbot found the following crash on: > > > > HEAD commit:5d1365940a68 Merge > > git://git.kernel.org/pub/scm/linux/kerne... > > git tree: net-next > > cons

Re: [PATCH] sctp: fix spelling mistake: "max_retans" -> "max_retrans"

2018-05-09 Thread Neil Horman
ns > count"; > size_t payload_len = sizeof(error) + sizeof(struct sctp_errhdr); > struct sctp_chunk *retval; > > -- > 2.17.0 > > Acked-by: Neil Horman

Re: [PATCH] sctp: fix a potential missing-check bug

2018-05-03 Thread Neil Horman
On Wed, May 02, 2018 at 08:07:17PM -0500, Wenwen Wang wrote: > Hi Marcelo, > > I guess I worked on an old version of the kernel. I will re-submit the > patch. Sorry :( > You don't have to resubmit the patch, this isn't broken. As marcelo points out, a value of zero in this socket option is speci

Re: net: hang in unregister_netdevice: waiting for lo to become free

2018-04-13 Thread Neil Horman
On Thu, Apr 12, 2018 at 02:15:30PM +0200, Dmitry Vyukov wrote: > On Wed, Feb 21, 2018 at 3:53 PM, Tommi Rantala > wrote: > > On 20.02.2018 18:26, Neil Horman wrote: > >> > >> On Tue, Feb 20, 2018 at 09:14:41AM +0100, Dmitry Vyukov wrote: > >>> > >&

Re: [PATCH] [net-next] sctp: fix unused lable warning

2018-03-28 Thread Neil Horman
> > +#ifdef CONFIG_PROC_FS > err_init_proc: > cleanup_sctp_mibs(net); > +#endif > err_init_mibs: > sctp_sysctl_net_unregister(net); > err_sysctl_register: > -- > 2.9.0 > > Acked-by: Neil Horman

Re: BUG: corrupted list in sctp_association_free

2018-03-13 Thread Neil Horman
On Tue, Mar 13, 2018 at 02:09:09PM +0300, Dmitry Vyukov wrote: > On Tue, Mar 13, 2018 at 1:44 PM, Xin Long wrote: > > On Tue, Mar 13, 2018 at 3:34 PM, syzbot > > wrote: > >> Hello, > >> > >> syzbot hit the following crash on net-next commit > >> fd372a7a9e5e9d8011a0222d10edd3523abcd3b1 (Thu Mar 8

Re: KASAN: use-after-free Read in sctp_association_free (2)

2018-03-12 Thread Neil Horman
On Mon, Mar 12, 2018 at 04:16:27PM +0800, Xin Long wrote: > On Sun, Mar 11, 2018 at 3:04 AM, Neil Horman wrote: > > On Sun, Mar 11, 2018 at 12:22:32AM +0800, Xin Long wrote: > >> On Sat, Mar 10, 2018 at 9:13 PM, Neil Horman wrote: > >> > On Sat, Mar 10, 2018 at 03

Re: KASAN: use-after-free Read in sctp_association_free (2)

2018-03-10 Thread Neil Horman
On Sun, Mar 11, 2018 at 12:22:32AM +0800, Xin Long wrote: > On Sat, Mar 10, 2018 at 9:13 PM, Neil Horman wrote: > > On Sat, Mar 10, 2018 at 03:58:04PM +0800, Xin Long wrote: > >> On Sat, Mar 10, 2018 at 6:08 AM, Neil Horman wrote: > >> > On Fri, Mar 09, 2018 at

Re: KASAN: use-after-free Read in sctp_association_free (2)

2018-03-10 Thread Neil Horman
On Sat, Mar 10, 2018 at 03:58:04PM +0800, Xin Long wrote: > On Sat, Mar 10, 2018 at 6:08 AM, Neil Horman wrote: > > On Fri, Mar 09, 2018 at 12:59:06PM -0800, syzbot wrote: > >> Hello, > >> > >> syzbot hit the following crash on net-next commit > >>

Re: KASAN: use-after-free Read in sctp_association_free (2)

2018-03-09 Thread Neil Horman
On Fri, Mar 09, 2018 at 12:59:06PM -0800, syzbot wrote: > Hello, > > syzbot hit the following crash on net-next commit > fd372a7a9e5e9d8011a0222d10edd3523abcd3b1 (Thu Mar 8 19:43:48 2018 +) > Merge tag 'mlx5-updates-2018-02-28-2' of > git://git.kernel.org/pub/scm/linux/kernel/git/mellanox/linu

Re: net: hang in unregister_netdevice: waiting for lo to become free

2018-02-20 Thread Neil Horman
On Tue, Feb 20, 2018 at 09:14:41AM +0100, Dmitry Vyukov wrote: > On Tue, Feb 20, 2018 at 8:56 AM, Tommi Rantala > wrote: > > On 19.02.2018 20:59, Dmitry Vyukov wrote: > >> > >> On Sat, Feb 3, 2018 at 1:15 PM, Xin Long wrote: > > > > On 1/30/18 1:57 PM, David Ahern wrote: > >> > >>

Re: general protection fault in sctp_v6_get_dst

2018-01-11 Thread Neil Horman
t.c > @@ -353,15 +353,14 @@ static struct sctp_af *sctp_sockaddr_af(struct > sctp_sock *opt, > if (len < sizeof (struct sockaddr)) > return NULL; > > + if (!opt->pf->af_supported(addr->sa.sa_family, opt)) > + return NULL; > + > /* V4 mapped address are really of AF_INET family */ > if (addr->sa.sa_family == AF_INET6 && > ipv6_addr_v4mapped(&addr->v6.sin6_addr)) { > if (!opt->pf->af_supported(AF_INET, opt)) > return NULL; > - } else { > - /* Does this PF support this AF? */ > - if (!opt->pf->af_supported(addr->sa.sa_family, opt)) > - return NULL; > } > This looks reasonable Acked-by: Neil Horman

Re: general protection fault in sctp_stream_free

2017-11-06 Thread Neil Horman
On Sun, Nov 05, 2017 at 01:35:02AM -0700, syzbot wrote: > Hello, > > syzkaller hit the following crash on > 2a171788ba7bb61995e98e8163204fc7880f63b2 > git://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git/master > compiler: gcc (GCC) 7.1.1 20170620 > .config is attached > Raw console ou

Re: [PATCH][next] sctp: make array sctp_sched_ops static

2017-10-11 Thread Neil Horman
t; +static struct sctp_sched_ops *sctp_sched_ops[] = { > &sctp_sched_fcfs, > &sctp_sched_prio, > &sctp_sched_rr, > -- > 2.14.1 > > Acked-by: Neil Horman

Re: [PATCH 0/2] i2c: ismt: Fix length handling for SMBus block reads

2017-08-29 Thread Neil Horman
; audience. for-current (v4.13) might have been also applicable, but I > don't want to apply the patches there without the driver maintainer > acks. I hope this works for you. > Sorry, I've been on vacation, yes, the patches look good to me Acked-by: Neil Horman

  1   2   3   4   5   6   7   >