linux-kernel - Re: [PATCH 1/7] viafb: package often used basic io functions

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] [day] [month] [year] [list]

Date:	Sat, 17 Apr 2010 22:55:23 +0200
From:	Florian Tobias Schandinat <FlorianSchandinat@....de>
To:	Jonathan Corbet <corbet@....net>
CC:	linux-kernel@...r.kernel.org, linux-fbdev@...r.kernel.org,
	JosephChan@....com.tw, ScottFang@...tech.com.cn
Subject: Re: [PATCH 1/7] viafb: package often used basic io functions

Hi Jon,

Jonathan Corbet schrieb:
> On Sat, 17 Apr 2010 19:44:51 +0000
> Florian Tobias Schandinat <FlorianSchandinat@....de> wrote:
> 
>> This patch puts redesigned versions of the basic io functions that
>> are used overall the driver in an extra header.
> 
> I don't object to the basic idea of these patches, but...as I'm sure
> you can imagine, this stuff will conflict hard with the work I'm trying
> to prepare for submission.  I, too, have had to go in to those
> functions and do things like add locking.  Is there any chance you
> could hold off for just a *little* bit while I get things together?  

Huh, I really didn't expect that to conflict with anything you were 
doing as those are basic functions that are only changed a bit and I 
didn't find anything that would cause any disturbance in some of your trees.
As for locking I'd rather suggest doing it in the caller of these 
functions as they are often bundled together.
Feel free to ignore theses patches if they get in your way. As I said 
the next merge window is mostly for you, I just try to get some stuff in 
that has a near zero regression potential and that I don't expect to 
conflict with yours.

> As I said before, I was traveling this last week.  Now I'm home and can
> get back to this...

Thanks,

Florian Tobias Schandinat
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/