From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH V2] debugfs: don't assume sizeof(bool) to be 4 bytes Date: Tue, 15 Sep 2015 08:30:55 +0200 Message-ID: <11729827.zzMdq0fFyE@wuerfel> References: <81516fb9c662cc338b5af5b63fbbcde5374e0893.1442202447.git.viresh.kumar@linaro.org> <20150914160348.GA7290@kroah.com> <20150915020438.GG32551@linux> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: , "moderated list:SOUND - SOC LAYER / DYNAMIC AUDIO POWER MANAGEM..." , Avri Altman , Stanislaw Gruszka , Jiri Slaby , "open list:DOCUMENTATION" , Peter Zijlstra , Catalin Marinas , Sebastian Andrzej Siewior , Joerg Roedel , Jaroslav Kysela , "open list:MEMORY MANAGEMENT" , Kalle Valo , Emmanuel Grumbach , Luciano Coelho , Richard Fitzgerald , Ingo Molnar , Hariprasad S , Thomas Gleixner , Davidlohr Bueso , Eliad Peller Return-path: In-Reply-To: <20150915020438.GG32551@linux> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: iommu-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org List-Id: netdev.vger.kernel.org On Tuesday 15 September 2015 07:34:38 Viresh Kumar wrote: > On 14-09-15, 09:03, Greg KH wrote: > > What ever ones you think it is relevant for :) > > "Relevant" is a relevant term :) > > So, the patch which defined the type bool as _Bool was added in v2.6.19 :) > > 6e2182874324 ("[PATCH] Generic boolean") > > So, I will try at least for v3.10+ as they are used by a lot of > people, as they are LTS kernels. But wasn't sure if I should do it > right from 2.6.19. I don't think there is any use in backporting the global type change, that is too invasive and does not change much. The specific bug in the acpi code dates back to when acpi_ec_add_debugfs() was added in 2.6.36, so that is the earliest point it could be backported to. Then again, as I said it is only a problem on big-endian 64-bit architectures, and on ACPI that would specifically mean arm64, which only very recently gained ACPI support, so backporting to 4.1 is probably enough. Arnd