blk-settings.c integer truncation

From: Sergey Senozhatsky
Date: Fri Jun 19 2009 - 05:18:47 EST


Hello.
I was asking
> Should BLK_BOUNCE_ANY be (-1UL)?

No. It should not.

git annotate gives for blkdev.h:
Andi Kleen 2008-04-21 09:51:05 +0200 705)#define BLK_BOUNCE_ANY (-1ULL)
...

...
(Martin K. Petersen 2009-05-22 17:17:51 -0400 311) unsigned long bounce_pfn;
...
(Martin K. Petersen 2009-05-22 17:17:50 -0400 1020)static inline unsigned long queue_bounce_pfn(struct request_queue *q)
(Martin K. Petersen 2009-05-22 17:17:50 -0400 1021){
(Martin K. Petersen 2009-05-22 17:17:51 -0400 1022) return q->limits.bounce_pfn;
(Martin K. Petersen 2009-05-22 17:17:50 -0400 1023)}
...

I think, commit need to be reworked a bit.

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