Potential concurrency bug in ide-disk.c ?

2005-09-02 Thread Tushar Adeshara
Hi, The way file ide-disk.c handles usage count, it seems to me that its concurrency bug. In open method and release, it uses code as follows static int idedisk_open(struct inode *inode, struct file *filp) { ide_drive_t *drive = inode->i_bdev->bd_disk->private_data; drive->usage++

Re: the difference between irq save and the irq disable ?

2005-09-05 Thread Tushar Adeshara
On 9/5/05, mandy london <[EMAIL PROTECTED]> wrote: > in my mind, irq save only store the conditions of that time , and the > following code can access the shared region and change it ,so modify > irq states . > > while , disable irq keeps the states of interrupt unchangable . > > but , I have

Re: Kevent POSIX timers support.

2006-12-13 Thread Tushar Adeshara
On 11/23/06, Evgeniy Polyakov <[EMAIL PROTECTED]> wrote: On Wed, Nov 22, 2006 at 01:44:16PM +0300, Evgeniy Polyakov ([EMAIL PROTECTED]) wrote: +static int posix_kevent_init_timer(struct k_itimer *tmr, int fd) +{ + struct ukevent uk; + struct file *file; + struct kevent_user *u;