From: Peter Chubb <peter@chubb.wattle.id.au>
To: Benjamin LaHaise <bcrl@redhat.com>
Cc: Pavel Machek <pavel@suse.cz>,
Peter Chubb <peter@chubb.wattle.id.au>,
torvalds@transmeta.com, linux-kernel@vger.kernel.org
Subject: Re: Large block device patch, part 1 of 9
Date: Wed, 28 Aug 2002 09:40:12 +1000 [thread overview]
Message-ID: <15724.3548.862436.514142@wombat.chubb.wattle.id.au> (raw)
In-Reply-To: <20020827185833.B26573@redhat.com>
>>>>> "Ben" == Benjamin LaHaise <bcrl@redhat.com> writes:
Ben> On Tue, Aug 27, 2002 at 03:23:04PM +0000, Pavel Machek wrote:
>> Hi!
>>
>> > Then the following works properly without ugly casts or warnings:
>> >
>> > __u64 val = 1;
>> >
>> > printk("at least "PFU64" of your u64s are belong to us\n", val);
>>
>> Casts are ugly but this looks even worse. I'd go for casts.
Ben> Casts override the few type checking abilities the compiler gives
Ben> us. At least with the PFU64 style, we'll get warnings when
Ben> someone changes a variable into a pointer without remembering to
Ben> update the printk.
We did go through all this after the first or second incarnation of
this patch, back in May. After going around in circles for a bit, the
use of casts seemed then to win the day.
If Linus has a major objection I'll redo things.
The main advantage of casts, to me at present, is it allows the
patches to be incremental. Not all places that should be sector_t are
yet --- some are int, and some are long. Casting them all to unsigned
long long and printing with %llu works now, before all the places are
changed (in patch 3 or 4 IIRC).
See the thread that starts at
http://marc.theaimsgroup.com/?l=linux-kernel&m=102100347212072&w=2
--
Dr Peter Chubb peterc@gelato.unsw.edu.au
You are lost in a maze of BitKeeper repositories, all almost the same.
next prev parent reply other threads:[~2002-08-27 23:36 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-08-23 5:47 Large block device patch, part 1 of 9 Peter Chubb
2002-08-23 7:07 ` Andreas Dilger
2002-08-27 15:23 ` Pavel Machek
2002-08-27 22:58 ` Benjamin LaHaise
2002-08-27 23:40 ` Peter Chubb [this message]
2002-09-03 10:01 ` Neil Brown
2002-09-03 16:04 ` Linus Torvalds
2002-09-03 16:21 ` Thunder from the hill
2002-09-03 23:06 ` Neil Brown
2002-12-03 0:03 ` H. Peter Anvin
2002-09-03 19:42 ` Daniel Phillips
2002-09-03 19:50 ` Benjamin LaHaise
2002-09-03 20:02 ` Daniel Phillips
2002-09-03 20:04 ` Benjamin LaHaise
2002-09-03 20:19 ` Daniel Phillips
2002-09-03 20:25 ` Daniel Phillips
2002-09-03 20:05 ` Thunder from the hill
2002-09-03 20:19 ` Daniel Phillips
2002-12-03 0:05 ` H. Peter Anvin
[not found] <15732.34929.657481.777572@notabene.cse.unsw.edu.au.suse.lists.linux.kernel>
[not found] ` <Pine.LNX.4.44.0209030900410.1997-100000@home.transmeta.com.suse.lists.linux.kernel>
2002-09-03 16:23 ` Andi Kleen
2002-09-03 8:19 ` Linus Torvalds
2002-09-03 16:43 ` Nikita Danilov
2002-09-03 8:45 ` Linus Torvalds
2002-12-03 0:06 ` H. Peter Anvin
2002-12-05 10:58 ` Pavel Machek
2002-12-05 21:15 ` H. Peter Anvin
[not found] <825516963@toto.iv>
[not found] ` <15734.37217.686498.162782@wombat.chubb.wattle.id.au>
2002-09-05 0:04 ` Benjamin LaHaise
2002-09-05 0:38 ` Daniel Phillips
2002-09-05 1:31 ` Benjamin LaHaise
2002-09-05 4:05 ` Daniel Phillips
[not found] <fa.l4d1mqv.1ghm1h2@ifi.uio.no>
[not found] ` <fa.j8nq6dv.14lihor@ifi.uio.no>
2002-12-03 8:45 ` Giacomo Catenazzi
2002-12-03 17:40 ` H. Peter Anvin
2002-12-03 17:48 ` Giacomo Catenazzi
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=15724.3548.862436.514142@wombat.chubb.wattle.id.au \
--to=peter@chubb.wattle.id.au \
--cc=bcrl@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=pavel@suse.cz \
--cc=torvalds@transmeta.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).