Re: [PATCH V2 3/6] mpcore_wdt: Fix WDIOC_SETOPTIONS handling
From: Arnd Bergmann
Date: Wed Jul 06 2011 - 10:19:39 EST
On Wednesday 06 July 2011, Russell King - ARM Linux wrote:
> That looks like something which needs fixing in watchdog land. And it's
> not too difficult:
>
> 1. Change the existing WDIOC_SETOPTIONS to WDIOC_SETOPTIONS_OLD
> 2. Add a new, correctly defined WDIOC_SETOPTIONS.
> 3. Update all drivers to use WDIOC_SETOPTIONS.
> 4. Provide a helper which deals with WDIOC_SETOPTIONS_OLD and translates
> it to the proper WDIOC_SETOPTIONS call (this could be just:
>
> if (cmd == WDIOC_SETOPTIONS_OLD) {
> static int first = 1;
> if (first)
> pr_warn("%s:%d used old WDIOC_SETOPTIONS call. Please rebuild\n",
> current->comm, current->pid);
> first = 0;
> cmd = WDIOC_SETOPTIONS;
> }
>
> before any argument copies.
>
> This means you preserve existing compatibility with userspace, yet gain
> a path to a non-broken ioctl interface.
>
> As stuff gets rebuilt and replaced, you'll eventually be able to remove
> the above. That period used to be two years for such simple changes.
Since Wim is currently doing a new core driver to be used by watchdog
drivers anyway, that sounds like a good idea to fix afterwards.
Note that the mpcore_wdt is the only driver that checks the direction
bit, so it's unlikely to cause problems for anything else.
> Hopefully, someone can let Wim know his email is broken...
This should get through to him I hope. Wim, see below.
> Also deleted
> the broken arm@xxxxxxxxxx address.
Ok, sorry about that. I need to finally ask the kernel.org team to
set that up, I announced it too early.
Arnd
> This is the mail system at host ylaen.iguana.be.
>
> I'm sorry to have to inform you that your message could not
> be delivered to one or more recipients. It's attached below.
>
> For further assistance, please send mail to <postmaster>
>
> If you do so, please include this problem report. You can
> delete your own text from the attached returned message.
>
> The mail system
>
> <wim@xxxxxxxxxxxxxxxxx> (expanded from <wim@xxxxxxxxx>): Host or domain name
> not found. Name service error for name=infomag.iguana.be type=AAAA: Host
> found but no data record of requested type
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/