linux-kernel - Re: [PATCH v4 39/44] sparc: dma-mapping: Use unsigned long for dma

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:	Fri, 10 Jun 2016 22:03:25 +0200
From:	Krzysztof Kozlowski <k.kozlowski@...sung.com>
To:	David Miller <davem@...emloft.net>
Cc:	k.kozlowski@...sung.com, akpm@...ux-foundation.org,
	hch@...radead.org, b.zolnierkie@...sung.com,
	julian.calaby@...il.com, adam.buchbinder@...il.com,
	sparclinux@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4 39/44] sparc: dma-mapping: Use unsigned long for
 dma_attrs

On Fri, Jun 10, 2016 at 10:43:47AM -0700, David Miller wrote:
> From: Krzysztof Kozlowski <k.kozlowski@...sung.com>
> Date: Fri, 10 Jun 2016 12:11:56 +0200
> 
> > Split out subsystem specific changes for easier reviews. This will be
> > squashed with main commit.
> > 
> > Signed-off-by: Krzysztof Kozlowski <k.kozlowski@...sung.com>
> 
> If you split the method signatures from the type changes to the pointers
> that they are assigned to, this breaks bisection.
> 
> You have to change them all at once, or use some other transitional
> mechanism for the intervening changes.

Yes, this is why I wrote the commit message above. The split here is only
for making code review easier for respective maintainers.

All these smaller patches will be squashed together with the main commit
(patch 1/44 [1] [2]). Unfortunately the CC-list would be too huge to
send it to all maintainers.

Best regards,
Krzysztof

[1] https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg1164335.html
[2] https://github.com/krzk/linux/commit/72dee332b0bedb5cbfc51646d3c03ecbc4a5fe1e