Re: [PATCH V2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-09-14 Thread Arnd Bergmann
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

Re: [PATCH v4 5/6] iommu/mediatek: Add mt8173 IOMMU driver

2015-09-14 Thread Yong Wu
On Fri, 2015-09-11 at 16:33 +0100, Robin Murphy wrote: > On 03/08/15 11:21, Yong Wu wrote: > > This patch adds support for mediatek m4u (MultiMedia Memory Management > > Unit). > > > > Signed-off-by: Yong Wu > > --- > [...] > > +/* > > + * There is only one iommu domain called the m4u domain that

Re: [PATCH V2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-09-14 Thread Viresh Kumar
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 o

Re: [PATCH 0/4] Separate the IOVA library from the intel-iommu driver

2015-09-14 Thread Robin Murphy
Hi David, On 28/07/15 14:25, David Woodhouse wrote: On Mon, 2015-07-13 at 14:31 +0300, Sakari Ailus wrote: Hi folks, This set separates the IOVA library from the intel-iommu driver, and makes it usable for other implementations (such as DMA mapping) as well. Thanks. I've applied patches ##1-

Re: [PATCH V2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-09-14 Thread Mark Brown
On Mon, Sep 14, 2015 at 09:21:54AM +0530, Viresh Kumar wrote: > Long back 'bool' type used to be a typecast to 'int', but that changed > in v2.6.19. And that is a typecast to _Bool now, which (mostly) takes > just a byte. Anyway, the bool type is implementation defined, and better > we don't assume

Re: [PATCH V2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-09-14 Thread Viresh Kumar
On 14-09-15, 17:39, Arnd Bergmann wrote: > On Monday 14 September 2015 09:21:54 Viresh Kumar wrote: > > diff --git a/drivers/acpi/ec_sys.c b/drivers/acpi/ec_sys.c > > index b4c216bab22b..bea8e425a8de 100644 > > --- a/drivers/acpi/ec_sys.c > > +++ b/drivers/acpi/ec_sys.c > > @@ -128,7 +128,7 @@ stat

[PATCH V2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-09-14 Thread Viresh Kumar
Long back 'bool' type used to be a typecast to 'int', but that changed in v2.6.19. And that is a typecast to _Bool now, which (mostly) takes just a byte. Anyway, the bool type is implementation defined, and better we don't assume its size to be 4 bytes or 1. The problem with current code is that i

Re: [PATCH V2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-09-14 Thread Greg KH
On Mon, Sep 14, 2015 at 09:17:08PM +0530, Viresh Kumar wrote: > On 14-09-15, 17:39, Arnd Bergmann wrote: > > On Monday 14 September 2015 09:21:54 Viresh Kumar wrote: > > > diff --git a/drivers/acpi/ec_sys.c b/drivers/acpi/ec_sys.c > > > index b4c216bab22b..bea8e425a8de 100644 > > > --- a/drivers/ac

Re: [PATCH V2] debugfs: don't assume sizeof(bool) to be 4 bytes

2015-09-14 Thread Arnd Bergmann
On Monday 14 September 2015 09:21:54 Viresh Kumar wrote: > diff --git a/drivers/acpi/ec_sys.c b/drivers/acpi/ec_sys.c > index b4c216bab22b..bea8e425a8de 100644 > --- a/drivers/acpi/ec_sys.c > +++ b/drivers/acpi/ec_sys.c > @@ -128,7 +128,7 @@ static int acpi_ec_add_debugfs(struct acpi_ec *ec, > uns

Re: [PATCH v3 3/6] iommu: add ARM short descriptor page table allocator.

2015-09-14 Thread Yong Wu
On Tue, 2015-07-21 at 18:11 +0100, Will Deacon wrote: [...] > > +static int arm_short_map(struct io_pgtable_ops *ops, unsigned long iova, > > +phys_addr_t paddr, size_t size, int prot) > > +{ > > + struct arm_short_io_pgtable *data = io_pgtable_ops_to_data(ops); > > +

Re: [PATCH 00/39] drop null test before destroy functions

2015-09-14 Thread SF Markus Elfring
> Recent commits to kernel/git/torvalds/linux.git have made the following > functions able to tolerate NULL arguments: > > kmem_cache_destroy (commit 3942d29918522) > mempool_destroy (commit 4e3ca3e033d1) > dma_pool_destroy (commit 44d7175da6ea) How do you think about to extend an other SmPL scrip

RE: [PATCH v7 15/17] KVM: Update Posted-Interrupts Descriptor when vCPU is blocked

2015-09-14 Thread Wu, Feng
First of all, Paolo, thanks a lot for your review on this series, it really means a lot!:) > -Original Message- > From: linux-kernel-ow...@vger.kernel.org > [mailto:linux-kernel-ow...@vger.kernel.org] On Behalf Of Paolo Bonzini > Sent: Friday, September 11, 2015 7:21 PM > To: Wu, Feng; al