linux-kernel - Re: [PATCHv2 3/6] rds: check for excessive looping in rds_send

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20150402.140850.1775436517754499795.davem@davemloft.net>
Date:	Thu, 02 Apr 2015 14:08:50 -0400 (EDT)
From:	David Miller <davem@...emloft.net>
To:	sowmini.varadhan@...cle.com
Cc:	netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
	chien.yen@...cle.com, rds-devel@....oracle.com, agrover@...hat.com,
	clm@...com, zab@...bo.net, ajaykumar.hotchandani@...cle.com,
	sergei.shtylyov@...entembedded.com
Subject: Re: [PATCHv2 3/6] rds: check for excessive looping in rds_send_xmit

From: Sowmini Varadhan <sowmini.varadhan@...cle.com>
Date: Thu,  2 Apr 2015 10:19:37 -0400

> @@ -177,6 +178,18 @@ int rds_send_xmit(struct rds_connection *conn)
>  
>  		rm = conn->c_xmit_rm;
>  
> +		if (!rm) {
> +			same_rm = 0;
> +		} else {
> +			same_rm++;
> +			if (same_rm >= 4096) {
> +				printk_ratelimited(KERN_ERR "RDS: Stuck rm\n");
> +				cond_resched();
> +				ret = -EAGAIN;
> +				break;
> +			}
> +		}
> +

I'm not accepting junk like this.

I want to know exactly how this situation can occur, in detail, and
then and only then can we start to talk about more reasonable
solutions to this bug.

This looks like a minimal effort hack to make a hang go away, and
even more concerning is that a change like this is submitted with
a commit message that is essentially empty.

Sorry, this is not acceptable.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/