Re: [PATCH] msdos fs: remove unsettable atari option

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

 



On Tue, 3 Jun 2008, OGAWA Hirofumi wrote:
Petr Stehlik <pstehlik@xxxxxxxxxx> writes:
Geert Uytterhoeven wrote:
Geert Uytterhoeven has a patch in his POSTPONED queue that is supposed
to fix Atari FAT.  I can't speak to whether it works or why it's
postponed, but it seems to compile.
I see. Although I don't know the patch, I'd like to add this part to
Geert's patch if possible. And when the patch can be posted, we can
re-add this part again.

Geert, what do you think?

It's in POSTPONED state because nobody speaks up for/against it.
Personally, I don't know how much it is (still) needed...
But a few years ago somebody contacted me that he used it succesfully
on PA-RISC and i386 ;-)

BTW, the patch is
http://linux-m68k-cvs.ubb.ca/~geert/linux-m68k-patches-2.6/POSTPONED/134-atari-fat.diff

Atari people, I think now is the time to let it live or die...

The patch looks good, how can I let it live?

Could you test whether it works well? If it works fine, I'll merge and
clean it up. If it doesn't work, please fix it.

Petr, as there has been no further response anymore, it went into Linus' tree:

    commit 7557bc66be629d19a402e752673708bfbb8b5e86
    Author: Rene Scharfe <rene.scharfe@xxxxxxxxxxxxxx>
    Date:   Fri Jul 25 01:46:45 2008 -0700

	msdos fs: remove unsettable atari option
	
	It has been impossible to set the option 'atari' of the MSDOS filesystem
	for several years.  Since nobody seems to have missed it, let's remove its
	remains.

I'm inclined to drop it, if nobody seems to really care...

BTW,

+		if (sbi->fat_bits != 32 && total_clusters+2 > sbi->
+			fat_length*SECTOR_SIZE*8/sbi->fat_bits)
+			sbi->fat_bits = 12;

SECTOR_SIZE should be sb->s_blocksize? Or Atari doesn't support the
sector size other than 512 bytes?

Gr{oetje,eeting}s,

						Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
							    -- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Video for Linux]     [Yosemite News]     [Linux S/390]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux