Re: [PATCH] fix some compiler warnings (field width, unused variable)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thursday 01 June 2006 18:54, Atsushi Nemoto wrote:
>  			printk("initrd extends beyond end of memory "
> -			       "(0x%0*Lx > 0x%0*Lx)\ndisabling initrd\n",
> -			       width,
> -			       (unsigned long long) CPHYSADDR(initrd_end),
> -			       width,
> -			       (unsigned long long) PFN_PHYS(max_low_pfn));
> +			       "(0x%p > 0x%p)\ndisabling initrd\n",
> +			       (void *)CPHYSADDR(initrd_end),
> +			       (void *)PFN_PHYS(max_low_pfn));

I'm not so sure if this is a good idea because some systems have 36 bit 
physical addresses while they only have 32 bit void pointers, so long long is 
probably really the better solution.

I'm wondering if it would be worth having a special flag in printk to indicate 
a physical address ("%lp" perhaps?) so that you don't get the unimportant 
leading zeroes for the bits 36 to 63 for above mentioned platforms.

Uli


****************************************************
Visit our website at <http://www.domino-printing.com/>
****************************************************
This Email and any files transmitted with it are confidential and intended solely for the use of the individual or entity to whom they are addressed. If you have received this Email in error please notify the system manager.

This footnote also confirms that this Email message has been swept by MailSweeper for the presence of computer viruses.
****************************************************



[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux