udf uses 16-bit year numbers for inode timestamps, which will work for the 15000 years, but the VFS uses struct timespec for timestamps, and the Linux udf implementation internally uses a time_t, both of which are only good until 2038 on 32-bit CPUs. This gets us one small step closer to lifting the VFS limit by using struct inode_time in udf, but the implementation still depends on the 'year_seconds' array that needs to be extended or replaced by an algorithm that calculates the correct time for every year. Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx> Cc: Jan Kara <jack@xxxxxxx> --- fs/udf/udf_i.h | 2 +- fs/udf/udf_sb.h | 2 +- fs/udf/udfdecl.h | 7 ++++--- fs/udf/udftime.c | 7 ++++--- 4 files changed, 10 insertions(+), 8 deletions(-) diff --git a/fs/udf/udf_i.h b/fs/udf/udf_i.h index b5cd8ed..ba12313 100644 --- a/fs/udf/udf_i.h +++ b/fs/udf/udf_i.h @@ -27,7 +27,7 @@ struct udf_ext_cache { */ struct udf_inode_info { - struct timespec i_crtime; + struct inode_time i_crtime; /* Physical address of inode */ struct kernel_lb_addr i_location; __u64 i_unique; diff --git a/fs/udf/udf_sb.h b/fs/udf/udf_sb.h index 1f32c7b..20ab0ca 100644 --- a/fs/udf/udf_sb.h +++ b/fs/udf/udf_sb.h @@ -135,7 +135,7 @@ struct udf_sb_info { rwlock_t s_cred_lock; /* Root Info */ - struct timespec s_record_time; + struct inode_time s_record_time; /* Fileset Info */ __u16 s_serial_number; diff --git a/fs/udf/udfdecl.h b/fs/udf/udfdecl.h index be7dabb..88320da 100644 --- a/fs/udf/udfdecl.h +++ b/fs/udf/udfdecl.h @@ -237,8 +237,9 @@ extern struct long_ad *udf_get_filelongad(uint8_t *, int, uint32_t *, int); extern struct short_ad *udf_get_fileshortad(uint8_t *, int, uint32_t *, int); /* udftime.c */ -extern struct timespec *udf_disk_stamp_to_time(struct timespec *dest, - struct timestamp src); -extern struct timestamp *udf_time_to_disk_stamp(struct timestamp *dest, struct timespec src); +extern struct inode_time *udf_disk_stamp_to_time(struct inode_time *dest, + struct timestamp src); +extern struct timestamp *udf_time_to_disk_stamp(struct timestamp *dest, + struct inode_time src); #endif /* __UDF_DECL_H */ diff --git a/fs/udf/udftime.c b/fs/udf/udftime.c index 1f11483..1ab6fe7 100644 --- a/fs/udf/udftime.c +++ b/fs/udf/udftime.c @@ -60,6 +60,7 @@ static const unsigned short int __mon_yday[2][13] = { #define SPD 0x15180 /*3600*24 */ #define SPY(y, l, s) (SPD * (365 * y + l) + s) +/* FIXME: convert this to a 64-bit type */ static time_t year_seconds[MAX_YEAR_SECONDS] = { /*1970*/ SPY(0, 0, 0), SPY(1, 0, 0), SPY(2, 0, 0), SPY(3, 1, 0), /*1974*/ SPY(4, 1, 0), SPY(5, 1, 0), SPY(6, 1, 0), SPY(7, 2, 0), @@ -86,8 +87,8 @@ extern struct timezone sys_tz; #define SECS_PER_HOUR (60 * 60) #define SECS_PER_DAY (SECS_PER_HOUR * 24) -struct timespec * -udf_disk_stamp_to_time(struct timespec *dest, struct timestamp src) +struct inode_time * +udf_disk_stamp_to_time(struct inode_time *dest, struct timestamp src) { int yday; u16 typeAndTimezone = le16_to_cpu(src.typeAndTimezone); @@ -119,7 +120,7 @@ udf_disk_stamp_to_time(struct timespec *dest, struct timestamp src) } struct timestamp * -udf_time_to_disk_stamp(struct timestamp *dest, struct timespec ts) +udf_time_to_disk_stamp(struct timestamp *dest, struct inode_time ts) { long int days, rem, y; const unsigned short int *ip; -- 1.8.3.2 -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html