Subject: Re: [PATCH 09/44 take 2] [UBI] debug unit header
From: Artem Bityutskiy <dedekind@infradead.org>
Reply-To: dedekind@infradead.org
To: Josh Boyer <jwboyer@linux.vnet.ibm.com>
Cc: Arnd Bergmann <arnd@arndb.de>,
       Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
       Christoph Hellwig <hch@infradead.org>,
       Frank Haverkamp <haver@vnet.ibm.com>,
       Thomas Gleixner <tglx@linutronix.de>,
       David Woodhouse <dwmw2@infradead.org>
In-Reply-To: <1171893769.24204.15.camel@zod.rchland.ibm.com>
References: <20070217165424.5845.4390.sendpatchset@localhost.localdomain>
	 <20070217165509.5845.12977.sendpatchset@localhost.localdomain>
	 <200702172218.22709.arnd@arndb.de>  <1171888399.13817.21.camel@sauron>
	 <1171893769.24204.15.camel@zod.rchland.ibm.com>
Content-Type: text/plain; charset=utf-8
Date: Mon, 19 Feb 2007 16:04:29 +0200
Message-Id: <1171893869.14817.2.camel@sauron>
Mime-Version: 1.0
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 802
Lines: 19

On Mon, 2007-02-19 at 08:02 -0600, Josh Boyer wrote:
> It is not the comment that annoys people.  Arnd was saying that the
> entire debug unit should not exist.

There is many per-subsystem debugging stuff in kernel. I do not see
_objective_ any reason to prohibit use have our own debugging stuff in
UBI. It is compiled out when disabled. It helps to quickly fix bugs. It
helps to check if new patches introduce bugs (assertions, paranoid
checks). These are _objective_ reasons to keep it.

-- 
Best regards,
Artem Bityutskiy (Битюцкий Артём)

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