+ ppc-e500_tlb-memset-clears-nothing.patch added to -mm tree

+ ppc-e500_tlb-memset-clears-nothing.patch added to -mm tree

[akpm]

The patch titled
     Subject: ppc: e500_tlb memset clears nothing
has been added to the -mm tree.  Its filename is
     ppc-e500_tlb-memset-clears-nothing.patch

Before you just go and hit "reply", please:
   a) Consider who else should be cc'ed
   b) Prefer to cc a suitable mailing list as well
   c) Ideally: find the original patch on the mailing list and do a
      reply-to-all to that, adding suitable additional cc's

*** Remember to use Documentation/SubmitChecklist when testing your code ***

The -mm tree is included into linux-next and is updated
there every 3-4 working days

------------------------------------------------------
From: Alan Cox <alan@linux.intel.com>
Subject: ppc: e500_tlb memset clears nothing

Put the parameters the right way around

Addresses https://bugzilla.kernel.org/show_bug.cgi?id=44031

Reported-by: David Binderman <dcb314@hotmail.com>
Signed-off-by: Alan Cox <alan@linux.intel.com>
Cc: Avi Kivity <avi@redhat.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Alexander Graf <agraf@suse.de>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Paul Mackerras <paulus@samba.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
---

 arch/powerpc/kvm/e500_tlb.c |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff -puN arch/powerpc/kvm/e500_tlb.c~ppc-e500_tlb-memset-clears-nothing arch/powerpc/kvm/e500_tlb.c
--- a/arch/powerpc/kvm/e500_tlb.c~ppc-e500_tlb-memset-clears-nothing
+++ a/arch/powerpc/kvm/e500_tlb.c
@@ -322,11 +322,11 @@ static inline void kvmppc_e500_ref_relea
 static void clear_tlb1_bitmap(struct kvmppc_vcpu_e500 *vcpu_e500)
 {
 	if (vcpu_e500->g2h_tlb1_map)
-		memset(vcpu_e500->g2h_tlb1_map,
-		       sizeof(u64) * vcpu_e500->gtlb_params[1].entries, 0);
+		memset(vcpu_e500->g2h_tlb1_map, 0,
+		       sizeof(u64) * vcpu_e500->gtlb_params[1].entries);
 	if (vcpu_e500->h2g_tlb1_rmap)
-		memset(vcpu_e500->h2g_tlb1_rmap,
-		       sizeof(unsigned int) * host_tlb_params[1].entries, 0);
+		memset(vcpu_e500->h2g_tlb1_rmap, 0,
+		       sizeof(unsigned int) * host_tlb_params[1].entries);
 }
 
 static void clear_tlb_privs(struct kvmppc_vcpu_e500 *vcpu_e500)
_
Subject: Subject: ppc: e500_tlb memset clears nothing

Patches currently in -mm which might be from alan@linux.intel.com are

origin.patch
linux-next.patch
ppc-e500_tlb-memset-clears-nothing.patch
fs-make-dumpable=2-require-fully-qualified-path.patch
coredump-warn-about-unsafe-suid_dumpable-core_pattern-combo.patch
kernel-sysc-avoid-argv_freenull.patch
drivers-message-i2o-i2o_procc-the-pointer-returned-from-chtostr-points-to-an-array-which-is-no-longer-valid.patch

Re: + ppc-e500_tlb-memset-clears-nothing.patch added to -mm tree

[Avi Kivity]

On 07/25/2012 12:00 AM, akpm@linux-foundation.org wrote:
> The patch titled
>      Subject: ppc: e500_tlb memset clears nothing
> has been added to the -mm tree.  Its filename is
>      ppc-e500_tlb-memset-clears-nothing.patch
> 

The kvm tree is maintained.  Alex, please pick up this patch; I guess
it's 3.6 material.


> 
> *** Remember to use Documentation/SubmitChecklist when testing your code ***
> 
> The -mm tree is included into linux-next and is updated
> there every 3-4 working days
> 
> ------------------------------------------------------
> From: Alan Cox <alan@linux.intel.com>
> Subject: ppc: e500_tlb memset clears nothing
> 
> Put the parameters the right way around
> 
> Addresses https://bugzilla.kernel.org/show_bug.cgi?id=44031
> 
> Reported-by: David Binderman <dcb314@hotmail.com>
> Signed-off-by: Alan Cox <alan@linux.intel.com>
> Cc: Avi Kivity <avi@redhat.com>
> Cc: Marcelo Tosatti <mtosatti@redhat.com>
> Cc: Alexander Graf <agraf@suse.de>
> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
> Cc: Paul Mackerras <paulus@samba.org>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> ---
> 
>  arch/powerpc/kvm/e500_tlb.c |    8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff -puN arch/powerpc/kvm/e500_tlb.c~ppc-e500_tlb-memset-clears-nothing arch/powerpc/kvm/e500_tlb.c
> --- a/arch/powerpc/kvm/e500_tlb.c~ppc-e500_tlb-memset-clears-nothing
> +++ a/arch/powerpc/kvm/e500_tlb.c
> @@ -322,11 +322,11 @@ static inline void kvmppc_e500_ref_relea
>  static void clear_tlb1_bitmap(struct kvmppc_vcpu_e500 *vcpu_e500)
>  {
>  	if (vcpu_e500->g2h_tlb1_map)
> -		memset(vcpu_e500->g2h_tlb1_map,
> -		       sizeof(u64) * vcpu_e500->gtlb_params[1].entries, 0);
> +		memset(vcpu_e500->g2h_tlb1_map, 0,
> +		       sizeof(u64) * vcpu_e500->gtlb_params[1].entries);
>  	if (vcpu_e500->h2g_tlb1_rmap)
> -		memset(vcpu_e500->h2g_tlb1_rmap,
> -		       sizeof(unsigned int) * host_tlb_params[1].entries, 0);
> +		memset(vcpu_e500->h2g_tlb1_rmap, 0,
> +		       sizeof(unsigned int) * host_tlb_params[1].entries);
>  }
>  
>  static void clear_tlb_privs(struct kvmppc_vcpu_e500 *vcpu_e500)
> _
> Subject: Subject: ppc: e500_tlb memset clears nothing
> 

-- 
error compiling committee.c: too many arguments to function

Re: + ppc-e500_tlb-memset-clears-nothing.patch added to -mm tree

[Alexander Graf]

On 25.07.2012, at 10:32, Avi Kivity <avi@redhat.com> wrote:

> On 07/25/2012 12:00 AM, akpm@linux-foundation.org wrote:
>> The patch titled
>>     Subject: ppc: e500_tlb memset clears nothing
>> has been added to the -mm tree.  Its filename is
>>     ppc-e500_tlb-memset-clears-nothing.patch
>> 
> 
> The kvm tree is maintained.  Alex, please pick up this patch; I guess
> it's 3.6 material.

Yeah, not sure what's going on here. Alan sent me the patch a while back in a private mail, so I asked him to resend it to the ML, so it's available for review. Next thing that happens is this mail a few weeks later.

Either way, will bypass the normal process this time and pull it in my queue. It is certainly 3.6 material.


Alex

Re: + ppc-e500_tlb-memset-clears-nothing.patch added to -mm tree

[Avi Kivity]

On 07/25/2012 12:00 PM, Alexander Graf wrote:
> 
> On 25.07.2012, at 10:32, Avi Kivity <avi@redhat.com> wrote:
> 
>> On 07/25/2012 12:00 AM, akpm@linux-foundation.org wrote:
>>> The patch titled
>>>     Subject: ppc: e500_tlb memset clears nothing
>>> has been added to the -mm tree.  Its filename is
>>>     ppc-e500_tlb-memset-clears-nothing.patch
>>> 
>> 
>> The kvm tree is maintained.  Alex, please pick up this patch; I guess
>> it's 3.6 material.
> 
> Yeah, not sure what's going on here. Alan sent me the patch a while back in a private mail, so I asked him to resend it to the ML, so it's available for review. Next thing that happens is this mail a few weeks later.
> 
> Either way, will bypass the normal process this time and pull it in my queue. It is certainly 3.6 material.

I wonder how many such bugs a memzero()/bzero() will prevent.

-- 
error compiling committee.c: too many arguments to function

Re: + ppc-e500_tlb-memset-clears-nothing.patch added to -mm tree

[Alan Cox]

> Yeah, not sure what's going on here. Alan sent me the patch a while
> back in a private mail, so I asked him to resend it to the ML, so
> it's available for review. Next thing that happens is this mail a few
> weeks later.

I don't recall seeing your your reply and I really wasn't sure where it
needed to go. I mostly sent it to PPC people. In the end I sent it
to Andrew with a question as to where it should actually go.

> Either way, will bypass the normal process this time and pull it in
> my queue. It is certainly 3.6 material.

Alan

Re: + ppc-e500_tlb-memset-clears-nothing.patch added to -mm tree

[Alan Cox]

> I wonder how many such bugs a memzero()/bzero() will prevent.

If the compiler-foo is possible with gcc then a 0 length constant
memset warning and a warning if the set value is > 255 would both
probably be useful.

Fortunately a lot of other validation/verification tools do pick it up
already.

Alan
 

Re: + ppc-e500_tlb-memset-clears-nothing.patch added to -mm tree

[Avi Kivity]

On 07/25/2012 01:12 PM, Alan Cox wrote:
>> I wonder how many such bugs a memzero()/bzero() will prevent.
> 
> If the compiler-foo is possible with gcc then a 0 length constant
> memset warning and a warning if the set value is > 255 would both
> probably be useful.

It's pretty easy with BUILD_BUG_ON(), unfortunately that emits an error
and there may be legitimate uses for memset(p, 255, 0).

> Fortunately a lot of other validation/verification tools do pick it up
> already.

Also a lot of kernel developers whose time would be better spent on
other things.

-- 
error compiling committee.c: too many arguments to function