All of lore.kernel.org
 help / color / mirror / Atom feed
From: Suzuki.Poulose@arm.com (Suzuki K. Poulose)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V2] drivers: CCI: fix used_mask init in validate_group()
Date: Thu, 30 Apr 2015 11:55:20 +0100	[thread overview]
Message-ID: <55420A18.7080203@arm.com> (raw)
In-Reply-To: <20150413124121.GF2869@arm.com>

On 13/04/15 13:41, Will Deacon wrote:
> On Thu, Apr 09, 2015 at 04:36:29PM +0100, Mark Rutland wrote:
>> On Thu, Apr 09, 2015 at 03:57:05PM +0100, Mark Salter wrote:
>>> Currently in validate_group(), there is a static initializer
>>> for fake_pmu.used_mask which is based on CPU_BITS_NONE but
>>> the used_mask array size is based on CCI_PMU_MAX_HW_EVENTS.
>>> CCI_PMU_MAX_HW_EVENTS is not based on NR_CPUS, so CPU_BITS_NONE
>>> is not correct and will cause a build failure if NR_CPUS
>>> is set high enough to make CPU_BITS_NONE larger than used_mask.
>>>
>>> Signed-off-by: Mark Salter <msalter@redhat.com>
>>
>> Reviewed-by: Mark Rutland <mark.rutland@arm.com>
>>
>> Arnd, Olof, are you happy to take this via arm-soc?
>>
>> My (broken) patch went via Will's tree because of a perf dependency, but
>> other CCI patches have gone via you guys.
>
> Yeah, arm-soc is the best place for this. You should resend to
> arm at kernel.org as a new patch with the relevant acks.
Mark

Could you please send this to arm-soc as suggested by Will, with the 
relevant acks/reviews  ?

Thanks
Suzuki

WARNING: multiple messages have this Message-ID (diff)
From: "Suzuki K. Poulose" <Suzuki.Poulose@arm.com>
To: "msalter@redhat.com" <msalter@redhat.com>
Cc: Will Deacon <will.deacon@arm.com>,
	Mark Rutland <Mark.Rutland@arm.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	Punit Agrawal <Punit.Agrawal@arm.com>,
	"arm@kernel.org" <arm@kernel.org>
Subject: Re: [PATCH V2] drivers: CCI: fix used_mask init in validate_group()
Date: Thu, 30 Apr 2015 11:55:20 +0100	[thread overview]
Message-ID: <55420A18.7080203@arm.com> (raw)
In-Reply-To: <20150413124121.GF2869@arm.com>

On 13/04/15 13:41, Will Deacon wrote:
> On Thu, Apr 09, 2015 at 04:36:29PM +0100, Mark Rutland wrote:
>> On Thu, Apr 09, 2015 at 03:57:05PM +0100, Mark Salter wrote:
>>> Currently in validate_group(), there is a static initializer
>>> for fake_pmu.used_mask which is based on CPU_BITS_NONE but
>>> the used_mask array size is based on CCI_PMU_MAX_HW_EVENTS.
>>> CCI_PMU_MAX_HW_EVENTS is not based on NR_CPUS, so CPU_BITS_NONE
>>> is not correct and will cause a build failure if NR_CPUS
>>> is set high enough to make CPU_BITS_NONE larger than used_mask.
>>>
>>> Signed-off-by: Mark Salter <msalter@redhat.com>
>>
>> Reviewed-by: Mark Rutland <mark.rutland@arm.com>
>>
>> Arnd, Olof, are you happy to take this via arm-soc?
>>
>> My (broken) patch went via Will's tree because of a perf dependency, but
>> other CCI patches have gone via you guys.
>
> Yeah, arm-soc is the best place for this. You should resend to
> arm@kernel.org as a new patch with the relevant acks.
Mark

Could you please send this to arm-soc as suggested by Will, with the 
relevant acks/reviews  ?

Thanks
Suzuki


  reply	other threads:[~2015-04-30 10:55 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-08 18:21 [PATCH] drivers: CCI: fix used_mask init in validate_group() Mark Salter
2015-04-08 18:21 ` Mark Salter
2015-04-09 11:06 ` Suzuki K. Poulose
2015-04-09 11:06   ` Suzuki K. Poulose
2015-04-09 13:51 ` Mark Rutland
2015-04-09 13:51   ` Mark Rutland
2015-04-09 14:11   ` Mark Salter
2015-04-09 14:11     ` Mark Salter
2015-04-09 14:20     ` Mark Rutland
2015-04-09 14:20       ` Mark Rutland
2015-04-09 14:26       ` Mark Salter
2015-04-09 14:26         ` Mark Salter
2015-04-09 14:40         ` Mark Rutland
2015-04-09 14:40           ` Mark Rutland
2015-04-09 14:57           ` [PATCH V2] " Mark Salter
2015-04-09 14:57             ` Mark Salter
2015-04-09 15:36             ` Mark Rutland
2015-04-09 15:36               ` Mark Rutland
2015-04-13 12:41               ` Will Deacon
2015-04-13 12:41                 ` Will Deacon
2015-04-30 10:55                 ` Suzuki K. Poulose [this message]
2015-04-30 10:55                   ` Suzuki K. Poulose
2015-04-30 13:26                   ` Mark Salter
2015-04-30 13:26                     ` Mark Salter
2015-04-30 13:33                     ` Mark Rutland
2015-04-30 13:33                       ` Mark Rutland
2015-04-30 14:03                       ` Mark Salter
2015-04-30 14:03                         ` Mark Salter
2015-04-30 14:38                         ` Mark Rutland
2015-04-30 14:38                           ` Mark Rutland
2015-04-30 14:46                           ` Mark Salter
2015-04-30 14:46                             ` Mark Salter
2015-04-30 14:52                             ` Arnd Bergmann
2015-04-30 14:52                               ` Arnd Bergmann
2015-04-15 10:44             ` Suzuki K. Poulose
2015-04-15 10:44               ` Suzuki K. Poulose
2015-04-15 11:58               ` Will Deacon
2015-04-15 11:58                 ` Will Deacon
2015-04-15 12:50                 ` Suzuki K. Poulose
2015-04-15 12:50                   ` Suzuki K. Poulose

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=55420A18.7080203@arm.com \
    --to=suzuki.poulose@arm.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.