public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
* Can't change permissions (etc) of FIFOs on JFFS.
@ 2000-08-18 15:30 David Vrabel
  2000-08-18 15:47 ` David Woodhouse
  0 siblings, 1 reply; 2+ messages in thread
From: David Vrabel @ 2000-08-18 15:30 UTC (permalink / raw)
  To: mtd, jffs-dev

Hi,

Attempting to change permissions and times of FIFOs in JFFS results in
    jffs_notify_change(): Invalid inode number = nnn

David Vrabel



To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Can't change permissions (etc) of FIFOs on JFFS.
  2000-08-18 15:30 Can't change permissions (etc) of FIFOs on JFFS David Vrabel
@ 2000-08-18 15:47 ` David Woodhouse
  0 siblings, 0 replies; 2+ messages in thread
From: David Woodhouse @ 2000-08-18 15:47 UTC (permalink / raw)
  To: David Vrabel; +Cc: mtd, jffs-dev


dvrabel@arcom.co.uk said:
>  Attempting to change permissions and times of FIFOs in JFFS results
> in
>     jffs_notify_change(): Invalid inode number = nnn

Fixed. jffs_notify_change() was assuming that the inode would be a JFFS 
inode, when obviously it's a FIFO inode in this case, so it can't just grab 
the pointer to the struct jffs_file out of it.

Fixed committed - now we use jffs_find_file() to find the file by its inum.

A quick scan shows no more such paths which might expect JFFS inodes but 
actually get other types. You know what to do if you find them, though.

Someone who isn't as pissed off with this game as I am right now probably 
wants to port the last 24 hours worth of fixes from inode-v22.c to 
inode-v23.c


--
dwmw2




To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2000-08-18 15:49 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2000-08-18 15:30 Can't change permissions (etc) of FIFOs on JFFS David Vrabel
2000-08-18 15:47 ` David Woodhouse

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox