linux-kernel - Re: [PATCH 3/3] scsi : megaraid_{mm,mbox}: a fix on "kernel unaligned access address" issue

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <200607251705.26361.eike-kernel@sf-tec.de>
Date:	Tue, 25 Jul 2006 17:05:25 +0200
From:	Rolf Eike Beer <eike-kernel@...tec.de>
To:	"Ju, Seokmann" <Seokmann.Ju@...l.com>
Cc:	sakurai_hiro@...t.fujitsu.com, James.Bottomley@...eleye.com,
	akpm@...l.org, linux-scsi@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	"Patro, Sumant" <Sumant.Patro@...enio.com>,
	"Yang, Bo" <Bo.Yang@...enio.com>
Subject: Re: [PATCH 3/3] scsi : megaraid_{mm,mbox}: a fix on "kernel unaligned access address" issue

Ju, Seokmann wrote:
> Hi,
>
> This is a third patch which follows prevous two patches ([PATCH 1/3] and
> [PATCH 2/3]).

Either use a [0/3] mail that describes the complete changeset or just send 
each patch as a reply to the previous one. This way they are grouped together 
in the mail programs. That keeps the inbox clean and the relationsship is 
clearer.

> Signed-Off By: Seokmann Ju <seokmann.ju@...l.com>
> ---
> diff -Naur inqwithevpd/Documentation/scsi/ChangeLog.megaraid
> unaligned/Documentation/scsi/ChangeLog.megaraid
> --- inqwithevpd/Documentation/scsi/ChangeLog.megaraid	2006-07-24
> 15:35:02.000000000 -0400
> +++ unaligned/Documentation/scsi/ChangeLog.megaraid	2006-07-24
> 15:41:49.000000000 -0400
> @@ -66,6 +66,61 @@
>  	Fix: MegaRAID F/W has fixed the problem and being process of
> release,
>  	soon. Meanwhile, driver will filter out the request.
>
> +3.	One of member in the data structure of the driver leads unaligne
                                                                ^^^^^^^^
> +	issue on 64-bit platform.
> +	Customer reporeted "kernel unaligned access addrss" issue when
                                                   ^^^^^^

Typos.

> +	> -----Original Message-----
[...]

This is IMHO too much data for an in-kernel changelog. I would vote for 
including this in your commit comments, then it will be available as git 
comment and not inflate the kernel tree itself with text that's useless for 
most users. This is really only of interest if someone tries to find out 
something about the changes in this driver and then he's normally also 
interested in the diffs itself.

Eike

Content of type "application/pgp-signature" skipped